Bug 210672 - java/aparapi: Remove i386 option
Summary: java/aparapi: Remove i386 option
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: i386 Any
: --- Affects Only Me
Assignee: Koop Mast
URL:
Keywords: easy, needs-qa, patch
Depends on:
Blocks:
 
Reported: 2016-06-28 20:20 UTC by Johannes M Dieterich
Modified: 2016-09-25 21:41 UTC (History)
2 users (show)

See Also:
koobs: maintainer-feedback+
kwm: merge-quarterly-


Attachments
svn diff in java/aparapi (313 bytes, patch)
2016-06-28 20:20 UTC, Johannes M Dieterich
koobs: maintainer-approval? (jmd)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Johannes M Dieterich freebsd_committer 2016-06-28 20:20:06 UTC
Created attachment 171921 [details]
svn diff in java/aparapi

Attached is a patch to remove i386 from the list of viable platforms for java/aparapi. The compilation causes OpenJDK8 on 10.1/i386 to dump core with an internal error.

Also, i386 does not (and most likely will not) support OpenCL, so this is not a big loss.
Comment 1 John Marino freebsd_committer 2016-07-22 03:52:51 UTC
add maintainer to CC list.
Comment 2 Johannes M Dieterich freebsd_committer 2016-07-22 04:16:34 UTC
I am the maintainer of this port, I sent the patch.
Comment 3 Kubilay Kocak freebsd_committer freebsd_triage 2016-07-22 04:18:12 UTC
Apologies Dieterich. Please:

Set maintainer-approval to + on attachments for ports you are MAINTAINER of. Attachment -> Details -> maintainer-approval [+]

Confirm this change passes QA (portlint, poudriere)
Comment 4 Johannes M Dieterich freebsd_committer 2016-07-22 04:34:12 UTC
No problem, I shall flag submissions in the future, thanks for letting me know!

Passes portlint and current port already passes poudriere on all other platforms.
Comment 5 Johannes M Dieterich freebsd_committer 2016-07-28 00:43:41 UTC
OK, re-ran poudriere with the latest version on 9.3 amd64, 10.1 amd64, and 10.3 amd64. All worked. Works w/o poudriere on my CURRENT.

Please let me know what is missing so that the patch can go into the tree.
Comment 6 commit-hook freebsd_committer 2016-09-25 21:41:14 UTC
A commit references this bug:

Author: kwm
Date: Sun Sep 25 21:40:54 UTC 2016
New revision: 422757
URL: https://svnweb.freebsd.org/changeset/ports/422757

Log:
  Remove i386 from the viable platforms list.

  The compiliation of aparapi causes OpenJDK8 on 10.1/i386 to dump core with a
  internal error.

  PR:		210672
  Submitted by:	maintainer@

Changes:
  head/java/aparapi/Makefile