Bug 668326

Summary: mash config only allows one config section per file
Product: [Fedora] Fedora Reporter: Bradley <bbaetz>
Component: mashAssignee: Bill Nottingham <notting>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: low Docs Contact:
Priority: low    
Version: 14CC: dcantrell, notting, rvokal
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-07-11 01:35:28 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:
Attachments:
Description Flags
Patch none

Description Bradley 2011-01-10 04:00:48 UTC
Created attachment 472495 [details]
Patch

Description of problem:



Version-Release number of selected component (if applicable):

mash-0:0.5.20-1.fc14.noarch

How reproducible:

Always

Steps to Reproduce:
1. Create a file /etc/mash/test.mash:

[taga]

[tagb]

2. Run:

mash -o /tmp/tagb

Actual results:

ERROR: No configuration named 'tagb'!

Expected results:

configuration is found

Additional info:

This is because the MashDistroConfig is only set up per file, not per section, so any attributes set on the first section also apply to subsequent sections.

the mash config has:

                if not thisdistro.name:
                    thisdistro.name = sect

so they all end up with the name of the first section. Setting the |name| attribute is not a workaround; all that happens in the *last* section's name then gets used because the same thisdistro object is appended multiple times to config.distros.

Same bug is in the epel5 mash version

Comment 1 Fedora Update System 2012-06-22 20:18:22 UTC
mash-0.5.28-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/FEDORA-EPEL-2012-6209/mash-0.5.28-1.el6

Comment 2 Fedora Update System 2012-06-25 13:34:07 UTC
Package mash-0.5.28-1.el6:
* should fix your issue,
* was pushed to the Fedora EPEL 6 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=epel-testing mash-0.5.28-1.el6'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-EPEL-2012-6209/mash-0.5.28-1.el6
then log in and leave karma (feedback).

Comment 3 Fedora Update System 2012-07-11 01:35:28 UTC
mash-0.5.28-1.el6 has been pushed to the Fedora EPEL 6 stable repository.  If problems still persist, please make note of it in this bug report.