Bug 510639

Summary: Revisor 2.1.6-1 fails to build Fedora 11 images
Product: [Fedora] Fedora Reporter: Philip Heuer <pheuer>
Component: revisorAssignee: Jeroen van Meeuwen <vanmeeuwen+fedora>
Status: CLOSED CURRENTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: 11CC: jonathansteffan, vanmeeuwen+fedora
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: 2009-08-12 23:03:39 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
Fix for a regular expression issue in kickstart.py
none
Fix for buildinstall call in pungi.py none

Description Philip Heuer 2009-07-10 01:14:46 UTC
Created attachment 351202 [details]
Fix for a regular expression issue in kickstart.py

Description of problem:

Revisor failed to parse a kickstart file and build iso images for Fedora 11.

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

2.1.6-1

How reproducible:

always

Steps to Reproduce:
1. Create a kickstart file with a %packages section.
2. Attempt to build.
  
Actual results:

Revisor failed to parse the kickstart file and create the images.

Expected results:

Images would be created successfully.

Additional info:

I found that two issues prevented the proper creation of the iso images. I am attaching patches below.

Comment 1 Philip Heuer 2009-07-10 01:18:59 UTC
Created attachment 351203 [details]
Fix for buildinstall call in pungi.py

This patch fixes the syntax of the buildinstall call (adds --output before the output directory).

Comment 2 Jeroen van Meeuwen 2009-07-11 11:23:10 UTC
These have been applied in GIT already, just not releases in the form of a package yet. Thanks!

I'll leave this bug open until after the release of an update. I'm still working on some other issues as well.

Comment 3 Jeroen van Meeuwen 2009-08-12 23:03:39 UTC
revisor-2.1.7 with these fixes should have landed by now.