Thanks for the info, Danek.
Do you know if this standard is already in place as a convention right now or 
is it still planned?
It certainly seems pretty straightforward, but I'd do this as a separate change 
to be clean.

Martin

On 7/28/2011 4:13 PM, Danek Duvall wrote:
Martin Widjaja wrote:

Hi,
I'm requesting a quick code review for a fix to CR 
7026148<http://monaco.sfbay.sun.com/detail.jsf?cr=7026148>- install exec_attr and 
prof_attr entries should be flagged 
read-only<http://monaco.sfbay.sun.com/detail.jsf?cr=7026148>

https://cr.opensolaris.org/action/browse/caiman/widjaja/7026148/

Frame version of the webrev doesn't work well for files with URL encoding
embedded in their names, such as
exec_attr.system%2Finstall%2Fmedia%2Finternal.
For consolidations transitioning to IPS, Comay is suggesting that the
*_attr fragments they deliver be delivered as full package names, but with
colons replacing slashes.  This is more readable than the %2f approach, and
wouldn't have this issue in webrev, either.

I know it's not strictly related to this change, but it might be relatively
straightforward to implement.

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

Reply via email to