Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing v3.6.1 tag on master branch #1657

Closed
robertoleksy opened this issue Jun 28, 2019 · 8 comments
Closed

Missing v3.6.1 tag on master branch #1657

robertoleksy opened this issue Jun 28, 2019 · 8 comments

Comments

@robertoleksy
Copy link

Found by [email protected]

@nlohmann
Copy link
Owner

You're right. Can you give me a hint how to set it after the fact?

@MartB
Copy link

MartB commented Jun 29, 2019

Switch to master and issue the following:

git tag v3.6.1 1126c9c 
git push origin v3.6.1

@nlohmann
Copy link
Owner

This does not work unfortunately:

$ git tag v3.6.1 1126c9c

fatal: tag 'v3.6.1' already exists

$ git push origin v3.6.1

Everything up-to-date

@MartB
Copy link

MartB commented Jun 29, 2019

Edit nvm im wrong here, tags are bound to commits not to branches so it shouldnt matter.

You can close this issue as invalid then, if master doesnt differ from develop

@nlohmann
Copy link
Owner

Alright. Thanks for reporting anyway!

@MartB
Copy link

MartB commented Jun 29, 2019

Okay actually, your master branch differs from the develop branch which is not really a great idea, the tag points to a commit which is only available in the develop branch.

@nlohmann
Copy link
Owner

I am using the same release script for a year now, so I am confused that tags behave differently now.

@MartB
Copy link

MartB commented Jun 29, 2019

This is not specific to 3.6.1 a few more i checked are also only in develop.

It entirely depends on how you want to tag, if you want the master branch to be used as a way for users to build releases based on tags, the current tagging wont work and one must use develop for it.

If you want the master branch to have the same tags, you need to tag a commit that is in both branches.

You can clearly see why they are not the same atm:
https://github.com/nlohmann/json/compare/master

all the release merge commits are missing on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants