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 <[email protected]>
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 <[email protected]>
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 <[email protected]>
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 <[email protected]>
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 <[email protected]>
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 <[email protected]>
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 <[email protected]>
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 <[email protected]>
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/
signature.asc
Description: PGP signature
