Follow-up Comment #6, bug #67207 (group groff): Hi Branden,
I have a problem with your "diffy". I tested it by running this command (to
produce a Japanese man page):-
[derij@pip build (master)]$ test-groff -Tpdf -man -fG -petk -F
/usr/share/groff/site-font/ -mja groff.7 |okular -
In the download file in that directory:-
SauceHanSansJP-B
/usr/share/groff/site-font/devps/SauceHanSansJP-B.pfb
SauceHanSansJP-BI
/usr/share/groff/site-font/devps/SauceHanSansJP-BI.pfb
SauceHanSansJP-BV
/usr/share/groff/site-font/devps/SauceHanSansJP-BV.pfb
SauceHanSansJP-I
/usr/share/groff/site-font/devps/SauceHanSansJP-I.pfb
SauceHanSansJP-R
/usr/share/groff/sitefont/devps/SauceHanSansJP-R.pfb
SauceHanSansJP-V
/usr/share/groff/site-font/devps/SauceHanSansJP-V.pfb
I introduced a typo in the entry for "SauceHanSansJP-R" (missing hyphen in the
filename). When I ran with the current gropdf I get:-
[derij@pip build (master)]$ groff -Tpdf -man -fG -petk -F
/usr/share/groff/site-font/ -mja groff.7 |okular -
gropdf: warning: The download file in '/usr/share/groff/site-font/devpdf' has
erroneous entry for 'SauceHanSansJP-R (GR)'
troff:groff.7:1084: warning: special character 'u3002' not defined
But if I run with the diff applied, I get:-
[derij@pip build (master)]$ test-groff -Tpdf -man -fG -petk -F
/usr/share/groff/site-font/ -mja groff.7 |okular -
troff:groff.7:1084: warning: special character 'u3002' not defined
No message about the problem entry. I realise your code is more complex than
the original, but I don't quite understand what you are trying to "improve".
To my mind the original message identifies the exact download file the exact
entry which caused the issue and the groff font name affected (GR). I don't
need to know which other download files I have read before finding the
problem, since that information is superfluous to fixing the issue. Please can
you explain your thinking behind adding the complexity in your code.
Apologies if I'm barking up the wrong tree, but I do consider a typo in the
filename to be worthy of a warning. Perhaps if you explained what you were
trying to achieve I can knock up some code for gropdf.
_______________________________________________________
Reply to this item at:
<https://savannah.gnu.org/bugs/?67207>
_______________________________________________
Message sent via Savannah
https://savannah.gnu.org/
signature.asc
Description: PGP signature
