- Status Closed
- Percent Complete
- Task Type Patches
- Category Infrastructure → Build environment
- Assigned To No-one
- Operating System All players
- Severity Low
- Priority Very Low
- Reported Version Release 3.0
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#9630 - Improvements for tools/svnversion.sh for git users
This patch modifies tools/svnversion.sh to work with git repositories that lack git-svn information. It will still work with git-svn repos, as it extract the svn revision from the most recent git log entry that originates in svn. Also added is support for a docs/LOCALVERSION file to be appended to the version, useful because most git users will probably want to add a .gitignore, which would cause attempts to detect local modification to always return true.
EDIT: corrected testing junk that did not belong in patch.
Closed by Unhelpful
2009-01-19 16:54
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
2009-01-19 16:54
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
r19799 committed the patch from
FS#9758
, with equivalent functionality
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
nice, but I don’t get the reason for the LOCALVERSION – the VERSION file that gets checked isn’t in svn anyway, so one could use that instead. Or am I missing something here?
As I read it, using VERSION gets a version that reads as <VERSION>-<DATE>. LOCALVERSION is instead appended after <REVISION>-<DATE>, so that the final version displayed gives the SVN revision, the build date, and whatever local version tag you’ve added. Perhaps it would be more sensible to remove LOCALVERSION, and make VERSION work this way? Another option, if git is detected, is to append the branch being built from, but the label may not be meaningful, or may contain characters that need to be sanitized, etc.