Package: creoleparser
Version: 0.7.2-3
Severity: normal
Tags: patch
User: ubuntu-de...@lists.ubuntu.com
Usertags: origin-ubuntu maverick ubuntu-patch

Hi,

I thinks this patch is cleaner, as it just move the test after the 
installation, to have a predictable fix path. Is it ok?

Thanks,
Fabrice

*** /tmp/tmpyIeasc


-- System Information:
Debian Release: squeeze/sid
  APT prefers lucid-updates
  APT policy: (500, 'lucid-updates'), (500, 'lucid-security'), (500, 
'lucid-proposed'), (500, 'lucid')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-25-generic (SMP w/2 CPU cores)
Locale: LANG=es_ES.UTF-8, LC_CTYPE=es_ES.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
diff -u creoleparser-0.7.2/debian/rules creoleparser-0.7.2/debian/rules
--- creoleparser-0.7.2/debian/rules
+++ creoleparser-0.7.2/debian/rules
@@ -5,6 +5,7 @@
 
 PKGNAME:=python-creoleparser
 PYVERS:=$(shell pyversions -r)
+PYVERSONLY:=$(shell pyversions -rv)
 
 include /usr/share/dpatch/dpatch.make
 -include /usr/share/python/python.mk
@@ -18,13 +19,6 @@
 	    $$py setup.py build; \
 	done
 
-ifeq (,$(findstring nocheck,$(DEB_BUILD_OPTIONS)))
-	set -e ;\
-	for py in $(PYVERS); do  \
-	    PYTHONPATH=$(CURDIR)/build/lib $$py $(CURDIR)/build/lib/creoleparser/__init__.py; \
-	done
-endif
-
 	touch $@
 
 clean: clean-patched unpatch
@@ -49,6 +43,13 @@
 	for py in $(PYVERS); do  \
 	    $$py setup.py install --root=debian/$(PKGNAME) $(py_setup_install_args); \
 	done
+ifeq (,$(findstring nocheck,$(DEB_BUILD_OPTIONS)))
+	set -e ;\
+	for py in $(PYVERSONLY); do  \
+	    PYTHONPATH=$(CURDIR)/debian/$(PKGNAME)/usr/lib/python$$py/$(call py_sitename_sh, $$py) python$$py $(CURDIR)/debian/$(PKGNAME)/usr/lib/python$$py/$(call py_sitename_sh, $$py)/creoleparser/__init__.py; \
+	done
+endif
+
 
 	dh_installdocs -A
 	touch install-stamp

Reply via email to