Page MenuHomeFreeBSD

Update benchmarks/spp to v0.4
ClosedPublic

Authored by lstewart on Nov 26 2018, 7:52 AM.
Tags
None
Referenced Files
F81551081: D18336.id.diff
Wed, Apr 17, 10:14 PM
F81517695: D18336.id51102.diff
Wed, Apr 17, 10:56 AM
Unknown Object (File)
Wed, Apr 3, 11:42 PM
Unknown Object (File)
Thu, Mar 28, 1:58 PM
Unknown Object (File)
Wed, Mar 27, 10:25 AM
Unknown Object (File)
Jan 25 2024, 6:40 AM
Unknown Object (File)
Jan 25 2024, 6:40 AM
Unknown Object (File)
Jan 25 2024, 6:40 AM
Subscribers

Details

Summary

Upstream recently tagged v0.4 which includes some important bug fixes. Update our port to the new version sourced from the public Bitbucket repo, and make myself maintainer while I'm here.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Please use devel/arcanist, or generate a diff with full context like it does, with svn diff -x -U9999 or git diff -U9999.

benchmarks/spp/Makefile
8

It makes no sense to set DISTVERSION while also setting PORTVERSION. See 5.2.2. Versions, DISTVERSION or PORTVERSION.

12

Wrong place in the Makefile. See 15. Order of Variables in Port Makefiles.

Full context diff, shuffle variable order and ditch DISTVERSION in favour of a custom variable to hold the commit hash used in the tarball directory name.

benchmarks/spp/pkg-descr
8–9

Only 1 WWW line.

lstewart marked 2 inline comments as done.

Checked with one of the authors. Preference is to reference the Bitbucket URL in pkg-descr if it can only list a single URL.

Thanks all for the feedback... I'm a bit rusty on working with ports. Will commit sometime this week with my src commit bit hat on and "review/approved by:" if no further feedback materialises.

Thanks all for the feedback... I'm a bit rusty on working with ports. Will commit sometime this week with my src commit bit hat on and "review/approved by:" if no further feedback materialises.

Is anything blocking this? Please do not hesitate to commit this.

This revision was not accepted when it landed; it landed in state Needs Review.Feb 25 2019, 7:33 PM
This revision was automatically updated to reflect the committed changes.