Re: CVS commit: pkgsrc/databases

[Frédéric Fauberteau <triaxx%NetBSD.org@localhost>]

Le 2019-10-25 02:38, Takahiro Kambe a écrit :
> In message <20191024080821.9CE6584D6F%mail.netbsd.org@localhost>
>         on Thu, 24 Oct 2019 10:07:54 +0200,
>         Leonardo Taccari <leot%NetBSD.org@localhost> wrote:
> > Frédéric Fauberteau writes:
> > > [...]
> > > Modified Files:
> > >         pkgsrc/databases/postgresql12: Makefile.common options.mk
> > >         pkgsrc/databases/postgresql12-client: PLIST
> > >         pkgsrc/databases/postgresql12-plperl: PLIST
> > >         pkgsrc/databases/postgresql12-plpython: PLIST
> > >         pkgsrc/databases/postgresql12-pltcl: PLIST
> > >         pkgsrc/databases/postgresql12-server: Makefile PLIST
> > >
> > > Log Message:
> > > postgresql12: make nls support optional
> > > [...]
> >
> > Given that the default options are changed that would need a 
> > PKGREVISION
> > bump to all packages too.  Can you please bump PKGREVISION to them?
> Or nls option enable default?
>
> It is fine to make nls support to optional, but why it is disabled
> default?

Maybe I should open the question on pkgsrc-users@ or tech-pkg@.

I think nls is not an indispensable feature to make a package working 
well. If I build packages with PREFER_PKGSRC=yes or if gettext is not 
available on the system, it removes dependencies to devel/gettext-lib 
and converters/libiconv (that is a poor implementation). If I pbulk a 
set of packages for servers, I get a /usr/pkg subtree with no 
/usr/pkg/share/locale directory. But If a want package for my laptop or 
desktop, I build them with PKG_DEFAULT_OPTIONS=nls.

But if most people prefer having locales by default, I can move nls 
option to PKG_SUGGESTED_OPTIONS.