Hi Jan --
I only really have one comment. The rest looks ok.
pre_pkg_img_mod.py
-----------------------------
ln 144 save_etc_files function -- I was wondering....would it make sense to
just get rid of this function. Instead, define the save_list in the
__init__
and then just call the save_files_directories function where
save_etc_files is called (at ln 730 and 787).
sinc that's the only thing the save_etc_files function does.
That's really my only comment, although if you choose to make this
change, then you
might be able to take advantage of the save_list in the test setup.
Instead of having
to create a list (self.pi_filelist), you could use the list in the
PrePkgImgMod class.
thanks,
ginnie
On 04/24/12 04:42 AM, Jan Damborsky wrote:
Hi,
I would appreciate code review of changes for
7162079 CPIO installation propagates install root password to the target
webrev:
https://cr.opensolaris.org/action/browse/caiman/dambi/cr-7162079/webrev/
Thank you,
Jan
Testing done:
* built text/AI/LiveCD install images using modified Distro Const.
Workarounds for 7162039 and7162553were applied on LiveCD.
* tested text, AI, LiveCD installations.
Verified that shadow(4) was restored from 'save' area in case
when CPIO transfer method was used for installation (text, LiveCD).
* run unit tests (new unit test added for introduced
method PrePkgImgMod.save_etc_files()
_______________________________________________
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
