[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: get-deps.sh patch, was Re: Subversion 1.8.5 released

From: Nico Kadel-Garcia <nkadel_at_gmail.com>
Date: Thu, 28 Nov 2013 08:02:46 -0500

On Thu, Nov 28, 2013 at 6:35 AM, Daniel Shahaf <d.s_at_daniel.shahaf.name> wrote:
> Ben Reser wrote on Wed, Nov 27, 2013 at 22:32:53 -0800:
>> On 11/27/13 9:04 PM, Nico Kadel-Garcia wrote:
>> > * You mentioned that I should not use "$()" syntax. It was helpful to
>> > get the "printf" for SQLITE_VERSION handling into a single line,
>> > because printf parsing is a bit odd.. I'm startled if any system
>> > modern enough to compose Subversion 1.8.5';s dependencies do not
>> > support that syntax, but but it's easy to unroll that change.
>>
>> Solaris's /bin/sh doesn't support $() unfortunately. Yes $() is part of POSIX.
>> We've had users complain about this not working. Unfortunately that means you
>> have to split that onto 2 lines. I wasn't thrilled about doing that but
>> couldn't find a portable way to do it in one line.
>
> Would this not work?
>
> % echo `echo \`echo foo\` | sed s/foo/bar/`
> bar

Brother, the problem is "printf". It doesn't deal with content piped
to it well, and it starts becoming illegible and unmaintenable if you
start playing the "how many backslashes do I need in front of the `"
game. That's why I used "$()", which allows nested subshell
operations..

Oh, well, 2 lines it is.!
Received on 2013-11-28 14:03:21 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.