Re: git: 51e825758420 - main - www/firefox: unbreak after b00578b9b208

From: Tijl Coosemans <tijl_at_FreeBSD.org>
Date: Wed, 09 Feb 2022 20:38:22 UTC
On Wed, 9 Feb 2022 20:59:41 +0100 Christoph Moench-Tegeder
<cmt@burggraben.net> wrote:
> ## Tijl Coosemans (tijl@FreeBSD.org):
>>> This variable isn't defined when DEFAULT_VERSIONS contains llvm and the
>>> LTO options is off, which then breaks the dependency checks in
>>> www/firefox.  You can simply remove this variable if you replace the
>>> dependency in www/firefox with  
> 
> Basically the whole check there is a problem: we need the wasi-libcxx
> to go with the llvm port, which means that we can only use llvm13 as
> of now. I suggested to brooks@ that the wasi-stuff should be folded
> into the llvm ports, which would simplify the gecko-side a lot.
> But it works somewhat, as long as you don't screw around with the
> default llvm version (which I don't, so it's not that high-prio
> for me, as I have real brokeness in other ports to tend to).

The only place (afaict) where LLVM_VERSION is used is in BUILD_DEPENDS
in www/firefox.  If you change that dependency line you can remove
LLVM_VERSION and stop worrying about keeping it in sync.