Re: news/pan and mk/tools/gettext.mk

[Joerg Sonnenberger <joerg%bec.de@localhost>]

On Fri, Nov 16, 2018 at 02:56:21PM +0100, Rhialto wrote:
> On Fri 16 Nov 2018 at 14:19:02 +0100, Joerg Sonnenberger wrote:
> > On Fri, Nov 16, 2018 at 10:46:39AM +0100, Rhialto wrote:
> > > I noticed that news/pan fails to build on the netbsd builder even though
> > > it builds fine for me.
> > > 
> > > The problem is that it wants the pkgsrc version of msgfmt; the version
> > > on NetBSD 8 isn't new enough. But it picks up the pkgsrc version for me,
> > > so I didn't notice.
> > 
> > *Why* does it want a newer version? It is *very* undesirable to pull in
> > pkgsrc gettext.
> 
> According to
> http://nyftp.netbsd.org/pub/pkgsrc/packages/reports/2018Q3/NetBSD-8.0-x86_64/20181109.1909/pan-0.145nb1/build.log
> it uses msgfmt --xml and msgfmt --desktop.
> 
> I can find that in pan-0.145/Makefile.am:
> 
> pan.desktop: pan.desktop.in
>         $(AM_V_GEN)$(MSGFMT) --desktop --template $< -d $(top_srcdir)/po -o $@
> [...]
> pan.appdata.xml: pan.appdata.xml.in
>         $(AM_V_GEN)$(MSGFMT) --xml --template $< -d $(top_srcdir)/po -o $@
> 
> I suppose I can do without those files (I don't use any "desktop"
> anyway), but it would make the package incomplete for others.

I'd say we can just ship them statically.

Joerg