Hi Arne,

I don't quite agree with this. I think the commit is find and modifies a system configuration file that is not modified in any manner by the web interface or any other script.

Users that put their own modifications into those files will sooner or later crash their systems any way. That is why we usually include files where they can put their own modifications. I find it always incredibly unnecessary to go through this procedure for crontab just because some users cannot create their own one. The process is faulty and takes a lot of time to do.

I don't insist on keeping this commit in the repository, so if you insist on inserting it on the fly I will do that instead.

-Michael

On Thu, 2014-07-10 at 15:49 +0200, Arne Fitzenreiter wrote:

Hi Micha, hi Dirk,

I think it is not a good idea to complete overwrite the logrotate.conf at update because there are some user configurable settings inside. It is better to use sed for adding the include line...

http://git.ipfire.org/?p=ipfire-2.x.git;a=commit;h=23b810171806939a23c3d5df7...

Arne

---

Development mailing list Development@lists.ipfire.org http://lists.ipfire.org/mailman/listinfo/development