[Bug 225983] Merge Review: less

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: less


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225983





------- Additional Comments From ville.skytta at iki.fi  2007-02-19 14:03 EST -------
Why would executable bits for /etc/profile.d/*sh scripts be correct?  They do
not have a shebang, so they cannot be executed alone, and /etc/bashrc and
/etc/csh.cshrc check for them using -r (readability, not executability), then
source (not execute) them.

Some 5+ years ago it was necessary to have them executable because of bugs in
the setup package, but that has been fixed since around Red Hat Linux 7.1, see
bug 35714.  644 should be the correct permissions for them these days.

Having them with incorrect permissions in CVS is an orthogonal issue which
shouldn't prevent them from being shipped with correct permissions in the actual
package.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.