Bug 211382 - [NEW PORT]: devel/php70-APCu
Summary: [NEW PORT]: devel/php70-APCu
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: Guido Falsi
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-07-26 14:05 UTC by Daniel Ylitalo
Modified: 2016-09-14 12:06 UTC (History)
2 users (show)

See Also:


Attachments
Port shar file (1.44 KB, text/plain)
2016-07-26 14:05 UTC, Daniel Ylitalo
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Ylitalo 2016-07-26 14:05:01 UTC
Created attachment 172999 [details]
Port shar file

php 7.0 compatible version of pecl-APCu
Comment 1 Guido Falsi freebsd_committer freebsd_triage 2016-08-01 09:42:55 UTC
I'm testing your submission.
Comment 2 commit-hook freebsd_committer freebsd_triage 2016-08-01 16:32:15 UTC
A commit references this bug:

Author: madpilot
Date: Mon Aug  1 16:31:47 UTC 2016
New revision: 419442
URL: https://svnweb.freebsd.org/changeset/ports/419442

Log:
  APCu - APC User Cache

  WWW: http://pecl.php.net/package/apcu

  PR:		211382
  Submitted by:	daniel@blodan.se

Changes:
  head/devel/Makefile
  head/devel/php70-APCu/
  head/devel/php70-APCu/Makefile
  head/devel/php70-APCu/distinfo
  head/devel/php70-APCu/pkg-descr
Comment 3 Guido Falsi freebsd_committer freebsd_triage 2016-08-01 16:33:27 UTC
New port added.

You can be added to the Additional FreeBSD Contributors [1]

Is it Ok if I add you as Daniel Ylitalo <daniel@blodan.se>?
Comment 4 Daniel Ylitalo 2016-08-01 16:37:16 UTC
Sure! :)
Comment 5 commit-hook freebsd_committer freebsd_triage 2016-08-01 16:47:17 UTC
A commit references this bug:

Author: madpilot
Date: Mon Aug  1 16:47:09 UTC 2016
New revision: 49204
URL: https://svnweb.freebsd.org/changeset/doc/49204

Log:
  Add Daniel Ylitalo for devel/php70-APCu.

  PR:		211382

Changes:
  head/en_US.ISO8859-1/articles/contributors/contrib.additional.xml
Comment 6 Guido Falsi freebsd_committer freebsd_triage 2016-08-01 16:47:54 UTC
Added.

Thanks!
Comment 7 joshruehlig 2016-09-14 11:25:13 UTC
This package does not seem to be building.

I'm wondering if "PHP_DEFAULT= 7.0" should be set? I noticed this was in the php70-redis port, which does properly.
Comment 8 Guido Falsi freebsd_committer freebsd_triage 2016-09-14 11:44:25 UTC
(In reply to joshruehlig from comment #7)

Please do not reply to closed bug reports, but file a new one. Especally when the issue is not directly related.

This bug report was a new port proposal which has been added to the tree. You now have problems building that package, it's another issue.

Also, maybe even more important, such replies are easily lost, it's just a coincidence I noticed this one.

> This package does not seem to be building.
> 
> I'm wondering if "PHP_DEFAULT= 7.0" should be set? I noticed this was in the
> php70-redis port, which does properly.

The reason an official package for this port is not being build is exactly that the cluster uses php 5.6 by default and this port is not compatible.

The PHP_DEFAULT= 7.0 flag is really a user flag and should not be used by a port, adding it is a kind of a hack.

Please file a new bug report reporting this same issue, so it can be correctly assigned, the maintainer can send feedback and we can properly track the issue.

You can add me to the CC list of the new bug, so I can assign it too myself and follow this issue.
Comment 9 joshruehlig 2016-09-14 11:48:13 UTC
(In reply to Guido Falsi from comment #8)
Thanks for the quick reply, OK sounds good I'll create a new bug report.

Also, you said I had problems building the port. It actually builds fine for me, but it is not available on the package server.
Comment 10 joshruehlig 2016-09-14 12:05:44 UTC
Hmm, so I see what you mean by 'PHP_DEFAULT=' being a hack since it lies to 'Mk/Uses/php.mk'

Is there a better way to do this? 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 11 joshruehlig 2016-09-14 12:06:25 UTC
sorry, I commented on the wrong bug report, I'll post my last comment to the new one as well...