5.3/5.4 got upgraded via @pkgpath when they stopped being the "main" ports version of php. 5.5 isn't going to be installed via dependencies any more so people who installed it will have asked for it specifically, so it doesn't make much sense to auto update it via @pkgpath, so adding a quirk to say that it's obsolete is going to be enough and is fairly simple.

--
 Sent from a phone, apologies for poor formatting.



On 28 August 2017 2:56:44 pm Martijn van Duren <openbsd+po...@list.imperialat.at> wrote:

On 08/28/17 14:02, Stuart Henderson wrote:
On 2017/08/27 19:37, Martijn van Duren wrote:
On 08/27/17 19:31, Robert Nagy wrote:
Yeah let's keep php as is for 6.2, you can remove 5.5 and add 7.1 but that's it.
After unlock we can unleash hell on the tree.

Then I'd like to propose to just remove 5.5.

Adding 7.1 would either backporting extra fixes from my patch, or keep it close
to the way 7.0 is right now (including some ugly hacks in Makefile.inc), which
would further complicate things for me later on.

OK?

OK as far as it goes, but it needs quirks parts too.

I'm not quite clear on the quirks part:
The removal of PHP 5.3 and 5.4 don't seem to have any quirks entry
(based on both Quirks.pm and ports-changes@) and Quirks.pm doesn't seem
to have any transition based upon the removal of a specific version. So
what would these entries look like (if still applicable), and should it
be based on a removal (obsolete_reason), or a transition to 5.6
(stem_extensions)?

martijn@

On (2017-08-27 13:50), Stuart Henderson wrote:
I would be ok with killing 5.5 and adding 7.1 but think it's the wrong time
in the release cycle for anything more complex.
--
 Sent from a phone, apologies for poor formatting.



Index: Makefile
===================================================================
RCS file: /cvs/ports/lang/php/Makefile,v
retrieving revision 1.11
diff -u -p -r1.11 Makefile
--- Makefile    28 Apr 2016 18:19:23 -0000      1.11
+++ Makefile    27 Aug 2017 17:36:28 -0000
@@ -1,7 +1,6 @@
 # $OpenBSD: Makefile,v 1.11 2016/04/28 18:19:23 sthen Exp $

 SUBDIR =
-SUBDIR += 5.5
 SUBDIR += 5.6
 SUBDIR += 7.0

Index: Makefile.inc
===================================================================
RCS file: /cvs/ports/lang/php/Makefile.inc,v
retrieving revision 1.96
diff -u -p -r1.96 Makefile.inc
--- Makefile.inc        24 Aug 2017 10:03:25 -0000      1.96
+++ Makefile.inc        27 Aug 2017 17:36:28 -0000
@@ -78,11 +78,7 @@ CONFIGURE_ARGS+=     --enable-shared \
                        --enable-sqlite-utf8 \
                        --with-sqlite3 \
                        --program-suffix=-${PV}
-
-# readline is broken in PHP-5.3
-.if ${PV} != 5.3
 CONFIGURE_ARGS +=      --with-readline
-.endif

 # apache module
 CONFIGURE_ARGS+=       --with-apxs2=${LOCALBASE}/sbin/apxs2
@@ -221,10 +217,8 @@ MULTI_PACKAGES+=   -odbc
 COMMENT-odbc=          odbc database access extensions for php5
 CONFIGURE_ARGS+=        --with-iodbc=shared,${LOCALBASE}
 LIB_DEPENDS-odbc=      databases/iodbc
-WANTLIB-odbc=          iodbc>=2 pthread
-.if ${PV} != "5.3"
-WANTLIB-odbc+=         iodbcinst
-.endif
+WANTLIB-odbc=          iodbc>=2 pthread \
+                       iodbcinst

 # pcntl
 MULTI_PACKAGES+=       -pcntl
@@ -347,7 +341,6 @@ PHPXS_SUBST+= -e 's,${i},${${i}},'
 WANTLIB-main+=         c crypto iconv intl lzma m pthread ssl xml2>=8 z
 WANTLIB-main+=         ncurses readline ${COMPILER_LIBCXX}

-# php 5.4/5.5 : WANTLIB-main += ${COMPILER_LIBCXX}
 WANTLIB-fastcgi=       ${WANTLIB-main}
 LIB_DEPENDS-main=      devel/gettext \
                        textproc/libxml
@@ -384,11 +377,7 @@ pre-configure:
 MODULE_NAME-${i}=      ${i}
 DESCR-${i}=            ${.CURDIR}/../files/DESCR-${i}
 PKGNAME-${i}=          php-${i}-${V}
-.if ${V:M5.4*}
-PKGSPEC-${i}=          php-${i}->=5.4,<5.5
-.elif ${V:M5.5*}
-PKGSPEC-${i}=          php-${i}->=5.5,<5.6
-.elif ${V:M5.6*}
+.if ${V:M5.6*}
 PKGSPEC-${i}=          php-${i}->=5.6,<5.7
 .elif ${V:M7.0*}
 PKGSPEC-${i}=          php-${i}->=7.0,<7.1



Reply via email to