Page MenuHomeFreeBSD

devel/binutils: enable shared for riscv64_none_elf
ClosedPublic

Authored by mhorne on Jul 19 2020, 10:41 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jul 10, 10:31 AM
Unknown Object (File)
Tue, Jul 9, 4:40 PM
Unknown Object (File)
Sat, Jul 6, 3:45 AM
Unknown Object (File)
Wed, Jul 3, 3:49 AM
Unknown Object (File)
Tue, Jul 2, 9:12 PM
Unknown Object (File)
Sun, Jun 23, 12:50 PM
Unknown Object (File)
Sat, Jun 22, 5:23 PM
Unknown Object (File)
Sat, Jun 22, 4:45 AM
Subscribers
None

Details

Summary

Shared library support is incomplete for the RISC-V embedded elf target,
and was disabled some time ago [1]. However, it appears that it is
functional enough to support basic usage, and is required to
successfully build u-boot for RISC-V. This toolchain doesn't have
widespread usage, so any risk of re-enabling this support is outweighed
by the benefit of being able to build important ports like u-boot.
Override this default.

[1] https://sourceware.org/git/?p=binutils-gdb.git;a=commit;h=be1dce26256aa9f6c7742e81c8c3db9eacff079d

Test Plan

The linked commit doesn't offer much of a description, but I spoke with
the author who told me a subset of unit tests fail for this target, so
he simply disabled the shared library support until someone motivated
comes along to address them.

I've been running this patch locally for several months, and u-boot builds and runs fine.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

mhorne created this revision.

Hi, any opposition to this? I will need at least one ports/maintainer approval in order to commit.

I still have a lukewarm ports commit bit. :)

This looks good. Approved.

This revision is now accepted and ready to land.Jul 30 2020, 1:40 AM
This revision was automatically updated to reflect the committed changes.