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
F81912437: D7527.id24822.diff
Tue, Apr 23, 1:41 AM
F81912434: D7527.id19344.diff
Tue, Apr 23, 1:41 AM
F81912432: D7527.id19346.diff
Tue, Apr 23, 1:41 AM
F81912428: D7527.id.diff
Tue, Apr 23, 1:41 AM
F81912426: D7527.id24821.diff
Tue, Apr 23, 1:41 AM
F81912424: D7527.id19340.diff
Tue, Apr 23, 1:41 AM
F81912422: D7527.id19348.diff
Tue, Apr 23, 1:41 AM
F81912419: D7527.id24839.diff
Tue, Apr 23, 1:41 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
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 4818
Build 4875: arc lint + arc unit

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

This is not needed

9–13

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

11

This is not needed

20

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

emulators/riscv-fesvr/distinfo
1–2

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

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.