gach-dev November 2008

gach-dev@lists.fedorahosted.org

4 participants
6 discussions

[review-o-matic] TODO list
by Orcan Ogetbil 12 Dec '08

12 Dec '08

Hi folks, >From the template I use for my reviews I prepared this TODO list for the implementation of all the checks that need to be implemented on r-o-m: http://oget.fedorapeople.org/r-o-m/review-o-matic The list is pretty much complete. I think it covers almost all of the guidelines, still there might be a few things that we can add. The boxes on the left hand side of any check indicate which of the following files is needed to do that check: SPEC, SRPM, RPMs, buildlog. I tried to be accurate by putting those x's but there might be minor flaws. Feel free to correct me. I think this would be a good starting point. Note that the implementation of certain guidelines is very hard. Examples: - The SPEC file must be written in American English - If there is a -devel package, check for the header files inside the package to figure out "Requires:" for the -devel package. - If applicable, package must obey the specific Guidelines (python, java etc) I think what we should do is to create minimal check files for all the items in this list. Whenever the script runs, the minimal check file should tell the reviewer what needs to be checked manually. In time, whenever we implement some of these checks, we will replace those directives with the proper ones. Does everybody agree? If yes, I will write the minimal check files rightaway. -Orcan

4 8

[review-o-matic] [PATCH]: SrpmCheck.py: New file
by Debarshi Ray 26 Nov '08

26 Nov '08

Here is an initial version of a new class SrpmCheck: http://rishi.fedorapeople.org/0001--SrpmCheck.py-New-file.patch It is meant to be the abstract parent class of all SRPM based checks just like SpecCheck is for SPEC based checks. Apart from a few cosmetic things, it is similar to SpecCheck.py. Comments? Happy hacking, Debarshi

2 1

[review-o-matic] [PATCH] Srpm.py: New file
by Debarshi Ray 26 Nov '08

26 Nov '08

Here is a patch to add an initial version of a new class named Srpm to represent SRPMs: http://rishi.fedorapeople.org/0001--Srpm.py-New-file.patch The plan is to keep adding more get* methods as and when we need them. Comments? Happy hacking, Debarshi

2 1

[review-o-matic] New check: BuildRootCheck
by Orcan Ogetbil 26 Nov '08

26 Nov '08

Hi folks, I wrote this check a few days ago. Tonight I sat down and polished it. It is actually a very simple one that checks if the Buildroot tag is specified correctly in the specfile. Here is the link: http://oget.fedorapeople.org/r-o-m/BuildRootCheck.py I like the way you designed the API. I will write more checks whenever I have time. Right now I can only do SPEC file check. It would be nice if you could tell me how I can work with the SRPM of the package that is being reviewed. I can write more checks with that in hand. Please let me know if you have objections, comments, warnings, suggestions, ideas or some extra beer. Cheers, Orcan

2 2

[review-o-matic] Folks - are you following this thread on fedora devel ? [Re: Detecting binaries in rpmbuild]
by sankarshan 05 Nov '08

05 Nov '08

https://www.redhat.com/archives/fedora-devel-list/2008-November/msg00246.ht… -- http://www.gutenberg.net - Fine literature digitally re-published http://www.plos.org - Public Library of Science http://www.creativecommons.org - Flexible copyright for creative work

1 0

[review-o-matic] rpmgrok and r-o-m
by sankarshan 04 Nov '08

04 Nov '08

Would it be a good idea to look at a possible integration ? -- http://www.gutenberg.net - Fine literature digitally re-published http://www.plos.org - Public Library of Science http://www.creativecommons.org - Flexible copyright for creative work

1 0

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

gach-dev November 2008