Re: ant wrapper script testing

2016-11-03 Thread Jeff Adamson
s out what is wrong with the awk patterns listed. Thanks for all your help. Regards, Jeff Adamson From: Johan Corveleyn To: Ant Developers List Date: 11/03/2016 06:37 AM Subject: Re: ant wrapper script testing On Mon, Oct 31, 2016 at 9:20 PM, Jeff Adamson wrote: > > W

Re: ant wrapper script testing

2016-11-03 Thread Johan Corveleyn
On Mon, Oct 31, 2016 at 9:20 PM, Jeff Adamson wrote: > > We will give this more thought. Ant only needs one replacement engine to work > on all platforms, but I don't know of a specific reason that nawk should not > be just as capable as sed. > > To summarize, > > Using 4-backslash replacement n

Re: ant wrapper script testing

2016-11-02 Thread Stefan Bodewig
On 2016-10-31, Jeff Adamson wrote: > We will give this more thought. Ant only needs one replacement engine to > work on all platforms, but I don't know of a specific reason that nawk > should not be just as capable as sed. If we can get a non-awk solution cross plaform then it may be a waste of y

Re: ant wrapper script testing

2016-10-31 Thread Jeff Adamson
t;\\$", "\\$"); print }' )" esc_arg="$(printf '%s' "$esc_arg" | "$awk_exec" '{ gsub("\"", "\\\""); print }' )" esc_arg="$(printf '%s' "$esc_arg" | "$awk

Re: ant wrapper script testing

2016-10-28 Thread Johan Corveleyn
On Fri, Oct 28, 2016 at 10:41 PM, Johan Corveleyn wrote: ... > Yes, that works perfectly! On Solaris 11, with your latest version of > the ant wrapper. When I run your test.sh, with the test.xml, I get > exactly the correct output. > > However, with esc_tool=awk it fails. Don't know why yet, but t

Re: ant wrapper script testing

2016-10-28 Thread Johan Corveleyn
On Wed, Oct 26, 2016 at 4:16 PM, Jeff Adamson wrote: > > I am feeling pretty good as the introduction of the newline to the printf > does not fundamentally change much about the updated processing. My main > concern was if that additional newline was going to make it into the final > arguments as

Re: ant wrapper script testing

2016-10-26 Thread Jeff Adamson
o] _ [echo] nl= [echo] _ [echo] nlnl= [echo] [echo] _ [echo] nlnlnl= [echo] [echo] [echo] _ [echo] doublespace= _ [echo] end Regards, Jeff Adamson Johan Corveleyn wrote on 10/26/2016 06:53:18 AM: > From: Johan Corveleyn > To: Ant Develop

Re: ant wrapper script testing

2016-10-26 Thread Stefan Bodewig
On 2016-10-26, Johan Corveleyn wrote: > Now, I only tested 'ant -version'. I've actually not been following > the original discussion that lead to all the changes in the wrapper > script (I only noticed that 1.9.7 was broken on Solaris for simple > inputs). Are there some specific inputs you would

Re: ant wrapper script testing

2016-10-26 Thread Stefan Bodewig
On 2016-10-25, Jeff Adamson wrote: > should have been > https://github.com/jwadamson/ant-1/commit/c1ac08abcaad0030064d62933dda278b5326f92b which is now the git master version. Many thanks to both of you so far, Jeff and Johan Stefan -

Re: ant wrapper script testing

2016-10-26 Thread Johan Corveleyn
On Tue, Oct 25, 2016 at 9:29 PM, Jeff Adamson wrote: > > Oops, had the nawk check inverted. > > should have been > https://github.com/jwadamson/ant-1/commit/c1ac08abcaad0030064d62933dda278b5326f92b > Great, it works on Solaris 11, both with 'sed' and 'awk' as esc_tool. Nice job! Now, I only tes

Re: ant wrapper script testing

2016-10-25 Thread Jeff Adamson
6 02:40 PM Subject: Re: ant wrapper script testing Probably helps if I include a link to the updated change https://github.com/jwadamson/ant-1/commit/f3a0faecc464e899ec77cd7f2052b81a7c52be35 Please let me know if this resolves the problems with every crazy input you can think of. Reg

Re: ant wrapper script testing

