Opened 20 years ago

Closed 20 years ago

Last modified 18 years ago

#843 closed defect (fixed)

problems with chowning the package dir

Reported by: archaic@… Owned by: alfs-log@…
Priority: highest Milestone:
Component: Profiles (LFS) Version: SVN
Severity: normal Keywords:
Cc:

Description

Chown --recursive in addinguser.xml is at best unnecessary and at worst, can be dangerous. The reasoning is since the defacto standard in at least the 5.0-3 and current testing profiles is to copy the tarballs, and the umask is set to 022, it will not matter if the packages dir is chowned since user lfs only needs 755 on the dir and 644 on the tarballs (which will happen with umask 022). Now in a copy itself, it doesn't really matter (but doesn't add any value, either). But if you mount --bind your sources then either 2 things will happen. One, if you keep sources on RO media (like a CD) the build will die. Two, if the sources are on one of the host dirs, you are effectively modifying the host, which can cause unnecessary problems and should probably be avoided. The last problem is that in creatingtoolsdir.xml, the comment about mount --bind refers to a umount command which isn't in the profile anywhere.

Change History (4)

comment:1 by archaic@…, 20 years ago

I don't know why it formatted that way. I'll redo it:

Chown --recursive in addinguser.xml is at best unnecessary and at worst, can be dangerous. The reasoning is since the defacto standard in at least the 5.0-3 and current testing profiles is to copy the tarballs, and the umask is set to 022, it will not matter if the packages dir is chowned since user lfs only needs 755 on the dir and 644 on the tarballs (which will happen with umask 022). Now in a copy itself, it doesn't really matter (but doesn't add any value, either). But if you mount --bind your sources then either 2 things will happen. One, if you keep sources on RO media (like a CD) the build will die. Two, if the sources are on one of the host dirs, you are effectively modifying the host, which can cause unnecessary problems and should probably be avoided. The last problem is that in creatingtoolsdir.xml, the comment about mount --bind refers to a umount command which isn't in the profile anywhere.

comment:2 by jwrober@…, 20 years ago

Version: CVSSVN

comment:3 by kpfleming@…, 20 years ago

Resolution: fixed
Status: newclosed

Has been fixed (the ownership change is no longer recursive).

comment:4 by Thomas, 18 years ago

Component: LFS ProfilesLFS Tasks
Note: See TracTickets for help on using tickets.