[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#29522: rustc-1.16.0 broken after jemalloc updated to 5.0.1
From: |
Adam Van Ymeren |
Subject: |
bug#29522: rustc-1.16.0 broken after jemalloc updated to 5.0.1 |
Date: |
Mon, 04 Dec 2017 11:10:10 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux) |
address@hidden (Ludovic Courtès) writes:
> Hi,
>
> Jelle Licht <address@hidden> skribis:
>
>> 2017-11-30 23:41 GMT+01:00 Adam Van Ymeren <address@hidden>:
>>
>>> I haven't had time to dig in to this further, but in case anyone wants
>>> to fix rustc-1.16.0, it's broken after the upgrade of jemalloc to 5.0.1.
>>>
>>> Reverting commit 475b99fa5cf402430aa93a40e406e854ad2ff6e4 which reverts
>>> jemalloc back to 4.5.0 causes rustc to build successfully again. It has
>>> been broken for some time.
>>>
>>> https://hydra.gnu.org/job/gnu/master/rustc-1.16.0.x86_64-linux
>>>
>>>
>>>
>>>
>> It seems that the bundled copy of jemalloc in the rustc repo is currently
>> pinned at 4.5.0 partially
>> because of this specific issue as well.
>>
>> I did find an issue on the rust GH repo [0], and it seems this also affects
>> the nix-rust project,
>> who seem to have the same errors as our currently failing build [1].
>>
>> A temporary workaround could be to have a custom version of jemalloc with
>> the c++ features disabled
>> by building with `--disable-cxx'. Alternatively, we could just make use of
>> jemalloc 4.5.0 for rustc only
>> until this is all sorted our by upstream.
>
> Using a --disable-cxx variant of the latest jemalloc sounds preferable
> to me over running an old jemalloc.
I feel like if rust is pegged at jemalloc 4.5.0 then that's what we
should be feeding it. The changelog suggestst that jemalloc 5 has some
pretty significant changes, changing rust to use that theoretically lead
to some subtle bugs, I feel like I'd rather wait for upstream to make
the upgrade themselves.
>
> How does that sound?
>
> Thanks,
> Ludo’.