This is the mail archive of the gdb@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: git is live


On Wed, Oct 30, 2013 at 08:53:32AM +1030, Alan Modra wrote:
> I tried to push some testsuite changes today, and got
> 
> remote: ld/testsuite/ld-powerpc/tlsexe.r:50: trailing whitespace.
> remote: + +03 +\.tdata \.dynamic \.opd \.got \.plt 
> remote: ld/testsuite/ld-powerpc/tlsexe.r:90: trailing whitespace.
> remote: +.* SECTION +LOCAL +DEFAULT +13 
> ...
> remote: error: hook declined to update refs/heads/master
> 
> I agree that it's good to reject trailing whitespace generally, but
> the testsuite is a little special.  We're matching tool output.  If,
> as in this example, readelf emits trailing whitespace then we
> shouldn't need to write a regex to match it.  Can this be fixed
> easily?

Come to think of it, maybe no trailing whitespace in the testsuite
isn't such a bad thing.  If the testsuite matched all trailing spaces
with " *" then the bar to updating readelf and objdump -d to not emit
trailing spaces is that much lower.

-- 
Alan Modra
Australia Development Lab, IBM


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]