On Nov 20, 2014, at 4:55 AM, KO Myung-Hun <[email protected]> wrote: > > Hi/2.
Hi, Thanks for the do-over in light of my edits on branch-1.4 :) > These are OS/2 patches for master branch. Patches for 1.4 branch are > reflected. > > [PATCH 1/5] configury: support DLLs on OS/2 Agreed. But these flags apply to more than just OS/2, so I turned on -avoid-version and -no-undefined unconditionally. > [PATCH 2/5] m4: support to load modules on OS/2 In principle, these seem like good changes, but the implementation is not as clean as it could be, considering many of these changes certainly may apply on more than just OS/2. I've rewritten equivalents to everything except module name truncation to 8 characters maximum, which I'll figure out how to do nicely tomorrow. > [PATCH 3/5] configury: append $EXEEXT suffix to the executable, > [PATCH 4/5] configury: add -Zargs-resp to LDFLAGS on OS/2. > [PATCH 5/5] esyscmd: fdopen() with a text mode explicitly on OS/2. Ack. Effectively applied and pushed everything here, or equivalents - save module name truncation. Cheers, -- Gary V. Vaughan (gary AT gnu DOT org) _______________________________________________ M4-patches mailing list [email protected] https://lists.gnu.org/mailman/listinfo/m4-patches
