[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: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Wed, 2 Aug 2017 14:07:20 +0000

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.

Or do you just mean changing:
   printf %s 'foo $(bindir) bar'
to:
   printf 'foo %s bar' $(bindir)
?

Cheers,

Daniel

[1]
% grep -h -w bindir ${abs_srcdir}/build-outputs.mk ./Makefile
        $(MKDIR) $(DESTDIR)$(bindir)
        cd subversion/svn ; $(INSTALL_BIN) svn$(EXEEXT) $(DESTDIR)$(bindir)/svn$(EXEEXT)
        cd subversion/svnadmin ; $(INSTALL_BIN) svnadmin$(EXEEXT) $(DESTDIR)$(bindir)/svnadmin$(EXEEXT)
        cd subversion/svnbench ; $(INSTALL_BIN) svnbench$(EXEEXT) $(DESTDIR)$(bindir)/svnbench$(EXEEXT)
        cd subversion/svndumpfilter ; $(INSTALL_BIN) svndumpfilter$(EXEEXT) $(DESTDIR)$(bindir)/svndumpfilter$(EXEEXT)
        cd subversion/svnfsfs ; $(INSTALL_BIN) svnfsfs$(EXEEXT) $(DESTDIR)$(bindir)/svnfsfs$(EXEEXT)
        cd subversion/svnlook ; $(INSTALL_BIN) svnlook$(EXEEXT) $(DESTDIR)$(bindir)/svnlook$(EXEEXT)
        cd subversion/svnmucc ; $(INSTALL_BIN) svnmucc$(EXEEXT) $(DESTDIR)$(bindir)/svnmucc$(EXEEXT)
        cd subversion/svnrdump ; $(INSTALL_BIN) svnrdump$(EXEEXT) $(DESTDIR)$(bindir)/svnrdump$(EXEEXT)
        cd subversion/svnserve ; $(INSTALL_BIN) svnserve$(EXEEXT) $(DESTDIR)$(bindir)/svnserve$(EXEEXT)
        cd subversion/svnsync ; $(INSTALL_BIN) svnsync$(EXEEXT) $(DESTDIR)$(bindir)/svnsync$(EXEEXT)
        cd subversion/svnversion ; $(INSTALL_BIN) svnversion$(EXEEXT) $(DESTDIR)$(bindir)/svnversion$(EXEEXT)
bindir = ${exec_prefix}/bin
  $(MKDIR) $(DESTDIR)$(bindir); \
  ln -sf svnmucc$(EXEEXT) $(DESTDIR)$(bindir)/svnsyitf$(EXEEXT); \
  if test "$(DESTDIR)$(bindir)" != "$(DESTDIR)$(toolsdir)"; then \
    ln -sf $(bindir)/svnmucc$(EXEEXT) $(DESTDIR)$(toolsdir)/svnmucc$(EXEEXT); \
    ln -sf $(bindir)/svnbench$(EXEEXT) $(DESTDIR)$(toolsdir)/svn-bench$(EXEEXT); \
    'exec $(bindir)/svnfsfs stats $${2:+"-M"} $$2 $$1' \
Received on 2017-08-02 16:07:30 CEST

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