Thanks, Dave !
Jan

On 06/04/12 17:24, Dave Miner wrote:
Jan, this looks OK.

Dave

On 06/04/12 05:44, Jan Damborsky wrote:
Hi,

could I please ask one more pair of eyes to take a look at the fix ?
If testing does not reveal new issues, I would like to integrate into 18, so that net booted text installer does not remain broken for another build.

Webrev with Karen's comments addressed is available at:

incremental:
https://cr.opensolaris.org/action/browse/caiman/dambi/cr-7172606-1-diff/webrev-1-diff/

full:
https://cr.opensolaris.org/action/browse/caiman/dambi/cr-7172606-1/webrev-1/

Thank you very much,
Jan


On 06/ 1/12 01:07 PM, Jan Damborsky wrote:
Hi,

could I please ask two pairs of eyes to take a look at fix for

7172606 Network text install fails trying to download /etc/shadow

webrev:
https://cr.opensolaris.org/action/browse/caiman/dambi/cr-7172606/webrev/

Thank you,
Jan


Testing done:
* built text/AI/LiveCD Sparc/x86 install images using modified Distro
Const.

* Verified that 'save area' now resides under /mnt/misc/save directory
   (delivered by solarismisc.zlib archive)

* tested text (booted from media and network),
   AI (booted from media and network),
   LiveCD  installations (Sparc and x86).

* run related unit tests (two unit tests modified)
_______________________________________________
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


_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

Reply via email to