Re: [PATCH] htaccess file and permissions

2013-03-01 Thread Ben Reser
On Thu, Feb 28, 2013 at 8:13 PM, Daniel Shahaf d...@daniel.shahaf.name wrote: 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). One thing to add here is to make sure you click

[PATCH] htaccess file and permissions

2013-02-28 Thread Gabriela Gibson
[[[ Update web server configuration file and change file permissions to remove the executable bit. * site/publish/.htaccess(): remove XBitHack On. * site/* remove svn:executable property from all files, with the exception of 'download/download.cgi'. Patch by: Gabriela Gibson

Re: [PATCH] htaccess file and permissions

2013-02-28 Thread C. Michael Pilato
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. On 02/28/2013 06:31 AM, Gabriela Gibson wrote: [[[ Update web server configuration

Re: [PATCH] htaccess file and permissions

2013-02-28 Thread Gabriela Gibson
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

Re: [PATCH] htaccess file and permissions

2013-02-28 Thread C. Michael Pilato
Perfect! It ensures that a dev such as myself who isn't closely following the related discussion threads has a hope of understanding why the change was made. On 02/28/2013 04:45 PM, Gabriela Gibson wrote: On 28/02/13 14:42, C. Michael Pilato wrote: IMO, your log message should explain the

Re: [PATCH] htaccess file and permissions

2013-02-28 Thread C. Michael Pilato
Perfect! It ensures that a dev such as myself who isn't closely following the related discussion threads has a hope of understanding why the change was made. On 02/28/2013 04:45 PM, Gabriela Gibson wrote: On 28/02/13 14:42, C. Michael Pilato wrote: IMO, your log message should explain the

Re: [PATCH] htaccess file and permissions

2013-02-28 Thread Daniel Shahaf
Gabriela Gibson wrote on Thu, Feb 28, 2013 at 21:45:10 +: 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

Re: [PATCH] htaccess file and permissions

2013-02-28 Thread Daniel Shahaf
Daniel Shahaf wrote on Fri, Mar 01, 2013 at 06:13:54 +0200: +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). BTW, you can check whether the web site has updated by retrieving