Bug 233538 - devel/arachne-pnr: fix build with GCC-based architectures
Summary: devel/arachne-pnr: fix build with GCC-based architectures
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Mark Linimon
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-11-26 17:10 UTC by Piotr Kubaj
Modified: 2018-12-07 00:04 UTC (History)
2 users (show)

See Also:
bugzilla: maintainer-feedback? (jsorocil)

Attachments
patch (824 bytes, patch)
2018-11-26 17:10 UTC, Piotr Kubaj 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Piotr Kubaj freebsd_committer freebsd_triage 2018-11-26 17:10:51 UTC 
Created attachment 199574 [details]
patch

GCC requires including sys/param.h for u_int definitions.

Tested on powerpc64 and amd64.

Hardware sponsored by IntegriCloud.
Comment 1 Mark Linimon freebsd_committer freebsd_triage 2018-11-28 08:21:41 UTC 
On powerpc64 I get:

pkg-static: Unable to access file /wrkdirs/usr/ports/java/java3d/work/stage/usr/local/openjdk8/jre/lib/powerpc64/libj3dcore-ogl.so:No such file or directory
Comment 2 Mark Linimon freebsd_committer freebsd_triage 2018-11-28 08:23:30 UTC 
Sorry, wrong log.

On powerpc64 I get:

gmake[1]: *** No rule to make target '/usr/local/share/icebox/chipdb-lm4k.txt', needed by 'share/arachne-pnr/chipdb-lm4k.bin'.  Stop.
Comment 3 Johnny Sorocil 2018-11-30 07:39:53 UTC 
(In reply to Mark Linimon from comment #2)
Can you try to update devel/icestorm first and then build arachne-pnr?
Comment 4 Piotr Kubaj freebsd_committer freebsd_triage 2018-12-01 11:48:09 UTC 
(In reply to Mark Linimon from comment #2)
Built with Poudriere:
https://talos.anongoth.pl/data/powerpc64-default/2018-11-30_13h34m38s/logs/arachne-pnr-g20181021.log
Comment 5 Mark Linimon freebsd_committer freebsd_triage 2018-12-07 00:02:12 UTC 
Forcing a build of icestorm allows it to build with the patch.
Comment 6 commit-hook freebsd_committer freebsd_triage 2018-12-07 00:04:19 UTC 
A commit references this bug:

Author: linimon
Date: Fri Dec  7 00:04:10 UTC 2018
New revision: 486811
URL: https://svnweb.freebsd.org/changeset/ports/486811

Log:
  Archs using base gcc require including sys/param.h for u_int definitions.

  PR:		233538
  Submitted by:	Piotr Kubaj
  Reviewed by:	maintainer

Changes:
  head/devel/arachne-pnr/files/
  head/devel/arachne-pnr/files/patch-src_util.cc