Hi Seth,

On 07/16/11 10:31 AM, Seth Goldberg wrote:
Hey there,

  On fresh install, they have mode 0341.

Here's the bug in usr/src/lib/install_boot/boot.py:

871                    os.makedirs(grub_dir, 755)

  That second argument should be "0755".

Oops :-)

Since we're on the subject and you've presumably noticed the comments above,
I'm of the opinion that pybootmgmt should be creating this directory itself since the current situation requires the installer boot checkpoint to have intimate knowledge
about the layout of the legacy GRUB configuration file directory structure.

I think that's reasonable, if the caller requests to commit the boot config without specifying a directory (i.e. a "real" commit). I'll put it on my list :).

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

Reply via email to