Bug 201981 - [MAINTAINER] sysutils/b2sum: build fix
Summary: [MAINTAINER] sysutils/b2sum: build fix
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Bartek Rutkowski
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-07-30 01:04 UTC by Matthew Rezny
Modified: 2015-08-06 17:41 UTC (History)
1 user (show)

See Also:


Attachments
sysutils/b2sum build fix (1.10 KB, patch)
2015-07-30 01:04 UTC, Matthew Rezny
rezny: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Matthew Rezny freebsd_committer freebsd_triage 2015-07-30 01:04:20 UTC
Created attachment 159377 [details]
sysutils/b2sum build fix

Build with 9.x's older Clang fails in SSE path (default on amd64) without -flax-vector-conversions. Ideally the addition of that CFLAG could be conditional upon CHOSEN_COMPILER_TYPE == "clang" && COMPILER_VERSION < 34, but those variables cannot be checked without a USES=compiler, which we only have in the OPENMP case, but in that case we know CHOSEN_COMPILER_TYPE == "gcc". Or, at least it should... compiler:openmp selects GCC on 10.x but selects Clang on 9.x. So, just add -flax-vector-conversions to CFLAGS for the whole SSE path, it should be harmless for newer compilers.

While here, update the distfile. Upstream rolled a new one dated a few days later. The previous distfile had a few binaries in it.
Comment 1 commit-hook freebsd_committer freebsd_triage 2015-08-06 17:41:06 UTC
A commit references this bug:

Author: robak
Date: Thu Aug  6 17:40:09 UTC 2015
New revision: 393664
URL: https://svnweb.freebsd.org/changeset/ports/393664

Log:
  sysutils/b2sum: update 0.0.20150529 -> 0.0.20150531

  - Fix build on 9.x systems

  PR:		201981
  Submitted by:	Matthew Rezny <matthew@reztek.cz> (maintainer)

Changes:
  head/sysutils/b2sum/Makefile
  head/sysutils/b2sum/distinfo
Comment 2 Bartek Rutkowski freebsd_committer freebsd_triage 2015-08-06 17:41:39 UTC
Committed with minor change, thanks!