[Bug 217016] graphics/libGL: update to 17.0.1 to get llvm40 support

[bugzilla-noreply at freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=217016

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

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

--- Comment #9 from Jan Beich (mail not working) <jbeich at FreeBSD.org> ---
(In reply to Matthew Rezny from comment #8)
> That said, a patch to require llvm39 but use llvm40 if it is already installed
> 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?=native in make.conf myself, so this was tested.

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

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

https://cgit.freedesktop.org/mesa/mesa/commit/?id=9ab1704f42b

> 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*)

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