[php-pecl-radius] - run upstream test suite - install doc in pecl_docdir - install tests in pecl_testdir - add missing

Mon Mar 17 09:25:17 UTC 2014

commit 60f40c10b3caf0c62a474d76f826c45f4e58a8dc
Author: Remi Collet <remi at fedoraproject.org>
Date:   Mon Mar 17 10:25:11 2014 +0100

    - run upstream test suite
    - install doc in pecl_docdir
    - install tests in pecl_testdir
    - add missing License file (extracted from headers)

 LICENSE              |   28 +++++++++++++
 php-pecl-radius.spec |  111 +++++++++++++++++++++++++++++++++++++++-----------
 2 files changed, 115 insertions(+), 24 deletions(-)
---

diff --git a/LICENSE b/LICENSE
new file mode 100644
index 0000000..18492fb
--- /dev/null
+++ b/LICENSE
@@ -0,0 +1,28 @@
+Copyright (c) 2003, Michael Bretterklieber <michael at bretterklieber.com>
+All rights reserved.
+
+Redistribution and use in source and binary forms, with or without
+modification, are permitted provided that the following conditions
+are met:
+
+1. Redistributions of source code must retain the above copyright
+   notice, this list of conditions and the following disclaimer.
+2. Redistributions in binary form must reproduce the above copyright
+   notice, this list of conditions and the following disclaimer in the
+   documentation and/or other materials provided with the distribution.
+3. The names of the authors may not be used to endorse or promote products
+   derived from this software without specific prior written permission.
+
+THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND
+ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
+WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED.
+IN NO EVENT SHALL THE COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT,
+INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING,
+BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY
+OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
+NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE,
+EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+
+This code cannot simply be copied and put under the GNU Public License or
+any other GPL-like (LGPL, GPL2) License.
diff --git a/php-pecl-radius.spec b/php-pecl-radius.spec
index 568b7fe..c0a2dcd 100644
--- a/php-pecl-radius.spec
+++ b/php-pecl-radius.spec
@@ -1,22 +1,39 @@
-%{!?php_inidir: %{expand: %%global php_inidir  %{_sysconfdir}/php.d}}
-%{!?__pecl:     %{expand: %%global __pecl      %{_bindir}/pecl}}
-
-%global pecl_name radius
-%global with_zts  0%{?__ztsphp:1}
+# spec file for php-pecl-radius
+#
+# Copyright (c) 2009-2014 Remi Collet
+# Copyright (c) 2006-2009 Christopher Stone
+#
+# License: MIT
+# http://opensource.org/licenses/MIT
+#
+# Please, preserve the changelog entries
+#
+
+%{!?php_inidir:  %global php_inidir   %{_sysconfdir}/php.d}
+%{!?__pecl:      %global __pecl       %{_bindir}/pecl}
+%{!?__php:       %global __php        %{_bindir}/php}
+
+%global pecl_name  radius
+%global with_zts   0%{?__ztsphp:1}
 
 Name:           php-pecl-radius
 Version:        1.2.7
-Release:        2%{?dist}
+Release:        3%{?dist}
 Summary:        Radius client library
 
-# https://bugs.php.net/65156 - request LICENSE file
 License:        BSD
 Group:          Development/Languages
 URL:            http://pecl.php.net/package/radius
 Source0:        http://pecl.php.net/get/radius-%{version}.tgz
 
+# https://bugs.php.net/65156 ask license file
+# file created from the radius.c headers
+Source1:        LICENSE
+
 BuildRequires:  php-devel
 BuildRequires:  php-pear
+BuildRequires:  php-posix
+BuildRequires:  php-sockets
 
 Requires(post): %{__pecl}
 Requires(postun): %{__pecl}
@@ -29,9 +46,11 @@ Provides:       php-pecl(%{pecl_name}) = %{version}
 Provides:       php-pecl(%{pecl_name})%{?_isa} = %{version}
 
 
+%if 0%{?fedora} < 20 && 0%{?rhel} < 7
 # Filter private shared
-%{?filter_provides_in: %filter_provides_in %{php_extdir}/.*\.so$}
+%{?filter_provides_in: %filter_provides_in %{_libdir}/.*\.so$}
 %{?filter_setup}
+%endif
 
 
 %description
@@ -45,7 +64,13 @@ OS (for example against Windows Active-Directory via IAS).
 %prep
 %setup -qc
 
-cd %{pecl_name}-%{version}
+sed -e '/CREDITS/s/role="src"/role="doc"/' -i package.xml
+
+mv %{pecl_name}-%{version} NTS
+cd NTS
+
+cp %{SOURCE1} LICENSE
+
 extver=$(sed -n '/#define PHP_RADIUS_VERSION/{s/.* "//;s/".*$//;p}' php_radius.h)
 if test "x${extver}" != "x%{version}"; then
    : Error: Upstream version is ${extver}, expecting %{version}.
@@ -59,18 +84,19 @@ extension=%{pecl_name}.so
 EOF
 
 %if %{with_zts}
