[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] gnu: Add mcelog.
From: |
Marius Bakke |
Subject: |
Re: [PATCH] gnu: Add mcelog. |
Date: |
Wed, 14 Sep 2016 16:32:10 +0100 |
Danny Milosavljevic <address@hidden> writes:
> DESTDIR is a feature which lets you install into a tempdir. The idea
> is that you install it all there and then atomically replace the
> ORIGINAL version at the non-DESTDIR location.
Oops, I meant DOCDIR! Sorry for the confusion!
>> >> + (home-page "http://mcelog.org/")
>> > Nit-pick: the trailing slash is unnecessary :)
>
> If you don't include the trailing slash it will require an additional
> round-trip to the server in order to add it via a (302) redirect.
That's a very good point, although all HTTP clients I've used add it
automatically. But, explicit is better than implicit and all that.
Now I'll have to start doing it too :)
Thanks!
Marius
Re: [PATCH] gnu: Add mcelog., Ludovic Courtès, 2016/09/14