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

Build number policy #255

Closed
AntoineTurmel opened this issue Jan 23, 2014 · 7 comments
Closed

Build number policy #255

AntoineTurmel opened this issue Jan 23, 2014 · 7 comments

Comments

@AntoineTurmel
Copy link
Member

I was wondering if we should keep the build number... it's not really useful in my opinion since you can check version + date already...

Buildnumber was used on Songbird and incremented at each commit (if I remember well), and we don't really do so.

@mook
Copy link
Member

mook commented Jan 23, 2014

The build number was incremented as part of the build process (in buildbot,
not in normal offline/developer builds). That was useful when you have
multiple build per day (e.g. spinning out RCs in preparation for a release).

For reference, Komodo currently uses the subversion revision number, since
that atomically increases automatically. In the git world, git describe
might be a good alternative. Of course, these two methods lose the ability
to differentiate builds using the same source (e.g., different dependency
binaries or build process changes). At this point, though, actually getting
builds might be higher priority.

@ilikenwf
Copy link
Member

I'd be one to vote for using git describe...it's what I'm using for the versioning on the Archlinux PKGBUILD.

@rsjtdrjgfuzkfg
Copy link
Member

Agree on the git describe for the buildnumber.

@freaktechnik freaktechnik added this to the 1.12.2 milestone Mar 18, 2014
rsjtdrjgfuzkfg added a commit to rsjtdrjgfuzkfg/nightingale-hacking that referenced this issue Feb 8, 2015
Build numbers are now generated from git. I'm not yet certain how portable this
solution is, though. Tested on Linux; not sure if this is compatible with
Windows and/or OSX.
@rsjtdrjgfuzkfg rsjtdrjgfuzkfg self-assigned this Feb 8, 2015
@rsjtdrjgfuzkfg
Copy link
Member

#334 -- not sure if one can pull-request on an existing issue. If that is possible, I successfully failed to do that.

@freaktechnik
Copy link
Member

No, you can't. You could make github corss refer to the issue by mentioning it in the pull's description, like it did for the commit. I'm also wondering if this kind of mention in the commit will auto-fix this bug as soon as it's merged.

freaktechnik added a commit that referenced this issue May 11, 2015
@freaktechnik
Copy link
Member

As I commented in #334:

Should we check for git's presence if --enable-official or --enable-nightly is set?

@rsjtdrjgfuzkfg
Copy link
Member

@freaktechnik probably a good idea, if you know a fail-safe way to do that cross-platform? I'm not too sure what's possible in all shells used to build Nightingale. But it is not really bad if we don't imho.

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

5 participants