Benjy Weinberger got a fish trophy!
Make rglobs follow symlinked directories by default.
Review Request #1881 - Created March 7, 2015 and submitted
|dturner-tw, jsirois, patricklaw, stuhood, zundel|
It seems unintuitive that it doesn't, and this has in fact led
to problems in Foursquare's codebase.
Feel free to push back if you think this should not be the default.
CI passes: https://travis-ci.org/pantsbuild/pants/builds/53424050
I've stated my objections to this defult elsewhere, but the objections aside, presumably you agree rglobs and zglobs should have a uniform default.
Its confusing to use bare asserts when you're testing in a unittest.TestCase class. Its unclear whether the py.test juicing of assert is in play or not. I think its best to stick to self.assertXXX or else switch a pure py.test style top-level function test everywhere in the file.
Updated to include zglobs.
Revision 2 (+33 -6)