Bug 28075 - anaconda's documentation of "part" is incorrect
Summary: anaconda's documentation of "part" is incorrect
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Red Hat Linux
Classification: Retired
Component: anaconda
Version: 7.0
Hardware: i386
OS: Linux
medium
high
Target Milestone: ---
Assignee: Tammy Fox
QA Contact: Brock Organ
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2001-02-17 02:37 UTC by Jim Wright
Modified: 2007-04-18 16:31 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2001-03-05 17:11:27 UTC
Embargoed:


Attachments (Terms of Use)

Description Jim Wright 2001-02-17 02:37:12 UTC
From Bugzilla Helper:
User-Agent: Mozilla/4.75 [en] (X11; U; Linux 2.2.16-22 i686)


The documentation of "part" in RH7.0 reference guide implies that the
correct format would be similar to:

    part /boot --size 20   --asprimary 1

However, looking at the code (and based on my experience) the correct
format is

    part /boot --size=20   --asprimary=1

The python code is using getopt, and the "--" arguments expect an "=" with
no whitespace.

Reproducible: Always
Steps to Reproduce:
1. try a kickstart file with the incorrect syntax

	

Actual Results:  anaconda complains that 

  partition command requires one anonymous argument

A cryptic message that is meant to inform the user that all options should
be parsed by getopt, with only the name of the filesystem left as an
"extra"
argument.

Expected Results:  Documentation should correctly show what is expected.

It would also be nice to have it documented that the numbers should be
integers which represent filesystem size in megabytes.  It quickly became
clear that "20m" is incorrect.

Comment 1 Glen Foster 2001-02-21 18:22:52 UTC
We (Red Hat) should really try to resolve this before next release.

Comment 2 Glen Foster 2001-03-02 18:05:37 UTC
Our installer team-lead thinks we should really fix this before next release.

Comment 3 Brent Fox 2001-03-05 17:04:16 UTC
tfox, I think he is right in that there must be an "=" sign.  Can you correct
the documentation?

Comment 4 Brent Fox 2001-03-05 17:11:23 UTC
Wait a minute, that's not right.  "--onprimary" expects an "=" sign, but
"--asprimary" takes no arguments.  I think the original documentation is
wrong...it needs to be explicit that asprimary does not take any arguements.
Trying to pass one will cause python to complain.

Comment 5 Tammy Fox 2001-03-05 17:22:08 UTC
I corrected the docs to reflect the difference between asprimary and onprimary
and added an example of onprimary. I also added a note that the size and maxsize
are in MB and only expect an integer value.

Fixed in CVS


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