Bug 218497 - mark databases/mariadb100-client and 101-client unbroken on powerpc64
Summary: mark databases/mariadb100-client and 101-client unbroken on powerpc64
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: Bernard Spil
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-04-08 16:30 UTC by Mark Linimon
Modified: 2017-05-20 20:37 UTC (History)
0 users

See Also:
brnrd: maintainer-feedback+

Attachments
patch to mariadb100-server and mariadb101-server (1.47 KB, patch)
2017-04-08 16:30 UTC, Mark Linimon 		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 Mark Linimon freebsd_committer freebsd_triage 2017-04-08 16:30:10 UTC 
Created attachment 181604 [details]
patch to mariadb100-server and mariadb101-server

While doing a TRYBROKEN on powerpc64, it seems that the two mariadb client ports actually build.  The server ports, which are masterports, do not.

This patch moves the BROKEN_powerpc64 definition inside one of the conditional blocks.  I have verified it with portlint and make describe but not a build yet.

I would do this commit myself, but it's strange-looking enough that I think the maintainer should review it.
Comment 1 commit-hook freebsd_committer freebsd_triage 2017-05-20 20:36:55 UTC 
A commit references this bug:

Author: brnrd
Date: Sat May 20 20:36:43 UTC 2017
New revision: 441334
URL: https://svnweb.freebsd.org/changeset/ports/441334

Log:
  databases/mariadb101-client: Mark not-broken for client on power64

   - Move BROKEN_powerpc64 block to server-only block
   - Move non-option to non-option block (10.0 only)

  PR:		218497
  Submitted by:	linimon

Changes:
  head/databases/mariadb100-server/Makefile
  head/databases/mariadb101-server/Makefile
Comment 2 Bernard Spil freebsd_committer freebsd_triage 2017-05-20 20:37:49 UTC 
Thanks Mark! Sorry for the delay.