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

Note You need to log in before you can comment on or make changes to this bug.
Description Piotr Kubaj 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 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 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