Hi Keith,
grub_setup.py:
----------------------
lines 185-191: "installadm_entry" is a new optional argument you introduced.
Why not explicitly checking for that argument since that's the only thing
that gets interpreted at this time? The way it is coded now will allow
other arguments to be specified, but then, people will be confused as
to why the values they specified are being ignored.
pkg_img_mod.py
------------------------
- line 381: Why did you add a call to create_save_list() here? My
understanding
is that the file created by the create_save_list() call will be used as
"instructions" on what files to download from the server when the text
installer
boots up. I don't think this file is needed for the case of the LiveCD.
net-fs-root
---------------
- line 380: Nit: I was confused for a little bit when I read this line,
because you specified
the output file first, which is different convention than the rest of
the wget calls in the file where
you first specify which file to download, and then, specify where to put
it. Perhaps it will
be good to be consistent?
Thanks,
--Karen
On 01/ 6/11 03:57 PM, Keith Mitchell wrote:
This might help:
http://cr.opensolaris.org/~kemitche/6971585.final/
On 01/ 6/11 03:56 PM, Keith Mitchell wrote:
Hi,
Can I get one or two sets of eyes to look at (what I hope to be) the
last webrev for netboot text installer? This work was started by
Andrew and Chris last fall, and I've merged it with the updated DC,
as well as ironed out a few last minute bugs (plus tossed in a fix to
7010312 for good measure).
The files that have changed since last are:
grub_setup.py
pkg_img_mod.py
net-fs-root
ti_install.py
Though comments on the other files are of course welcome as well.
Thanks,
Keith
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss
