[Bug 1125225] Review Request: alot - MUA based on notmuch mail

https://bugzilla.redhat.com/show_bug.cgi?id=1125225 Tomas Tomecek changed: What|Removed |Added Flags||fedora-cvs? --- Comment #10 from Toma

[Bug 1125225] Review Request: alot - MUA based on notmuch mail

https://bugzilla.redhat.com/show_bug.cgi?id=1125225 --- Comment #9 from Tomas Tomecek --- Flo, thank you for doing the review! -- 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 __

[Bug 1128366] Review Request: lis - A library for solving linear equations and eigenvalue problems

https://bugzilla.redhat.com/show_bug.cgi?id=1128366 --- Comment #3 from Christopher Meng --- 1. 1.4.55(japanese version page) is out. 2. Can't "make check" work? 3. I'm not sure if --enable-longlong is needed on 64 bits arch, you'd better drop an email to the upstream. 4. WRONG: %package do

[Bug 1128373] Review Request: perl-Audio-Cuefile-Parser - Perl cuefile parser module

https://bugzilla.redhat.com/show_bug.cgi?id=1128373 --- Comment #1 from Christopher Meng --- 1. Missing BRs of module itself: perl(Carp) perl(Class::Struct) perl(IO::File) perl(strict) perl(warnings) 2. Missing BR of tests: perl(File::Basename) 3. Drop this in the SPEC: find $RPM_BUILD_ROO

[Bug 1128373] Review Request: perl-Audio-Cuefile-Parser - Perl cuefile parser module

https://bugzilla.redhat.com/show_bug.cgi?id=1128373 Christopher Meng changed: What|Removed |Added Flags||fedora-review? -- You are receivi

[Bug 1128366] Review Request: lis - A library for solving linear equations and eigenvalue problems

https://bugzilla.redhat.com/show_bug.cgi?id=1128366 Christopher Meng changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|nob...@f

[Bug 960645] Review Request: sanewall - A powerful firewall builder

https://bugzilla.redhat.com/show_bug.cgi?id=960645 Christopher Meng changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|---

[Bug 728302] Review Request: pjproject - Libraries written in C language for building embedded/non-embedded VoIP applications

https://bugzilla.redhat.com/show_bug.cgi?id=728302 --- Comment #82 from Christopher Meng --- (In reply to Tristan Matthews from comment #81) > FYI, ilbc is already packaged for Fedora: > https://apps.fedoraproject.org/packages/ilbc Thanks. Then I think ilbc in Fedora needs an update maybe. --

[Bug 1128378] Review Request: cv - Coreutils Viewer

https://bugzilla.redhat.com/show_bug.cgi?id=1128378 --- Comment #9 from Christopher Meng --- 1. Please, unset %global _hardened_build 1, I don't know why you prefer adding this to all packages. 2. LDFLAGS still not set: cc -Wall cv.o sizes.o hlist.o -o cv -lncurses -lm In your spec: LDFLAGS

[Bug 728302] Review Request: pjproject - Libraries written in C language for building embedded/non-embedded VoIP applications

https://bugzilla.redhat.com/show_bug.cgi?id=728302 --- Comment #81 from Tristan Matthews --- FYI, ilbc is already packaged for Fedora: https://apps.fedoraproject.org/packages/ilbc -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes

[Bug 728302] Review Request: pjproject - Libraries written in C language for building embedded/non-embedded VoIP applications

https://bugzilla.redhat.com/show_bug.cgi?id=728302 --- Comment #80 from Christopher Meng --- Notes: I've seen ilbc in many opensource projects, chromium, freeswitch. As ilbc is part of webrtc now, I still can download it from the original homepage: http://www.soft-switch.org/downloads/voipcod

[Bug 728302] Review Request: pjproject - Libraries written in C language for building embedded/non-embedded VoIP applications

https://bugzilla.redhat.com/show_bug.cgi?id=728302 --- Comment #79 from Sandro Mani --- Reading through the above I see that unbundling the ilbc is simply a matter of hacking the build scripts. This seems to be feasible, I'll try to post a patch by tomorrow. -- You are receiving this mail bec

[Bug 728302] Review Request: pjproject - Libraries written in C language for building embedded/non-embedded VoIP applications

https://bugzilla.redhat.com/show_bug.cgi?id=728302 Sandro Mani changed: What|Removed |Added Assignee|pa...@hubbitus.info |manisan...@gmail.com -- You are receivi

[Bug 728302] Review Request: pjproject - Libraries written in C language for building embedded/non-embedded VoIP applications

