Bug 213321 - [NEW PORT] net-im/pecl-stomp2: PECL extension of stomp client
Summary: [NEW PORT] net-im/pecl-stomp2: PECL extension of stomp client
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: Torsten Zuehlsdorff
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-10-08 22:38 UTC by Danilo G. Baio
Modified: 2016-11-30 16:20 UTC (History)
1 user (show)

See Also:


Attachments
pecl-stomp2.shar (1.57 KB, text/plain)
2016-10-08 22:38 UTC, Danilo G. Baio
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Danilo G. Baio freebsd_committer freebsd_triage 2016-10-08 22:38:02 UTC
Created attachment 175543 [details]
pecl-stomp2.shar

pecl-stomp allows php applications to communicate with any
Stomp compliant Message Brokers through easy object oriented
and procedural interfaces.

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

This version is specific to php70 compared to net-im/pecl-stomp.

portlint: OK (looks fine.)
testport: 
	poudriere: i386,  9.3   (OK)
	poudriere: amd64, 9.3   (OK)
	poudriere: i386,  10.3  (OK)
	poudriere: amd64, 10.3  (OK)
	poudriere: i386,  12    (OK)
	poudriere: amd64, 12    (OK)
Comment 1 Torsten Zuehlsdorff freebsd_committer freebsd_triage 2016-11-17 11:46:56 UTC
Hello Danilo,

you defined "IGNORE_WITH_PHP=	55 56" and set a conflict to net-im/pecl-stomp2. The changelog [1] states, that it now supports PHP 7. This is the only change stated.

So: wouldn't it be better to update pecl-stomp to version 2? Or are the incompatible in any way? Did version 2 really did not build with PHP 5.5 and 5.6?

Also just a short notice: you did a good amount of poudriere-tests, which is very fine! But your list misses FreeBSD 11. It was there in PR 213320 - so i assume you just forgot to notice it, did you?

Greetings,
Torsten

[1] https://pecl.php.net/package-changelog.php?package=stomp&release=2.0.0
Comment 2 Danilo G. Baio freebsd_committer freebsd_triage 2016-11-17 23:33:46 UTC
Hi Torsten.

Thanks for your help.

2.0.0 release build's only on PHP 7.
1.0.9 release build's on PHP 55 and 56.

The same happens with these ports:

net/pecl-oauth
net/pecl-oauth2

math/pecl-stats
math/pecl-stats2

textproc/pecl-xdiff
textproc/pecl-xdiff2

mail/pecl-mailparse
mail/pecl-mailparse3 - open bug 213319

When I submitted this port, I didn't have a 11.0 release Jail.
Now I have. =)
I've just did another round of poudriere testport:

portlint: 
WARN: /usr/local/poudriere/data/.m/103-amd64-pdbaio/ref/usr/ports
/net-im/pecl-stomp2/distinfo: [1]: TIMESTAMP is over 30 days old
0 fatal errors and 1 warning found.

testport: 
	poudriere: i386,  9.3   (OK)
	poudriere: amd64, 9.3   (OK)
	poudriere: i386,  10.3  (OK)
	poudriere: amd64, 10.3  (OK)
	poudriere: i386,  11    (OK)
	poudriere: amd64, 11    (OK)
	poudriere: i386,  12    (OK)
	poudriere: amd64, 12    (OK)

Do you want me update this .shar to fix the TIMESTAMP on distinfo ?

Best regards.

Build logs here:

http://poudriere.bs2.com.br/poudriere/data/93-i386-pdbaio/2016-11-17_20h48m06s/logs/pecl-stomp2-2.0.0.log
http://poudriere.bs2.com.br/poudriere/data/93-amd64-pdbaio/2016-11-17_20h45m11s/logs/pecl-stomp2-2.0.0.log

http://poudriere.bs2.com.br/poudriere/data/103-i386-pdbaio/2016-11-17_20h42m49s/logs/pecl-stomp2-2.0.0.log
http://poudriere.bs2.com.br/poudriere/data/103-amd64-pdbaio/2016-11-17_20h40m13s/logs/pecl-stomp2-2.0.0.log

http://poudriere.bs2.com.br/poudriere/data/11-i386-pdbaio/2016-11-17_21h01m59s/logs/pecl-stomp2-2.0.0.log
http://poudriere.bs2.com.br/poudriere/data/11-amd64-pdbaio/2016-11-17_20h59m01s/logs/pecl-stomp2-2.0.0.log

http://poudriere.bs2.com.br/poudriere/data/12-i386-pdbaio/2016-11-17_20h55m02s/logs/pecl-stomp2-2.0.0.log
http://poudriere.bs2.com.br/poudriere/data/12-amd64-pdbaio/2016-11-17_20h51m00s/logs/pecl-stomp2-2.0.0.log
Comment 3 Torsten Zuehlsdorff freebsd_committer freebsd_triage 2016-11-18 14:33:07 UTC
Hello Danilo,

i see now. The changelog is a "little" misguiding. :D

No need to update the shar just because of the timestamp.

Greetings,
Torsten
Comment 4 commit-hook freebsd_committer freebsd_triage 2016-11-30 16:19:02 UTC
A commit references this bug:

Author: tz
Date: Wed Nov 30 16:17:59 UTC 2016
New revision: 427434
URL: https://svnweb.freebsd.org/changeset/ports/427434

Log:
  New port: net-im/pecl-stomp2

  pecl-stomp allows php applications to communicate with any
  Stomp compliant Message Brokers through easy object oriented
  and procedural interfaces.

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

  PR:           213321
  Submitted by: Danilo G. Baio <dbaio@bsd.com.br>
  Approved by:  mentors (implicit)

Changes:
  head/net-im/Makefile
  head/net-im/pecl-stomp2/
  head/net-im/pecl-stomp2/Makefile
  head/net-im/pecl-stomp2/distinfo
  head/net-im/pecl-stomp2/pkg-descr
Comment 5 Torsten Zuehlsdorff freebsd_committer freebsd_triage 2016-11-30 16:20:03 UTC
Committed, thanks! :)

I just added PHP_VER= 70. So the port will build clearly with PHP 7.0.