Hey,

So i think llvm should not be in depends and instead it should be checked if it is installed or not like wayland

Like this:
"
cpt l -q llvm && llvm=enabled || llvm=disabled

cl-meson \
-Dplatforms="$platforms" \
-Dzstd=false \
-D shared-llvm=$llvm \
-D llvm=$llvm \
. output
"

Thanks!

WolfsDream <WolfsDream@proton.me> writes:

> Hey,
>
> So i think llvm should not be in depends and instead it should be
> checked if it is installed or not like wayland

Hello,

Even though LLVM is *technically* optional to mesa, some of the drivers
that mesa builds by default depend on LLVM. As far as I'm aware, you do
need LLVM for your video drivers as long as you are not using an AMD
GPU. If that's the case for you, my advice would be to fork the package
and modify the build so that it only builds the drivers for your
specific GPU.


The reason that I cannot make such a change to the package is that:

- There is no way to modify the `depends` file before the build starts,
  so if you happen to need LLVM for your drivers (which is highly
  likely) but it's not there, the build would fail. I don't want the
  packages on the repository to require modification to work.
- Even if you could modify the dependencies beforehand, doing this would
  require some hardware detection which adds extra complexity to the build.

Hope it clarifies the issue a bit, I certainly had thought of it before!

--
Best,
Cem