svn commit: r551634 - head/databases/postgresql13-server

[Piotr Kubaj]

On 20-10-08 04:34:47, Adam Weinberger wrote:
> On Thu, Oct 8, 2020 at 4:24 AM Piotr Kubaj <pkubaj at anongoth.pl> wrote:
> >
> > It was previously committed in https://svnweb.freebsd.org/ports?view=revision&revision=527871 and it seemed to work. Or is it that just no one noticed it?
> >
> > I'm trying to understand why this issue appeared and how to fix it without reverting to the previous (also broken) behaviour.
> 
> The problem happens because of the extra level of indirection. The
> ${CHOSEN_COMPILER_TYPE} causes it to be evaluated too late,
> overwriting PORT_OPTIONS.
> 
> As to why it wasn't happening before, I'm not sure. Why not just
> exclude it on gcc archs instead?
> OPTIONS_DEFINE= ... LLVM
> OPTIONS_EXCLUDE_fooarch= LLVM

That wouldn't work. There are no architectures that use base GCC on head, because base GCC was removed. So it should be excluded on 11 and 12, but it should also be enabled on 13.

So ideally, it would need to be something like BROKEN:
BROKEN_FreeBSD_12_powerpc64=

I'll look whether something like this works:
OPTIONS_EXCLUDE_powerpc64=	${OPTIONS_EXCLUDE_powerpc64_${OSREL:R}}
OPTIONS_EXCLUDE_powerpc64_12=	LLVM

> 
> I can certainly appreciate the clever approach you're taking here, but
> it's apparently too clever for options.mk.
> 
> # Adam
> 
> 
> > On 20-10-08 04:11:05, Adam Weinberger wrote:
> > > On Wed, Oct 7, 2020 at 7:06 AM Piotr Kubaj <pkubaj at freebsd.org> wrote:
> > > >
> > > > Author: pkubaj
> > > > Date: Wed Oct  7 13:06:19 2020
> > > > New Revision: 551634
> > > > URL: https://svnweb.freebsd.org/changeset/ports/551634
> > > >
> > > > Log:
> > > >   databases/postgresql13-server: fix build on GCC architectures
> > > >
> > > >   Add the previously removed disabling of LLVM when it's not in base.
> > >
> > >
> > > > +OPTIONS_DEFAULT+=      ${OPTIONS_DEFAULT_${CHOSEN_COMPILER_TYPE}}
> > > > +OPTIONS_DEFAULT_clang= LLVM
> > >
> > > This forces the LLVM option on for all clang users, and prevents it
> > > from ever getting turned off. Please, turn the LLVM forcing off.
> 
> 
> -- 
> Adam Weinberger
> adamw at adamw.org
> https://www.adamw.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/svn-ports-all/attachments/20201008/8a1e0ad5/attachment.sig>