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
