Re: [patch] ftp(1): change mtime for http/https links

2017-10-05 Thread Jesper Wallin
...and another bump. :-) *ping sthen@* On Sun, Sep 24, 2017 at 07:25:33PM +0200, Jesper Wallin wrote: > On Sat, Sep 23, 2017 at 11:50:46PM +0200, Jesper Wallin wrote: > > Woups, seems like I managed to break ftp(1) in the installer due to > > pledge being a bit too tight. Here is an updated

Re: [patch] ftp(1): change mtime for http/https links

2017-09-24 Thread Jesper Wallin
On Sat, Sep 23, 2017 at 11:50:46PM +0200, Jesper Wallin wrote: > Woups, seems like I managed to break ftp(1) in the installer due to > pledge being a bit too tight. Here is an updated version of the patch > and with Philips changes as well. ...and hopefully a final version, sorry for the noise.

Re: [patch] ftp(1): change mtime for http/https links

2017-09-23 Thread Jesper Wallin
> On Sat, Sep 23, 2017 at 09:43:12AM +0200, Jesper Wallin wrote: > Yeah, fully understandable. I'll bump this once 6.2 is out. > > I'll apply the changes Philip suggested (thanks!) and then build a > release to verify that everything works. > > Woups, seems like I managed to break ftp(1) in the

Re: [patch] ftp(1): change mtime for http/https links

2017-09-23 Thread Raf Czlonka
On Sat, Sep 23, 2017 at 10:12:32AM BST, Ted Unangst wrote: > Raf Czlonka wrote: > > This is unrelated to your diff but what I do instead is to check > > the BUILDINFO file - it's tiny and all the information you need, > > is already there. > > While we're on the subject, I'll point out that

Re: [patch] ftp(1): change mtime for http/https links

2017-09-23 Thread Jesper Wallin
Yeah, the SHA256.sig should be enough and just veify that everything is correct. If not, fetch the incorrect sets again. I just wanted to give a brief explaination on how I noticed this inconsistency. But yeah, thanks for pointing me to a better solution. On Sat, Sep 23, 2017 at 05:12:32AM

Re: [patch] ftp(1): change mtime for http/https links

2017-09-23 Thread Ted Unangst
Raf Czlonka wrote: > This is unrelated to your diff but what I do instead is to check > the BUILDINFO file - it's tiny and all the information you need, > is already there. While we're on the subject, I'll point out that sometimes only the base sets change, but not the x sets. If you download

Re: [patch] ftp(1): change mtime for http/https links

2017-09-23 Thread Jesper Wallin
Yeah, fully understandable. I'll bump this once 6.2 is out. I'll apply the changes Philip suggested (thanks!) and then build a release to verify that everything works. On Fri, Sep 22, 2017 at 11:35:03PM +0100, Stuart Henderson wrote: > I like this, but it's rather late in the release cycle to

Re: [patch] ftp(1): change mtime for http/https links

2017-09-23 Thread Anton Lindqvist
On Fri, Sep 22, 2017 at 11:56:51PM +0100, Raf Czlonka wrote: > On Fri, Sep 22, 2017 at 11:01:57PM BST, Jesper Wallin wrote: > > Hi all, > > > > My morning routine consists of downloading the latest snapshot files and > > running the upgrade. To avoid wasting bandwidth and time, I check the > >

Re: [patch] ftp(1): change mtime for http/https links

2017-09-22 Thread Philip Guenther
On Sat, 23 Sep 2017, Jesper Wallin wrote: ... > The patch below will use the Last-Modified header in order to set the > modification time for http or https links. I also added, what to me > looked like a missing "\n" on the error message in ftp.c. ... > --- fetch.c 7 Mar 2017 08:00:23 -

Re: [patch] ftp(1): change mtime for http/https links

2017-09-22 Thread Raf Czlonka
On Fri, Sep 22, 2017 at 11:01:57PM BST, Jesper Wallin wrote: > Hi all, > > My morning routine consists of downloading the latest snapshot files and > running the upgrade. To avoid wasting bandwidth and time, I check the > local modification time of INSTALL.amd64, fetch the remote one and check >

Re: [patch] ftp(1): change mtime for http/https links

2017-09-22 Thread Stuart Henderson
I like this, but it's rather late in the release cycle to be adding features to ftp(1), could you send a reminder when we're at 6.2-current? (Also, if you haven't already done so, check that "make release" and the installer still work). On 2017/09/22 22:01, Jesper Wallin wrote: > Hi all, > >

[patch] ftp(1): change mtime for http/https links

2017-09-22 Thread Jesper Wallin
Hi all, My morning routine consists of downloading the latest snapshot files and running the upgrade. To avoid wasting bandwidth and time, I check the local modification time of INSTALL.amd64, fetch the remote one and check if the modification time has changed. A few days ago, the mirror I use