Re: [PATCH] Preserve timestamps during 'make install'

2016-09-12 Thread Wietse Venema
Ondrej Lysonek: > And what about 'touch -r ref'? I checked man pages of the systems > listed on the Postfix home page, and the option '-r' is everywhere. > I'm attaching a new patch. Adopted, but I added the requirement that one says "postfix-install -keep-new-mtime ..." because this is a visible

Re: [PATCH] Preserve timestamps during 'make install'

2016-09-12 Thread Ondřej Lysoněk
Can you confirm that this patch is going to get merged in? Thanks. Ondra On 09/04/2016 04:47 PM, Viktor Dukhovni wrote: On Sun, Sep 04, 2016 at 04:25:36AM -0400, Ondrej Lysonek wrote: And what about 'touch -r ref'? I checked man pages of the systems listed on the Postfix home page, and the o

Re: [PATCH] Preserve timestamps during 'make install'

2016-09-04 Thread Viktor Dukhovni
On Sun, Sep 04, 2016 at 04:25:36AM -0400, Ondrej Lysonek wrote: > And what about 'touch -r ref'? I checked man pages of the systems listed > on the Postfix home page, and the option '-r' is everywhere. I'm attaching > a new patch. Yes, that should be quite portable, as it is in POSIX, both the 19

Re: [PATCH] Preserve timestamps during 'make install'

2016-09-04 Thread Ondrej Lysonek
unday, August 28, 2016 3:19:20 AM Subject: Re: [PATCH] Preserve timestamps during 'make install' Michael Orlitzky: > On 08/27/2016 07:42 PM, Wietse Venema wrote: > >> > >> Thanks. The "cp -p" feature was not portable in the days that this script > >>

Re: [PATCH] Preserve timestamps during 'make install'

2016-09-04 Thread Ondrej Lysonek
3:37:40 AM Subject: Re: [PATCH] Preserve timestamps during 'make install' > On Aug 27, 2016, at 8:39 PM, Michael Orlitzky wrote: > > On 08/27/2016 07:42 PM, Wietse Venema wrote: >>> >>> Thanks. The "cp -p" feature was not portable in the days that this sc

Re: [PATCH] Preserve timestamps during 'make install'

2016-08-27 Thread Viktor Dukhovni
> On Aug 27, 2016, at 8:39 PM, Michael Orlitzky wrote: > > On 08/27/2016 07:42 PM, Wietse Venema wrote: >>> >>> Thanks. The "cp -p" feature was not portable in the days that this script >>> was written, but it should be safe to use now. >> >> Unfortunately, I have to roll back this change, bec

Re: [PATCH] Preserve timestamps during 'make install'

2016-08-27 Thread Wietse Venema
Michael Orlitzky: > On 08/27/2016 07:42 PM, Wietse Venema wrote: > >> > >> Thanks. The "cp -p" feature was not portable in the days that this script > >> was written, but it should be safe to use now. > > > > Unfortunately, I have to roll back this change, because it may > > install files with non

Re: [PATCH] Preserve timestamps during 'make install'

2016-08-27 Thread Michael Orlitzky
On 08/27/2016 07:42 PM, Wietse Venema wrote: >> >> Thanks. The "cp -p" feature was not portable in the days that this script >> was written, but it should be safe to use now. > > Unfortunately, I have to roll back this change, because it may > install files with non-root ownership. > > Those who

Re: [PATCH] Preserve timestamps during 'make install'

2016-08-27 Thread Wietse Venema
Wietse Venema: > Ond?ej Lyson?k: > > Hi, > > > > I'm sending a patch that eliminates certain (unwanted) warnings caused > > by unnecessary update of modification time during 'make install'. See > > the downstream bug report at [1] for more details. The attached patch > > was created by Robert S

Re: [PATCH] Preserve timestamps during 'make install'

2016-08-09 Thread Wietse Venema
Ond?ej Lyson?k: > Hi, > > I'm sending a patch that eliminates certain (unwanted) warnings caused > by unnecessary update of modification time during 'make install'. See > the downstream bug report at [1] for more details. The attached patch > was created by Robert Scheck. Thanks. The "cp -p" f

[PATCH] Preserve timestamps during 'make install'

2016-08-08 Thread Ondřej Lysoněk
Hi, I'm sending a patch that eliminates certain (unwanted) warnings caused by unnecessary update of modification time during 'make install'. See the downstream bug report at [1] for more details. The attached patch was created by Robert Scheck. [1] https://bugzilla.redhat.com/show_bug.cgi?id