Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=915095

--- Comment #4 from Tom Hughes <t...@compton.nu> ---
Fixed python macro, and removed hyphen from parseable. Fowler's seems clear the
hyphen is not needed and is more concerned with whether the e should be
removed.

I've fixed the tests as you suggest, though it's not very nice, but I don't
think there's anything better. Ideally RPM would run tests against the build
root after %install I guess but that would be hard.

The failures you're seeing aren't exactly failures, but then lint.js isn't
exactly a test suite - it's more of a coverage test that generates a list of
any attributes that don't have documentation yet. Possibly we shouldn't run it
- it will never fail (in the exit non-zero sense) anyway.

New spec and SRPM:

Spec URL: http://download.compton.nu/nodejs/nodejs-mapnik-reference.spec
SRPM URL:
http://download.compton.nu/nodejs/nodejs-mapnik-reference-5.0.4-2.fc18.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=n1QH035hoR&a=cc_unsubscribe
_______________________________________________
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to