Update of bug #67268 (group groff):

                  Status:             In Progress => Fixed
             Open/Closed:                    Open => Closed
         Planned Release:                    None => 1.24.0

    _______________________________________________________

Follow-up Comment #8:


commit b53278be5deb5b6bc4a290ee73ba1e9e1d326134
Author: G. Branden Robinson <g.branden.robin...@gmail.com>
Date:   Fri Jul 4 04:14:25 2025 -0500

    [gropdf]: Add `-W` option.
    
    ... to make font embedding warnings fatal.
    
    * src/devices/gropdf/gropdf.pl: Add global scalar `makeWarningsFatal`.
      Recognize option when calling `GetOptions()`.
    
      (LoadFont): Fall over dead if embedding fails.
    
    * src/devices/gropdf/gropdf.pl (usage):
    * src/devices/gropdf/gropdf.1.man (Options): Document it.
    
    Begins fixing Savannah #67268.

commit bfd77135f31b067e4a21ccb9f3f496cc3672365d
Author: G. Branden Robinson <g.branden.robin...@gmail.com>
Date:   Fri Jul 4 08:56:00 2025 -0500

    [doc]: Use new gropdf `-W` option.
    
    * doc/doc.am (DOC_PDFMOM): Append `-P -W` to `pdfmom` arguments.
    
    Building from a distribution archive fails at this commit because this
    change exposes a latent error that should be fatal in a groff build.
    
    Log:
    
    Use of uninitialized value $file in substr at
.../groff-1.23.0.3459-bb549/_build/sub/gropdf line 1188, <__ANONIO__> line 4.
    Use of uninitialized value $file in substr at
.../groff-1.23.0.3459-bb549/_build/sub/gropdf line 1195, <__ANONIO__> line 4.
    Use of uninitialized value $file in concatenation (.) or string at
.../groff-1.23.0.3459-bb549/_build/sub/gropdf line 1195, <__ANONIO__> line 4.
      GEN      doc/groff-man-pages-cover.groff
      GROFF    doc/groff-man-pages.pdf
    Use of uninitialized value $file in substr at
.../groff-1.23.0.3459-bb549/_build/sub/gropdf line 1188, <__ANONIO__> line 4.
    Use of uninitialized value $file in substr at
.../groff-1.23.0.3459-bb549/_build/sub/gropdf line 1195, <__ANONIO__> line 4.
    Use of uninitialized value $file in concatenation (.) or string at
.../groff-1.23.0.3459-bb549/_build/sub/gropdf line 1195, <__ANONIO__> line 4.
    gropdf: fatal error: unable to embed font file for 'Times-Roman' (TR)
(missing entry in 'download' file?)
    
    Continues fixing Savannah #67268.

commit 2738ad12bb6db9d1f8f0e338797b8d8b61ebe7ce
Author: G. Branden Robinson <g.branden.robin...@gmail.com>
Date:   Fri Jul 4 08:56:56 2025 -0500

    contrib/mom/mom.am: Run `pdfmom` with `-P -W`.
    
    * contrib/mom/mom.am (MOMPDFMOM): Append `-P -W` to `pdfmom` arguments.
    
    Continues fixing Savannah #67268.

commit 9aac02a7c38ed00341ae9e8044bbca1b3e6f195e
Author: G. Branden Robinson <g.branden.robin...@gmail.com>
Date:   Fri Jul 4 09:10:49 2025 -0500

    [gropdf]: Skip blank lines in "download" file.
    
    * src/devices/gropdf/gropdf.pl (LoadDownload): Skip lines that consist
      only of whitespace.
    
    Continues fixing Savannah #67268.

commit d5832598ec8bb3968269a88eed80930365e764d4
Author: G. Branden Robinson <g.branden.robin...@gmail.com>
Date:   Fri Jul 4 09:28:19 2025 -0500

    [devpdf]: Add debugging diags to BuildFoundries.
    
    * font/devpdf/util/BuildFoundries.pl: Throw more `Notice` (debugging)
      diagnostic messages...
      (RunAfmtodit): ...when running `afmtodit`...
      (UseGropsVersion): ...when trying to open a font description file for
      writing...
      (LoadDownload): ...and when the download file cannot be opened.
    
    Continues fixing Savannah #67268.

commit d4302d172486c33d6cd59495f8a37906404de8be
Author: G. Branden Robinson <g.branden.robin...@gmail.com>
Date:   Fri Jul 4 09:35:07 2025 -0500

    [devpdf]: Make `--strict` option stricter.
    
    * font/devpdf/util/BuildFoundries.pl: Make `--strict` option stricter,
      turning more warnings fatal...
    
      (LoadFoundry): ...when an embeddable font file is known to be
      unavailable, when a grops(1) font description file is unreadable, when
      afmtodit(1) fails [the inference is indirect]...
    
      (RunAfmtodit): ...when a font description file has an unexpected
      format...
    
      (UseGropsVersion): ...when a font description file can't be created,
      when one being read lacks an `internalname` directive, and when the
      file can't be opened for reading.
    
    Continues fixing Savannah #67268.

commit bf35831c05e4b8bc26cf95cf819fcb47b5b29480
Author: G. Branden Robinson <g.branden.robin...@gmail.com>
Date:   Fri Jul 4 09:18:59 2025 -0500

    [devpdf]: Give BuildFoundries `--download` option.
    
    * font/devpdf/util/BuildFoundries.pl: Support new `--download` option,
      taking an input download file as argument and storing it to new
      `downloadFile` scalar.  Default to "download" as before.
    
      (LoadDownload): Throw `notice` diagnostic if we can't open the
      "download" file, and report the error encountered by `open()`.
    
    Continues fixing Savannah #67268.

commit 20d59d263fed685cdf8007910a2aeff3079d3786
Author: G. Branden Robinson <g.branden.robin...@gmail.com>
Date:   Wed Jul 2 09:30:10 2025 -0500

    [devpdf]: Fix Savannah #67268.
    
    * font/devpdf/devpdf.am: Tidy up for better build reliability.
    
      (font/devpdf/util/BuildFoundries): Stop handling the "download" file.
    
      (font/devpdf/download): Move the file's comment header from here...
    
    * font/devpdf/download.in: ...to here.
    
    * font/devpdf/devpdf.am (font/devpdf/download): Use "BuildFoundries"'s
      new `--download` option to construct the target from "download.in".
      Stop setting write permission on the generated download file in the
      build tree; it's no longer necessary.
    
    Fixes <https://savannah.gnu.org/bugs/?67268>.  Thanks to Dave Kemper for
    the report.




    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?67268>

_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/

Attachment: signature.asc
Description: PGP signature

Reply via email to