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

Ryan Lerch <rle...@redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Blocks|                            |177841 (FE-NEEDSPONSOR)



--- Comment #16 from Ryan Lerch <rle...@redhat.com> ---
(In reply to Ankur Sinha (FranciscoD) from comment #15)
> 
> PACKAGE IS *** APPROVED ***
> 
> Tiny issues you should correct before importing:
> 
> - No need to remove fonts from the ./data directory. They aren't included in
>   the tar any more.

Okay, will remove the following lines from the spec:
"""
# in separate package
rm -rf %{buildroot}%{_datadir}/fonts
"""
> - Please co-own the appdata directory

Not 100% sure what to do to achieve this.


Also, I originally forgot to make this block the NEED_SPONSOR bug, as this is
my first package. 

cheers,
ryanlerch


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to