[Bug 915095] Review Request: nodejs-mapnik-reference - Reference for Mapnik Styling Options

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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

--- Comment #4 from Tom Hughes <tom@xxxxxxxxxx> ---
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@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]