[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: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Thu, 28 Nov 2013 13:35:21 +0200

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
Received on 2013-11-28 12:36:02 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.