-cp -pr %{pecl_name}-%{version} %{pecl_name}-%{version}-zts
+# Duplicate source tree for NTS / ZTS build
+cp -pr NTS ZTS
 %endif
 
 
 %build
-cd %{pecl_name}-%{version}
+cd NTS
 %{_bindir}/phpize
 %configure  --with-php-config=%{_bindir}/php-config
 make %{?_smp_mflags}
 
 %if %{with_zts}
-cd ../%{pecl_name}-%{version}-zts
+cd ../ZTS
 %{_bindir}/zts-phpize
 %configure  --with-php-config=%{_bindir}/zts-php-config
 make %{?_smp_mflags}
@@ -78,8 +104,7 @@ make %{?_smp_mflags}
 
 
 %install
-make -C %{pecl_name}-%{version} \
-     install INSTALL_ROOT=%{buildroot}
+make -C NTS install INSTALL_ROOT=%{buildroot}
 
 # Install XML package description
 install -Dpm 644 package.xml %{buildroot}%{pecl_xmldir}/%{name}.xml
@@ -88,25 +113,56 @@ install -Dpm 644 package.xml %{buildroot}%{pecl_xmldir}/%{name}.xml
 install -Dpm644 %{pecl_name}.ini %{buildroot}%{php_inidir}/%{pecl_name}.ini
 
 %if %{with_zts}
-make -C %{pecl_name}-%{version}-zts \
-     install INSTALL_ROOT=%{buildroot}
-
+make -C ZTS install INSTALL_ROOT=%{buildroot}
 install -Dpm644 %{pecl_name}.ini %{buildroot}%{php_ztsinidir}/%{pecl_name}.ini
 %endif
 
+# Test & Documentation
+cd NTS
+for i in $(grep 'role="test"' ../package.xml | sed -e 's/^.*name="//;s/".*$//')
+do install -Dpm 644 $i %{buildroot}%{pecl_testdir}/%{pecl_name}/$i
+done
+for i in LICENSE  $(grep 'role="doc"' ../package.xml | sed -e 's/^.*name="//;s/".*$//')
+do install -Dpm 644 $i %{buildroot}%{pecl_docdir}/%{pecl_name}/$i
+done
+
 
 %check
+DEP=
+[ -f %{php_extdir}/posix.so ]   && DEP="$DEP -d extension=posix.so"
+[ -f %{php_extdir}/sockets.so ] && DEP="$DEP -d extension=sockets.so"
+
+%if "%{php_version}" > "5.5"
+# used fake_server is not compatible with php 5.5
+# Deprecated: Non-static method xxx should not be called statically
+rm -f $(grep -l fake_server ?TS/tests/*phpt)
+%endif
+
 # simple module load test
-php --no-php-ini \
-    --define extension_dir=%{pecl_name}-%{version}/modules \
-    --define extension=%{pecl_name}.so \
+%{__php} --no-php-ini \
+    --define extension=%{buildroot}%{php_extdir}/%{pecl_name}.so \
     --modules | grep %{pecl_name}
 
+: Upstream test suite for NTS extension
+cd NTS
+TEST_PHP_EXECUTABLE=%{_bindir}/php \
+TEST_PHP_ARGS="-n $DEP -d extension=$PWD/modules/%{pecl_name}.so" \
+NO_INTERACTION=1 \
+REPORT_EXIT_STATUS=1 \
+%{_bindir}/php -n run-tests.php
+
 %if %{with_zts}
 %{__ztsphp} --no-php-ini \
-    --define extension_dir=%{pecl_name}-%{version}-zts/modules \
-    --define extension=%{pecl_name}.so \
+    --define extension=%{buildroot}%{php_ztsextdir}/%{pecl_name}.so \
     --modules | grep %{pecl_name}
+
+: Upstream test suite for ZTS extension
+cd ../ZTS
+TEST_PHP_EXECUTABLE=%{__ztsphp} \
+TEST_PHP_ARGS="-n $DEP -d extension=$PWD/modules/%{pecl_name}.so" \
+NO_INTERACTION=1 \
+REPORT_EXIT_STATUS=1 \
+%{__ztsphp} -n run-tests.php
 %endif
 
 
@@ -121,9 +177,10 @@ fi
 
 
 %files
-%defattr(-,root,root,-)
-%doc %{pecl_name}-%{version}/{CREDITS,examples}
+%doc %{pecl_docdir}/%{pecl_name}
+%doc %{pecl_testdir}/%{pecl_name}
 %{pecl_xmldir}/%{name}.xml
+
 %config(noreplace) %{php_inidir}/%{pecl_name}.ini
 %{php_extdir}/%{pecl_name}.so
 
@@ -134,6 +191,12 @@ fi
 
 
 %changelog
+* Sun Mar 16 2014 Remi Collet <remi at fedoraproject.org> - 1.2.7-2
+- run upstream test suite
+- install doc in pecl_docdir
+- install tests in pecl_testdir
+- add missing License file (extracted from headers)
+
 * Sun Aug 04 2013 Fedora Release Engineering <rel-eng at lists.fedoraproject.org> - 1.2.7-2
 - Rebuilt for https://fedoraproject.org/wiki/Fedora_20_Mass_Rebuild
 
