Bug#861637: sassphp: src:sassphp explicitly creates a php7.0 binary package

2017-05-04 Thread Ondřej Surý
Yes, correct. Also you can kill the warnings and $(mkdir should be $(shell mkdir I need to fix that in tideways package... Ondřej On 4 May 2017 21:00:21 Nish Aravamudan wrote: On 04.05.2017 [10:40:50 +0200], Ondřej Surý wrote: Source: sassphp Followup-For:

Bug#861637: sassphp: src:sassphp explicitly creates a php7.0 binary package

2017-05-04 Thread Nish Aravamudan
On 04.05.2017 [10:40:50 +0200], Ondřej Surý wrote: > Source: sassphp > Followup-For: Bug #861637 > > Rhonda, > > like in the attached patch. > > Sorry for not having a better documentation, but I am extremely bad at > documenting my own work. (Would be happy to accept any patches that > makes

Bug#861637: sassphp: src:sassphp explicitly creates a php7.0 binary package

2017-05-04 Thread Nish Aravamudan
On Thu, May 4, 2017 at 11:40 AM, Ondřej Surý wrote: > That's: > > PECL_SOURCE=$(filter-out debian $(DIR_TARGETS),$(wildcard *)) +$(foreach > ver,$(DH_PHP_VERSIONS),$(eval PECL_SOURCE_$(ver) := $(PECL_SOURCE))) > > So something else must be going on there. I have successfully

Bug#861637: sassphp: src:sassphp explicitly creates a php7.0 binary package

2017-05-04 Thread Ondřej Surý
That's: PECL_SOURCE=$(filter-out debian $(DIR_TARGETS),$(wildcard *)) +$(foreach ver,$(DH_PHP_VERSIONS),$(eval PECL_SOURCE_$(ver) := $(PECL_SOURCE))) So something else must be going on there. I have successfully built sass for coinstallable php 5.6, 7.0 and 7.1. Cheers, Ondřej On 4 May

Bug#861637: sassphp: src:sassphp explicitly creates a php7.0 binary package

2017-05-04 Thread Nish Aravamudan
Hi Ondřej, It appears that this patch assumes a normal PECL extension and I'm not sure sass is one? The build eventually fails with: cp -a undefined build-7.1 cp: cannot stat 'undefined': No such file or directory /usr/share/dh-php/pkg-pecl.mk:60: recipe for target 'configure-7.1-stamp' failed

Bug#861637: sassphp: src:sassphp explicitly creates a php7.0 binary package

2017-05-04 Thread Ondřej Surý
Source: sassphp Followup-For: Bug #861637 Rhonda, like in the attached patch. Sorry for not having a better documentation, but I am extremely bad at documenting my own work. (Would be happy to accept any patches that makes the documentation of dh-php better though.) O. -- System Information:

Bug#861637: sassphp: src:sassphp explicitly creates a php7.0 binary package

2017-05-02 Thread Rhonda D'Vine
Hi there, Am Montag, den 01.05.2017, 21:42 -0700 schrieb Nishanth Aravamudan: > Package: sassphp > Version: 0.5.10-2 > Severity: normal > > It would appear that sassphp doesn't follow many other PHP packages > and > explicitly names it's binary package php7.0-sassphp rather than >

Bug#861637: sassphp: src:sassphp explicitly creates a php7.0 binary package

2017-05-01 Thread Nishanth Aravamudan
Package: sassphp Version: 0.5.10-2 Severity: normal Dear Maintainer, It would appear that sassphp doesn't follow many other PHP packages and explicitly names it's binary package php7.0-sassphp rather than php-sassphp. This requires renaming the binary package and explicitly adding transitional