package-review July 2012

package-review@lists.fedoraproject.org

1 participants
1403 discussions

[Bug 658754] Review Request: CUBRID - a very fast and reliable open source SQL database server.
by Red Hat Bugzilla 12 Jul '12

12 Jul '12

https://bugzilla.redhat.com/show_bug.cgi?id=658754 --- Comment #14 from Michael Scherer <misc(a)zarb.org> --- The spectool --get-files command work ok for me, and the file is correct. Can you look with file if that's not some html error message ? ( and then, look at it ) ? -- You are receiving this mail because: You are on the CC list for the bug.

1 0

[Bug 835103] New: Review Request: perl-Pod-Parser - Basic perl modules for handling Plain Old Documentation (POD)
by Red Hat Bugzilla 12 Jul '12

12 Jul '12

https://bugzilla.redhat.com/show_bug.cgi?id=835103 Bug ID: 835103 QA Contact: extras-qa(a)fedoraproject.org Severity: medium Version: rawhide Priority: medium CC: notting(a)redhat.com, package-review(a)lists.fedoraproject.org Assignee: nobody(a)fedoraproject.org Summary: Review Request: perl-Pod-Parser - Basic perl modules for handling Plain Old Documentation (POD) Regression: --- Story Points: --- Classification: Fedora OS: Linux Reporter: ppisar(a)redhat.com Type: --- Documentation: --- Hardware: All Mount Type: --- Status: NEW Component: Package Review Product: Fedora Spec URL: http://ppisar.fedorapeople.org/perl-Pod-Parser/perl-Pod-Parser.spec SRPM URL: http://ppisar.fedorapeople.org/perl-Pod-Parser/perl-Pod-Parser-1.51-1.fc18.… Description: This software distribution contains the packages for using Perl5 POD (Plain Old Documentation). See the "perlpod" and "perlsyn" manual pages from your Perl5 distribution for more information about POD. Fedora Account System Username: ppisar This is dual-lived Perl package. It will get built into F18 after sub-packaging the module from perl-5.16.0. -- You are receiving this mail because: You are on the CC list for the bug.

1 7

[Bug 835064] New: Review Request: perl-Devel-DProf - Deprecated Perl code profiler
by Red Hat Bugzilla 12 Jul '12

12 Jul '12

https://bugzilla.redhat.com/show_bug.cgi?id=835064 Bug ID: 835064 QA Contact: extras-qa(a)fedoraproject.org Severity: medium Version: rawhide Priority: medium CC: notting(a)redhat.com, package-review(a)lists.fedoraproject.org Assignee: nobody(a)fedoraproject.org Summary: Review Request: perl-Devel-DProf - Deprecated Perl code profiler Regression: --- Story Points: --- Classification: Fedora OS: Linux Reporter: ppisar(a)redhat.com Type: --- Documentation: --- Hardware: All Mount Type: --- Status: NEW Component: Package Review Product: Fedora Spec URL: http://ppisar.fedorapeople.org/perl-Devel-DProf/perl-Devel-DProf.spec SRPM URL: http://ppisar.fedorapeople.org/perl-Devel-DProf/perl-Devel-DProf-20110802.0… Description: The Devel::DProf package is a Perl code profiler. This will collect information on the execution time of a Perl script and of the subs in that script. This information can be used to determine which subroutines are using the most time and which subroutines are being called most often. This information can also be used to create an execution graph of the script, showing subroutine relationships. This module is deprecated and new users are advised to use Devel::NYTProf instead. Fedora Account System Username: ppisar ---- This package is intended to provide compatibility to perl 5.16.0 which is aiming F18. It conflicts with perl 5.14 in files location. -- You are receiving this mail because: You are on the CC list for the bug.

1 10

[Bug 710383] Review Request: Agda - Commandline for dependently typed functional language
by Red Hat Bugzilla 12 Jul '12

12 Jul '12

