Bug 215378 - [NEW PORT] security/pecl-ssh2-1: PECL extension to the libssh2 library (php 7)
Summary: [NEW PORT] security/pecl-ssh2-1: PECL extension to the libssh2 library (php 7)
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: Danilo G. Baio
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-12-18 13:50 UTC by Alexander Ushakov
Modified: 2017-05-17 23:09 UTC (History)
2 users (show)

See Also:
dbaio: maintainer-feedback+


Attachments
pecl-ssh2-1.shar (1.78 KB, text/plain)
2016-12-20 23:33 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 Alexander Ushakov 2016-12-18 13:50:38 UTC
Current version (0.13) of security/pecl-ssh2 doesn't support php7 though there is already 1.0 version with such support - see project homepage http://pecl.php.net/package/ssh2

As new version (1.0) doesn't work with php5 it may be needed to split port to the old and the new one.
Comment 1 Danilo G. Baio freebsd_committer freebsd_triage 2016-12-20 23:33:31 UTC
Created attachment 178163 [details]
pecl-ssh2-1.shar

I submitted other pecl ports like this but I have a doubt about the name of this one.

security/pecl-ssh2-1 | security/pecl-ssh2-php7 ? Any idea ?

The patch attached hasn't PACKAGENAMESUFFIX, so packages will be like this:

pecl-ssh2-1.0.txz   -  security/pecl-ssh2-1  
pecl-ssh2-0.13.txz  -  security/pecl-ssh2

I don't know if this is the right approach.


Could you change this bug title for: 
[NEW PORT] security/pecl-ssh2-1: PECL extension to the libssh2 library

PHP bindings to the libssh2 library which provide access to resources
(shell, remote exec, tunneling, file transfer) on a remote machine using a
secure cryptographic transport.

WWW: https://pecl.php.net/package/ssh2

This version is specific to php7 compared to security/pecl-ssh2

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,  11    (OK)
	poudriere: amd64, 11    (OK)	
	poudriere: i386,  12    (OK)
	poudriere: amd64, 12    (OK)
Comment 2 Alexander Ushakov 2016-12-23 10:05:52 UTC
Thank you for your answer. I've changed bug ticket name.
As about name, security/pecl-ssh2-1 looks good for me
Comment 3 Danilo G. Baio freebsd_committer freebsd_triage 2017-05-12 16:49:19 UTC
I'll take it.
Comment 4 commit-hook freebsd_committer freebsd_triage 2017-05-17 23:03:22 UTC
A commit references this bug:

Author: dbaio
Date: Wed May 17 23:02:42 UTC 2017
New revision: 441122
URL: https://svnweb.freebsd.org/changeset/ports/441122

Log:
  security/pecl-ssh2: Update to 1.0

  - Create security/pecl-ssh2-0
  - Add UPDATING note

  security/pecl-ssh2 (v1.0)  --> PHP 7 only.
  security/pecl-ssh2-0 (v0.13)  --> PHP 5 compatible.

  Changes: https://pecl.php.net/package-changelog.php?package=ssh2&release=1.0

  PR:		215378
  Reported by:	Alexander Ushakov <alexander@polyvizor.com>
  Approved by:	garga (mentor)
  Differential Revision:	https://reviews.freebsd.org/D10702

Changes:
  head/UPDATING
  head/security/pecl-ssh2/Makefile
  head/security/pecl-ssh2/distinfo
  head/security/pecl-ssh2-0/
  head/security/pecl-ssh2-0/Makefile
Comment 5 Danilo G. Baio freebsd_committer freebsd_triage 2017-05-17 23:09:37 UTC
Committed, thanks.

security/pecl-ssh2   (v1.0)
security/pecl-ssh2-0 (v0.13)