New port net-p2p/mt-recorder: Rust based hashing program for the Bitmark Property System
Needs ReviewPublic
Actions

Authored by hsw_bitmark.com on Mar 3 2020, 7:06 AM.

Details

Reviewers

araujo

Group Reviewers

Contributor Reviewers (ports)

Summary

This is a Rust based hashing program for the Bitmark Property System
(net-p2p/bitmark).

It has two flavors the default which compiles with simd for modern
CPUs and mt-recorder@nosimd for older CPUs without the AVX feature

Also update UIDs and GIDs for mt-recorder user/group

Test Plan

poudriere (12.1-RELEASE amd64)

poudriere testport -j 12_1-amd64 net-p2p/mt-recorder
poudriere testport -j 12_1-amd64 net-p2p/mt-recorder@nosimd

Diff Detail

Repository

rP FreeBSD ports repository

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 29751
Build 27589: arc lint + arc unit

Event Timeline

hsw_bitmark.com created this revision.Mar 3 2020, 7:06 AM

Herald added a subscriber: mat. · View Herald TranscriptMar 3 2020, 7:06 AM

Harbormaster completed remote builds in B29750: Diff 69105.Mar 3 2020, 7:06 AM

upload port files

Harbormaster completed remote builds in B29751: Diff 69106.Mar 3 2020, 7:07 AM

tobik added a subscriber: tobik.Mar 8 2020, 4:59 PM

tobik added inline comments.

net-p2p/mt-recorder/Makefile
14–15	Wrong order. See https://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/porting-order.html Also please do not lock it to llvm80 and use ${LLVM_DEFAULT} here.
17	This should probably be an option not a flavor. When compiled with SIMD support the argonautica crate passes `-march=native` to the build [1], which means the package would be specific to the package builder hardware, i.e., the package would be broken on a large number of systems [2]. It is a waste of resources to build such packages. [1] https://github.com/bcmyers/argonautica/blob/v0.2.0/argonautica-rs/build.rs#L67 [2] https://www.freebsd.org/doc/en_US.ISO8859-1/books/porters-handbook/dads-arch.html
22	USES=pkgconfig is already implied by CARGO_CRATES=pkgconfig. Please remove it.
155	Nop with USES=cargo ports. Please remove it. If the binary needs to be stripped put a ${STRIP_CMD} in post-install.
163–168	There is no reason to use `+=` anywhere in this port.
179	No need for the quotes.

update to address the problems in review by @tobik

Harbormaster completed remote builds in B29850: Diff 69320.Mar 9 2020, 8:25 AM

hsw_bitmark.com added a reviewer: Contributor Reviewers (ports).Jun 16 2020, 4:13 AM

mikael added a subscriber: mikael.Jun 16 2020, 4:18 PM

mikael added inline comments.Jun 16 2020, 4:21 PM

net-p2p/mt-recorder/Makefile
20	simd exists on arm, ppc. Is it limited to amd64?

My present understanding is that SIMD exists on certain CPUs and that the way the Rust compiler works in that it does a native build
if SIMD feature is enabled which would mean that the build machine would have to support SIMD. Originally I had it as a flavour so
both kinds would have been built. I was recommended to convert to an option defaulted off so that build machines without SIMD
would not fail the SIMD version

linimon retitled this revision from New port net-p2p/mt-recorder (v0.5.3) to New port net-p2p/mt-recorder: Rust based hashing program for the Bitmark Property System.Jun 26 2020, 8:14 AM