Bug#821715: PHP 7.0 transition - PHP bindings removed and uploaded to DELAYED/2

2016-06-16 Thread Ondřej Surý 
On Thu, Jun 16, 2016, at 09:15, Torsten Landschoff wrote:
> On 06/16/2016 08:36 AM, Ondřej Surý wrote:
> > Hi Torsten,
> >
> > attached are patches on top of latest unstable swig.
> >
> > TL;DR it's the php5-* in d/control that prevents the transition;
> >
> *blush* Dumb me. I was sure that the build depends had php-dev or
> similiar (like libperl-dev, python-dev, ...).
> No idea why I hardwired php5.

Nah, no need to feel silly. There was no "php-dev" before packaging
rewrite that happened with switch to PHP 7.0, so php5-dev *was* the
correct value.

Cheers,
-- 
Ondřej Surý 
Knot DNS (https://www.knot-dns.cz/) – a high-performance DNS server
Knot Resolver (https://www.knot-resolver.cz/) – secure, privacy-aware,
fast DNS(SEC) resolver
Vše pro chleba (https://vseprochleba.cz) – Potřeby pro pečení chleba
všeho druhu

Bug#821715: PHP 7.0 transition - PHP bindings removed and uploaded to DELAYED/2

2016-06-16 Thread Ondřej Surý 
Anyway feel free to upload 3.0.8-1 with the PHP remove directly to
unstable. That would actually help speed up the process as of now :)

Cheers,
-- 
Ondřej Surý 
Knot DNS (https://www.knot-dns.cz/) – a high-performance DNS server
Knot Resolver (https://www.knot-resolver.cz/) – secure, privacy-aware,
fast DNS(SEC) resolver
Vše pro chleba (https://vseprochleba.cz) – Potřeby pro pečení chleba
všeho druhu

On Thu, Jun 16, 2016, at 09:16, Ondřej Surý wrote:
> On Thu, Jun 16, 2016, at 09:15, Torsten Landschoff wrote:
> > On 06/16/2016 08:36 AM, Ondřej Surý wrote:
> > > Hi Torsten,
> > >
> > > attached are patches on top of latest unstable swig.
> > >
> > > TL;DR it's the php5-* in d/control that prevents the transition;
> > >
> > *blush* Dumb me. I was sure that the build depends had php-dev or
> > similiar (like libperl-dev, python-dev, ...).
> > No idea why I hardwired php5.
> 
> Nah, no need to feel silly. There was no "php-dev" before packaging
> rewrite that happened with switch to PHP 7.0, so php5-dev *was* the
> correct value.
> 
> Cheers,
> -- 
> Ondřej Surý 
> Knot DNS (https://www.knot-dns.cz/) – a high-performance DNS server
> Knot Resolver (https://www.knot-resolver.cz/) – secure, privacy-aware,
> fast DNS(SEC) resolver
> Vše pro chleba (https://vseprochleba.cz) – Potřeby pro pečení chleba
> všeho druhu

Bug#821715: PHP 7.0 transition - PHP bindings removed and uploaded to DELAYED/2

2016-06-16 Thread Torsten Landschoff 
On 06/16/2016 08:36 AM, Ondřej Surý wrote:
> Hi Torsten,
>
> attached are patches on top of latest unstable swig.
>
> TL;DR it's the php5-* in d/control that prevents the transition;
>
*blush* Dumb me. I was sure that the build depends had php-dev or
similiar (like libperl-dev, python-dev, ...).
No idea why I hardwired php5.

Greetings, Torsten

Bug#821715: PHP 7.0 transition - PHP bindings removed and uploaded to DELAYED/2

2016-06-16 Thread Ondřej Surý 
Hi Torsten,

attached are patches on top of latest unstable swig.

TL;DR it's the php5-* in d/control that prevents the transition;

Cheers,
-- 
Ondřej Surý 
Knot DNS (https://www.knot-dns.cz/) – a high-performance DNS server
Knot Resolver (https://www.knot-resolver.cz/) – secure, privacy-aware,
fast DNS(SEC) resolver
Vše pro chleba (https://vseprochleba.cz) – Potřeby pro pečení chleba
všeho druhu

On Thu, Jun 16, 2016, at 00:39, Torsten Landschoff wrote:
> Hi Ondrej,
> 
> On 06/15/2016 10:24 AM, Ondřej Surý wrote:
> > in an effort to finally finish PHP 7.0 transition started in April, I
> > removed PHP bindings from swig as it doesn't look like swig will support
> > PHP 7.0 anytime soon and also from the remaining packages using swig to
> > build PHP bindings.
> not sure if there is a misunderstanding: SWIG can generate bindings for
> all supported languages (which does not currently include PHP7)
> regardless of configure arguments.
> AFAICT you just passed --without-php to configure in the new SWIG
> release. The resulting binary will still allow creating PHP bindings
> (albeit broken).
> 
> So I don't see how this helps in the transition - looks like I am
> missing something!?
> 
> tracker.debian.org states
> > This package is part of the ongoing testing transition known as php7.0
> > . Please
> > avoid uploads unrelated to this transition
> which is why I uploaded 3.0.8 to experimental only to not disturb the
> php7.0 transition hoping that upstream will update the PHP generation.
> So, sorry. Just uploading swig dropping the php dependencies is
> something I should have done to help I guess...
> 
> Greetings, Torsten
> 
From 0727ffe95aa59b6cf250d5e28130809166764d79 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ond=C5=99ej=20Sur=C3=BD?= 
Date: Wed, 15 Jun 2016 10:17:14 +0200
Subject: [PATCH 2/2] swig 3.0.7-2.1

---
 debian/changelog | 7 +++
 1 file changed, 7 insertions(+)

diff --git a/debian/changelog b/debian/changelog
index 0ff4e97..1c6da16 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
+swig (3.0.7-2.1) unstable; urgency=medium
+
+  * Remove swig PHP support as swig doesn't support PHP 7.0 yet
+(Closes: #821715)
+
+ -- Ondřej Surý   Wed, 15 Jun 2016 10:17:00 +0200
+
 swig (3.0.7-2) unstable; urgency=medium
 
   * Reintroduce symlinks swig and ccache-swig for the default version of
-- 
2.1.4

From 5eb3ea6ca9df7958bc1d3bbeedb12a6112ba5635 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ond=C5=99ej=20Sur=C3=BD?= 
Date: Wed, 15 Jun 2016 10:14:59 +0200
Subject: [PATCH 1/2] Remove swig PHP support as swig doesn't support PHP 7.0
 yet (Closes: #821715)

---
 debian/control   | 6 ++
 debian/rules | 2 +-
 debian/swig3.0-examples.examples | 1 -
 3 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/debian/control b/debian/control
index 0f686b2..f4f63f8 100644
--- a/debian/control
+++ b/debian/control
@@ -16,8 +16,6 @@ Build-Depends: dpkg-dev (>= 1.16.1~),
libchicken-dev [!m68k !mips !mipsel],
libperl-dev,
libpcre3-dev,
-   php5-cgi,
-   php5-dev,
python-dev,
ruby,
ruby-dev,
@@ -33,7 +31,7 @@ Conflicts: swig2.0
 Replaces: swig2.0
 Description: Generate scripting interfaces to C/C++ code
  SWIG is a compiler that makes it easy to integrate C and C++ code
- with other languages including Perl, PHP, Tcl, Ruby, Python, Java,
+ with other languages including Perl, Tcl, Ruby, Python, Java,
  Guile, Mzscheme, Chicken, OCaml, Pike, and C#.
  .
  Swig takes a set of ANSI C/C++ declarations and generates an
@@ -48,7 +46,7 @@ Multi-Arch: foreign
 Suggests: swig3.0-examples, swig3.0-doc
 Description: Generate scripting interfaces to C/C++ code
  SWIG is a compiler that makes it easy to integrate C and C++ code
- with other languages including Perl, PHP, Tcl, Ruby, Python, Java, 
+ with other languages including Perl, Tcl, Ruby, Python, Java, 
  Guile, Mzscheme, Chicken, OCaml, Pike, and C#.
  .
  Swig takes a set of ANSI C/C++ declarations and generates an 
diff --git a/debian/rules b/debian/rules
index 1ae0cb1..e953341 100755
--- a/debian/rules
+++ b/debian/rules
@@ -18,6 +18,7 @@ include /usr/share/dpkg/buildflags.mk
 MAKEOPTS =
 CONFIGURE_ARGS := \
 	--prefix=/usr --mandir=/usr/share/man	\
+	--without-php	\
 	--with-tclconfig=/usr/lib			\
 	--with-python=/usr/bin/python			\
 	--with-ruby=/usr/bin/ruby			\
@@ -68,7 +69,6 @@ configure-stamp: $(QUILT_STAMPFN)
 	dh_testdir
 	dh_autoreconf ./autogen.sh
 	mkdir -p builddir
-	PHP4INC="`php-config --includes`" \
 	cd builddir && ../configure $(CONFIGURE_ARGS)
 	touch configure-stamp
 
diff --git a/debian/swig3.0-examples.examples b/debian/swig3.0-examples.examples
index 25264d3..b3e4ea2 100644
--- a/debian/swig3.0-examples.examples
+++

Bug#821715: PHP 7.0 transition - PHP bindings removed and uploaded to DELAYED/2

2016-06-15 Thread Torsten Landschoff 
Hi Ondrej,

On 06/15/2016 10:24 AM, Ondřej Surý wrote:
> in an effort to finally finish PHP 7.0 transition started in April, I
> removed PHP bindings from swig as it doesn't look like swig will support
> PHP 7.0 anytime soon and also from the remaining packages using swig to
> build PHP bindings.
not sure if there is a misunderstanding: SWIG can generate bindings for
all supported languages (which does not currently include PHP7)
regardless of configure arguments.
AFAICT you just passed --without-php to configure in the new SWIG
release. The resulting binary will still allow creating PHP bindings
(albeit broken).

So I don't see how this helps in the transition - looks like I am
missing something!?

tracker.debian.org states
> This package is part of the ongoing testing transition known as php7.0
> . Please
> avoid uploads unrelated to this transition
which is why I uploaded 3.0.8 to experimental only to not disturb the
php7.0 transition hoping that upstream will update the PHP generation.
So, sorry. Just uploading swig dropping the php dependencies is
something I should have done to help I guess...

Greetings, Torsten