Bug 251448
| Summary: | benchmarks/iperf: htonl argument issue on armv6 | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Ports & Packages | Reporter: | Martin Birgmeier <d8zNeCFG> | ||||
| Component: | Individual Port(s) | Assignee: | Po-Chuan Hsieh <sunpoet> | ||||
| Status: | Closed FIXED | ||||||
| Severity: | Affects Only Me | CC: | dim | ||||
| Priority: | --- | Flags: | bugzilla:
maintainer-feedback?
(sunpoet) |
||||
| Version: | Latest | ||||||
| Hardware: | Any | ||||||
| OS: | Any | ||||||
| Attachments: |
|
||||||
Comment on attachment 220059 [details]
patch for fixing compilation of benchmarks/iperf on armv6
It seems the patch does not work anymore, there are compile issues on both armv6 and i386 now.
-- Martin
See also bug 252280 for the i386 fixes. A commit references this bug: Author: sunpoet Date: Sun Jan 3 20:44:50 UTC 2021 New revision: 560245 URL: https://svnweb.freebsd.org/changeset/ports/560245 Log: Fix build on armv6 PR: 251448 Submitted by: Martin Birgmeier <d8zNeCFG@aon.at> Changes: head/benchmarks/iperf/files/patch-src-Settings.cpp Committed. Thanks! A commit references this bug: Author: sunpoet Date: Wed Jan 6 19:05:47 UTC 2021 New revision: 560558 URL: https://svnweb.freebsd.org/changeset/ports/560558 Log: MFH: r560244 r560245 Fix build on i386 PR: 252280 Reported by: Victor Sudakov <vas@sibptus.ru> Submitted by: dim Fix build on armv6 PR: 251448 Submitted by: Martin Birgmeier <d8zNeCFG@aon.at> Changes: _U branches/2021Q1/ branches/2021Q1/benchmarks/iperf/files/patch-src-Settings.cpp branches/2021Q1/benchmarks/iperf/files/patch-src-histogram.c |
Created attachment 220059 [details] patch for fixing compilation of benchmarks/iperf on armv6 Scenario: - Raspberry Pi 1B+ or qemu-user-static (compiling natively or via qemu) - FreeBSD 13.0-CURRENT #0 r367245M - Trying to compile benchmarks/iperf Result: - The build fails with a type error in the arguments to htonl in src/Settings.cpp, regardless of compiling on the RPI-1B+ or via qemu Note: - On amd64 and i386 running release/12.2, the build succeeds A (lightly tested) patch is attached, to be placed in benchmarks/iperf/files. -- Martin