bugGNU roff - Bugs: bug #64114, [pdfpic] when falling back to EPS,...

 
 

bug #64114: [pdfpic] when falling back to EPS, should create file in temporary directory, but doesn't

Submitter:  G. Branden Robinson <gbranden>
Submitted:  Wed 26 Apr 2023 10:51:18 AM UTC
   
 
Category:  Macro - others/general Severity:  3 - Normal
Item Group:  Incorrect behaviour Status:  None
Privacy:  Public Assigned to:  None
Open/Closed:  Open Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Mon 10 Jul 2023 08:54:54 AM UTC, comment #2: 

Cross reference.


commit c3428341b64521d18c73d5f3cc064e3e05ddf95a
Author: G. Branden Robinson <g.branden.robinson@gmail.com>
Date:   Wed Apr 26 04:36:05 2023 -0500

    [pdfpic]: Refactor (`sy` -> `pso`).

    * tmac/pdfpic.tmac: Migrate gathering of image dimensions from `sy` and
      a temporary file to `pso`.

      (pdfpic@cleanup): Drop `pdfpic*temporary-file` string.

      (pdfpic@get-image-dimensions): Remove redirection.  Invoke `pso`, not
      `sy`.

      (PDFPIC): Stop constructing `pdfpic*temporary-file` string.  Stop
      testing `systat` register.  Stop sourcing and deleting temporary file.

    We keep the temporary directory handling because we will need it for the
    `PSPIC` fallback logic, but it also promises to be really painful to fix
    that before we have more formatter support for string traversal.  See
    <https://savannah.gnu.org/bugs/index.php?64114>.


G. Branden Robinson <gbranden>
Group administrator
Thu 27 Apr 2023 01:47:38 PM UTC, comment #1: 

Oy, vey, I need to implement basename(1) in *roff.

I think I'll stick this behind bug #62264.

G. Branden Robinson <gbranden>
Group administrator
Wed 26 Apr 2023 10:51:18 AM UTC, original submission:  

Affects groff 1.22.4.


$ cat peng.ms
.LP
Here is a picture of a penguin.
.PDFPIC unwritable/penguin.pdf
$ ls -dl unwritable
dr-x------ 2 branden branden 4096 Apr 26 05:42 unwritable
$ ls unwritable/
penguin.pdf
$ groff -U -ms peng.ms >| peng.ps
I/O Error: Couldn't open PostScript file 'unwritable/penguin.eps'
troff: peng.ms:3: can't open 'unwritable/penguin.eps': No such file or directory


Switching to the 'pdf' device...


$ groff -U -ms -Tpdf peng.ms >| peng.pdf
$ echo $?
0


This is because a temporary directory was correctly used for construction of the file that was sourced to read the image dimensions.  See bug #64061 for more on that process than you ever wanted to know.

The program anonymously complaining of an I/O error would appear to be pdf2ps.  From poppler-utils--try to contain your surprise.

G. Branden Robinson <gbranden>
Group administrator

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Digest:
   bug dependencies.

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by gbranden (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2023-08-10 gbranden StatusPostponed None
        Dependencies- Depends on bugs #62264
    2023-04-27 gbranden StatusNone Postponed

    Back to the top

    Powered by Savane 3.13-4448.
    Corresponding source code