Page MenuHomeFreeBSD

[New PORTS] emulators/riscv-fesvr emulators/riscv-isa-sim
ClosedPublic

Authored by lwhsu on Aug 16 2016, 12:37 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 4, 4:01 PM
Unknown Object (File)
Tue, Oct 15, 8:27 AM
Unknown Object (File)
Tue, Oct 15, 8:27 AM
Unknown Object (File)
Tue, Oct 15, 8:27 AM
Unknown Object (File)
Tue, Oct 15, 8:27 AM
Unknown Object (File)
Tue, Oct 15, 8:27 AM
Unknown Object (File)
Tue, Oct 15, 8:26 AM
Unknown Object (File)
Tue, Oct 15, 8:26 AM
Subscribers

Details

Summary

RISC-V Emulator Support Packages

Add these two packages from github for support of RISC-V development

Diff Detail

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

Event Timeline

sbruno retitled this revision from to RISC-V Emulator Support Packages.
sbruno updated this object.
sbruno edited the test plan for this revision. (Show Details)
sbruno added a reviewer: br.

Try using LIB_DEPENDS not BUILD_DEPENDS

Fixup pkg-plist duplicate entries.

Fixup and explicitly require a c++11 compiler for versions of
FreeBSD that don't have a newish clang by default.

sbruno retitled this revision from RISC-V Emulator Support Packages to [New PORTS] emulators/riscv-fesvr emulators/riscv-isa-sim.Aug 16 2016, 1:01 PM
sbruno updated this object.

Theoretically add all the svn keywords

The exact same remarks apply to the second port :-)

emulators/riscv-fesvr/Makefile
7 ↗(On Diff #19348)

This is not needed

9–13 ↗(On Diff #19348)

All this should be at the end, after the USES/USE block

11 ↗(On Diff #19348)

This is not needed

20 ↗(On Diff #19348)

I don't care, but someone from the sort police will tell you these should be sorted.

emulators/riscv-fesvr/distinfo
1–2 ↗(On Diff #19348)

This is missing a TIMESTAMP line, as it is enforced at commit time, you should add it now. (run make makesum)

emulators/riscv-fesvr/pkg-plist
15–17 ↗(On Diff #19348)

This looks like strangely sorted.

@br are our patches going upstream?

many thanks I plan to test this soon
@emaste hopefully

lwhsu added a reviewer: sbruno.
lwhsu edited edge metadata.

try to improve a little bit

lwhsu marked 4 inline comments as done.Feb 7 2017, 5:46 AM
lwhsu marked 2 inline comments as done.Feb 7 2017, 5:50 AM
br edited edge metadata.
This revision is now accepted and ready to land.Feb 7 2017, 10:41 AM
This revision was automatically updated to reflect the committed changes.