Hi Jack,

On 11/01/10 10:57, Jack Schwartz wrote:
Hi Karen.

Thanks for your review.

On 11/01/10 10:35, Karen Tung wrote:
Hi Jack,

A couple of comments:

- Is there a reason for naming the file as exec_attr.system%2Finstall%2Fmedia%2Finternal,
instead of exec_attr.system-install-media-internal?
The names of these files are supposed to mimic the names of the packages they represent. (Take a look at the filenames in /etc/security/exec_attr.d for more examples.) In my case, the package name is
    system/install/media/internal
and the "%2F" represents the "/".  The filename properly reflects this.
Thanks for the explanation. I was looking at how the GUI installer added the RBAC entry
and the name for that file is simply exec_attr.SUNWgui-install even though
the package name is system/install/gui-install. I guess that's not a good example. :-)


- In system-install-media-internal.mf, need to add some entries to make sure
all the directories up to /etc/security/exec_attr.d are created.
Thanks.  Fixed.

Webrev respun, same location:
http://cr.opensolaris.org/~schwartz/101031.1/webrev/

Updated webrev looks good.

Thanks,

--Karen


    Thanks,
    Jack


Thanks,

--Karen
*
*On 10/31/10 17:25, Jack Schwartz wrote:
Hi everyone.

Here is a quick webrev to enable the
  /usr/lib/install/live_img_pkg5_prep
script to run with privilege on the live CD. This script copies the /var/pkg tree to its own tmpfs file system so pkg can work. Without privilege, the DDU run as the jack user on the live CD cannot install pkg packages.

Webrev: http://cr.opensolaris.org/~schwartz/101031.1/webrev/
Bug report: http://monaco.sfbay.sun.com/detail.jsf?cr=6996315

Please review ASAP to make it into 151a.

This fix has been approved for 151a.

Tested by booting a live CD image, verifying /etc/security/exec_attr, running the script as the jack user, and installing a pkg-delivered driver with the DDU.

    Thanks,
    Jack

P.S. The DDU team has made other changes on their side to pfexec this script instead of just call it. (See bug 6996173.)


_______________________________________________
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