Re: pkg/58668: gettext fails to build if base compiler is clang

[" devel.v9zjk%slmail.me@localhost via gnats" <gnats-admin%NetBSD.org@localhost>]

The following reply was made to PR pkg/58668; it has been noted by GNATS.

From: devel.v9zjk%slmail.me@localhost
To: Tobias Nygren <tnn%netbsd.org@localhost>
Cc: gnats-bugs%netbsd.org@localhost
Subject: Re: pkg/58668: gettext fails to build if base compiler is clang
Date: Sat, 14 Sep 2024 15:21:51 +0000

 Hi Tobias,
 sorry for the late answer but I couldn't try again with your suggestions.
 
 Yes, setting those options solved the problem.
 That said, in the documentation I think this requirements it's not that
 clear.
 
 Here what is documented:
 (source: https://wiki.netbsd.org/tutorials/clang/)
 MKGCC=3Dno		needed as building GCC alongside is not maintained
 MKLLVM=3Dyes 		builds LLVM libraries
 HAVE_LLVM=3Dyes 		controls clang being built
 PKGSRC_COMPILER=3Dclang 	instructs pkgsrc it's clang
 CLANGBASE=3D/usr 		instructs pkgsrc where to look for the compiler
 
 As you can see, HAVE_LLVM is considered something to set just to build
 clang and not to set the latter also as the default compiler for pkgsrc.
 I think it would be better to describe such option as mandatory in the
 wiki, since it's not always true that the compilation environment for
 the base is the same of pkgsrc's.
 
 Anyway, thank you very much for your help.
 
 
 Best regards,
 Luca De Pandis
 
 
 On Wednesday, September 11th, 2024 at 12:00, Tobias Nygren - tnn at netbsd.=
 org <tnn%netbsd.org@localhost> wrote:
 
 > Hi,
 >=20
 > I don't think the patch is is correct. Rather I suspect this is a
 > problem due to interaction between pkgsrc and /usr/share/mk on NetBSD.
 >=20
 > On a NetBSD system with clang as the base compiler you should explicitly
 > set these options in mk.conf:
 >=20
 > HAVE_LLVM=3Dyes
 > MKLLVM=3Dyes
 > MKGCC=3Dno
 >=20
 > The last two might be optional but they will be required also for
 > rebuilding your userland with ./build.sh so it's best to set them.
 > Please let us know if this solves the issue.
 >=20
 > -Tobias