https://bugzilla.redhat.com/show_bug.cgi?id=728302 --- Comment #78 from Sandro Mani --- Created attachment 925597 --> https://bugzilla.redhat.com/attachment.cgi?id=925597&action=edit licensecheck I'll take over the review if no-one minds. - Group unnecessary - lconfig on devel unnecessary -

[Bug 1128480] Review Request: python-pipsi - Wraps pip and virtualenv to install scripts

https://bugzilla.redhat.com/show_bug.cgi?id=1128480 --- Comment #1 from Matej Stuchlik --- Spec URL: https://mstuchli.fedorapeople.org/python-pipsi.spec SRPM URL: https://mstuchli.fedorapeople.org/python-pipsi-0.4-1.fc20.src.rpm -- You are receiving this mail because: You are on the CC list f

[Bug 1128480] New: Review Request: python-pipsi - Wraps pip and virtualenv to install scripts

https://bugzilla.redhat.com/show_bug.cgi?id=1128480 Bug ID: 1128480 Summary: Review Request: python-pipsi - Wraps pip and virtualenv to install scripts Product: Fedora Version: rawhide Component: Package Review Sev

[Bug 1048493] Review Request: icecat - GNU version of Firefox browser

https://bugzilla.redhat.com/show_bug.cgi?id=1048493 --- Comment #34 from Antonio Trande --- Spec URL: http://sagitter.fedorapeople.org/Icecat/icecat.spec SRPM URL: http://sagitter.fedorapeople.org/Icecat/icecat-24.0-12.fc20.src.rpm HTTPS-everywhere and RequestPolicy are not packaged in Icecat

[Bug 1093500] Review Request: clasp - A conflict-driven nogood learning answer set solver

https://bugzilla.redhat.com/show_bug.cgi?id=1093500 --- Comment #4 from Scott K Logan --- Christopher - Thanks for the feedback. (1) and (2) have been fixed as described. For (3), I assume you were looking for the -devel package to depend on the base. I added that. For (0), I'm sorry to rep

[Bug 1111388] Review Request: gmsh - A three-dimensional finite element mesh generator

https://bugzilla.redhat.com/show_bug.cgi?id=388 --- Comment #21 from Fedora Update System --- gmsh-2.8.5-2.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/gmsh-2.8.5-2.fc20 -- You are receiving this mail because: You are on the CC list for the b

[Bug 1111294] Review Request: engrid - Mesh generation tool

https://bugzilla.redhat.com/show_bug.cgi?id=294 --- Comment #11 from Fedora Update System --- engrid-1.4.0-3.gite6d55f5.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/engrid-1.4.0-3.gite6d55f5.fc20 -- You are receiving this mail because: You ar

[Bug 1111294] Review Request: engrid - Mesh generation tool

https://bugzilla.redhat.com/show_bug.cgi?id=294 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving

[Bug 1128378] Review Request: cv - Coreutils Viewer

https://bugzilla.redhat.com/show_bug.cgi?id=1128378 --- Comment #8 from Florian "der-flo" Lehner --- ok - here we are again. I hope I have understood you and implemented the points correctly. Spec URL: https://flo.fedorapeople.org/cv.spec SRPM URL: https://flo.fedorapeople.org/cv-0.4.1-3.fc20

[Bug 197565] Review Request: buildbot

https://bugzilla.redhat.com/show_bug.cgi?id=197565 Björn "besser82" Esser changed: What|Removed |Added CC||bjoern.es...@gmail.com

[Bug 1128393] Review Request: qhexedit2 - Binary Editor for Qt

https://bugzilla.redhat.com/show_bug.cgi?id=1128393 --- Comment #2 from Sandro Mani --- I see. I guess [1] would be an alternative, though Qt5 only. Is there somewhere where one can follow the upstream discussion? [1] https://github.com/Dax89/QHexEdit -- You are receiving this mail because:

[Bug 1128378] Review Request: cv - Coreutils Viewer

https://bugzilla.redhat.com/show_bug.cgi?id=1128378 --- Comment #7 from Christopher Meng --- If you don't use my patch, simply hack from command line won't work. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product a

[Bug 1128378] Review Request: cv - Coreutils Viewer

https://bugzilla.redhat.com/show_bug.cgi?id=1128378 --- Comment #6 from Christopher Meng --- (In reply to Ralf Corsepius from comment #4) > 2. Passing LDFLAGS is not required. > => Remove LDFLAGS+="%{?__global_ldflags}" No. It's linked against ncurses. It's a MUST. (In reply to Florian "der-

[Bug 1048493] Review Request: icecat - GNU version of Firefox browser

