Closed
Bug 188655
Opened 23 years ago
Closed 23 years ago
dist/sdk/mozilla-config.h is installed with perm=444
Categories
(SeaMonkey :: Build Config, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla1.3beta
People
(Reporter: roland.mainz, Assigned: netscape)
Details
Attachments
(1 file)
|
851 bytes,
patch
|
timeless
:
review+
|
Details | Diff | Splinter Review |
2003-01-06-08-trunk build with Sun Workshop 7 on Solaris 2.7/SPARC.
While removing the objdir for a full rebuild I noticed this minor nit:
dist/sdk/mozilla-config.h is installed with perm=444 - as the _only_ file with
this permission flag combination (/usr/bin/rm then bails-out with "rm:
2003-01-06-08-trunk/objdir_ws7_gtk/dist/sdk/mozilla-config.h: override
protection 444 (yes/no)?") in objdir/ - all other files are writeable by the
user.
Is that intentional ?
| Assignee | ||
Comment 1•23 years ago
|
||
The mode setting was intentional as there was no reason for the headers to be
writable once it was installed.
Summary: dist/sdk/mozilla-config.h is installed with perm=444 → dist/sdk/mozilla-config.h is installed with perm=444
| Reporter | ||
Comment 2•23 years ago
|
||
Christopher Seawood wrote:
> The mode setting was intentional as there was no reason for the headers to be
> writable once it was installed.
OK, but why is this the only header with u-w ? All other header in dist/ are
writeable by the owner...
| Assignee | ||
Comment 3•23 years ago
|
||
Because, afaik, that's one of the few headers that are not installed using one
of the common rulesets and this is the first time I've seen a complaint about
the header not being removable. Even the solaris tinderboxes haven't complained.
IMO, that's a bug in that version of solaris rm. You don't have to be able to
write to the file to remove it. You just have to be able to write to the directory.
| Assignee | ||
Comment 4•23 years ago
|
||
Attachment #111828 -
Flags: review+
| Assignee | ||
Comment 5•23 years ago
|
||
The patch has been checked in.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.3beta
Updated•21 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•