Paresh Mutha wrote: % Hello, % % Commit 7a18b250b07ff4ed0c34fa48e69029c114ec3ab1 introduced the check % on ownership & permission of the config file so that it doesn't % log/display the diff it the config file is not readable by all. % % Here it is checking the stat for source. But source is always the % temp file which is created based on the config file content received % from the Server and it is always under root ownership and not % readable by all. Thus for all config files which are not owned by % root or are readable by all, the diff result is not displayed. % Instead there is the message "Differences exist in a file that is % not readable by all. Re-deployment of configuration file is % recommended" % % Attached is the proposed patch to check the stat of dst i.e the file % present on the system. With this change, for the files which are % readable by all the actual diff result would be displayed.
You're correct :). Reviewed and pushed patch. Thanks. % Regards, % Paresh -- Michael Mráka Satellite Engineering, Red Hat _______________________________________________ Spacewalk-devel mailing list Spacewalk-devel@redhat.com https://www.redhat.com/mailman/listinfo/spacewalk-devel