all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Noam Postavsky <npostavs@users.sourceforge.net>
Cc: stefan@merten-home.de, 30063@debbugs.gnu.org, simenheg@gmail.com
Subject: bug#30063: 26.0.90; Silent fail with `rst-compile-pdf-preview'
Date: Tue, 30 Jan 2018 15:49:47 +0200	[thread overview]
Message-ID: <83lggfh2x0.fsf@gnu.org> (raw)
In-Reply-To: <87k1w1lup5.fsf@users.sourceforge.net> (message from Noam Postavsky on Sun, 28 Jan 2018 13:11:34 -0500)

> From: Noam Postavsky <npostavs@users.sourceforge.net>
> Date: Sun, 28 Jan 2018 13:11:34 -0500
> Cc: stefan@merten-home.de, 30063@debbugs.gnu.org
> 
> Simen Heggestøyl <simenheg@gmail.com> writes:
> 
> >  	 (command (format "%s %s %s && %s %s ; rm %s"
> > +			  pdf-compile-program
> >  			  buffer-file-name tmp-filename
> >  			  rst-pdf-program tmp-filename tmp-filename)))
> > +    (unless (executable-find pdf-compile-program)
> > +      (error "Cannot find executable `%s'" pdf-compile-program))
> > +    (unless (executable-find rst-pdf-program)
> > +      (error "Cannot find executable `%s'" rst-pdf-program))
> 
> It's possible to have PATH and exec-path desynchronized, such that the
> above code could throw an error even though the
> start-process-shell-command call later would succeed.  Maybe we should
> should just consider that a misconfiguration on the user's part though.

Yes, we could do that.  But can we signal an error only when the
command fails?  That should minimize false negatives.

Also please note that the shell command as written is unportable: the
";" part will not work on MS-Windows, we need to use "&" instead.  But
that's a separate issue.





  parent reply	other threads:[~2018-01-30 13:49 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-01-10 13:57 bug#30063: 26.0.90; Silent fail with `rst-compile-pdf-preview' Simen Heggestøyl
2018-01-27 11:59 ` Simen Heggestøyl
2018-01-28 18:11   ` Noam Postavsky
2018-01-29 17:43     ` Glenn Morris
2018-01-30 13:49     ` Eli Zaretskii [this message]
2018-01-30 14:30       ` Noam Postavsky
2018-01-30 15:40         ` Eli Zaretskii
2019-09-29 12:09   ` Lars Ingebrigtsen
2019-10-03 20:53     ` Stefan Merten
2019-10-07  3:25       ` Lars Ingebrigtsen
2019-10-09 16:13         ` Simen Heggestøyl

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=83lggfh2x0.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=30063@debbugs.gnu.org \
    --cc=npostavs@users.sourceforge.net \
    --cc=simenheg@gmail.com \
    --cc=stefan@merten-home.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.