Bug 215378

Summary: [NEW PORT] security/pecl-ssh2-1: PECL extension to the libssh2 library (php 7)
Product: Ports & Packages Reporter: Alexander Ushakov <alexander>
Component: Individual Port(s)Assignee: Danilo G. Baio <dbaio>
Status: Closed FIXED    
Severity: Affects Many People CC: dbaio, miwi
Priority: --- Flags: dbaio: maintainer-feedback+
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
pecl-ssh2-1.shar none

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)