Red Hat Bugzilla – Bug 845848
rhts-devel scripts fail when path name contains space character
Last modified: 2015-11-22 22:57:45 EST
Description of problem:
When you have made some automated test for beaker and want to do "make tag; git push --tags; make bkradd", then make bkradd fails when you have a space character in the path. (e.g. you have directoty "Red Hat", where you have other working subdirectories, then it fails because of the space in directory name "Red Hat".)
It also fails if you substitute "make noarch-rpm" instead of "make bkradd".
(The "make arch-rpm" isn't working at all at the time, as far as I know. I will be adding new bug to bugzilla when I confirm it.)
Version-Release number of selected component (if applicable):
Steps to Reproduce:
1. Copy some test-case into new directory. Then rename the directory so it contains space character (ASCII #32 Dec).
2. Dive into directory and run these commands: "make tag; git push --tags; make bkradd" or "make tag; git push --tags; make noarch-rpm".
make bkradd/noarch-rpm fails with exit code 2. Make reports:
make: *** [bkradd] Error 1
make bkradd/noarch-rpm doesn't fail and exit code is 0. Then the requested operation is completed sucessfully.
I'm proposing a patch...
Thanks for this patch, David. At first glance it looks good, although there are some unrelated changes bundled in there too (adding usage messages, changing exit statuses) which look useful, but I would rather they be separated out into their own patches.
I'm clearing the hss_*_ack flags since we aren't using those in Beaker at present. And there's no need for this bug or the comments to be private, since there is no confidential information here.
(In reply to comment #5)
> ...although there are some unrelated changes bundled in there too (adding usage > messages, changing exit statuses) which look useful, but I would rather they be
> separated out into their own patches.
Ah, sorry for that. I didn't know if that would be necessary. I've made some other changes, that you are writing about, to keep the rhts scripts consistent. (E.g. that usage message was added because argument processing in one those script were calling usage function in case wrong arguments were used. However the script hadn't any function usage... So I've "patched" these things too. I've also added quotes to some variables that are actually not required right now, but it might be useful in future, if there'll be some other updates.)
> And there's no need for this bug or the comments to be private, since there is > no confidential information here.
At the beginning I wasn't sure if this is or is not an internal component... Anyway, why shouldn't be there License agreemenet like GPL and other information if headers of the scripts? Some of them has only #! shebang and they're not consistent of what shell they're using... ;)
Bulk reassignment of issues as Bill has moved to another team.
The problem is still present in current version of scripts.
Created attachment 733280 [details]
Patches to fix the issue
Set of patches to rhts-mk-get-test-package-name and rhts-build-package scripts to fix handling of spaces in working directory name.