Re: [libsanitizer] a script to help merging asan from upstream

2012-11-22 Thread Konstantin Serebryany
Done, thanks! r193743. On Fri, Nov 23, 2012 at 2:50 AM, Xinliang David Li wrote: > Looks fine to me. > > David > > On Thu, Nov 22, 2012 at 11:06 AM, Konstantin Serebryany > wrote: >> Anything else? >> >> On Thu, Nov 22, 2012 at 6:42 AM, Konstantin Serebryany >> wrote: >>> Added to README.gcc: >

Re: [libsanitizer] a script to help merging asan from upstream

2012-11-22 Thread Xinliang David Li
Looks fine to me. David On Thu, Nov 22, 2012 at 11:06 AM, Konstantin Serebryany wrote: > Anything else? > > On Thu, Nov 22, 2012 at 6:42 AM, Konstantin Serebryany > wrote: >> Added to README.gcc: >> The merges from upstream should be done with the aid of the merge.sh script; >> it will also upd

Re: [libsanitizer] a script to help merging asan from upstream

2012-11-22 Thread Konstantin Serebryany
Anything else? On Thu, Nov 22, 2012 at 6:42 AM, Konstantin Serebryany wrote: > Added to README.gcc: > The merges from upstream should be done with the aid of the merge.sh script; > it will also update the file MERGE to contain the upstream revision > we merged with. > > > On Wed, Nov 21, 2012 at

Re: [libsanitizer] a script to help merging asan from upstream

2012-11-21 Thread Konstantin Serebryany
Added to README.gcc: The merges from upstream should be done with the aid of the merge.sh script; it will also update the file MERGE to contain the upstream revision we merged with. On Wed, Nov 21, 2012 at 11:03 PM, Xinliang David Li wrote: > How about also documenting this in README.gcc? > > Da

Re: [libsanitizer] a script to help merging asan from upstream

2012-11-21 Thread Xinliang David Li
How about also documenting this in README.gcc? David On Wed, Nov 21, 2012 at 10:56 AM, Kostya Serebryany wrote: > Done both. > > > +fatal() { > + echo "$1" > + exit 1; > +} > + > +pwd | grep 'libsanitizer$' || \ > + fatal "Run this script from libsanitizer dir" > > > +rm -rf upstream > > > >

Re: [libsanitizer] a script to help merging asan from upstream

2012-11-21 Thread Kostya Serebryany
Done both. +fatal() { + echo "$1" + exit 1; +} + +pwd | grep 'libsanitizer$' || \ + fatal "Run this script from libsanitizer dir" +rm -rf upstream On Wed, Nov 21, 2012 at 10:49 PM, Xinliang David Li wrote: > Suggestions: > > 1) make sure current local dir is libsanitizer -- exit if it

Re: [libsanitizer] a script to help merging asan from upstream

2012-11-21 Thread Xinliang David Li
Suggestions: 1) make sure current local dir is libsanitizer -- exit if it is not 2) clean up the upstream directory after the merge is done. David On Wed, Nov 21, 2012 at 10:25 AM, Kostya Serebryany wrote: > Hi, > > A dummy script to help merging asan from upstream. > Not 100% complete, but sh

[libsanitizer] a script to help merging asan from upstream

2012-11-21 Thread Kostya Serebryany
Hi, A dummy script to help merging asan from upstream. Not 100% complete, but should be enough to complete the current merge. You suggestions on how to improve it (or how to do w/o it) are welcome, but I really wish to do the first merge tomorrow to unblock other folks. Thanks, --kcc merge.pat