Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=591190 Oron Peled <oron@xxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends on| |642160, 591389 --- Comment #8 from Oron Peled <oron@xxxxxxxxxxxx> 2010-10-12 05:14:23 EDT --- That was quick, thanks, fixed. Details below. SPEC URL: http://oron.fedorapeople.org/deb-package/debhelper.spec SRPM URL: http://oron.fedorapeople.org/deb-package/debhelper-8.0.0-2.fc13.src.rpm Fixed issues: 1. BR dpkg-devel, re-added. /me should mail Debian BTS. They may have forgotten about this. ( http://packages.debian.org/source/squeeze/debhelper ) 2. Made examples non-executable (with a comment in the .spec) 3. Added %check: * The failure is due to a bug in dpkg-architecture (from dpkg-devel): - Opened a Fedora bug 642160 (contains a 2 liner fix) - Marked bug 642160 as a blocker to this review - Tested with a locally fixed dpkg-architecture -- all PASS * Added perl-Test-Pod as BR, so 'make test' does not skip some tests. 3. Added another patch to Makefile so installation paths are parametric and used it in the %install phase. Another issue: 4. Requires: po-debconf - This means Fedora bug 591389 (review request) prevent debhelper from really being installed/tested. - Marked bug 591389 as blocker to this review - More work... -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review