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@actcom.co.il changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends on| |642160, 591389
--- Comment #8 from Oron Peled oron@actcom.co.il 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...