Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 09 Mar 2017 23:24:30 +0000
From:      bugzilla-noreply@freebsd.org
To:        x11@FreeBSD.org
Subject:   [Bug 217016] graphics/libGL: update to 17.0.1 to get llvm40 support
Message-ID:  <bug-217016-7141-rFuLpbFfIh@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-217016-7141@https.bugs.freebsd.org/bugzilla/>
References:  <bug-217016-7141@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D217016

Jan Beich (mail not working) <jbeich@FreeBSD.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jbeich@FreeBSD.org

--- Comment #9 from Jan Beich (mail not working) <jbeich@FreeBSD.org> ---
(In reply to Matthew Rezny from comment #8)
> That said, a patch to require llvm39 but use llvm40 if it is already inst=
alled
> would be nice, but that is not what the proposed patch does.

Detecting dependencies at runtime is undesired.

https://www.freebsd.org/doc/en/books/porters-handbook/makefile-depend.html#=
makefile-automatic-dependencies

> Instead, it merely allows MESA_LLVM_VER to be more easily overridden
> while also dropping the min version and bumping the default version

I wanted the knob hidden, so it can allow footshooting. Not really a feature
supported by the maintainer.

> dropping part of post-patch,

[[:graph:]]* wasn't dropped but plain incorrect. According to re_format(7)
negating character class is perfectly fine but BSD regex doesn't support
shorthand variants. This may change with the import of libtre in future.

I do use CPUTYPE?=3Dnative in make.conf myself, so this was tested.

> rolling the post-patch changes or a subset thereof into patch-configure a=
nd
> patch-configure.ac (which need not be patched),

17.0.1 requires more extensive changes. I've tried to use the syntax compat=
ible
with both GNU sed and BSD sed. files/configure.ac was out of date, so it sh=
ould
either be removed or submitted upstream. Leaving it as is is confusing.

https://cgit.freedesktop.org/mesa/mesa/commit/?id=3D9ab1704f42b

> and there is some other change to configure patches I have no looked
> at close enough to see whether the make sense for the 13 -> 17 bump
> given the rest is incorrect.

What else? For one, the following change was made to be consistent with the
rest of the file.

  -+linux*|freebsd*)
  ++linux* | freebsd*)

--=20
You are receiving this mail because:
You are the assignee for the bug.=



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-217016-7141-rFuLpbFfIh>