[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#33344: 26.1; doc-view bounding-box recognition doesn't work on path
From: |
Eli Zaretskii |
Subject: |
bug#33344: 26.1; doc-view bounding-box recognition doesn't work on path names with spaces |
Date: |
Tue, 13 Nov 2018 19:08:54 +0200 |
> From: Glenn Morris <rgm@gnu.org>
> Date: Mon, 12 Nov 2018 16:36:48 -0500
> Cc: 33344@debbugs.gnu.org
>
> > Please change line 1218 from
> >
> > (format "-dFirstPage=%s -dLastPage=%s %s"
> >
> > to
> >
> > (format "-dFirstPage=%s -dLastPage=%s \"%s\""
>
> Thanks for the report.
> I don't see a need for the shell to be involved at all here.
> The following seems to work for me - how about for you?
Thanks, but call-process won't DTRT if the default-directory is
remote, would it?
And I wonder how many more subtle incompatibilities will such a change
cause. All that because we need to run a single string through
shell-quote-argument (and not just enclose it in double quotes)? Is
it really worth it?
bug#33344: 26.1; doc-view bounding-box recognition doesn't work on path names with spaces, Robert Spillner, 2018/11/22