https://bugzilla.redhat.com/show_bug.cgi?id=710383 Shakthi Kannan <shakthimaan(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |VERIFIED Flags| |fedora-review+ --- Comment #7 from Shakthi Kannan <shakthimaan(a)gmail.com> --- Package Review ============== Key: - = N/A x = Pass ! = Fail ? = Not evaluated ==== 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. Note: The package did not built BR could therefore not be checked or the package failed to build because of missing BR [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. [x]: MUST Requires correct, justified where necessary. [x]: MUST Rpmlint output is silent. [x]: MUST Sources used to build the package match the upstream source, as provided in the spec URL. /home/shaks/710383/Agda-executable-2.3.0.1.tar.gz : MD5SUM this package : a9c803f0a829cf54d35b1a82f0ba6181 MD5SUM upstream package : a9c803f0a829cf54d35b1a82f0ba6181 [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. [x]: 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). [x]: SHOULD Package functions as described. [x]: SHOULD Latest version is packaged. [x]: SHOULD Package does not include license text files separate from upstream. [x]: SHOULD SourceX is a working URL. [x]: SHOULD Description and summary sections in the package spec file contains translations for supported Non-English languages, if available. [x]: SHOULD Package should compile and build into binary rpms on all supported architectures. [-]: SHOULD %check is present and all tests pass. [x]: SHOULD Packages should try to preserve timestamps of original installed files. [x]: SHOULD Spec use %global instead of %define. Generated by fedora-review 0.1.3 External plugins: Package approved. -- You are receiving this mail because: You are on the CC list for the bug.

1 0

[Bug 825496] New: Review Request: python-django-staticfiles - A Django app that provides helpers for serving static files
by Red Hat Bugzilla 12 Jul '12

12 Jul '12

https://bugzilla.redhat.com/show_bug.cgi?id=825496 Bug ID: 825496 QA Contact: extras-qa(a)fedoraproject.org Severity: medium Version: rawhide Priority: medium CC: notting(a)redhat.com, package-review(a)lists.fedoraproject.org Assignee: nobody(a)fedoraproject.org Summary: Review Request: python-django-staticfiles - A Django app that provides helpers for serving static files Regression: --- Story Points: --- Classification: Fedora OS: Linux Reporter: domingobecker(a)gmail.com Type: --- Documentation: --- Hardware: All Mount Type: --- Status: NEW Component: Package Review Product: Fedora Spec URL: http://beckerde.fedorapeople.org/transifex/python-django-staticfiles.spec SRPM URL: http://beckerde.fedorapeople.org/transifex/python-django-staticfiles-0.3.2-… Description: This is a Django app that provides helpers for serving static files. The main website for django-staticfiles is github.com/jezdez/django-staticfiles where you can also file tickets. Fedora Account System Username: beckerde -- You are receiving this mail because: You are on the CC list for the bug.

1 9

[Bug 658754] Review Request: CUBRID - a very fast and reliable open source SQL database server.
by Red Hat Bugzilla 12 Jul '12

12 Jul '12

https://bugzilla.redhat.com/show_bug.cgi?id=658754 --- Comment #13 from Cristian Ciupitu <cristian.ciupitu(a)yahoo.com> --- I redownloaded cubrid.spec[1] then the source files using `spectool --get-files ~/rpmbuild/SPECS/cubrid.spec`. Unfortunately `tar tzf cubrid-8.4.1.2032.3.tar.gz` says: gzip: stdin: unexpected end of file tar: Unexpected EOF in archive tar: Error is not recoverable: exiting now [1] http://switch.dl.sourceforge.net/project/cubrid/CUBRID-8.4.1/Linux/Fedora-R… -- You are receiving this mail because: You are on the CC list for the bug.

1 0

[Bug 658754] Review Request: CUBRID - a very fast and reliable open source SQL database server.
by Red Hat Bugzilla 12 Jul '12

12 Jul '12

