[Bug 957520] Review Request: rf - read feeds from any source

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

 



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

Antonio Trande <anto.trande@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Assignee|nobody@xxxxxxxxxxxxxxxxx    |anto.trande@xxxxxxxxx
              Flags|                            |fedora-review+

--- Comment #3 from Antonio Trande <anto.trande@xxxxxxxxx> ---
(In reply to comment #2)
> Spec URL: http://juanmabc.fedorapeople.org/packages/rf/rf.spec
> SRPM URL:
> http://juanmabc.fedorapeople.org/packages/rf/rf-0.4.16-1.fc18.src.rpm
> Description: command line scriptable feed reader with default feeds (
> http://code.google.com/p/readfeed )
> Fedora Account System Username: juanmabc
> 
> *** --> Your package needs just
> 
> BuildRequires:       xmlstarlet
> Requires:            curl
> Requires:            lynx
> Requires(post):      info
> Requires(preun):     info
> 
> bash, sed, ...  can be omitted.
> 
> http://fedoraproject.org/wiki/Packaging:Guidelines#Exceptions_2
> 
> ***:
> Quoting: "Exceptions:There is no need to include the following packages or
> their dependencies as *BuildRequires* because they would occur too often.
> These 
> packages are considered the minimum build environment."
> 
> I use them in *Requires*, which is completely different and has no
> exceptions. Note that BuildRequires xmlstarlet i assume is a typo, since
> there is no BuildRequire dependency for it, and the fact that has no
> BuildRequires is precissely because of the Exceptions. Requires are what is
> needed at runtime, BuildRequires at compile time.
> 

Gee whiz! I'm sorry, I have mixed them. :P

I think your package can be approved.

-- 
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=0gm0KMbLJl&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]