Bug 212684 - devel/php70-APCu Port not available on package server
Summary: devel/php70-APCu Port not available on package server
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: Guido Falsi
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-09-14 11:58 UTC by joshruehlig
Modified: 2016-09-14 19:41 UTC (History)
3 users (show)

See Also:
madpilot: maintainer-feedback+


Attachments
Force build with php70 as php56 is default in ports (451 bytes, patch)
2016-09-14 13:38 UTC, Daniel Ylitalo
daniel: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description joshruehlig 2016-09-14 11:58:23 UTC
This package is not available from any package servers. It is not being built because PHP5.6 is currently the default while this port requires PHP7.0.

I am not sure the best solution but I noticed the databases/php70-redis port has "PHP_DEFAULT= 7.0" and it is available from the package servers. I then noticed I authored this port, so I'm not 100% sure if I did it correctly but somehow it is working.
Comment 1 joshruehlig 2016-09-14 12:06:58 UTC
I see  that 'PHP_DEFAULT=' being a hack since it lies to 'Mk/Uses/php.mk'

Maybe APCu needs to be added to PHP70's list of available extensions? While we're at it, we should probably fix databases/php70-redis as well.
Comment 2 Daniel Ylitalo 2016-09-14 13:26:43 UTC
I'll upload a patch in a moment, just going to try to build it with the new line added.

Regarding changing the extensions port, I doubt that is going to happened as it only contains stock php modules provided by php themselfes.

Perhaps someone can create a "php70-thirdparty-extensions" meta port which can contain all pecl modules and others.
Comment 3 Daniel Ylitalo 2016-09-14 13:38:51 UTC
Created attachment 174780 [details]
Force build with php70 as php56 is default in ports
Comment 4 joshruehlig 2016-09-14 14:03:05 UTC
Ahh, I didn't know that those were only stock extensions.
Comment 5 commit-hook freebsd_committer freebsd_triage 2016-09-14 19:14:07 UTC
A commit references this bug:

Author: madpilot
Date: Wed Sep 14 19:13:07 UTC 2016
New revision: 422151
URL: https://svnweb.freebsd.org/changeset/ports/422151

Log:
  Add PHP_DEFAULT=7.0 so that a package is build on the cluster with
  default ports configuration.

  PR:		212684
  Submitted by:	joshruehlig at gmail.com
  Approved by:	Daniel Ylitalo <daniel at blodan.se> (maintainer)

Changes:
  head/devel/php70-APCu/Makefile
Comment 6 Guido Falsi freebsd_committer freebsd_triage 2016-09-14 19:14:26 UTC
Committed, with a minor change, there is no need to bump PORTREVISION in this case. The change does not cause any difference for people already building this and no package is available on the cluster so, no need to regenerate it.

Thanks!
Comment 7 joshruehlig 2016-09-14 19:18:04 UTC
Awesome thanks guys!
This will help out a bunch of people using my ownCloud tutorial on the FreeNAS forums, because I say to "pkg install php70-APCu". =]
Comment 8 Guido Falsi freebsd_committer freebsd_triage 2016-09-14 19:41:44 UTC
(In reply to joshruehlig from comment #7)
> Awesome thanks guys!
> This will help out a bunch of people using my ownCloud tutorial on the
> FreeNAS forums, because I say to "pkg install php70-APCu". =]

Just for the sake of completeness, please note that FreeBSD releases by default use the quarterly branch packages, which do not include this port, because it was added to the tree after the last quarterly was branched. So those instructions will not work out of the box with FreeBSD releases.

This will be automatically rectified around October 1st when a new quarterly will be branched, including this port and this commit.