Bug 77631

Summary: XFree86 upgrade leaves /etc/X11/XF86Config-4.rpmsave turd without warning about it
Product: [Fedora] Fedora Reporter: Jonathan Kamens <jik>
Component: XFree86Assignee: Mike A. Harris <mharris>
Status: CLOSED RAWHIDE QA Contact: David Lawrence <dkl>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhide   
Target Milestone: ---   
Target Release: ---   
Hardware: i386   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2004-09-21 08:04:10 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Jonathan Kamens 2002-11-11 07:24:22 UTC
Note this code excerpt from the XFree86 preinstall scriptlet:

    CONFFILE=/etc/X11/XF86Config-4
    if [ -e $CONFFILE ] ; then
      SAVECONF=$CONFFILE.rpmsave
      cp -p $CONFFILE $SAVECONF
      grep -iv 'Load[[:space:]]*"\(pex5\|xie\)"' $SAVECONF > $CONFFILE
    fi

There are two problems here:

1) If XF86Config-4 hasn't been changed by the code above, then
XF86Config-4.rpmsave should be removed.

2) IF XF86Config-4 *has* been changed by the code above, then the script should
print a warning about it, just like rpm does when it creates a rpmsave file.

Comment 1 Mike A. Harris 2002-11-11 18:25:11 UTC
Yeah, the XFree86 scripts are quite an ugly unmaintainable mess currently.
I've got plans to completely overhaul them sometime, but it is something
that must be done very very carefully.  I want to put an intense amount
of thought into re-engineering the scripts to be cleaner and do much more
to ensure better installations and upgrades, etc.  But I want to minimize
the system damage potential even to crazy rawhide testers because even
in rawhide, I can get nailed with tonnes of bug reports.

Since the scripts more or less "work" even though they're not very pretty
per se, it has gotten a low priority each release, to do any major changes.

The changes you're asking for seem relatively sane at first glance and
most likely I'll do something to clean it up.

Thanks.

Comment 2 Mike A. Harris 2004-01-10 16:36:06 UTC
Status update to indicate this is still present in current
development.  Changed OS version.

Comment 3 Mike A. Harris 2004-09-21 08:04:10 UTC
The rpm script processing of the config file has been changed
in Fedora Core 3 development, and should not have this problem
anymore.  Fedora Core 2 erratum release that will be released
soon should also fix this issue.

Setting status to "RAWHIDE".  If the problem persists at all
with Fedora Core 3 development rpms from test2 onward, please
reopen and we'll reinvestigate the issue.

Thanks.