PATCH: ignore .rpm* debris in directories included in config files
list Japheth Cleaver
This patch alters hobbit-wide "directory" include reading logic to skip files that have been replaced or left over by an RPM install. Specifically, we're splitting hobbitlaunch.cfg into a directory of files, but we don't want to read in things tweaked by a package upgrade; and there's no way to exclude those in the syntax... Regards, Japheth Cleaver
Attachments (1)
list Buchan Milne
▸
On Tuesday 24 March 2009 23:51:35 Cleaver, Japheth wrote:
This patch alters hobbit-wide "directory" include reading logic to skip files that have been replaced or left over by an RPM install. Specifically, we're splitting hobbitlaunch.cfg into a directory of files, but we don't want to read in things tweaked by a package upgrade; and there's no way to exclude those in the syntax...
My original proposal for this feature was to support an include syntax, (e.g. include /etc/xymon/hobbitlaunch.d/*.cfg), but this was not accepted and instead implemented with the include syntax. However, a hard-coded rpm-specific solution is probably not going to be any more attractive to merge ... Henrik, you really should try and solve this problem if you want to see Xymon adopted by more distros ....
list Japheth Cleaver
-----Original Message----- From: Buchan Milne [mailto:user-9b139aff4dec@xymon.invalid] Sent: Thursday, March 26, 2009 1:52 AM To: user-ae9b8668bcde@xymon.invalid Cc: Cleaver, Japheth Subject: Re: [hobbit] PATCH: ignore .rpm* debris in directories included in config files
▸
On Tuesday 24 March 2009 23:51:35 Cleaver, Japheth wrote:This patch alters hobbit-wide "directory" include reading logic to skip files that have been replaced or left over by an RPM install. Specifically, we're splitting hobbitlaunch.cfg into a directory of files, but we don't want to read in things tweaked by a package upgrade; and there's no way to exclude those in the syntax...My original proposal for this feature was to support an include syntax, (e.g. include /etc/xymon/hobbitlaunch.d/*.cfg), but this was not accepted > and instead implemented with the include syntax. However, a hard-coded rpm-specific solution is probably not going to be any more attractive to merge ...
True, a specific include (a la apache's default "include conf.d/*.conf") would be nice. But there are already exceptions in the code for things like RCS and .dotfiles. The jump from properly ignoring RCS ",v" stuff to coping with package-managed versioning as well isn't too far, I don't think. -jc