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

From: Tijl Coosemans <tijl_at_FreeBSD.org>
Date: Tue, 08 Feb 2022 20:02:04 UTC
On Tue, 8 Feb 2022 21:00:20 +0100 Tijl Coosemans <tijl@coosemans.org>
wrote:
> On Tue, 8 Feb 2022 19:28:18 GMT Christoph Moench-Tegeder
> <cmt@FreeBSD.org> wrote:
>> The branch main has been updated by cmt:
>> 
>> URL: https://cgit.FreeBSD.org/ports/commit/?id=51e8257584202f96099f31adf71577099fbb12ce
>> 
>> commit 51e8257584202f96099f31adf71577099fbb12ce
>> Author:     Christoph Moench-Tegeder <cmt@FreeBSD.org>
>> AuthorDate: 2022-02-08 19:26:13 +0000
>> Commit:     Christoph Moench-Tegeder <cmt@FreeBSD.org>
>> CommitDate: 2022-02-08 19:26:13 +0000
>> 
>>     www/firefox: unbreak after b00578b9b208
>>     
>>     the webasm components must follow llvm (the llvm we use for building
>>     firefox), and wasi-compiler-rt and wasi-libcxx are really just
>>     llvm components and should be at the same version as the llvm we need.
>> ---
>>  Mk/bsd.gecko.mk                   | 2 +-
>>  devel/wasi-compiler-rt13/Makefile | 2 +-
>>  devel/wasi-compiler-rt13/distinfo | 6 +++---
>>  devel/wasi-libcxx/Makefile        | 3 +--
>>  devel/wasi-libcxx/distinfo        | 6 +++---
>>  www/firefox/Makefile              | 2 +-
>>  6 files changed, 10 insertions(+), 11 deletions(-)
>> 
>> diff --git a/Mk/bsd.gecko.mk b/Mk/bsd.gecko.mk
>> index 52107a1c0c4c..fdc1effe8672 100644
>> --- a/Mk/bsd.gecko.mk
>> +++ b/Mk/bsd.gecko.mk
>> @@ -98,7 +98,7 @@ MOZ_EXPORT+=	LLVM_OBJDUMP="${LOCALBASE}/bin/llvm-objdump${LLVM_DEFAULT}"
>>  # Ignore Mk/bsd.default-versions.mk but respect make.conf(5) unless LTO is enabled
>>  .if !defined(DEFAULT_VERSIONS) || ! ${DEFAULT_VERSIONS:Mllvm*} || ${PORT_OPTIONS:MLTO}
>>  LLVM_DEFAULT=	13 # chase bundled LLVM in lang/rust for LTO
>> -LLVM_VERSION=	13.0.0 # keep in sync with devel/wasi-compiler-rt${LLVM_DEFAULT}
>> +LLVM_VERSION=	13.0.1 # keep in sync with devel/wasi-compiler-rt${LLVM_DEFAULT}  
> 
> 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

wasi-compiler-rt${LLVM_DEFAULT}>0:devel/wasi-compiler-rt${LLVM_DEFAULT}