subversion-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bert Huijben" <b...@qqmail.nl>
Subject RE: svn commit: r904435 - in /subversion/site/publish: ./ dev/ docs/ docs/release-notes/ security/ style/
Date Fri, 29 Jan 2010 11:05:13 GMT


> -----Original Message-----
> From: Bert Huijben [mailto:bert@qqmail.nl]
> Sent: vrijdag 29 januari 2010 11:54
> To: dongsheng@apache.org
> Cc: dev@subversion.apache.org; commits@subversion.apache.org
> Subject: RE: svn commit: r904435 - in /subversion/site/publish: ./ dev/ docs/
> docs/release-notes/ security/ style/
> 
> 
> 
> > -----Original Message-----
> > From: dongsheng@apache.org [mailto:dongsheng@apache.org]
> > Sent: vrijdag 29 januari 2010 10:38
> > To: commits@subversion.apache.org
> > Subject: svn commit: r904435 - in /subversion/site/publish: ./ dev/ docs/
> > docs/release-notes/ security/ style/
> >
> > Author: dongsheng
> > Date: Fri Jan 29 09:37:39 2010
> > New Revision: 904435
> >
> > URL: http://svn.apache.org/viewvc?rev=904435&view=rev
> > Log:
> > Remove 'svn:executable' from html files, add 'svn:mime-type' to css file
> 
> Why did you remove the 'svn:executable' from the html files?
> 
> This breaks the server side includes... and  this commit broke the entire UI on
> our website (http://subversion.apache.org/)
> It removes the header, menu, etc.

I reverted r904435 in r904457.

See http://httpd.apache.org/docs/2.2/mod/mod_include.html#xbithack for the details on why
the svn:executable was/is necessary.

I can see why you determined that this commit was a trivial patch outside your partial commit
area (which doesn't require approval), but I would recommend that you consult the list first
before repairing things if you are not 100% sure :)

+1 to apply the other changes (content types) again.

Thanks,

	Bert


Mime
View raw message