Matt, two questions:

- Why shouldn't we just deliver the legacy /var/sadm... path as a symlink in one of the packages?

- Why do we still have the UPGRADE_FAILURE_PANEL.txt file? We haven't allowed upgrades from the installer since the Dwarf Caiman days.

Dave

On 02/01/12 04:48, Matt Keenan wrote:
Karen,

Thanks for the review.

The first paragraph of the Evaluation was in response to this comment,
indicating that the symlink is already being created by all three
installers.

Do you still feel I need to add more ?

cheers

Matt

On 01/30/12 18:12, Karen Tung wrote:
Hi Matt,

The code changes look good to me.

I have a question about Comment entry 1 that Dave made in the bug.
The comment talks about deliver /var/sadm/system/logs as a link to
/var/install/log.
I do not see any response to that comment or the code changes
implementing that.
Can you update the bug to talk about what you are doing for Dave's comment?

Thanks,

--Karen

On 01/30/12 09:50, Matt Keenan wrote:
Hi,

Can I get CR for following bug :
7130956 - Change default log location to /var/log/install
http://monaco.us.oracle.com/detail.jsf?cr=7130956

Webrev :
https://cr.opensolaris.org/action/browse/caiman/mattman/7130956/


This fix is a follow on from the recently integrated bug 7107775, and
turned out to be a lot less painful than envisioned, in fact rather
trivial.

Testing :
- Built all three ISOS ai/text/gui and test installed to ensure logs
are being installed to /var/log/install
- Ran complete set of unit tests and no regressions found.
_______________________________________________
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