GNU bug report logs - #46051
27.1; Nroff preview does not work with file that include space in path (quoting problem ?)

Previous Next

Full log

Message #17 received at 46051 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: roucaries.bastien <at> gmail.com
Cc: 46051 <at> debbugs.gnu.org
Subject: Re: bug#46051: 27.1;
 Nroff preview does not work with file that include space in path
 (quoting problem ?)
Date: Sat, 30 Jan 2021 11:00:04 +0200

Ping!  Please see if the patch below fixes the problem, so that we
could install it.  TIA.

> Date: Sat, 23 Jan 2021 13:30:06 +0200
> From: Eli Zaretskii <eliz <at> gnu.org>
> Cc: 46051 <at> debbugs.gnu.org
> 
> > From: Bastien ROUCARIES <roucaries.bastien <at> gmail.com>
> > Date: Sat, 23 Jan 2021 11:11:09 +0000
> > Cc: 46051 <at> debbugs.gnu.org
> > 
> > > > I believe this is a quoting problem but I am not a lisp expert.
> > >
> > > Do you mean the nroff-view command?  Or do you mean something else?
> > 
> > Yes I means this command.
> > 
> > It faill when running for instance from "/tmp/dir whith space/someman.1"
> 
> Does the patch below fix the problem?
> 
> diff --git a/lisp/textmodes/nroff-mode.el b/lisp/textmodes/nroff-mode.el
> index 8965785..7e54755 100644
> --- a/lisp/textmodes/nroff-mode.el
> +++ b/lisp/textmodes/nroff-mode.el
> @@ -319,7 +319,7 @@ nroff-view
>  	 (save-buffer))
>      (if viewbuf
>  	(kill-buffer viewbuf))
> -    (Man-getpage-in-background file)))
> +    (Man-getpage-in-background (shell-quote-argument file))))
>  
>  (provide 'nroff-mode)
>  
> 
> 
> 
> 

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.