Review app for Fedora
kevin at scrye.com
Tue Nov 29 20:18:27 UTC 2011
On Tue, 29 Nov 2011 14:51:39 +0100
Pierre-Yves Chibon <pingou at pingoured.fr> wrote:
> With the work recently done on fedora-review , I started to wonder
> if it would be interesting to work on an app dedicated to review RPM
> package for Fedora.
> I got myself day dreaming about:
So, the advantages would be:
- Could run the review tool on the package whenever it updated.
- git history could be kept.
- Could possibly require some level of completness before filing the
Disadvantages would be:
- Another service/code we need to maintain.
- Possibly having to deal with junk uploads unless we filter things
> I was wondering if it could rely on a similar infrastructure than what
> we are using via fedpkg/pyrpkg (a reviewpkg ?).
Possibly. I would personally not want to see this on the same
server/setup as real packages, just to be sure there would be no
leakage from these into real packages somehow.
But some kind of similar instance could be setup I suppose.
> I was curious if that would be interesting and since an example is
> worth many words, I tried to work on a small proof of concept . The
> README.txt contains the information on how to set it up for testing.
> It needs still a lot of changes:
> - All parts regarding the review itself (changing the status, adding
> one in cc, mail comments added...) is to be done
It makes me wonder if we couldn't just ditch bugzilla entirely if we
move to this? have comments and such in the app...
> - The design needs to be done (this has to be done, what's there atm
> is just good enough for testing :-))
> but as I said, it's more a PoC for the moment to collect
> I am starting the conversation here on the infra list to get the first
> reactions, it'll be opened to the devel list later on if worth it.
> So please shoot :-)
Above are my 2cents. ;)
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 198 bytes
Desc: not available
Url : http://lists.fedoraproject.org/pipermail/infrastructure/attachments/20111129/db65babd/attachment.bin
More information about the infrastructure