2016-10-25 Thread Jeff Adamson
@IBMUS To: "Ant Developers List" Cc: maarten_co...@yahoo.com Date: 10/25/2016 01:56 PM Subject: Re: ant wrapper script testing Also glad to know that awk is basically a no-go due to legacy versions. Suppose we could use `which nawk` to use that if present? So the traili

Re: ant wrapper script testing

2016-10-25 Thread Jeff Adamson
es. I will not be able to retest mac with bsd_sed until this evening. Regards, Jeff Adamson From: Jeff Adamson/Cleveland/IBM@IBMUS To: "Ant Developers List" Cc: maarten_co...@yahoo.com Date: 10/25/2016 01:56 PM Subject: Re: ant wrapper script testing Also glad to k

Re: ant wrapper script testing

2016-10-25 Thread Jeff Adamson
second option. Regards, Jeff Adamson From: Johan Corveleyn To: Ant Developers List Cc: maarten_co...@yahoo.com Date: 10/25/2016 10:13 AM Subject:Re: ant wrapper script testing On Tue, Oct 25, 2016 at 3:50 PM, Johan Corveleyn wrote: > On Sat, Oct 22, 2016 at 4:11 P

Re: ant wrapper script testing

2016-10-25 Thread Johan Corveleyn
On Tue, Oct 25, 2016 at 3:50 PM, Johan Corveleyn wrote: > On Sat, Oct 22, 2016 at 4:11 PM, Stefan Bodewig wrote: >> On 2016-10-18, Jeff Adamson wrote: >> >>> I made a new pull request based on the ongoing work. >> >>> https://github.com/apache/ant/pull/25 >> >> Yes, thanks. I've merged it. >> >

Re: ant wrapper script testing

2016-10-25 Thread Johan Corveleyn
On Sat, Oct 22, 2016 at 4:11 PM, Stefan Bodewig wrote: > On 2016-10-18, Jeff Adamson wrote: > >> I made a new pull request based on the ongoing work. > >> https://github.com/apache/ant/pull/25 > > Yes, thanks. I've merged it. > >> What do you think and can we get some users to test behaviors on

Re: ant wrapper script testing

2016-10-22 Thread Stefan Bodewig
On 2016-10-18, Jeff Adamson wrote: > I made a new pull request based on the ongoing work. > https://github.com/apache/ant/pull/25 Yes, thanks. I've merged it. > What do you think and can we get some users to test behaviors on a variety > of platforms? I really hope we can. Stefan -

Re: ant wrapper script testing

2016-10-18 Thread Jeff Adamson
Stefan Bodewig, I made a new pull request based on the ongoing work. https://github.com/apache/ant/pull/25 What do you think and can we get some users to test behaviors on a variety of platforms? Regards, Jeff Adamson From: Jeff Adamson/Cleveland/IBM To: dev@ant.apache.org Date: 10

Re: ant wrapper script testing

2016-10-14 Thread Jeff Adamson
squash this down first? Is it ready to test by our Solaris/AIX/HPUX users or not until we get it merged into the mainline repo? Regards, Jeff Adamson From: Jeff Adamson/Cleveland/IBM@IBMUS To: "Ant Developers List" Date: 10/13/2016 09:54 AM Subject: Re: ant wrapper scri

RE: ant wrapper script testing

2016-10-13 Thread Martin Gainty
MG>please see quick comment below Subject: Re: ant wrapper script testing To: dev@ant.apache.org From: jwadam...@us.ibm.com Date: Thu, 13 Oct 2016 09:53:19 -0400 Things I found out/rediscovered during experiments: `echo` bash built-in modifies the input (escape interpretation, trailing newl

Re: ant wrapper script testing

2016-10-13 Thread Jeff Adamson
the child tools do it. I have more comments inline https://github.com/jwadamson/ant-1/blob/shell-detection/src/script/ant as well as an awk pattern alternative to sed ( commit c591ade4aa7c19ef7e640ed9759fd95e8c4582d8) Regards, Jeff Adamson From: Stefan Bodewig To: dev@ant.apache.org Dat

Re: ant wrapper script testing

2016-10-12 Thread Stefan Bodewig
On 2016-10-11, Jeff Adamson wrote: > I have been doing some work on trying to correct errors in the ant shell > script with how special characters and whitespace and handled in arguments. Many thanks for that. > The only solution I have found to date that avoids all behavior quirks such > as ech