https://bugzilla.redhat.com/show_bug.cgi?id=658754 --- Comment #12 from Esen Sagynov <esen.sagynov(a)nhn.com> --- Hello Cristian, First of all, thank you for your continuos support on IRC. You feedback was very valuable for us to improve our CUBRID spec. Below please find our last changes we've made according to your comments. 1) Regarding macros, we've replaced %{_prefix}/share/ with %{_datarootdir} and %{_prefix}/include -> %{_includedir} 2) "Some files under /usr/include and share are owned by cubrid even if they shouldn't." /usr/include directory is not owned by cubrid user. Only share dir was. But when we tried to chown root:root, we found out that cubrid master service checks if the files are owned by the user who is running the process. When root:root, the cubrid master service fails to load saying “Current user does not match CUBRID user”. Therefore, we had to roll back and chown cubrid:cubrid the shared files. We have reported this issues in our Issue Tracker, so we will look into it in the future version. 4) Regarding "devel-file-in-non-devel-package", like I explained before some libs must be in the main package because CUBRID uses them, otherwise, CUBRID will not work; we moved all libs to devel package which were possible to move. 5) Regarding, "put all commands under one giant command command like git does", the "cubrid" command actually does exactly that. This is that giant command. 6) “strip --strip-unneded”: we have completely removed strip commands, which now gives us only warnings saying "W: unstripped-binary-or-object". 7) This allows us at least have the debuginfo package created and it has no errors related to "empty-debuginfo-package". 8) We now keep changelogs up to date. Also spec release number is always UP'ed and file version is updated if necessary. 9) According to your suggestion, we added conditionals to handle different Java versions for different versions of Fedora. 10) For new Fedora we use systemd now, and for old including RH distributions use "SysV scripts". 11) "Replace %{libdir} with the standard %{_libdir} macro". As I explained before, we cannot do this in this version because we have a hardcoded /usr/lib path which is used even on 64-bit system architecture. We will fix this in the CUBRID core in the future version. 12) We no longer start the program/service after installation. 13) "If possible, use a private /tmp in your systemd service unit". We didn't add "PrivateTmp=true" because "cubrid master" process failed to start if this line was set in cubrid.service. I am glad that so far CUBRID package is being successfully built at Fedora. Please let me know if there is anything we can do more to improve the CUBRID spec. Thank you Cristian! Regards, Esen Sagynov. -- You are receiving this mail because: You are on the CC list for the bug.

1 0

[Bug 838801] New: Review Request: rubygem-ref - Library that implements weak, soft, and strong references in Ruby
by Red Hat Bugzilla 12 Jul '12

12 Jul '12

https://bugzilla.redhat.com/show_bug.cgi?id=838801 Bug ID: 838801 QA Contact: extras-qa(a)fedoraproject.org Severity: medium Version: rawhide Priority: medium CC: notting(a)redhat.com, package-review(a)lists.fedoraproject.org Assignee: nobody(a)fedoraproject.org Summary: Review Request: rubygem-ref - Library that implements weak, soft, and strong references in Ruby Regression: --- Story Points: --- Classification: Fedora OS: Linux Reporter: vondruch(a)redhat.com Type: --- Documentation: --- Hardware: All Mount Type: --- Status: NEW Component: Package Review Product: Fedora Spec URL: http://people.redhat.com/vondruch/rubygem-ref.spec SRPM URL: http://people.redhat.com/vondruch/rubygem-ref-1.0.0-1.fc18.src.rpm Description: Library that implements weak, soft, and strong references in Ruby that work across multiple runtimes (MRI, REE, YARV, Jruby, Rubinius, and IronRuby). Also includes implementation of maps/hashes that use references and a reference queue. Fedora Account System Username: vondruch Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=4229359 -- You are receiving this mail because: You are on the CC list for the bug.

1 6

[Bug 710383] Review Request: Agda - Commandline for dependently typed functional language
by Red Hat Bugzilla 12 Jul '12

12 Jul '12

https://bugzilla.redhat.com/show_bug.cgi?id=710383 --- Comment #6 from Jens Petersen <petersen(a)redhat.com> --- I see Debian has an "agda" meta-package that pulls in Agda-executable, Agda and the stdlib. -- You are receiving this mail because: You are on the CC list for the bug.

1 0

[Bug 739088] New: Review Request: arandr - Simple GTK+ XRandR GUI
by Red Hat Bugzilla 12 Jul '12

12 Jul '12

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: arandr - Simple GTK+ XRandR GUI https://bugzilla.redhat.com/show_bug.cgi?id=739088 Summary: Review Request: arandr - Simple GTK+ XRandR GUI Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: unspecified Component: Package Review AssignedTo: nobody(a)fedoraproject.org ReportedBy: mzatko(a)redhat.com QAContact: extras-qa(a)fedoraproject.org CC: notting(a)redhat.com, package-review(a)lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: http://v3.sk/~hexo/rpm/arandr.spec SRPM URL: http://v3.sk/~hexo/rpm/arandr-0.1.4-1.fc15.src.rpm Description: ARandR is designed to provide a simple visual front end for XRandR 1.2/1.3. Relative monitor positions are shown graphically and can be changed in a drag-and-drop way. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.

1 23

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

package-review July 2012