- 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 Daily build (which?)
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
Attached to Project: Rockbox
Opened by Catalin Patulea - 2007-09-10
Last edited by Jens Arnold - 2007-09-11
Opened by Catalin Patulea - 2007-09-10
Last edited by Jens Arnold - 2007-09-11
FS#7748 - Fix svnversion.sh for localization and failure corner-cases
There were two problems with svnversion.sh@14659:
- If it was run on a machine with localization set to anything other than
English, the comparison to "exported" would get confused and print something like "rexportiert -070910". This caused build problems because the string isn't quoted before being passed to gcc.
- If svnversion fails (such as when invoked with a non-existent directory),
the output would look like "r-070910" because svnversion.sh didn't check for command failure.
This patch fixes both issues, and refactors svnversion error and output handling
into a shell functions.
Thanks to Rincewind, GodEater_ and amiconn for discovering the bug, helping fix
it and test the fix.
Minimal (quick-n-dirty) version of original path.