[Bug 823066] Review Request: php-symfony2-Validator - Symfony2 Validator Component
https://bugzilla.redhat.com/show_bug.cgi?id=823066 Remi Collet changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #15 from Remi Collet --- == APPROVED == -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 825414] Review Request: flexiport - Flexible communications library
https://bugzilla.redhat.com/show_bug.cgi?id=825414 Dan Callaghan changed: What|Removed |Added Whiteboard||BuildFails --- Comment #3 from Dan Callaghan --- This SRPM fails to build. Looks like it needs BuildRequires: cmake BuildRequires: doxygen BuildRequires: graphviz You can try building your SRPM with mock (or a scratch build in Koji) to catch these problems. A couple of other issues which also need fixing... Your instructions for building the source tarball aren't quite right. Presumably you meant something like this: # git clone git://github.com/gbiggs/flexiport.git ; cd flexiport # git archive 1b6103daa | bzip2 >flexiport-2.0.0.tar.bz2 which worked for me and produced a tarball matching the one you have. I'd strongly recommend you use GitHub's tarball generation feature instead, since it gives a usable source URL. See here: http://lists.fedoraproject.org/pipermail/devel/2012-April/166267.html You should also add %{?_isa} to the Requires for the -devel subpackage to make it arch-specific: https://fedoraproject.org/wiki/Packaging/Guidelines#Requiring_Base_Package Please send your gcc47.patch upstream and add a link to the upstream report in the .spec. I would also suggest adding a comment above this line explaining its reason: sed -i 's/extensions/#extensions/' doc/conf.py.in -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 825414] Review Request: flexiport - Flexible communications library
https://bugzilla.redhat.com/show_bug.cgi?id=825414 Dan Callaghan changed: What|Removed |Added Status|NEW |ASSIGNED CC||dcall...@redhat.com Assignee|nob...@fedoraproject.org|dcall...@redhat.com Flags||fedora-review? -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 830777] Review Request: clucy - Clojure interface to Lucene
https://bugzilla.redhat.com/show_bug.cgi?id=830777 --- Comment #7 from Fedora Update System --- clucy-0.3.0-2.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/clucy-0.3.0-2.fc16 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 830777] Review Request: clucy - Clojure interface to Lucene
https://bugzilla.redhat.com/show_bug.cgi?id=830777 --- Comment #6 from Fedora Update System --- clucy-0.3.0-2.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/clucy-0.3.0-2.fc17 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 830777] Review Request: clucy - Clojure interface to Lucene
https://bugzilla.redhat.com/show_bug.cgi?id=830777 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 831705] Review Request: jj - A FIFO and file-system based Jabber/XMPP client
https://bugzilla.redhat.com/show_bug.cgi?id=831705 --- Comment #3 from Dan Callaghan --- (In reply to comment #2) > [x]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm > -q > --requires). > /usr/bin/jj is overlinked: > > $ ldd -u -r /usr/bin/jj > Unused direct dependencies: > /lib64/libidn.so.11 > > however in this case it is loudmouth's fault (wrong pkgconfig Libs) so it is > waived. I filed bug 836761 against loudmouth for this. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 831705] Review Request: jj - A FIFO and file-system based Jabber/XMPP client
https://bugzilla.redhat.com/show_bug.cgi?id=831705 --- Comment #2 from Dan Callaghan --- Review checklist follows, there is only one small issue at the bottom to be addressed. Package Review == Key: - = N/A x = Pass ! = Fail ? = Not evaluated C/C++ [x]: MUST Package does not contain any libtool archives (.la) [x]: MUST Package does not contain kernel modules. [x]: MUST Package contains no static executables. [x]: MUST Rpath absent or only used for internal libs. [x]: MUST Package is not relocatable. Generic [x]: MUST Package is licensed with an open-source compatible license and meets other legal requirements as defined in the legal section of Packaging Guidelines. [x]: MUST Package successfully compiles and builds into binary rpms on at least one supported primary architecture. [x]: MUST %build honors applicable compiler flags or justifies otherwise. [x]: MUST All build dependencies are listed in BuildRequires, except for any that are listed in the exceptions section of Packaging Guidelines. [x]: MUST Buildroot is not present Note: Unless packager wants to package for EPEL5 this is fine [x]: MUST Package contains no bundled libraries. [x]: MUST Changelog in prescribed format. [x]: MUST Package has no %clean section with rm -rf %{buildroot} (or $RPM_BUILD_ROOT) Note: Clean would be needed if support for EPEL is required [x]: MUST Sources contain only permissible code or content. [x]: MUST Each %files section contains %defattr if rpm < 4.4 Note: Note: defattr macros not found. They would be needed for EPEL5 [x]: MUST Macros in Summary, %description expandable at SRPM build time. [x]: MUST Package requires other packages for directories it uses. [x]: MUST Package uses nothing in %doc for runtime. [x]: MUST Package is not known to require ExcludeArch. [x]: MUST Permissions on files are set properly. [x]: MUST Package does not contain duplicates in %files. [x]: MUST Spec file lacks Packager, Vendor, PreReq tags. [x]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the beginning of %install. Note: rm -rf would be needed if support for EPEL5 is required [-]: MUST Large documentation files are in a -doc subpackage, if required. [x]: MUST If (and only if) the source package includes the text of the license(s) in its own file, then that file, containing the text of the license(s) for the package is included in %doc. [x]: MUST License field in the package spec file matches the actual license. [x]: MUST Package consistently uses macros (instead of hard-coded directory names). [x]: MUST Package is named according to the Package Naming Guidelines. [x]: MUST Package does not generate any conflict. [x]: MUST Package obeys FHS, except libexecdir and /usr/target. [x]: MUST Package must own all directories that it creates. [x]: MUST Package does not own files or directories owned by other packages. [x]: MUST Package installs properly. [-]: MUST Requires correct, justified where necessary. [x]: MUST Rpmlint output is silent. Warning about missing man page is waived as upstream does not supply one. rpmlint jj-1-1.fc18.i686.rpm jj.i686: W: no-manual-page-for-binary jj 1 packages and 0 specfiles checked; 0 errors, 1 warnings. rpmlint jj-debuginfo-1-1.fc18.i686.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. rpmlint jj-1-1.fc18.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. [x]: MUST Sources used to build the package match the upstream source, as provided in the spec URL. /home/dan/fedora/reviews/jj/831705/jj-1.tar.gz : MD5SUM this package : 1a0d8c60d0bd4333a7aeb7d8d58b88bf MD5SUM upstream package : 1a0d8c60d0bd4333a7aeb7d8d58b88bf [x]: MUST Spec file is legible and written in American English. [x]: MUST Spec file name must match the spec package %{name}, in the format %{name}.spec. [-]: MUST Package contains a SysV-style init script if in need of one. [x]: MUST File names are valid UTF-8. [x]: MUST Useful -debuginfo package or justification otherwise. [x]: SHOULD Reviewer should test that the package builds in mock. [-]: SHOULD If the source package does not include license text(s) as a separate file from upstream, the packager SHOULD query upstream to include it. [x]: SHOULD Dist tag is present. [x]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin. [x]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q --requires). /usr/bin/jj is overlinked: $ ldd -u -r /usr/bin/jj Unused direct dependencies: /lib64/libidn.so.11 however in this case it is loudmouth's fault (wrong pkgconfig Libs) so it is waived. [x]: SHOULD Package functions as described. [x]: SHOULD Latest version is packaged. [x]: SHOULD Package does not include license text files separate from upstream. [!]: SHOULD Patches link to upstream bugs/comments/lists or are otherwise justified. [x
[Bug 830811] Review Request: perl-No-Worries - Perl coding without worries
https://bugzilla.redhat.com/show_bug.cgi?id=830811 --- Comment #19 from Fedora Update System --- perl-No-Worries-0.3-2.el5 has been pushed to the Fedora EPEL 5 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 823063] Review Request: php-symfony2-Templating - Symfony2 Templating Component
https://bugzilla.redhat.com/show_bug.cgi?id=823063 --- Comment #14 from Fedora Update System --- php-symfony2-Templating-2.0.15-1.el6 has been pushed to the Fedora EPEL 6 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 823050] Review Request: php-symfony2-EventDispatcher - Symfony2 EventDispatcher Component
https://bugzilla.redhat.com/show_bug.cgi?id=823050 --- Comment #14 from Fedora Update System --- php-symfony2-EventDispatcher-2.0.15-1.el6 has been pushed to the Fedora EPEL 6 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 739015] Review Request: erlang-riak_sysmon - Rate-limiting system_monitor event handler for Riak
https://bugzilla.redhat.com/show_bug.cgi?id=739015 --- Comment #18 from Fedora Update System --- erlang-riak_sysmon-1.1.2-3.el6 has been pushed to the Fedora EPEL 6 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 823058] Review Request: php-symfony2-Process - Symfony2 Process Component
https://bugzilla.redhat.com/show_bug.cgi?id=823058 --- Comment #14 from Fedora Update System --- php-symfony2-Process-2.0.15-1.el6 has been pushed to the Fedora EPEL 6 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 830811] Review Request: perl-No-Worries - Perl coding without worries
https://bugzilla.redhat.com/show_bug.cgi?id=830811 --- Comment #18 from Fedora Update System --- perl-No-Worries-0.3-2.el6 has been pushed to the Fedora EPEL 6 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 823043] Review Request: php-symfony2-Console - Symfony2 Console Component
https://bugzilla.redhat.com/show_bug.cgi?id=823043 --- Comment #24 from Fedora Update System --- php-symfony2-Console-2.0.15-1.el6 has been pushed to the Fedora EPEL 6 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 823047] Review Request: php-symfony2-DomCrawler - Symfony2 DomCrawler Component
https://bugzilla.redhat.com/show_bug.cgi?id=823047 --- Comment #14 from Fedora Update System --- php-symfony2-DomCrawler-2.0.15-2.el6 has been pushed to the Fedora EPEL 6 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 823062] Review Request: php-symfony2-Serializer - Symfony2 Serializer Component
https://bugzilla.redhat.com/show_bug.cgi?id=823062 --- Comment #14 from Fedora Update System --- php-symfony2-Serializer-2.0.15-1.el6 has been pushed to the Fedora EPEL 6 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 823051] Review Request: php-symfony2-Finder - Symfony2 Finder Component
https://bugzilla.redhat.com/show_bug.cgi?id=823051 --- Comment #14 from Fedora Update System --- php-symfony2-Finder-2.0.15-1.el6 has been pushed to the Fedora EPEL 6 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 831705] Review Request: jj - A FIFO and file-system based Jabber/XMPP client
https://bugzilla.redhat.com/show_bug.cgi?id=831705 Dan Callaghan changed: What|Removed |Added Status|NEW |ASSIGNED CC||dcall...@redhat.com Assignee|nob...@fedoraproject.org|dcall...@redhat.com Flags||fedora-review? --- Comment #1 from Dan Callaghan --- Sounds useful! -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 836756] New: Review Request: targetd - Service to make storage remotely configurable
https://bugzilla.redhat.com/show_bug.cgi?id=836756 Bug ID: 836756 QA Contact: extras...@fedoraproject.org Severity: medium Version: rawhide Priority: medium CC: nott...@redhat.com, package-review@lists.fedoraproject.org Assignee: nob...@fedoraproject.org Summary: Review Request: targetd - Service to make storage remotely configurable Regression: --- Story Points: --- Classification: Fedora OS: Linux Reporter: agro...@redhat.com Type: --- Documentation: --- Hardware: All Mount Type: --- Status: NEW Component: Package Review Product: Fedora Spec URL: http://fedorapeople.org/~grover/new/targetd.spec SRPM URL: http://fedorapeople.org/~grover/new/targetd-0.2.1-1.fc17.src.rpm Description: targetd turns the machine into a remotely-configurable storage appliance. It supports an HTTP/jsonrpc-2.0 interface to let a remote administrator allocate volumes from an LVM volume group, and export those volumes over iSCSI. For this F18 feature: https://fedoraproject.org/wiki/Features/StorageManagement Fedora Account System Username: grover -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 836749] Review Request: objconv - object file converter and disassembler
https://bugzilla.redhat.com/show_bug.cgi?id=836749 Tim Sally changed: What|Removed |Added Blocks||177841 (FE-NEEDSPONSOR) -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 833204] Review Request: python-pyramid-tm - Allows pyramid requests to join the active transaction
https://bugzilla.redhat.com/show_bug.cgi?id=833204 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #12 from Fedora Update System --- python-pyramid-tm-0.4-1.fc17 has been pushed to the Fedora 17 testing repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 830709] Review Request: robert-hooke - Extension mechanism for Clojure functions
https://bugzilla.redhat.com/show_bug.cgi?id=830709 --- Comment #11 from Fedora Update System --- robert-hooke-1.2.0-2.fc17 has been pushed to the Fedora 17 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 836749] Review Request: objconv - object file converter and disassembler
https://bugzilla.redhat.com/show_bug.cgi?id=836749 --- Comment #2 from Tim Sally --- There are 5 total warnings, 2 of which are unique. The spelling warning is incorrect, as disassembler is a word. The warning about a man page is valid, however no man page is provided upstream. Thankfully this package is heavily documented in other ways, included an extensive manual and detailed help on the command line. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 800930] Review Request: redeclipse - Multiplayer FPS game based on Cube2
https://bugzilla.redhat.com/show_bug.cgi?id=800930 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2012-06-30 18:09:42 --- Comment #56 from Fedora Update System --- redeclipse-1.2-10.fc17 has been pushed to the Fedora 17 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 823070] Review Request: php-symfony2-BrowserKit - Symfony2 BrowserKit Component
https://bugzilla.redhat.com/show_bug.cgi?id=823070 --- Comment #13 from Fedora Update System --- php-symfony2-BrowserKit-2.0.15-2.fc17 has been pushed to the Fedora 17 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 823070] Review Request: php-symfony2-BrowserKit - Symfony2 BrowserKit Component
https://bugzilla.redhat.com/show_bug.cgi?id=823070 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2012-06-30 18:08:59 --- Comment #12 from Fedora Update System --- php-symfony2-BrowserKit-2.0.15-2.fc16 has been pushed to the Fedora 16 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 836749] Review Request: objconv - object file converter and disassembler
https://bugzilla.redhat.com/show_bug.cgi?id=836749 --- Comment #1 from Tim Sally --- Here are the rpmlint and koji results. [tsally@goddard SPECS]$ rpmlint -i objconv.spec ../RPMS/x86_64/objconv-2.14-1.fc17.x86_64.rpm ../RPMS/x86_64/objconv-debuginfo-2.14-1.fc17.x86_64.rpm ../SRPMS/objconv-2.14-1.fc17.src.rpm objconv.x86_64: W: spelling-error Summary(en_US) disassembler -> disassemble, disassembles, disassembled The value of this tag appears to be misspelled. Please double-check. objconv.x86_64: W: spelling-error %description -l en_US disassembler -> disassemble, disassembles, disassembled The value of this tag appears to be misspelled. Please double-check. objconv.x86_64: W: no-manual-page-for-binary objconv Each executable in standard binary directories should have a man page. objconv.src: W: spelling-error Summary(en_US) disassembler -> disassemble, disassembles, disassembled The value of this tag appears to be misspelled. Please double-check. objconv.src: W: spelling-error %description -l en_US disassembler -> disassemble, disassembles, disassembled The value of this tag appears to be misspelled. Please double-check. 3 packages and 1 specfiles checked; 0 errors, 5 warnings. [tsally@goddard ~]$ koji build --scratch f17 rpmbuild/SRPMS/objconv-2.14-1.fc17.src.rpm Uploading srpm: rpmbuild/SRPMS/objconv-2.14-1.fc17.src.rpm [] 100% 00:00:02 489.23 KiB 201.80 KiB/sec Created task: 4210216 Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=4210216 Watching tasks (this may be safely interrupted)... 4210216 build (f17, objconv-2.14-1.fc17.src.rpm): free 4210216 build (f17, objconv-2.14-1.fc17.src.rpm): free -> open (x86-15.phx2.fedoraproject.org) 4210218 buildArch (objconv-2.14-1.fc17.src.rpm, i686): free 4210217 buildArch (objconv-2.14-1.fc17.src.rpm, x86_64): free 4210217 buildArch (objconv-2.14-1.fc17.src.rpm, x86_64): free -> open (x86-17.phx2.fedoraproject.org) 4210218 buildArch (objconv-2.14-1.fc17.src.rpm, i686): free -> open (x86-03.phx2.fedoraproject.org) 4210217 buildArch (objconv-2.14-1.fc17.src.rpm, x86_64): open (x86-17.phx2.fedoraproject.org) -> closed 0 free 2 open 1 done 0 failed 4210218 buildArch (objconv-2.14-1.fc17.src.rpm, i686): open (x86-03.phx2.fedoraproject.org) -> closed 0 free 1 open 2 done 0 failed 4210216 build (f17, objconv-2.14-1.fc17.src.rpm): open (x86-15.phx2.fedoraproject.org) -> closed 0 free 0 open 3 done 0 failed 4210216 build (f17, objconv-2.14-1.fc17.src.rpm) completed successfully -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 836749] New: Review Request: objconv - object file converter and disassembler
https://bugzilla.redhat.com/show_bug.cgi?id=836749 Bug ID: 836749 QA Contact: extras...@fedoraproject.org Severity: medium Version: rawhide Priority: unspecified CC: nott...@redhat.com, package-review@lists.fedoraproject.org Assignee: nob...@fedoraproject.org Summary: Review Request: objconv - object file converter and disassembler Regression: --- Story Points: --- Classification: Fedora OS: Linux Reporter: tsa...@atomicpeace.com Type: --- Documentation: --- Hardware: All Mount Type: --- Status: NEW Component: Package Review Product: Fedora Spec URL: http://www.atomicpeace.com/objconv/objconv.spec SRPM URL: http://www.atomicpeace.com/objconv/objconv-2.14-1.fc17.src.rpm Description: objconv can be used to convert between object files such as COFF/PE and ELF. It provides some object editing capabilities, including changing symbol names. It includes a disassembler that supports many x86 extensions. Fedora Account System Username: tsally -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 832443] Review Request: jbosgi-repository - JBossOSGi Repository
https://bugzilla.redhat.com/show_bug.cgi?id=832443 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #9 from Fedora Update System --- jbosgi-repository-1.0.5-2.fc17 has been pushed to the Fedora 17 testing repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 808258] Review Request: python-sh - Python module to simplify calling shell commands
https://bugzilla.redhat.com/show_bug.cgi?id=808258 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #20 from Fedora Update System --- python-sh-0.107-1.fc17 has been pushed to the Fedora 17 testing repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 830709] Review Request: robert-hooke - Extension mechanism for Clojure functions
https://bugzilla.redhat.com/show_bug.cgi?id=830709 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2012-06-30 18:04:12 --- Comment #10 from Fedora Update System --- robert-hooke-1.2.0-2.fc16 has been pushed to the Fedora 16 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 812758] Review Request: trader - Star Traders, a simple game of interstellar trading
https://bugzilla.redhat.com/show_bug.cgi?id=812758 --- Comment #7 from Jason Tibbitts --- It's no problem. I'll be out of the country in a couple of weeks as well. This ticket isn't going anywhere. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 829708] Review Request: clojure-compat - A dynamic programming language that targets the Java Virtual Machine
https://bugzilla.redhat.com/show_bug.cgi?id=829708 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2012-06-30 18:03:57 --- Comment #11 from Fedora Update System --- clojure-compat-1.2.1-2.fc17 has been pushed to the Fedora 17 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 825825] Review Request: struts - Web application framework
https://bugzilla.redhat.com/show_bug.cgi?id=825825 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #14 from Fedora Update System --- struts-1.3.10-3.fc17 has been pushed to the Fedora 17 testing repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 811849] Review Request: python-cdb - Constant database library adapted as a python extension module
https://bugzilla.redhat.com/show_bug.cgi?id=811849 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #8 from Fedora Update System --- Package python-cdb-0.34-3.fc17: * should fix your issue, * was pushed to the Fedora 17 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=updates-testing python-cdb-0.34-3.fc17' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-2012-10072/python-cdb-0.34-3.fc17 then log in and leave karma (feedback). -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 815098] Review Request: maven-processor-plugin - Maven Processor Plugin
https://bugzilla.redhat.com/show_bug.cgi?id=815098 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2012-06-30 17:50:11 --- Comment #10 from Fedora Update System --- maven-processor-plugin-2.0.5-2.fc17 has been pushed to the Fedora 17 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 819953] Review Request: lightdm - Lightweight Display Manager
https://bugzilla.redhat.com/show_bug.cgi?id=819953 Fedora Update System changed: What|Removed |Added Resolution|RAWHIDE |ERRATA --- Comment #19 from Fedora Update System --- lightdm-kde-0.1.1-6.fc17, lightdm-gtk-1.1.6-3.fc17, lightdm-1.2.2-15.fc17 has been pushed to the Fedora 17 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 835432] Review Request: python-pycxx - Write Python extensions in C++
https://bugzilla.redhat.com/show_bug.cgi?id=835432 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #36 from Fedora Update System --- python-pycxx-6.2.4-2.fc17 has been pushed to the Fedora 17 testing repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 812758] Review Request: trader - Star Traders, a simple game of interstellar trading
https://bugzilla.redhat.com/show_bug.cgi?id=812758 --- Comment #6 from John Zaitseff --- Thank you, Jason, for your detailed comments: I do appreciate them! Unfortunately, you've caught me at a bad time: I will be overseas and out of e-mail/web contact for almost seven weeks, starting from a few hours' time (I'm about to fly out and, yes, there are still places in the world like that). I will be responding to your comments in detail once I return. In the meantime, just a few comments: * I like the simplified RPM spec file, and given that I already have split off OpenSUSE specifics into another file, it will be easy to do, too. * I did not realise that bundling gnulib had to be declared. I'm also happy to work out a way NOT to bundle (or, rather, to use the Fedora gnulib instead of the bundled one)---once I return from overseas. * gperf is used (or, at least, was used) by gnulib to create iconv_open header files. The current lib/Makefile.am lists the files iconv_open-aix.h,iconv_open-hpux.h, iconv_open-irix.h, iconv_open-osf.h and iconv_open-solaris.h as requiring gperf; I think this list was larger in older versions of gnulib. I'd like to work on this now, but I have a flight to catch... -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 810859] Review Request: python3-dateutil - Powerful extensions to the standard datetime module
https://bugzilla.redhat.com/show_bug.cgi?id=810859 Thomas Spura changed: What|Removed |Added Flags||fedora-cvs? --- Comment #8 from Thomas Spura --- (In reply to comment #7) > [!]: SHOULD > http://fedoraproject.org/wiki/Packaging/ > Guidelines#All_patches_should_have_an_upstream_bug_link_or_comment > [!]: You may re-version the patch to 2.0. I'll do so, when importing, thanks again for the review. Also adding all current python-dateutil maintainers. If anyone doesn't want to (co-)maintain it, they need to remove themself again... New Package SCM Request === Package Name: python3-dateutil Short Description: Powerful extensions to the standard datetime module Owners: tomspur jspaleta limb Branches: f17 InitialCC: -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 812758] Review Request: trader - Star Traders, a simple game of interstellar trading
https://bugzilla.redhat.com/show_bug.cgi?id=812758 --- Comment #5 from Jason Tibbitts --- By the way, if I were to submit this, the spec would look something like the following, which is about as simple as a spec can get for a package with translations: http://www.math.uh.edu/~tibbs/rpms/trader.spec -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 812758] Review Request: trader - Star Traders, a simple game of interstellar trading
https://bugzilla.redhat.com/show_bug.cgi?id=812758 Jason Tibbitts changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|nob...@fedoraproject.org|ti...@math.uh.edu Flags||fedora-review? --- Comment #4 from Jason Tibbitts --- This is cool. The spec is... a bit overblown. To be honest, I'd edit out the comments and the excessive macro-ization to get a spec that's very readable and only 32 lines (excluding the changelog). In particular your macroization of Release: will interfere with our automatic rebuild stuff. In any case, this builds fine and rpmlint is silent. BuildRoot: is unnecessary in Fedora, as is the %clean section and the %defattr in the %files section. Your source bundles gnulib. That is permitted according to policy but it is required that you include Provides: bundled(gnulib). http://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries#Packages_granted_exceptions Is the dependency on gperf necessary? I can't see anywhere in the build where gperf is called, and when I build without it the package doesn't appear any different. I'll run through the rest of my checklist. * source files match upstream. sha256sum: b91e74b0fc5eee7dc61ca2d02e2bf04c64ac1c71bf858b5858fc408bfe9ad33a trader-7.4.tar.gz * package meets naming and versioning guidelines. X specfile is is a bit messy and has excessive macroization. * summary is OK. * description is OK. * dist tag is present. * license field matches the actual license. * license is open source-compatible. * license text included in package. * latest version is being packaged. ? BuildRequires has extra gperf. * compiler flags are appropriate. * package builds in mock (rawhide, x86_64). * package installs properly. * debuginfo package looks complete. * rpmlint is silent. X final provides and requires missing bundled(gnulib): trader = 7.4-1.fc18 trader(x86-64) = 7.4-1.fc18 = libncursesw.so.5()(64bit) libtinfo.so.5()(64bit) * %check is not present; no test suite upstream. Manually tests OK. X bundles gnulib without indicating such. * owns the directories it creates. * doesn't own any directories it shouldn't. * no duplicates in %files. * file permissions are appropriate. * no generically named files. * code, not content. * documentation is small, so no -doc subpackage is necessary. * %docs are not necessary for the proper functioning of the package. * no static libraries. * no libtool .la files. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 823075] Review Request: php-symfony2-Security - Symfony2 Security Component
https://bugzilla.redhat.com/show_bug.cgi?id=823075 Shawn Iwinski changed: What|Removed |Added Flags||fedora-cvs? --- Comment #11 from Shawn Iwinski --- New Package SCM Request === Package Name: php-symfony2-Security Short Description: Symfony2 Security Component Owners: siwinski Branches: f16 f17 el6 InitialCC: -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 823066] Review Request: php-symfony2-Validator - Symfony2 Validator Component
https://bugzilla.redhat.com/show_bug.cgi?id=823066 --- Comment #14 from Shawn Iwinski --- Updates per comment #12 - Removed APC require - Updated %description SPEC URL: http://siwinski.fedorapeople.org/rpmbuild/SPECS/php-symfony2-Validator.spec SRPM URL: http://siwinski.fedorapeople.org/rpmbuild/SRPMS/php-symfony2-Validator-2.0.15-4.fc17.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 825414] Review Request: flexiport - Flexible communications library
https://bugzilla.redhat.com/show_bug.cgi?id=825414 Rich Mattes changed: What|Removed |Added Whiteboard|NotReady| --- Comment #2 from Rich Mattes --- Whoops! Sorry about that. SRPM is at http://rmattes.fedorapeople.org/RPMS/flexiport/flexiport-2.0.0-1.20111231git.fc17.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 828188] Review Request: reprepro - Debian package repository producer
https://bugzilla.redhat.com/show_bug.cgi?id=828188 --- Comment #3 from Fabian Affolter --- Some additional comments to those from Cédric. - The latest release is 4.12.3 - Isn't openssl and gpgme picked automatically by rpm? Already mentioned in Comment 5 of 624023 - man pages doesn't need to be marked as %doc - %defattr doesn't follow the Fedora guidelines -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 828188] Review Request: reprepro - Debian package repository producer
https://bugzilla.redhat.com/show_bug.cgi?id=828188 Fabian Affolter changed: What|Removed |Added CC||kana...@kanarip.com --- Comment #2 from Fabian Affolter --- *** Bug 624023 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 803089] Review Request: whenjobs - Replacement for cron with dependencies
https://bugzilla.redhat.com/show_bug.cgi?id=803089 --- Comment #27 from Michael Scherer --- Yup, this laptop is 32 bits. Let me start again mock and go on with the review. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 810859] Review Request: python3-dateutil - Powerful extensions to the standard datetime module
https://bugzilla.redhat.com/show_bug.cgi?id=810859 Robin Lee changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #7 from Robin Lee --- [!]: SHOULD http://fedoraproject.org/wiki/Packaging/Guidelines#All_patches_should_have_an_upstream_bug_link_or_comment [!]: You may re-version the patch to 2.0. But those issues and the previous one are not serious blockers. Approved! -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 734248] Review Request: apf - Adventure PHP Framework
https://bugzilla.redhat.com/show_bug.cgi?id=734248 --- Comment #5 from Reiner Rottmann --- I've updated the apf.spec file and created new rpm packages. Spec URL: http://www.rottmann.it/apf/review/apf.spec SRPM URL: http://www.rottmann.it/apf/review/apf-1.15-1.fc17.src.rpm RPM URL: http://www.rottmann.it/apf/review/apf-1.15-1.fc17.noarch.rpm [rottmrei@fedora SPECS]$ rpmlint -i /home/rottmrei/rpmbuild/RPMS/noarch/apf-1.15-1.fc17.noarch.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. [rottmrei@fedora SPECS]$ rpmlint -i /home/rottmrei/rpmbuild/SRPMS/apf-1.15-1.fc17.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. [rottmrei@fedora SRPMS]$ mock apf-1.15-1.fc17.src.rpm INFO: mock.py version 1.1.22 starting... State Changed: init plugins INFO: selinux enabled State Changed: start INFO: Start(apf-1.15-1.fc17.src.rpm) Config(fedora-17-x86_64) State Changed: lock buildroot State Changed: clean INFO: chroot (/var/lib/mock/fedora-17-x86_64) unlocked and deleted State Changed: unlock buildroot State Changed: init State Changed: lock buildroot Mock Version: 1.1.22 INFO: Mock Version: 1.1.22 INFO: calling preinit hooks INFO: enabled root cache State Changed: unpacking root cache INFO: enabled yum cache State Changed: cleaning yum metadata INFO: enabled ccache State Changed: running yum State Changed: unlock buildroot INFO: Installed packages: State Changed: setup State Changed: build INFO: Done(apf-1.15-1.fc17.src.rpm) Config(default) 0 minutes 41 seconds INFO: Results and/or logs in: /var/lib/mock/fedora-17-x86_64/result State Changed: end -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 803089] Review Request: whenjobs - Replacement for cron with dependencies
https://bugzilla.redhat.com/show_bug.cgi?id=803089 --- Comment #26 from Richard W.M. Jones --- Koji scratch build which I guess proves that it builds on 32 bit ... http://koji.fedoraproject.org/koji/taskinfo?taskID=4209135 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 803089] Review Request: whenjobs - Replacement for cron with dependencies
https://bugzilla.redhat.com/show_bug.cgi?id=803089 --- Comment #25 from Richard W.M. Jones --- Spec URL: http://oirase.annexia.org/reviews/whenjobs/whenjobs.spec SRPM URL: http://oirase.annexia.org/reviews/whenjobs/whenjobs-0.7.3-1.fc17.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 803089] Review Request: whenjobs - Replacement for cron with dependencies
https://bugzilla.redhat.com/show_bug.cgi?id=803089 --- Comment #24 from Richard W.M. Jones --- Are you using 32 bit? This patch was required for 32 bit ARM: commit d6da1b74e241e79eb0af9c01e390e98ceead3a49 Author: Richard W.M. Jones Date: Sat Apr 28 20:40:39 2012 +0100 32 bit: Fix for 31 bit int overflow in time_t. so I'm going to do a 0.7.3 release in a moment which includes everything. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 836014] Review Request: templates_parser - template library from AWS
https://bugzilla.redhat.com/show_bug.cgi?id=836014 --- Comment #8 from Björn Persson --- This package looks OK to me, so I pass the ball to Peter. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 834239] Review Request: monobristol - frontend for britsol in mono
https://bugzilla.redhat.com/show_bug.cgi?id=834239 --- Comment #12 from Jørn Lomax --- SPEC:http://jvlomax.fedorapeople.org/packeging/monobristol.spec SRPMS: http://jvlomax.fedorapeople.org/packeging/monobristol-0.60.3-6.fc17.src.rpm Same rpmlint output as above, minus "percent-in-%post" -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 823661] Review Request: lessfs - An inline data deduplicating filesystem
https://bugzilla.redhat.com/show_bug.cgi?id=823661 Fabian Affolter changed: What|Removed |Added CC||m...@fabian-affolter.ch Summary|Lessfs is an inline data|Review Request: lessfs - An |deduplicating filesystem. |inline data deduplicating ||filesystem -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 825854] Review Request: zita-alsa-pcmi - alsa pcm libraries
https://bugzilla.redhat.com/show_bug.cgi?id=825854 Jørn Lomax changed: What|Removed |Added Flags||fedora-cvs? --- Comment #32 from Jørn Lomax --- New Package SCM Request === Package Name: zita-alsa-pcmi Short Description: alsa pcm libraries Owners: jvlomax Branches: f16 f17 InitialCC: -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 829713] Review Request: grive - An open source Linux client for Google Drive
https://bugzilla.redhat.com/show_bug.cgi?id=829713 Fabian Affolter changed: What|Removed |Added CC||m...@fabian-affolter.ch Summary|grive - An open source |Review Request: grive - An |Linux client for Google |open source Linux client |Drive |for Google Drive -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 834551] Review Request: poppler-sharp C sharp Bindings for Poppler
https://bugzilla.redhat.com/show_bug.cgi?id=834551 Fabian Affolter changed: What|Removed |Added CC||m...@fabian-affolter.ch --- Comment #2 from Fabian Affolter --- rpmlint says: [fab@laptop11 SRPMS]$ rpmlint poppler-sharp-0.0.3-1.fc17.src.rpm poppler-sharp.src: I: enchant-dictionary-not-found es poppler-sharp.src:56: W: libdir-macro-in-noarch-package (main package) %{_libdir}/%{name}/%{name}.dll* poppler-sharp.src:60: W: libdir-macro-in-noarch-package devel %{_libdir}/pkgconfig/%{name}.pc poppler-sharp.src:10: W: mixed-use-of-spaces-and-tabs (spaces: line 10, tab: line 1) 1 packages and 0 specfiles checked; 0 errors, 3 warnings. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 803089] Review Request: whenjobs - Replacement for cron with dependencies
https://bugzilla.redhat.com/show_bug.cgi?id=803089 --- Comment #23 from Michael Scherer --- It doesn't seems to build in mock ( who has been running since 2/3h ). It seems to block at : /bin/bash - ./test_run.sh ./t100_counter.ml Is this expected ( ie, it take a long time, I am running it on a macbook 2.1, core 2 duo cpu, 2.1 ghz ) ? ( to reproduce, "fedora-review -b 803089" on fedora 17 should do the trick ). -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 810859] Review Request: python3-dateutil - Powerful extensions to the standard datetime module
https://bugzilla.redhat.com/show_bug.cgi?id=810859 --- Comment #6 from Thomas Spura --- Thanks for the review! (In reply to comment #5) > Issues: > [!]: SHOULD SourceX / PatchY prefixed with %{name}. > Note: Patch0: python-dateutil-1.5-system-zoneinfo.patch (python- > dateutil-1.5-system-zoneinfo.patch) Hmm, this blocker is new to me... :) Spec URL: http://tomspur.fedorapeople.org/review/python3-dateutil.spec SRPM URL: http://tomspur.fedorapeople.org/review/python3-dateutil-2.0-2.fc16.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 835099] Review Request: kde-base-artwork - KDE Base Artwork
https://bugzilla.redhat.com/show_bug.cgi?id=835099 --- Comment #15 from Simon A. Erat --- True that @ Jason. Changed my review script to use koji instead, which saves also the hazzle to localy install the required packages to build. But imho also takes the ability to verify the installation, which i had forgotton to test. @ Kevin, so now i wait for the update do a final check? Or do you want to take over? -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 834069] Review Request: clean - The Clean language compiler
https://bugzilla.redhat.com/show_bug.cgi?id=834069 --- Comment #16 from Patrick Uiterwijk --- I have improved the package(s) a bit more. The changes are: - Packaging less pre-compiled files (only the really needed pre-compiled files like _startup* and _system are still packaged) - cg, cocl and linker moved to libexec as per HFS - Folder names made consistent (Clean -> clean) - Subpackages for optional modules added New SPEC: http://puiterwijk.fedorapeople.org/packages/Clean/clean-7.spec New SRPM: http://puiterwijk.fedorapeople.org/packages/Clean/clean-2.4-7.fc17.src.rpm New Koji link: http://koji.fedoraproject.org/koji/taskinfo?taskID=4208876 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 734248] Review Request: apf - Adventure PHP Framework
https://bugzilla.redhat.com/show_bug.cgi?id=734248 --- Comment #4 from Reiner Rottmann --- Thank you for having a look at the spec file and your greatly appreciated help. I'm still interested in having the package included in Fedora. I want to learn about the overall process so that I may help in packaging other software, too. Thanks for pointing out the one ticket per package rule. I will continue improving the apf.spec file and will create another ticket for the apf-doc documentation package. Regarding the forbidden things: I will remove the echo commands. What do you mean by conflicting macro usage? With unnecessary buildroot stuff, you are refering to the mkdir commands, aren't you? I suppose you mean that the directories are already present because of the prerequisite rpm packages. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 803089] Review Request: whenjobs - Replacement for cron with dependencies
https://bugzilla.redhat.com/show_bug.cgi?id=803089 --- Comment #22 from Richard W.M. Jones --- No, that's not normal. All I can think is that I must have done a 'make dist' twice and included another copy of the tarball. Here is a corrected SRPM: http://oirase.annexia.org/reviews/whenjobs/whenjobs-0.7.2-1.fc17.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 822491] Review Request: erlang-sext - Sortable Erlang Term Serialization
https://bugzilla.redhat.com/show_bug.cgi?id=822491 --- Comment #11 from Fedora Update System --- erlang-sext-0.4.1-2.fc16 has been pushed to the Fedora 16 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 828732] Review Request: feedstail - A tail-f-like utility for feeds
https://bugzilla.redhat.com/show_bug.cgi?id=828732 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #12 from Fedora Update System --- feedstail-0.4.0-3.fc17 has been pushed to the Fedora 17 testing repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 810928] Review Request: sticky-notes - Sticky notes is a free and open source paste-bin application
https://bugzilla.redhat.com/show_bug.cgi?id=810928 --- Comment #35 from Fedora Update System --- sticky-notes-0.3.09062012.4-5.fc16 has been pushed to the Fedora 16 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 827761] Review Request: vdr-skinenigmang - A skin for VDR based on the Enigma text2skin add on
https://bugzilla.redhat.com/show_bug.cgi?id=827761 --- Comment #14 from Fedora Update System --- vdr-skinenigmang-0.1.2-5.fc16 has been pushed to the Fedora 16 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 821356] Review Request: papaki - A Java annotation scanner and repository
https://bugzilla.redhat.com/show_bug.cgi?id=821356 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2012-06-30 04:32:36 --- Comment #14 from Fedora Update System --- papaki-1.0.0-0.1.Beta3.fc17 has been pushed to the Fedora 17 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 832446] Review Request: arquillian-osgi - Arquillian OSGi
https://bugzilla.redhat.com/show_bug.cgi?id=832446 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #9 from Fedora Update System --- arquillian-osgi-1.0.2-2.fc17 has been pushed to the Fedora 17 testing repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 832439] Review Request: jbosgi-resolver - Standalone OSGi Resolver
https://bugzilla.redhat.com/show_bug.cgi?id=832439 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #7 from Fedora Update System --- jbosgi-resolver-2.0.0-0.2.Beta2.fc17 has been pushed to the Fedora 17 testing repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 822491] Review Request: erlang-sext - Sortable Erlang Term Serialization
https://bugzilla.redhat.com/show_bug.cgi?id=822491 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2012-06-30 04:28:11 --- Comment #10 from Fedora Update System --- erlang-sext-0.4.1-2.fc17 has been pushed to the Fedora 17 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 828223] Review Request: properties-maven-plugin - Properties Maven Plugin
https://bugzilla.redhat.com/show_bug.cgi?id=828223 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2012-06-30 04:27:34 --- Comment #9 from Fedora Update System --- properties-maven-plugin-1.0-0.1.alpha2.fc17 has been pushed to the Fedora 17 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 822472] Review Request: erlang-merge_index - An Erlang library for storing ordered sets on disk
https://bugzilla.redhat.com/show_bug.cgi?id=822472 --- Comment #11 from Fedora Update System --- erlang-merge_index-1.1.0-2.fc16 has been pushed to the Fedora 16 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 833154] Review Request: eclipse-wtp-jeetools - Frameworks and tools for Eclipse, focused on the development of J2EE artifacts
https://bugzilla.redhat.com/show_bug.cgi?id=833154 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #11 from Fedora Update System --- eclipse-wtp-jeetools-3.4.0-0.2.20120618cvs.fc17 has been pushed to the Fedora 17 testing repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 824838] Review Request: arquillian-core - Arquillian is a revolutionary testing platform built on the JVM
https://bugzilla.redhat.com/show_bug.cgi?id=824838 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2012-06-30 04:26:36 --- Comment #14 from Fedora Update System --- arquillian-core-1.0.1-1.fc17 has been pushed to the Fedora 17 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 822472] Review Request: erlang-merge_index - An Erlang library for storing ordered sets on disk
https://bugzilla.redhat.com/show_bug.cgi?id=822472 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2012-06-30 04:25:32 --- Comment #10 from Fedora Update System --- erlang-merge_index-1.1.0-2.fc17 has been pushed to the Fedora 17 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 832810] Review Request: perl-Method-Signatures - Method and function declarations with signatures and no source filter
https://bugzilla.redhat.com/show_bug.cgi?id=832810 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #7 from Fedora Update System --- perl-Method-Signatures-20120523-2.fc16 has been pushed to the Fedora 16 testing repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 827761] Review Request: vdr-skinenigmang - A skin for VDR based on the Enigma text2skin add on
https://bugzilla.redhat.com/show_bug.cgi?id=827761 Fedora Update System changed: What|Removed |Added Resolution|NEXTRELEASE |ERRATA --- Comment #13 from Fedora Update System --- vdr-skinenigmang-0.1.2-5.fc17 has been pushed to the Fedora 17 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 810928] Review Request: sticky-notes - Sticky notes is a free and open source paste-bin application
https://bugzilla.redhat.com/show_bug.cgi?id=810928 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2012-06-30 04:23:37 --- Comment #34 from Fedora Update System --- sticky-notes-0.3.09062012.4-5.fc17 has been pushed to the Fedora 17 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 836708] New: Review Request: sugar-locosugar - a game for discovering how to use the mouse and keyboard
https://bugzilla.redhat.com/show_bug.cgi?id=836708 Bug ID: 836708 QA Contact: extras...@fedoraproject.org Severity: unspecified Version: rawhide Priority: unspecified CC: nott...@redhat.com, package-review@lists.fedoraproject.org Assignee: nob...@fedoraproject.org Summary: Review Request: sugar-locosugar - a game for discovering how to use the mouse and keyboard Regression: --- Story Points: --- Classification: Fedora OS: Linux Reporter: callka...@gmail.com Type: Bug Documentation: --- Hardware: Unspecified Mount Type: --- Status: NEW Component: Package Review Product: Fedora Spec URL: http://callkalpa.fedorapeople.org/sugar-locosugar/sugar-locosugar.spec SRPM URL: http://callkalpa.fedorapeople.org/sugar-locosugar/sugar-locosugar-3-1.fc17.src.rpm Description: sugar-locosugar is an activity for sugar which enables the users to discover how to use mouse and keyboard through a series of games. -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785471] Review Request: php-horde-Horde-Stream-Wrapper - Horde Stream wrappers
https://bugzilla.redhat.com/show_bug.cgi?id=785471 Remi Collet changed: What|Removed |Added Flags||fedora-review? --- Comment #3 from Remi Collet --- SHOULD: Use %{pear_name} where possible Consider adding %{pear_channel} MUST: [!]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the beginning of %install. [!]: MUST Requires correct, justified where necessary. BuildRequires and Requires same version of pear, according to package.xml BuildRequires: php-pear >= 1:1.7.0 Requires: php-pear >= 1:1.7.0 Also add php-common >= 5.2.0 (waiting for a better provides in php) [!]: MUST Package must own all directories that it creates. Missing %{pear_phpdir}/Horde %{pear_phpdir}/Horde/Stream %{pear_docdir}/Horde_Stream_Wrapper -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785471] Review Request: php-horde-Horde-Stream-Wrapper - Horde Stream wrappers
https://bugzilla.redhat.com/show_bug.cgi?id=785471 Remi Collet changed: What|Removed |Added Assignee|nob...@fedoraproject.org|fed...@famillecollet.com --- Comment #2 from Remi Collet --- Created attachment 595422 --> https://bugzilla.redhat.com/attachment.cgi?id=595422&action=edit php-horde-Horde-Stream-Wrapper-review.txt Generated by fedora-review 0.1.3 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 783294] Review Request: picolisp - Lisp Interpreter
https://bugzilla.redhat.com/show_bug.cgi?id=783294 --- Comment #9 from Kalpa Welivitigoda --- Spec URL: http://callkalpa.fedorapeople.org/picolisp/picolisp.spec SRPM URL: http://callkalpa.fedorapeople.org/picolisp/picolisp-3.1.0-1.fc17.src.rpm Now it seems ok -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 785442] Review Request: php-horde-Horde-Date - Horde Date package
https://bugzilla.redhat.com/show_bug.cgi?id=785442 --- Comment #6 from Remi Collet --- Please also update to 1.0.11 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 783294] Review Request: picolisp - Lisp Interpreter
https://bugzilla.redhat.com/show_bug.cgi?id=783294 --- Comment #8 from Kalpa Welivitigoda --- Spec URL: http://callkalpa.fedorapeople.org/picolisp/picolisp.spec SRPM URL: http://callkalpa.fedorapeople.org/picolisp/picolisp-3.1.0-1.fc16.src.rpm I did a scratch build for 3.1.0. 64 bit failed while 32 bit was a success. Can any of you help me spot the issue with 64 bit? -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review