https://bugzilla.redhat.com/show_bug.cgi?id=1048493 Antonio Trande changed: What|Removed |Added CC||niveusl...@niveusluna.org --- Commen

[Bug 961223] Review Request: ocrfeeder - A document layout analysis and optical character recognition system

https://bugzilla.redhat.com/show_bug.cgi?id=961223 Kalev Lember changed: What|Removed |Added CC||kalevlem...@gmail.com --- Comment #19 f

[Bug 1128378] Review Request: cv - Coreutils Viewer

https://bugzilla.redhat.com/show_bug.cgi?id=1128378 Björn "besser82" Esser changed: What|Removed |Added CC||bjoern.es...@gmail.com --- C

[Bug 1128378] Review Request: cv - Coreutils Viewer

https://bugzilla.redhat.com/show_bug.cgi?id=1128378 Ralf Corsepius changed: What|Removed |Added CC||rc040...@freenet.de --- Comment #4 f

[Bug 1128366] Review Request: lis - A library for solving linear equations and eigenvalue problems

https://bugzilla.redhat.com/show_bug.cgi?id=1128366 --- Comment #2 from Florian "der-flo" Lehner --- Spec URL: https://flo.fedorapeople.org/lis.spec SRPM URL: https://flo.fedorapeople.org/lis-1.4.53-2.fc20.src.rpm koji-Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=7269578 * Sun Aug

[Bug 1128378] Review Request: cv - Coreutils Viewer

https://bugzilla.redhat.com/show_bug.cgi?id=1128378 --- Comment #3 from Florian "der-flo" Lehner --- hi! Thanks for assigning! I know there is a patch merged already for a manpage. But it's not part of the current stable version. I am quite sure there will be a new release containing this man

[Bug 1015868] Review Request: python-qutepart - Source code text editor component based on Qt

https://bugzilla.redhat.com/show_bug.cgi?id=1015868 --- Comment #6 from Yajo --- (In reply to T.C. Hollingsworth from comment #4) > In the meantime, see if you can get going with koji. Also, try giving a few > other packages an informal review: > https://fedoraproject.org/wiki/ > How_to_get_sp

[Bug 1077301] Review Request: python-aaargh - An astonishingly awesome application argument helper

https://bugzilla.redhat.com/show_bug.cgi?id=1077301 Yajo changed: What|Removed |Added CC||yajo@gmail.com --- Comment #4 from Yajo -

[Bug 1127720] Review Request: perl-MooX-ConfigFromFile - Moo eXtension for initializing objects from config file

https://bugzilla.redhat.com/show_bug.cgi?id=1127720 David Dick changed: What|Removed |Added Depends On||1123336 Referenced Bugs: https://bugzi

[Bug 1123336] Review Request: perl-MooX-File-ConfigDir - Moo eXtension for File::ConfigDir

https://bugzilla.redhat.com/show_bug.cgi?id=1123336 David Dick changed: What|Removed |Added Blocks||1127720 Referenced Bugs: https://bugzi

[Bug 1128409] Review Request: perl-Module-Data - Introspect context information about modules in @INC

https://bugzilla.redhat.com/show_bug.cgi?id=1128409 David Dick changed: What|Removed |Added Depends On|1126598 |1124313 Referenced Bugs: https://bugzi

[Bug 1124313] Review Request: perl-Path-ScanINC - Emulate Perls internal handling of @INC

https://bugzilla.redhat.com/show_bug.cgi?id=1124313 David Dick changed: What|Removed |Added Blocks||1128409 Referenced Bugs: https://bugzi

[Bug 1128409] Review Request: perl-Module-Data - Introspect context information about modules in @INC

https://bugzilla.redhat.com/show_bug.cgi?id=1128409 David Dick changed: What|Removed |Added Depends On||1126598 --- Comment #1 from David Dick

[Bug 1128409] New: Review Request: perl-Module-Data - Introspect context information about modules in @INC

https://bugzilla.redhat.com/show_bug.cgi?id=1128409 Bug ID: 1128409 Summary: Review Request: perl-Module-Data - Introspect context information about modules in @INC Product: Fedora Version: rawhide Component: Package Review

[Bug 1128145] Review Request: perl-MooX-HasEnv - Making attributes based on ENV variables

https://bugzilla.redhat.com/show_bug.cgi?id=1128145 David Dick changed: What|Removed |Added Flags||fedora-cvs? --- Comment #4 from David Di

[Bug 1128145] Review Request: perl-MooX-HasEnv - Making attributes based on ENV variables

https://bugzilla.redhat.com/show_bug.cgi?id=1128145 Emmanuel Seyman changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #3 from