Bug 286791

Summary: RFE: support spaces in flavor name
Product: [Fedora] Fedora Reporter: David Cantrell <dcantrell>
Component: pungiAssignee: David Cantrell <dcantrell>
Status: CLOSED UPSTREAM QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: low Docs Contact:
Priority: medium    
Version: rawhide   
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-11-20 22:13:11 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description David Cantrell 2007-09-11 19:38:04 UTC
I want to be able to do --flavor="mortise restore"

Comment 1 Jesse Keating 2007-09-11 19:51:25 UTC
Eeew, I don't know if I want that, the Flavor is only used for the path, do you
really want spaces in your path?  The flavor doesn't show up anywhere in the
install.

Comment 2 David Cantrell 2007-09-11 20:01:04 UTC
Yeah, why not?  This isn't DOS or a wiki.  We can have spaces.  It's more
descriptive that way.  Filenames shouldn't have \0 or /, everything else is game.

I'd really like to name my flavor "\a\a\a mortise\trestore!\n"   :)

If you don't want to do that, pungi should be more clear about what --flavor is
used for and what characters are legal.

Comment 3 Jesse Keating 2007-09-11 21:49:57 UTC
Hrm, looks like pungi dealt with spaces OK until it came time to run
buildinstall.  Buildinstall just didn't like a space in the name.  I'll need to
investigate either a way to make the pass off to buildinstall happier with
regard to paths or just limit flavor to no spaces and no funky chars.  Thanks
os.path.join().

Comment 4 Jesse Keating 2007-11-20 22:13:11 UTC
Filed and tracked as https://hosted.fedoraproject.org/projects/pungi/ticket/62