Bug 432154 - coreutils install overwrites modified /etc/profile.d/colorls.sh
Summary: coreutils install overwrites modified /etc/profile.d/colorls.sh
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: coreutils
Version: 8
Hardware: All
OS: Linux
low
high
Target Milestone: ---
Assignee: Ondrej Vasik
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-02-09 05:17 UTC by JW
Modified: 2008-02-28 21:46 UTC (History)
2 users (show)

Fixed In Version: 6.9-13.fc8
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-02-28 21:46:21 UTC


Attachments (Terms of Use)

Description JW 2008-02-09 05:17:07 UTC
Description of problem:
coreutils install overwrites modified /etc/profile.d/colorls.sh

Version-Release number of selected component (if applicable):
coreutils-6.9-9

How reproducible:
Always

Steps to Reproduce:
1. modify /etc/profile.d/colorls.sh
2. upgrade coreutils

  
Actual results:
Old /etc/profile.d/colorls.sh disappears.
No rpmnew/old etc created

Expected results:
coreutils install should not replace a modified version of
/etc/profile.d/colorls.sh.  It should create colorls.sh.rpmnew or at least save
the old file.

Additional info:
rpm should be smarter

Comment 1 Jon Stanley 2008-02-14 03:46:15 UTC
SSSomewhat surprised that this made it through the merge review, but that was
completed awhile back.  These files should be marked %config IMHO.

Comment 2 Jon Stanley 2008-02-14 04:18:33 UTC
Hmmm...on second thought, looking at the script there is no configuration in
there.  Could go either way here - it sources it's config from /etc/DIR_COLORS
and friends, which *are* marked %config.  Changing colorls.sh is no different
than changing /bin/ls or the like.  What's the use case for changing this?  I
can't think of any., and I *can* think of valid use cases for the coreutils RPM
changing it - say that ls upstream changed and required new options.

Comment 3 JW 2008-02-14 04:33:22 UTC
Maybe people don't want carefully (globally) crafted ll/ls aliases overriden by
colorls.sh?  Maybe 'ls' aliases need to vary according to whether one is root or
not (eg -a option)?  Maybe /etc/profile.d/colorls should not exists at all.
Maybe the contents should be in user's ~/.bashrc or whatever.

What is the problem with preserving configuration files and scripts?  It costs
nothing, and provides outstanding benefits to everyone.



Comment 4 Ondrej Vasik 2008-02-14 09:06:20 UTC
Yes, I'm going to mark them %config - I know about this issue of overwritten
aliases. Problem is that I'm now not able to build rawhide package (because of
some recent change in rawhide one of tests fails on x86_64 architecture) and I
would like to have that in Rawhide before doing update for F8.

Comment 5 Ondrej Vasik 2008-02-14 15:39:21 UTC
Fixed in RAWHIDE coreutils-6.10-7.fc9 , will make F8 update next week.

Comment 6 Fedora Update System 2008-02-21 02:52:30 UTC
coreutils-6.9-13.fc8 has been pushed to the Fedora 8 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update coreutils'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F8/FEDORA-2008-1844

Comment 7 Fedora Update System 2008-02-28 21:46:16 UTC
coreutils-6.9-13.fc8 has been pushed to the Fedora 8 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.