Bug 323661 - revisor doesn't parse kickstart reboot command properly
Summary: revisor doesn't parse kickstart reboot command properly
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: revisor
Version: 7
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Jonathan Steffan
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-10-08 19:51 UTC by Vadym Chepkov
Modified: 2007-11-30 22:12 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-11-11 01:03:08 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Vadym Chepkov 2007-10-08 19:51:41 UTC
Description of problem:

When you create an image with your own kickstart file that contain command
'reboot --eject' and choose option to write kickstart file on the image revisor
produces wrong output:

# Reboot after installation
reboot
 --eject# System services

The command got split in two rows and produces wrong kickstart file

Version-Release number of selected component (if applicable):
revisor-2.0.4.1-2.fc7

Comment 1 Jonathan Steffan 2007-10-09 07:28:56 UTC
Strange. We use pykickstart to write out the kickstart into the tree the image
is composed from. Thanks for the report, we'll look into it.

Do you know what version of pykickstart you are using?

Comment 2 Vadym Chepkov 2007-10-09 11:51:53 UTC
It's pykickstart-1.1.1-1.fc7

Comment 3 Vadym Chepkov 2007-10-10 14:59:31 UTC
I have discovered another issue and I am not sure if I need to open another
ticket, but here is another command that is not getting parsed properly:

Source command:
part pv.1 --size=0 --grow --ondisk=sda --asprimary

After parsing, wrong one, since size has to be specified:
part pv.1 --asprimary --grow --ondisk=sda --bytes-per-inode=4096

Is there any reason why kickstart file is getting re-writen anyway, why not just
copy it as is ?

Comment 4 Jonathan Steffan 2007-11-11 01:03:08 UTC
Kickstart is now copied.


Note You need to log in before you can comment on or make changes to this bug.