Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Infrastructure → Build environment
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version Version 3.3
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by torne - 2009-07-08
Last edited by torne - 2009-10-15

FS#10414 - tools/version.sh support for bzr or bzr-svn repositories

tools/version.sh has some support for displaying the revision number for git-svn repositories, and their dirty status.

Attached patch adds similar support for bzr and bzr-svn. It uses the presence of a .bzr directory to detect bzr branches which should be fairly unambiguous. :)

1) A bzr-svn checkout of r12345 will output "r12345-date" or "r12345M-date" if there are uncommitted changes. This should make it behave identically to svn if bzr is being used as a direct replacement.

2) A bzr branch which has merged svn changes up to r12345 but is itself at bzr revision 13579 will output "r12345+bzr13579" or "r12345+bzr13579M" if there are uncommitted changes. This lets a developer using bzr keep track of what svn revision their branch is based from, but also shows them what revision of their own bzr-committed work they are at.

3) A bzr branch with no svn changes in its ancestry at revision 13579 will output "bzr13579" or "bzr13579M" if there are uncommitted changes. This may be useful if tailor or other VCS-converting schemes are in use instead of bzr-svn - the version number is meaningless to the rest of the world but at least means something to the person who made the build.

There should be no impact on anyone who doesn't have a .bzr directory in their copy of rockbox :)

Closed by  torne
2009-10-15 12:43
Reason for closing:  Fixed
Additional comments about closing:   Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407

Submitted in r23187

torne commented on 2009-07-08 19:46

Whoops, managed to upload an empty diff.

torne commented on 2009-07-08 19:49

…aaaand accidentally changed a line of the git support.

I assure everyone that this is just me making foolish mistakes and not any shortcoming of bzr :)

ajb commented on 2009-07-09 09:03

Looks OK to me.

I know some people complained about me adding a git commit -ish id when I added the git stuff which is why it only indicates the nearest svn version and the M for modified.

torne commented on 2009-07-09 09:25

My build comes from a loom of 12-15 branches stacked on top of each other so without the bzr revision I have no chance in hell of keeping track what's on my ipod at any given time :) Bzr revisions are at least short (but not global).

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing