Re: Virtual patch: `svn mv lib/Headers/Intrin.h lib/Headers/intrin.h`

2016-06-14 Thread Nico Weber via cfe-commits
r272701, thanks! On Tue, Jun 14, 2016 at 9:34 PM, Hans Wennborg wrote: > SGTM, then :-) > > On Tue, Jun 14, 2016 at 11:54 AM, Nico Weber wrote: > > I locally set up an SVN repo with a file called "File.txt" and then > renamed > > it with `svn mv` using a

Re: Virtual patch: `svn mv lib/Headers/Intrin.h lib/Headers/intrin.h`

2016-06-14 Thread Hans Wennborg via cfe-commits
SGTM, then :-) On Tue, Jun 14, 2016 at 11:54 AM, Nico Weber wrote: > I locally set up an SVN repo with a file called "File.txt" and then renamed > it with `svn mv` using a new svn client (1.8). > > Then I tried updating several local checkouts of my local repo with svn >

Re: Virtual patch: `svn mv lib/Headers/Intrin.h lib/Headers/intrin.h`

2016-06-14 Thread Nico Weber via cfe-commits
I locally set up an SVN repo with a file called "File.txt" and then renamed it with `svn mv` using a new svn client (1.8). Then I tried updating several local checkouts of my local repo with svn 1.6.6 clients. All methods of updating I tried (`svn up`, `svn co current_checkout_url@2`) worked

Re: Virtual patch: `svn mv lib/Headers/Intrin.h lib/Headers/intrin.h`

2016-06-08 Thread Nico Weber via cfe-commits
On Wed, Jun 8, 2016 at 1:27 PM, Hans Wennborg wrote: > On Wed, Jun 8, 2016 at 10:20 AM, Nico Weber wrote: > > It looks like we're starting to be more careful about header case. MSVC's > > intrin.h is called intrin.h with a lower-case 'i'; ours starts with

Re: Virtual patch: `svn mv lib/Headers/Intrin.h lib/Headers/intrin.h`

2016-06-08 Thread Hans Wennborg via cfe-commits
On Wed, Jun 8, 2016 at 10:20 AM, Nico Weber wrote: > It looks like we're starting to be more careful about header case. MSVC's > intrin.h is called intrin.h with a lower-case 'i'; ours starts with an > upper-case 'I' for no good reason. > > It looks like file-only renames

Virtual patch: `svn mv lib/Headers/Intrin.h lib/Headers/intrin.h`

2016-06-08 Thread Nico Weber via cfe-commits
It looks like we're starting to be more careful about header case. MSVC's intrin.h is called intrin.h with a lower-case 'i'; ours starts with an upper-case 'I' for no good reason. It looks like file-only renames work fine as of svn 1.7 without any workarounds (