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

Re: svn commit: r1803722 - /subversion/branches/1.9.x/STATUS

From: James McCoy <jamessan_at_apache.org>
Date: Wed, 2 Aug 2017 20:51:20 -0400

On Wed, Aug 02, 2017 at 02:07:20PM +0000, Daniel Shahaf wrote:
> jamessan_at_apache.org wrote on Wed, Aug 02, 2017 at 01:35:31 -0000:
> > * r1802032
> > Install 'fsfs-stats' as a wrapper to 'svnfsfs', to which it was renamed in
> > r1618848.
> > Justification:
> > Backwards compatibility with 1.8.x tools/.
> > Votes:
> > + -0: jamessan ($(bindir) and $$1 should be quoted in case they contain shell metacharacters)
> Thanks for the review. I'll fix $1 in a moment, but why does $(bindir)
> need to be quoted? The makefiles use $(bindir) unquoted [1],
> so I assumed what was safe for shell commands in Makefile is safe for
> shell commands on the installed system.

Just because no one has run into trouble with the existing code, doesn't
mean it's right. :) Sure, $(bindir) is _typically_ going to be safe to
use unquoted, but why rely on that? If I run "./configure --prefix
'/home/jamessan/things & stuff'" and then "make", the Makefile breaks
horribly. I don't even need to try installing. It fails while creating
the .la files.


GPG Key: 4096R/91BF BF4D 6956 BD5D F7B7  2D23 DFE6 91AE 331B A3DB
Received on 2017-08-03 02:51:33 CEST

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