Re: git: 9742a6346057 - main - Mk/bsd.default-versions.mk: bump LLVM_DEFAULT to 15

From: Piotr Kubaj <pkubaj_at_anongoth.pl>
Date: Tue, 03 Jan 2023 15:22:21 UTC
It's fine. I initially bumped the default to 10, because 9 had bugs
specificto powerpc that broke some packages.

On 23-01-03 16:16:52, Tobias C. Berner wrote:
> Well, if powerpc is fine with llvm15, then we can remove the if
> ```
> .  if ${ARCH} == powerpc
> LLVM_DEFAULT?=          10
> .  else
> LLVM_DEFAULT?=          15
> .  endif
> ```
> and just set
> ```
> LLVM_DFEAULT?= 15
> ```
> for everyone.
> 
> mfg Tobias
> 
> On Tue, 3 Jan 2023 at 16:11, Piotr Kubaj <pkubaj@anongoth.pl> wrote:
> >
> > Why wasn't the default for powerpc bumped?
> >
> > On 23-01-03 09:23:15, Tobias C. Berner wrote:
> > > The branch main has been updated by tcberner:
> > >
> > > URL: https://cgit.FreeBSD.org/ports/commit/?id=9742a6346057545688a02ae3eb537ac39c6b296c
> > >
> > > commit 9742a6346057545688a02ae3eb537ac39c6b296c
> > > Author:     Tobias C. Berner <tcberner@FreeBSD.org>
> > > AuthorDate: 2023-01-03 09:19:27 +0000
> > > Commit:     Tobias C. Berner <tcberner@FreeBSD.org>
> > > CommitDate: 2023-01-03 09:22:21 +0000
> > >
> > >     Mk/bsd.default-versions.mk: bump LLVM_DEFAULT to 15
> > >
> > >     - graphics/mesa* switched to LLVM version 15 some time ago
> > >     - desktop systems suffer from requiring many different LLVM versions
> > >     - having the default at something recent combined with the version
> > >       matching of llvm.mk (prefering $LLVM_DEFAULT) this should lessen
> > >       the number of LLVM installations on normal systems again alot.
> > >
> > >     - while here, remove outdated comment. x11@ switched to setting the
> > >       LLVM version explicitely as the default was way out of date.
> > >
> > >     PR:             263456
> > >     Exp-run by:     antoine
> > > ---
> > >  Mk/bsd.default-versions.mk | 6 ++----
> > >  1 file changed, 2 insertions(+), 4 deletions(-)
> > >
> > > diff --git a/Mk/bsd.default-versions.mk b/Mk/bsd.default-versions.mk
> > > index 8d8cdf71b807..91aafa427da3 100644
> > > --- a/Mk/bsd.default-versions.mk
> > > +++ b/Mk/bsd.default-versions.mk
> > > @@ -77,13 +77,11 @@ LIBRSVG2_DEFAULT?=        legacy
> > >  .  endif
> > >  # Possible values: c7
> > >  LINUX_DEFAULT?=              c7
> > > -# Possible values: 70, 80, 90, 10, 11, 12, 13, 14, -devel (to be used when non-base compiler is required)
> > > -# Please give notice to the Graphics Team (x11@FreeBSD.org) in advance before
> > > -# bumping the LLVM version.
> > > +# Possible values: 70, 80, 90, 10, 11, 12, 13, 14, 15, -devel (to be used when non-base compiler is required)
> > >  .  if ${ARCH} == powerpc
> > >  LLVM_DEFAULT?=               10
> > >  .  else
> > > -LLVM_DEFAULT?=               90
> > > +LLVM_DEFAULT?=               15
> > >  .  endif
> > >  # Possible values: 5.1, 5.2, 5.3, 5.4
> > >  LUA_DEFAULT?=                5.4