Re: Self Introduction

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

 



On 06/13/2011 09:27 AM, Ryan Lewis wrote:
> Hey Rahul,
>
> I few things are in the works:
> I am packing this program sketch, as a first taste of learning RPM's
> https://bugzilla.redhat.com/show_bug.cgi?id=711547
>
> I'm not sure why I haven't heard back from the reviewers after these
> few days have gone by, maybe I will hear this week?

Not a full review but a cursory look shows you need to do atleast the
following:

You haven't fixed the issues noted in comment 3.  You need not specify
the build dependencies if they are part of the default build root. 

http://fedoraproject.org/wiki/Packaging:Guidelines#Exceptions_2

You don't need to define a buildroot

http://fedoraproject.org/wiki/Packaging:Guidelines#BuildRoot_tag

No need to specify a clean section.  Yours is incorrect and empty

http://fedoraproject.org/wiki/Packaging:Guidelines#.25clean

You haven't written a changelog.  Make sure you run rpmlint on the spec
file, srpm and binary rpm

Rahul
-- 
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/devel


[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]
  Powered by Linux