Re: [PATCH 2/2] Fix t3404 assumption that `wc -l` does not use whitespace.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On Apr 28, 2008, at 5:41 AM, Jeff King <peff@xxxxxxxx> wrote:
BTW, how did you discover this bug? Through normal use, or was there a
failing test?

My small series of one-line patches were from trying to get t3404 (IIRC) to work. So our tests do help. ;-)

If a failing test, then I wonder if we could get a few people to set up
automated tests on alternate platforms. IIRC, Junio makes sure that
master always passes test on his Linux box and KO (Debian and Redhat, I
think?). Other platforms could "git pull && make test" daily. I could
probably do Solaris (once I get the tests to complete pass at all!) and
FreeBSD 6.

I have a script that boils down to 'make && make test && make install'. I pull, check changes, and then run it once or twice a week. I can't dedicate my laptop to automated testing, but I do raise a fuss when it fails.

A build farm would help, especially with simple portability errors like this. Would be good to get OS X, Solaris (our old nemesis), and a few variations for /bin/sh. But I don't really have a good machine to volunteer for the effort myself.

~~ Brian G.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux