[GitHub] ant issue #24: fix 60150 values containing backtick or $ character cause she...

2016-10-10 Thread jwadamson
Github user jwadamson commented on the issue: https://github.com/apache/ant/pull/24 I think dash may not support that substitution pattern :-( Still investigating --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your p

[GitHub] ant issue #24: fix 60150 values containing backtick or $ character cause she...

2016-10-10 Thread jwadamson
Github user jwadamson commented on the issue: https://github.com/apache/ant/pull/24 Will have another attempt at this later today. So far tested on ubunutu ksh, bash, dash as the bin/sh. Will try to get a true solarus and mac platforms to test since this is what led to back to thi

Any Solaris / HP-UX Users out here?

2016-10-10 Thread Stefan Bodewig
Hi all we've received several reports about the ant wrapper script of Ant 1.9.7 not working on the said systems. Most likely because some changes made to it use sed and the printf built-in in non-portable ways. Jeff Adamson is currently evaluating options inside of https://github.com/apache/ant/p

[GitHub] ant issue #24: fix 60150 values containing backtick or $ character cause she...

2016-10-10 Thread bodewig
Github user bodewig commented on the issue: https://github.com/apache/ant/pull/24 Thanks Jeff. I've asked for help on the dev and user mailing lists. It's not only Solaris but also HP/UX which may be even more difficult to get hold of. --- If your project is set up for it, you can re

[GitHub] ant issue #24: fix 60150 values containing backtick or $ character cause she...

2016-10-10 Thread jwadamson
Github user jwadamson commented on the issue: https://github.com/apache/ant/pull/24 Thanks, will try to find the threads on mailing list. Still making improvments, fixed an issue with mac sed that it doesn't support group-0. Annoying. Also finally figured out why so many backslashes,

[GitHub] ant issue #24: fix 60150 values containing backtick or $ character cause she...

2016-10-10 Thread jwadamson
Github user jwadamson commented on the issue: https://github.com/apache/ant/pull/24 So I think I figured out some of this in terms of why escaping was so funny. Also found an issue with sed on mac (no group-0 reference) in my earlier fix. The bash variable replacement s

[GitHub] ant issue #24: fix 60150 values containing backtick or $ character cause she...

2016-10-10 Thread jwadamson
Github user jwadamson commented on the issue: https://github.com/apache/ant/pull/24 Oh, and the final two lines of ant script needed quotes to preserve intra-argument whitespace properly --Jeff On Mon, Oct 10, 2016 at 3:36 PM, Jeff Adamson wrote: > S