On 6.4.2012 11:39, Petr Cvachoucek wrote:
Hi Mike, all,

I'd like to ask for a code review for these bugfixes for GNU make 3.82.

7159246 gmake target-specific variables fail if over 150 characters and contain semicolon
http://monaco.us.oracle.com/detail.jsf?cr=7159246

- this bug is not (yet) fixed in upstream, bugreport and patch
   are submitted here:
   http://savannah.gnu.org/bugs/index.php?36106


7158488 MAKEFLAGS not being honored always (regression) in 3.82
http://monaco.us.oracle.com/detail.jsf?cr=7158488

- these upstream bugs are related to this (fixed in upstream CVS):
   https://savannah.gnu.org/bugs/?30723
   https://savannah.gnu.org/bugs/?33873
   https://savannah.gnu.org/support/index.php?107487

Fix looks good, all tests passed (no regressions), and patched version now works against the 7159246 test case:

-bash-4.1$ ./make -f ~/test.mk
a; eeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee
-bash-4.1$

(Thats a pass :-)

As a side note, would it be possible to add a test to the test suite to specifically check these cases?
_______________________________________________
userland-discuss mailing list
userland-discuss@opensolaris.org
http://mail.opensolaris.org/mailman/listinfo/userland-discuss

Reply via email to