Gabriela Gibson wrote on Thu, Feb 28, 2013 at 21:45:10 +0000:
> On 28/02/13 14:42, C. Michael Pilato wrote:
>> IMO, your log message should explain the reason for and value of such a
>> sweeping change. The diff already tells us what you've done -- developers
>> coming a few years after us will want to know *why* it was done.
>>
> How about:
> [[[
> Update web server configuration file and change all the file
> permissions to remove the executable bit.
>
> "AddOutputFilter INCLUDES .html" is provided by httpd.conf
> and this makes the "XBitHack On" setting unnecessary. Similarly,
> the file permissions of the *.html files that are set to 'executable'
> are not necessary.
>
> See: http://svn.haxx.se/dev/archive-2013-02/0274.shtml for the discussion.
>
Please provide the Message-ID, or a link to mail-archives.apache.org
(haxx.se is operated by a third party, so we should try not to rely on
its indefinite existence).
> * site/publish/.htaccess():
> remove "XBitHack On".
> * site/*
> remove svn:executable property from all files.
>
> Patch by: Gabriela Gibson <gabriela.gibson{_AT_}gmail.com>
You should remove the "Patch by" line, and add an "Approved by" line
when you commit.
> Suggested by: danielsh
> ]]]
>
>
+1 to commit. The web site will update within a few seconds after your
commit, so please check then that it still works (and fix or revert if
not).
Thanks,
Daniel
Received on 2013-03-01 05:14:35 CET