[Bug 997678] Review Request: web-assets - A simple framework for bits pushed to browsers

[bugzilla at redhat.com]

https://bugzilla.redhat.com/show_bug.cgi?id=997678

T.C. Hollingsworth <tchollingsworth at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|package-review at lists.fedora |
                   |project.org                 |



--- Comment #3 from T.C. Hollingsworth <tchollingsworth at gmail.com> ---
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 at gmail.com> - 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