]> git.donarmstrong.com Git - debhelper.git/commit
Use ASCII RS for separating options in DH_INTERNAL_OPTIONS.
authorModestas Vainius <modestas@vainius.eu>
Mon, 29 Jun 2009 23:55:41 +0000 (02:55 +0300)
committerModestas Vainius <modestas@vainius.eu>
Mon, 29 Jun 2009 23:55:41 +0000 (02:55 +0300)
commite1669c2f4bec174ea29725430640bbbca950ff4c
tree3550e7e2175b5d7a31c493f38b34e151596a0cdc
parent2ddab1cefed6fb9a1ec173e32fd01b0ca7514abb
Use ASCII RS for separating options in DH_INTERNAL_OPTIONS.

Since now extra options via dh command line arguments are encouraged, dh will
break when a bit more complex option gets added to DH_INTERNAL_OPTIONS and it
gets misparsed by the debhelper command called from the override. E.g.
debian/rules:

| %:
|  dh --builddirectory="build dir"
|
| override_dh_install:
|  dh_install

Will fail with something like:

| ....
| make[1]: Entering directory `............'
| dh_install
| cp: cannot stat `debian/tmp/dir': No such file or directory
| dh_install: cp returned exit code 1
| make[1]: *** [override_dh_install] Error 1

So since DH_INTERNAL_OPTIONS is exclusively for internal use, why not to use an
old good ASCII unrepresentable control character as a separator? So I chose
ASCII 1E - RS Record Separator.

Signed-off-by: Modestas Vainius <modestas@vainius.eu>
Debian/Debhelper/Dh_Getopt.pm
dh