https://bugzilla.redhat.com/show_bug.cgi?id=997678 T.C. Hollingsworth <tchollingsworth@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC|package-review@lists.fedora | |project.org | --- Comment #3 from T.C. Hollingsworth <tchollingsworth@xxxxxxxxx> --- Thanks for the quick review! (In reply to Pete Travis from comment #1) > Personally, I'd appreciate seeing some packaged documentation to generally > outline the purpose of web-assets and references to packaging guidelines or > other resources explaining the use of these directories. Not a blocker, just > nice to have. I added a brief README for now. (In reply to Igor Gnatenko from comment #2) > I think: > Requires: web-assets-filesystem > should be > Requires: web-assets-filesystem%{?isa} = %{version}-%{release} I added %{version}-%{release}, but not %{?_isa} because that's wrong for noarch packages. (It'll just add whatever arch the noarch package happens to be building on at the time.) -- Spec: http://patches.fedorapeople.org/web-assets/web-assets.spec SRPM: http://patches.fedorapeople.org/web-assets/web-assets-4-2.fc20.src.rpm * Sat Aug 24 2013 T.C. Hollingsworth <tchollingsworth@xxxxxxxxx> - 4-2 - tighten dependency on filesystem from other packages - add brief README to -devel -- 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=aZhejba8yl&a=cc_unsubscribe _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review