Re: Marking files in /etc/cron.* as %config

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 2008-03-17 at 10:37 -0400, Jesse Keating wrote:
> On Mon, 2008-03-17 at 10:29 -0400, David Mansfield wrote:
> > 
> > Specifically to awstats: I have had to modify the cron.d script in order
> > to add environment variables necessary for some further customizations
> > we had made to the config for a few sites.
> > 
> > In general, I think the 'executableness' of these is irrelevant: they
> > are config files and should be marked so in rpm.  All files that could
> > conceivable be edited in /etc should be marked as config IMHO, and all
> > OTHER files should be removed from etc.  cf some previous discussion on
> > the hal/freedesktop files: they were moved to /usr because people kept
> > editing them and getting screwed.
> 
> FWIW I feel that awstats cron job should just be executing a script
> somewhere else, and that somewhere else would read config files from
> standard locations that are then marked as %config.  The config and the
> execution should not be in the same file.  Makes it very difficult to
> change the file over time without upsetting local configuration.
> 

I agree completely.  The standard sysconfig type thing would be great
here, but until it's done, the scripts DO need to be edited.

As to my other comment, I believe all /etc files should, as a rule, be %
config, and an effort should be made to take anything where there is a
reasonable 'do not edit this file' argument to made should be moved OUT
of /etc.

My 2.

David


-- 
fedora-devel-list mailing list
fedora-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/fedora-devel-list

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]
  Powered by Linux