[Bug 755275] New: Review Request: CSSLint - Detecting potential problems in CSS code
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: CSSLint - Detecting potential problems in CSS code https://bugzilla.redhat.com/show_bug.cgi?id=755275 Summary: Review Request: CSSLint - Detecting potential problems in CSS code Product: Fedora Version: rawhide Platform: Unspecified OS/Version: Unspecified Status: NEW Severity: unspecified Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: llaum...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: https://projects.llaumgui.com/p/rpmbuild/source/tree/master/SPECS/csslint.spec f SRPM URL: http://llaumgui.fedorapeople.org/review/csslint/csslint-0.8.1-1.fc16.noarch.rpm Project URL: https://github.com/stubbornella/csslint Description: CSSLint is a tool to help point out problems with your CSS code. It does basic syntax checking as well as applying a set of rules to the code that look for problematic patterns or signs of inefficiency. The rules are all pluggable, so you can easily write your own or omit ones you don't want. rpmlint output: builder@enterprise ~> rpmlint rpmbuild/**/*.rpm csslint.noarch: W: spelling-error %description -l en_US pluggable -> plug gable, plug-gable, plugged csslint.noarch: W: no-manual-page-for-binary csslint csslint.src: W: spelling-error %description -l en_US pluggable -> plug gable, plug-gable, plugged csslint.src: W: strange-permission csslint.sh 0755L 2 packages and 0 specfiles checked; 0 errors, 4 warnings. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728256] Review Request: rubygem-activesupport2.3 - rails 2 alongside rails 3
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=728256 --- Comment #4 from Emanuel Rietveld 2011-11-20 04:50:06 EST --- Tar gz of everything in my rails23 repo, most recent commit: http://xls01.freecult.org/pkg/?p=pkg.git;a=snapshot;h=rails23;sf=tgz Direct link to current source rpm: http://xls01.freecult.org/pkg/?p=pkg.git;a=blob_plain;f=SRPMS/rubygem-activesupport2.3-2.3.14-1.fc15.src.rpm;h=rails23 I do like keeping my packaging work in git, but I still need to get some more userfriendly interface. Thanks Emanuel -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 753597] Review Request: yazpp - C++ API for YAZ
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=753597
Martin Gieseking changed:
What|Removed |Added
CC||martin.giesek...@uos.de
--- Comment #3 from Martin Gieseking 2011-11-20
05:33:04 EST ---
Ryan, here are a couple of initial comments:
- please always check your rpms (source AND binary packages) with rpmlint:
$ rpmlint ./yazpp*.rpm
yazpp.x86_64: E: devel-dependency libyaz-devel
yazpp.x86_64: E: library-without-ldconfig-postin /usr/lib64/libyazpp.so.4.0.0
yazpp.x86_64: E: library-without-ldconfig-postun /usr/lib64/libyazpp.so.4.0.0
yazpp.x86_64: E: library-without-ldconfig-postin /usr/lib64/libzoompp.so.4.0.0
yazpp.x86_64: E: library-without-ldconfig-postun /usr/lib64/libzoompp.so.4.0.0
4 packages and 0 specfiles checked; 5 errors, 0 warnings.
- the Summary should be a bit more verbose ;)
- change the Group of the base package to "System Environment/Libraries"
- if you want to build the package for EPEL < 6 as well, adapt the BuildRoot
field according to
https://fedoraproject.org/wiki/EPEL/GuidelinesAndPolicies#BuildRoot_tag
If you targeting Fedora and/or EPEL 6 only, you can drop the BuildRoot tag,
the %clean section, the initial cleaning of the buildroot in %install, and
all %defattr lines
- Don't mix $RPM_BUILD_ROOT and %{buildroot}. Choose one variant and stick
with it.
- Why does the base package require libyaz-devel? I think you can drop it.
- replace "%package -n yazpp-devel" with "%package devel", and
"%description -n yazpp-devel" with "%description devel"
Same for %files:
replace "%files -n yazpp" with %files, and
"%files -n yazpp-devel" with "%files devel"
- if you want to keep the %defattr lines, change them to
%defattr(-,root,root,-)
- .la files must not be packaged in Fedora. Remove it in %install and drop it
from %files
- Static libraries must go to a separate -static package. However, Fedora
usually doesn't ship static libraries. If there's no good reason for
providing it, just add --disable-static to %configure, and remove the .a
file from %files.
- add "%post -p /sbin/ldconfig" and "%postun -p /sbin/ldconfig" between
%clean and %file section:
https://fedoraproject.org/wiki/Packaging/Guidelines#Shared_Libraries
- remove the echo line from the spec
- the devel package must require the base package this way:
https://fedoraproject.org/wiki/Packaging/Guidelines#Requiring_Base_Package
- Please be more specific in %files, especially if you add only few files.
This helps to avoid adding unwanted files and to get a quick overview of
what's actually put into the package. For example, instead of
%{_libdir}/*.so.* it's better to write:
%{_libdir}/libyazpp.so.*
%{_libdir}/libzoompp.so.*
- add file ChangeLog to the %docs
- The tarball contains a Doxyfile, so it's probably a good idea to build the
doxygen documentation and add it to the %docs. As there is quite a lot of
documentation available, I suggest to put it in a -doc subpackage.
- yazzpp provides some command-line tools, too (yaz-my-client, yaz-my-server,
zclient, zlint). Maybe you should add them to a -utils subpackage.
--
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 653917] Review Request: gimagereader - A tesseract OCR front-end
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=653917 --- Comment #4 from Sandro Mani 2011-11-20 07:12:56 EST --- I'm currently porting the application to gtk3, as soon as I'm done I'll revive this review request and perform informal reviews. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 755291] New: Review Request: -
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: - https://bugzilla.redhat.com/show_bug.cgi?id=755291 Summary: Review Request: - Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: cicku...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: SRPM URL: Description: -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 755291] Review Request: -
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=755291 Christopher Meng changed: What|Removed |Added Summary|Review Request: - |Review Request: || - -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 754879] Review Request: kup - Kernel.org Uploader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=754879 Josh Boyer changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #6 from Josh Boyer 2011-11-20 08:21:02 EST --- Thanks for the quick fixup. APPROVED. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 747051] Review Request: transmission-remote-gtk - GTK remote control for the Transmission BitTorrent client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=747051 --- Comment #4 from Praveen Kumar 2011-11-20 08:32:08 EST --- Done, Thanks for pointing out SPEC: http://kumarpraveen.fedorapeople.org/transmission-remote-gtk/transmission-remote-gtk.spec SRPM: http://kumarpraveen.fedorapeople.org/transmission-remote-gtk/transmission-remote-gtk-0.7-3.fc16.src.rpm -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 755093] Review Request: mactel-boot - boot tools for Intel Apple hardware
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=755093 --- Comment #5 from Matthew Garrett 2011-11-20 10:24:23 EST --- I didn't make the requirement on the logo hard because it's possible that some artwork packages don't have the bootloader icon, and enforcing the dependency there doesn't seem entirely reasonable. grub's obviously more reasonable, but given that we haven't done the transition yet it'd just be one more thing to update. Possibly justifiable. Blessing is a problem. It needs to be done every time the inode number changes (ie, every time the grub package gets updated), but we can't have any kind of pre-dependency in grub because this package is only going to be installed on a subset of machines. There may be a better way to do it, but I think it's something we'll only work out by iterating through a few implementations. I don't see a significant problem with /boot/grub2-efi, but we're going to have to disable module loading and just build everything in in order to support secure boot, so I don't see it being used for anything in an EFI world. In that case there's no reason to use grub2-efi-install instead of just doing what grub-efi does, and shipping the binary. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 747051] Review Request: transmission-remote-gtk - GTK remote control for the Transmission BitTorrent client
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=747051
Arun SAG changed:
What|Removed |Added
Flag||fedora-review+
--- Comment #5 from Arun SAG 2011-11-20 10:27:40 EST ---
You have forgot to remove
%preun
if [ $1 = 0 ] ; then
/sbin/install-info --delete %{_infodir}/%{name}.info %{_infodir}/dir || :
fi
You don't need this as you are not installing any info files in %post section.
Please remove above lines in spec file and rebuild them. Because of this line i
am getting following error while uninstalling
"install-info: No such file or directory for
/usr/share/info/transmission-remote-gtk.info"
Otherwise package looks fine
XX APPROVED XX
--
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 754749] Review Request: perl-Glib-Object-Introspection - Dynamically create Perl language bindings
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=754749 --- Comment #4 from Emmanuel Seyman 2011-11-20 10:32:50 EST --- This package fails to build so I can't properly start review. Daniel, you'll want to add a BuildRequires on perl-Glib-devel (Glib::MakeHelper is needed during build and that's the package it's in). FTR, It's probably best to test that your package builds, in mock and/or in a scratch koji build before submitting it. As Trond noted, you'll definitely want to include a MODULE_COMPAT Requires. Is there a reason you're forcing LANG=en_US.UTF8 ? If so, it's recommended to add a comment explaining why. Note that the defattr line is no longer needed in Fedora-only packages (but leaving it in isn't a blocker). -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 747051] Review Request: transmission-remote-gtk - GTK remote control for the Transmission BitTorrent client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=747051 Praveen Kumar changed: What|Removed |Added Flag||fedora-cvs? --- Comment #6 from Praveen Kumar 2011-11-20 11:07:55 EST --- New Package SCM Request === Package Name: transmission-remote-gtk Short Description: GTK remote control for the Transmission BitTorrent client Owners: kumarpraveen Branches: f15 f16 -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 695058] Review Request: transgui - An App to remotely control a Transmission Bit-Torrent client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=695058 --- Comment #30 from Fedora Update System 2011-11-20 11:21:34 EST --- transgui-3.2-5.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/transgui-3.2-5.fc16 -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 695058] Review Request: transgui - An App to remotely control a Transmission Bit-Torrent client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=695058 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 678634] Review Request: Saaghar - A Cross-Platform Persian Poetry Software
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=678634 --- Comment #5 from Hedayat Vatankhah 2011-11-20 11:34:24 EST --- Updated to the latest release 0.9.69 with some cleanups: SPEC: http://hedayat.fedorapeople.org/reviews/Saaghar/0.9.69-1/Saaghar.spec SRPM: http://hedayat.fedorapeople.org/reviews/Saaghar/0.9.69-1/Saaghar-0.9.69-1.fc16.src.rpm -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 749756] Review Request: zita-at1 - autotuner for JACK
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=749756 --- Comment #11 from Fedora Update System 2011-11-20 12:34:48 EST --- zita-at1-0.2.3-4.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/zita-at1-0.2.3-4.fc16 -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 749756] Review Request: zita-at1 - autotuner for JACK
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=749756 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 746848] Review Request: dbus-sharp - C# bindings for D-Bus
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=746848 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 746848] Review Request: dbus-sharp - C# bindings for D-Bus
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=746848 --- Comment #6 from Fedora Update System 2011-11-20 12:59:20 EST --- gnome-do-plugins-0.8.4-3.fc16.1,banshee-community-extensions-2.2.0-1.fc16,banshee-2.2.1-1.fc16,dbus-sharp-glib-0.5.0-1.fc16,dbus-sharp-0.7.0-3.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/gnome-do-plugins-0.8.4-3.fc16.1,banshee-community-extensions-2.2.0-1.fc16,banshee-2.2.1-1.fc16,dbus-sharp-glib-0.5.0-1.fc16,dbus-sharp-0.7.0-3.fc16 -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 748180] Review Request: dbus-sharp-glib - C# bindings for D-Bus glib main loop integration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=748180 Fedora Update System changed: What|Removed |Added Status|NEW |MODIFIED -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 748180] Review Request: dbus-sharp-glib - C# bindings for D-Bus glib main loop integration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=748180 --- Comment #13 from Fedora Update System 2011-11-20 12:59:15 EST --- gnome-do-plugins-0.8.4-3.fc16.1,banshee-community-extensions-2.2.0-1.fc16,banshee-2.2.1-1.fc16,dbus-sharp-glib-0.5.0-1.fc16,dbus-sharp-0.7.0-3.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/gnome-do-plugins-0.8.4-3.fc16.1,banshee-community-extensions-2.2.0-1.fc16,banshee-2.2.1-1.fc16,dbus-sharp-glib-0.5.0-1.fc16,dbus-sharp-0.7.0-3.fc16 -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 755291] Review Request: -
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=755291 Thomas Spura changed: What|Removed |Added CC||toms...@fedoraproject.org --- Comment #1 from Thomas Spura 2011-11-20 13:46:46 EST --- a) Did you want to propose this for others to include it in fedora? b) Or are you working on a spec and link it then here? In case of a: You can add it here: https://fedoraproject.org/wiki/Package_maintainers_wishlist In case of b: Here is documentation on how to package it on your own and becoming a contributor: https://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group#Submitting_quality_new_packages https://fedoraproject.org/wiki/How_to_create_an_RPM_package I hope you decide for case b :) -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 746754] Review request: pdfcrack - A Password Recovery Tool for PDF-files.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=746754 --- Comment #32 from Fedora Update System 2011-11-20 13:58:17 EST --- pdfcrack-0.11-7.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/pdfcrack-0.11-7.fc16 -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 746754] Review request: pdfcrack - A Password Recovery Tool for PDF-files.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=746754 --- Comment #31 from Fedora Update System 2011-11-20 13:58:08 EST --- pdfcrack-0.11-7.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/pdfcrack-0.11-7.fc15 -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 755155] Review Request: perl-TAP-Harness-Multiple - TAP::Harness::Muliple Perl module
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=755155 --- Comment #2 from Gerd Pokorra 2011-11-20 14:00:44 EST --- Thank you for starting the review. - all "Requires" are removed, only the "BuildRequires" are still in the specfile - buildroot is removed, no cleaning of buildroot in "install section" new SRPM URL: ftp://ftp.uni-siegen.de/pub/review/perl-TAP-Harness-Multiple-0.06-2.fc15.src.rpm The spec URL: ftp://ftp.uni-siegen.de/pub/review/perl-TAP-Harness-Multiple.spec is now a link to ftp://ftp.uni-siegen.de/pub/review/perl-TAP-Harness-Multiple.spec.2 F17-scratch-build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3528016 -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 705587] Review Request: android-tools - Android platform tools (adb, fastboot, etc)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=705587 --- Comment #20 from Ivan Afonichev 2011-11-20 14:22:01 EST --- Spec URL: https://github.com/vanaf/android-tools-fedora/blob/master/android-tools.spec SRPM URL: http://baldr.sgu.ru/rpm/android-tools-2020git4a25390-1.fc16.src.rpm - Versioning changes - Use only needed sources - Udev rules moved to lib - More license info added - adb and fastboot moved to provides from summary It seems that git tags have very little relation to platform tools part of this git repository. Git tags also have no info about how next release will be called. libzipfile and libcutils seems to be used only in android project and there are not currently packaged in Fedora... -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 705587] Review Request: android-tools - Android platform tools (adb, fastboot, etc)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=705587 --- Comment #21 from nucleo 2011-11-20 14:48:46 EST --- /lib/udev/rules.d/ is owned by udev, so "Requires: udev" should be added. There are two issues with libzipfile: 1. libzipfile compiled in both in adb and fastboot 2. libzipfile have ASL 2.0 license but fastboot is BSD So building at least libzipfile as shared library will eliminate its code duplication in adb and fastboot and made two binaries with different licenses. Is it possible to build libzipfile as shared library? -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 747051] Review Request: transmission-remote-gtk - GTK remote control for the Transmission BitTorrent client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=747051 --- Comment #7 from Jon Ciesla 2011-11-20 15:15:10 EST --- Git done (by process-git-requests). -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 594844] Review Request: iec16022 - Generate ISO/IEC 16022 2D barcodes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=594844 Fedora Update System changed: What|Removed |Added Fixed In Version|iec16022-0.2.4-7.el4|iec16022-0.2.4-7.el6 --- Comment #28 from Fedora Update System 2011-11-20 15:18:42 EST --- iec16022-0.2.4-7.el6 has been pushed to the Fedora EPEL 6 stable repository. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 594844] Review Request: iec16022 - Generate ISO/IEC 16022 2D barcodes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=594844 Fedora Update System changed: What|Removed |Added Fixed In Version|iec16022-0.2.4-6.fc12 |iec16022-0.2.4-7.el4 --- Comment #27 from Fedora Update System 2011-11-20 15:18:32 EST --- iec16022-0.2.4-7.el4 has been pushed to the Fedora EPEL 4 stable repository. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 755093] Review Request: mactel-boot - boot tools for Intel Apple hardware
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=755093 --- Comment #6 from Mads Kiilerich 2011-11-20 15:17:38 EST --- (Discussion of the new grub2-efi model might be a bit off topic here, but a review of the correctness of the packaging of this special package do in my opinion depend a lot on it.) In my opinion the "interface" of system-logos is to provide whatever logos is needed for various purposes in the distribution. Packages can always drop in a picture of their favourite junk food if they can't come up with something better. I see - HFS blessing works just like a grub2 blocklist? That bless has to be rerun when grub.efi is changed only makes it more clear that bless shouldn't be run in %post of this package. A trigger on grub2-efi would perhaps work in most cases. (But if bless has to be run every time grub.efi changes, then there is no real benefit from the symlink, and we could just as well also copy grub.efi to the HFS partition before blessing. There is thus no need for mounting the Fedora bootstrap HFS partition as a fake EFI system partition on /boot/efi. Mounting it somewhere else would in my opinion be more transparent and less surprising.) I don't know how you plan to implement "secure" boot, but I assume it is incompatible with the idea of having a simple editable boot loader configuration file. Most users will prefer that the EFI firmware starts a grub.efi that trusts "everything" and doesn't extend the verification/chain of trust. It seems to me like "secure" boot will have to be a special cumbersome mode anyway. Should fonts and background images and i18n and grubenv also move from /boot/grub2-efi to the EFI system partition - and how? It _is_ possible to include a "ram disk" with everything in grub.efi, but that would either require a grub2-efi-install-ish step "on demand", or that the grub2-efi rpm includes a pre-built inflexible "ram disk". Do you want to do something like that? -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 750461] Review Request: php-phpunit-PHP-Invoker - Utility class for invoking callables with a timeout
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=750461 Fedora Update System changed: What|Removed |Added Fixed In Version|php-phpunit-PHP-Invoker-1.0 |php-phpunit-PHP-Invoker-1.0 |.0-3.fc15 |.0-3.el6 --- Comment #12 from Fedora Update System 2011-11-20 15:18:10 EST --- php-phpunit-PHP-Invoker-1.0.0-3.el6 has been pushed to the Fedora EPEL 6 stable repository. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 594844] Review Request: iec16022 - Generate ISO/IEC 16022 2D barcodes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=594844 Fedora Update System changed: What|Removed |Added Fixed In Version|iec16022-0.2.4-7.el6|iec16022-0.2.4-7.el5 --- Comment #29 from Fedora Update System 2011-11-20 15:19:22 EST --- iec16022-0.2.4-7.el5 has been pushed to the Fedora EPEL 5 stable repository. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 755093] Review Request: mactel-boot - boot tools for Intel Apple hardware
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=755093 --- Comment #7 from Matthew Garrett 2011-11-20 15:25:41 EST --- Yes, a trigger is necessary, but it's still also necessary to do it on first install. The symlink is there so that we don't have two copies of grub.efi that can get out of sync. It's perfectly valid to have a configuration file that allows arbitrary configuration in a secure boot environment, but that configuration file should be living on the same filesystem as grub.efi. /boot/grub2-efi isn't necessary for that. Users may well prefer to have a grub.efi that trusts everything, and I'm sure we'll be able to provide tools for them to do that, but it's not going to be the default when run under a secure boot environment. Theming data can live where. For simplicity's sake it's still going to be easier to have it on the same filesystem as grub itself, but there's no hard requirement for that. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 187318] Review Request: mondo - A program which a Linux user can utilize to create a rescue/restore CD/tape
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=187318 Tadej Janež changed: What|Removed |Added CC||tadej.janez@tadej.hicsalta. ||si --- Comment #54 from Tadej Janež 2011-11-20 15:27:50 EST --- Hi! I don't want to sound very pessimistic, but I see at least two roadblocks here: - afio package (required by mondo) has been rejected from Fedora due to its licensing issues (see https://bugzilla.redhat.com/show_bug.cgi?id=449037#c26) - buffer package (also required by mindi) has been retired from Fedora on 2011-07-25 due to it being unable to build for multiple releases (see http://pkgs.fedoraproject.org/gitweb/?p=buffer.git;a=blob;f=dead.package;h=6c90bdc8768fc421bae28dabe5c08e13cbcd8a84;hb=4a2e8f905b7e3ee5d6c199b6d7e370f80cc9f18f) Does this still apply to the Mondo 3.0? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 755093] Review Request: mactel-boot - boot tools for Intel Apple hardware
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=755093 --- Comment #8 from Matthew Garrett 2011-11-20 15:27:07 EST --- Actually, having re-read the docs, doing this in a %trigger ought to work. I'll implement it that 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 651591] Review Request: partiwm - partitioning window manager and related tools
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=651591 --- Comment #11 from Pavel Alexeev (aka Pahan-Hubbitus) 2011-11-20 15:30:45 EST --- Sorry, but it is xpra included there is also it separate project http://code.google.com/p/partiwm/wiki/xpra ? Or not? -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 754064] Review Request: python-mimeparse - Python module providing basic functions for parsing mime-type names
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=754064
Jaroslav Škarvada changed:
What|Removed |Added
CC||lmac...@redhat.com,
||methe...@gmail.com,
||mru...@fedoraproject.org,
||pj.pan...@yahoo.co.in,
||sanjay.an...@gmail.com
--- Comment #1 from Jaroslav Škarvada 2011-11-20 15:55:46
EST ---
MUST items:
[YES] rpmplint is silent
[YES] Package meets naming guidelines.
[!] Package meets packaging guidelines.
CFLAGS are set for the python3 build but not for the python2* build.
Maybe it is not needed at all, but I am not sure about it.
[YES] Spec file matches base package name.
[YES] Licensing Guidelines are met.
[YES] Spec file is legible and in American English.
I would prefer summary like: "Python module for parsing mime-type names" or
similar.
[YES] Sources match upstream.
[YES] Package builds OK.
[!] BuildRequires are correct.
I cannot find the python-setuptools-devel in rawhide.
Is the python-setuptools really needed? It seems to build OK without it.
[YES] Package doesn't bundle copies of system libraries.
[YES] Package owns all the directories it creates.
[YES] Package has no duplicity in %files.
[YES] Permission on files are set properly.
[NO] Spec file has consistant macro usage.
Please use %{optflags} instead of $RPM_OPT_FLAGS or $RPM_BUILD_ROOT instead
of %{buildroot}.
[YES] Package is code or permissable content.
[YES] %doc files don't affect runtime.
[YES] Package doesn't own files/directories that other packages own.
[YES] All files are valid UTF-8.
Should items:
[YES] Package builds in mock.
[YES] Package uses sane scriptlets.
[NO] If the source package does not include license text(s) as a separate file
from upstream, the packager SHOULD query upstream to include it.
Please query the upstream about it.
[YES] Simple functionality test passed.
It seems there are several packages that bundle this lib:
django-authopenid
python-paste
python3-paste
django-authenticator
askbot
I am not sure if it is possible to unbundle the lib in all of them (maybe it is
forked), but I am CCing the maintainers to at least let them know.
Some more comments:
%{!?python_sitelib: %global python_sitelib %(%{__python} -c "from
distutils.sysconfig import get_python_lib; print get_python_lib()")}
is probably not needed any more.
The defattr is also not needed.
AFAIK the above have only sense if it is planned to package for RHEL-5 EPEL. If
so there should be also more additions (e.g. %clean section, ...).
Please consider running the included test as part of the build process.
--
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 691195] Review Request: wmnet - Network monitoring dockapp
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=691195 --- Comment #8 from Mario Blättermann 2011-11-20 16:22:37 EST --- Thanks for your hints. New files: Spec URL: http://mariobl.fedorapeople.org/Review/SPECS/wmnet.spec SRPM URL: http://mariobl.fedorapeople.org/Review/SRPMS/wmnet-1.06-5.fc15.src.rpm A *.desktop file is unneded for such dockapps. See bug #478744, comment #4. Usually, a Windowmaker dockapp will be started from a script rather than a menu. Unfortunately, I'm currently unable to do a scratch build on Koji, due to an expired certificate. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 691195] Review Request: wmnet - Network monitoring dockapp
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=691195 --- Comment #9 from Mario Blättermann 2011-11-20 16:53:25 EST --- My certificate has been updated, Koji scratch build is here: http://koji.fedoraproject.org/koji/taskinfo?taskID=3528616 -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 755358] New: Review Request: jcal - Unix cal-like interface to libjalali
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: jcal - Unix cal-like interface to libjalali https://bugzilla.redhat.com/show_bug.cgi?id=755358 Summary: Review Request: jcal - Unix cal-like interface to libjalali Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: heday...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: http://hedayat.fedorapeople.org/reviews/jcal/0.4.1-1/jcal.spec SRPM URL: http://hedayat.fedorapeople.org/reviews/jcal/0.4.1-1/jcal-0.4.1-1.fc16.src.rpm Description: This package provides two applications: jcal and jdate which use libjalali for calendar calculation. jcal is a UNIX cal-like tool to display calendar based on Jalali calendar system. jdate is UNIX date-like tool to display date and time based on Jalali calendar system. Notice: libjalali is also provided by this package. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 624182] Review Request: stardict-xmllittre - Authoritative 19th century French dictionary
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=624182
Mario Blättermann changed:
What|Removed |Added
Flag|fedora-review?, |fedora-review+, needinfo-
|needinfo?(mario.blaetterman |
|n...@gmail.com)|
--- Comment #8 from Mario Blättermann 2011-11-20
17:22:38 EST ---
Yes, I agree.
PACKAGE APPROVED.
(In reply to comment #6)
> If you don't want to provide your package for EPEL <= 6, you may drop the
> following parts of your spec file:
>
> - the BuildRoot declaration.
> - the initial cleaning of ${RPM_BUILD_ROOT} in %install
> - the %defattr line in %files
Consider to remove the mentioned parts if you don't plan to package for older
EPELs, before you commit it to the Git repo.
--
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 749885] Review Request: iris - A library for working with the XMPP/Jabber protocol
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=749885 --- Comment #24 from Rex Dieter 2011-11-20 17:38:37 EST --- Spec URL: http://rdieter.fedorapeople.org/rpms/iris/iris.spec SRPM URL: http://rdieter.fedorapeople.org/rpms/iris/iris-1.0.0-0.8.20110904svn812.fc16.src.rpm %changelog * Wed Nov 16 2011 Rex Dieter 1.0.0-0.8.20110904svn812 - fix Release -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 718502] Review Request: Temperature.app - Window Maker applet which fetches local temperature information
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=718502 --- Comment #5 from Mario Blättermann 2011-11-20 17:48:08 EST --- Here's the new spec: http://mariobl.fedorapeople.org/Review/SPECS/Temperature.app.spec But there remains the problem with the dead source location. The dockapps.org registration is expired, and the domain has been moved to a domain parking system, that's why I don't expect to get it back in the nearest future. In other cases, I've used the source pool at the Debian servers, but there's no such package. Well, there's another pool for unmaintained dockapps at http://repo.or.cz/w/dockapps.git, perhaps I should use it. It also includes some bugfixes, seems to be the best way for me. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 718502] Review Request: Temperature.app - Window Maker applet which fetches local temperature information
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=718502 Mario Blättermann changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|pikachu.2...@gmail.com -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 624182] Review Request: stardict-xmllittre - Authoritative 19th century French dictionary
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=624182 --- Comment #9 from François Cami 2011-11-20 18:27:31 EST --- Thank you Mario. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 624182] Review Request: stardict-xmllittre - Authoritative 19th century French dictionary
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=624182 François Cami changed: What|Removed |Added Flag||fedora-cvs? --- Comment #10 from François Cami 2011-11-20 18:31:50 EST --- New Package SCM Request === Package Name: stardict-xmllittre Short Description: Authoritative 19th century French dictionary Owners: fcami Branches: f15 f16 InitialCC: -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 700834] Review Request: wmsystemtray - System tray (freedesktop.org systray protocol) as a Window Maker dock app
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=700834 --- Comment #2 from Mario Blättermann 2011-11-20 18:06:30 EST --- Thanks for the hints. New files: Spec URL: http://mariobl.fedorapeople.org/Review/SPECS/wmsystemtray.spec SRPM URL: http://mariobl.fedorapeople.org/Review/SRPMS/wmsystemtray-1.2-2.fc15.src.rpm Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3528684 -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 750013] Review Request: clucene09 - A C++ port of Lucene
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=750013 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||clucene09-0.9.21b-2.fc16 Resolution||ERRATA Last Closed||2011-11-20 18:54:24 --- Comment #12 from Fedora Update System 2011-11-20 18:54:24 EST --- clucene09-0.9.21b-2.fc16 has been pushed to the Fedora 16 stable repository. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 748180] Review Request: dbus-sharp-glib - C# bindings for D-Bus glib main loop integration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=748180 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #14 from Fedora Update System 2011-11-20 18:53:32 EST --- Package gnome-do-plugins-0.8.4-3.fc16.1, banshee-community-extensions-2.2.0-1.fc16, banshee-2.2.1-1.fc16, dbus-sharp-glib-0.5.0-1.fc16, dbus-sharp-0.7.0-3.fc16: * should fix your issue, * was pushed to the Fedora 16 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=updates-testing gnome-do-plugins-0.8.4-3.fc16.1 banshee-community-extensions-2.2.0-1.fc16 banshee-2.2.1-1.fc16 dbus-sharp-glib-0.5.0-1.fc16 dbus-sharp-0.7.0-3.fc16' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-2011-16171/gnome-do-plugins-0.8.4-3.fc16.1,banshee-community-extensions-2.2.0-1.fc16,banshee-2.2.1-1.fc16,dbus-sharp-glib-0.5.0-1.fc16,dbus-sharp-0.7.0-3.fc16 then log in and leave karma (feedback). -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 746848] Review Request: dbus-sharp - C# bindings for D-Bus
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=746848 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #7 from Fedora Update System 2011-11-20 18:53:37 EST --- Package gnome-do-plugins-0.8.4-3.fc16.1, banshee-community-extensions-2.2.0-1.fc16, banshee-2.2.1-1.fc16, dbus-sharp-glib-0.5.0-1.fc16, dbus-sharp-0.7.0-3.fc16: * should fix your issue, * was pushed to the Fedora 16 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=updates-testing gnome-do-plugins-0.8.4-3.fc16.1 banshee-community-extensions-2.2.0-1.fc16 banshee-2.2.1-1.fc16 dbus-sharp-glib-0.5.0-1.fc16 dbus-sharp-0.7.0-3.fc16' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-2011-16171/gnome-do-plugins-0.8.4-3.fc16.1,banshee-community-extensions-2.2.0-1.fc16,banshee-2.2.1-1.fc16,dbus-sharp-glib-0.5.0-1.fc16,dbus-sharp-0.7.0-3.fc16 then log in and leave karma (feedback). -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 746754] Review request: pdfcrack - A Password Recovery Tool for PDF-files.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=746754 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #33 from Fedora Update System 2011-11-20 18:55:44 EST --- pdfcrack-0.11-7.fc15 has been pushed to the Fedora 15 testing repository. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 695058] Review Request: transgui - An App to remotely control a Transmission Bit-Torrent client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=695058 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #31 from Fedora Update System 2011-11-20 18:55:54 EST --- transgui-3.2-5.fc16 has been pushed to the Fedora 16 testing repository. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 744948] Review Request: monochrome-icon-theme - Icons for the panel, designed in a simplified monochrome style
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=744948 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #19 from Fedora Update System 2011-11-20 18:58:31 EST --- monochrome-icon-theme-0.0.38-1.fc16 has been pushed to the Fedora 16 testing repository. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 749756] Review Request: zita-at1 - autotuner for JACK
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=749756 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #12 from Fedora Update System 2011-11-20 19:01:32 EST --- zita-at1-0.2.3-4.fc16 has been pushed to the Fedora 16 testing repository. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 187318] Review Request: mondo - A program which a Linux user can utilize to create a rescue/restore CD/tape
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=187318 --- Comment #55 from Bruno Cornec 2011-11-20 20:26:53 EST --- mondo 3.0.0 is requirinrg either afio or star. Hopefully with that last one, it will be possible to make progress. Concerning buffer, it's mandatory for tape usage. Now I don't understand the concept of "unable to build for multiple releases" and the link doesn't give much more info. I'm building buffer for multiple Fedora release myself without issue, so I guess it's related to something else. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 755392] New: Review Request: chirp - A tool for programming two-way radio equipment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: chirp - A tool for programming two-way radio equipment https://bugzilla.redhat.com/show_bug.cgi?id=755392 Summary: Review Request: chirp - A tool for programming two-way radio equipment Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: randyn3...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: http://dp67.fedorapeople.org/pkgs/SPECS/chirp.spec SRPM URL: http://dp67.fedorapeople.org/pkgs/SRPMS/chirp-0.1.12-3.fc17.src.rpm 2 packages and 1 specfiles checked; 0 errors, 0 warnings. Description: Chirp is a tool for programming two-way radio equipment It provides a generic user interface to the programming data and process that can drive many radio models under the hood. Written entirely in Python. Project Page: http://chirp.danplanet.com I thought this would be very beneficial to Fedora Hams, but I do not have the equipment to test this application. I can only build it and confirm the basics. Anyone who has the radios this software claims to work with (list on project page) please test it and submit your findings here. Thanks. Test Options: http://dp67.fedorapeople.org/pkgs/RPMS/noarch/Rawhide/chirp-0.1.12-3.fc17.noarch.rpm http://dp67.fedorapeople.org/pkgs/RPMS/noarch/Fedora-16/chirp-0.1.12-3.fc16.noarch.rpm http://dp67.fedorapeople.org/pkgs/RPMS/noarch/Fedora-15/chirp-0.1.12-3.fc15.noarch.rpm *note packages are unsigned. Install with yum --nogpgcheck -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 753027] Review Request: pam-pgsql - PAM module to authenticate using a PostgreSQL database
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=753027 Robert Scheck changed: What|Removed |Added CC||redhat-bugzilla@linuxnetz.d ||e AssignedTo|nob...@fedoraproject.org|redhat-bugzilla@linuxnetz.d ||e Flag||fedora-review? --- Comment #6 from Robert Scheck 2011-11-20 21:17:27 EST --- Mathieu, your %post/%postun doesn't make sense for PAM at all, it's meant for libraries only. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 755155] Review Request: perl-TAP-Harness-Multiple - TAP::Harness::Muliple Perl module
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=755155 --- Comment #3 from Gerd Pokorra 2011-11-20 21:43:32 EST --- - a patch for the LICENSE file is added to fix the problem new SRPM URL: ftp://ftp.uni-siegen.de/pub/review/perl-TAP-Harness-Multiple-0.06-3.fc15.src.rpm The spec URL: ftp://ftp.uni-siegen.de/pub/review/perl-TAP-Harness-Multiple.spec is now a link to ftp://ftp.uni-siegen.de/pub/review/perl-TAP-Harness-Multiple.spec.3 F17-scratch-build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3528760 rpmlint on rpms now gave 0 errors and 0 warnings -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 755141] Review Request: gnome-system-log - A log viewer for GNOME
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=755141 Parag AN(पराग) changed: What|Removed |Added Summary|gnome-system-log - A log|Review Request: |viewer for GNOME|gnome-system-log - A log ||viewer for GNOME -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 705249] Review Request: ibus-tutcode - Japanese TUT-Code input method for ibus
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=705249 --- Comment #4 from Fedora Update System 2011-11-20 22:19:50 EST --- ibus-tutcode-1.0.1-1.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/ibus-tutcode-1.0.1-1.fc16 -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 705249] Review Request: ibus-tutcode - Japanese TUT-Code input method for ibus
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=705249 --- Comment #5 from Fedora Update System 2011-11-20 22:19:58 EST --- ibus-tutcode-1.0.1-1.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/ibus-tutcode-1.0.1-1.fc15 -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 705249] Review Request: ibus-tutcode - Japanese TUT-Code input method for ibus
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=705249 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 705249] Review Request: ibus-tutcode - Japanese TUT-Code input method for ibus
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=705249 Daiki Ueno changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution||RAWHIDE Last Closed||2011-11-20 22:26:51 -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 751917] Review Request: ghc-zlib-enum - Enumerator interface for zlib compression
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=751917 Jens Petersen changed: What|Removed |Added Flag||fedora-cvs? --- Comment #4 from Jens Petersen 2011-11-20 22:58:17 EST --- Thank you for reviewing! :) New Package SCM Request === Package Name: ghc-zlib-enum Short Description: Enumerator interface for zlib compression Owners: petersen Branches: f16 f15 el6 InitialCC: haskell-sig -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 751917] Review Request: ghc-zlib-enum - Enumerator interface for zlib compression
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=751917 Jens Petersen changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|lakshminaras2...@gmail.com Status Whiteboard|Ready | -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 755155] Review Request: perl-TAP-Harness-Multiple - TAP::Harness::Muliple Perl module
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=755155
--- Comment #4 from Parag AN(पराग) 2011-11-20 23:07:30 EST
---
1)upstream license file text should not be changed at all. The rpmlint warning
of incorrect-fsf-address is not a blocker or MUST fix. This should be fixed in
upstream only. Drop the patch and let the LICENSE file as it is. you just need
to report upstream that they should change the LICENSE text.
2)As I suggested only 2 requires are getting installed automatically. So you
need to keep following for running the tests.
Requires: perl(IO::CaptureOutput)
Requires: perl(Test::Simple)
3)Also, See /etc/rpmdevtools/spectemplate-perl.spec and add following in spec
%{?perl_default_filter}
--
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 755275] Review Request: CSSLint - Detecting potential problems in CSS code
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=755275 --- Comment #1 from Parag AN(पराग) 2011-11-20 23:11:24 EST --- Provide a valid SRPM URL -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 755275] Review Request: CSSLint - Detecting potential problems in CSS code
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=755275 Parag AN(पराग) changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com AssignedTo|nob...@fedoraproject.org|panem...@gmail.com Flag||fedora-review? -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 651591] Review Request: partiwm - partitioning window manager and related tools
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=651591 --- Comment #12 from Martin Dengler 2011-11-20 23:20:24 EST --- (In reply to comment #11) > Sorry, but it is xpra included there is also it separate project > http://code.google.com/p/partiwm/wiki/xpra ? Or not? xpra is included, in the sense that when one builds rpms from "partiwm.spec", xpra rpms are built too: http://www.martindengler.com/proj/partiwm/ -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 751722] Review Request: ghc-hakyll - Static website compiler library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=751722 --- Comment #5 from Fedora Update System 2011-11-20 23:40:03 EST --- ghc-hakyll-3.2.0.10-1.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/ghc-hakyll-3.2.0.10-1.fc15 -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 751722] Review Request: ghc-hakyll - Static website compiler library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=751722 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED Bug 751722 depends on bug 749499, which changed state. Bug 749499 Summary: Review Request: ghc-regex-pcre - Haskell PCRE binding https://bugzilla.redhat.com/show_bug.cgi?id=749499 What|Old Value |New Value Resolution||ERRATA Status|ON_QA |CLOSED -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 751722] Review Request: ghc-hakyll - Static website compiler library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=751722 --- Comment #6 from Fedora Update System 2011-11-20 23:40:11 EST --- ghc-hakyll-3.2.0.10-1.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/ghc-hakyll-3.2.0.10-1.fc16 -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 755176] Review Request : ghc-mwc-random - Pseudo random number generation library for Haskell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=755176 --- Comment #1 from Jens Petersen 2011-11-20 23:42:38 EST --- I think it is better not to indent the description. http://koji.fedoraproject.org/koji/taskinfo?taskID=3528845 -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 754892] Review Request: perl-DBIx-Array - This module is a wrapper around DBI with array interfaces
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=754892 --- Comment #2 from mrdvt92 2011-11-21 00:36:10 EST --- $ rpm -q --requires perl-DBIx-Array perl(:MODULE_COMPAT_5.14.1) perl(DBI) perl(DBI) perl(base) perl(strict) perl(warnings) rpmlib(CompressedFileNames) <= 3.0.4-1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1 I guess it is found. I don't understand why it matters but if that's what you need I'll drop it. I also fix three misspellings and will have the new version 0.23 on CPAN as soon as I burn it in. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728823] Review Request: django-tinymce - TinyMCE form field editor for Django applications
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=728823 Parag AN(पराग) changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com AssignedTo|nob...@fedoraproject.org|panem...@gmail.com Flag||fedora-review? --- Comment #8 from Parag AN(पराग) 2011-11-21 00:39:41 EST --- Every modification to SPEC file should bump the release tag and add a appropriate changelog. I see tarball checksum is still not matching. a181255e8efe7b57a22851ebb184f3b9bf867bac django-tinymce-1.5.tar.gz 9830507cd89f5cc3abea0fedea4a4cf9fbdeeb59 ../SOURCES/django-tinymce-1.5.tar.gz *** Always use the upstream source tarball and if required patch it in SPEC *** -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728826] Review Request: wadofstuff-django-serializers - Extended Django Serializer Module
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=728826 Parag AN(पराग) changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com AssignedTo|nob...@fedoraproject.org|panem...@gmail.com Flag||fedora-review+ --- Comment #9 from Parag AN(पराग) 2011-11-21 01:02:00 EST --- Review: + koji scratch build ->http://koji.fedoraproject.org/koji/taskinfo?taskID=3528874 + rpmlint on rpms gave wadofstuff-django-serializers.noarch: W: install-file-in-docs /usr/share/doc/wadofstuff-django-serializers-1.1.0/INSTALL 2 packages and 0 specfiles checked; 0 errors, 1 warnings. + source match with upstream as (sha1sum) cef278159d28d235b4eb06bc4b7864b9d9a8a9ed wadofstuff-django-serializers-1.1.0.tar.gz cef278159d28d235b4eb06bc4b7864b9d9a8a9ed ../SOURCES/wadofstuff-django-serializers-1.1.0.tar.gz + Follows packaging guidelines. suggestions: 1) we don't recommend to add INSTALL file as its only for manually installation from tarball. so remove that from spec. APPROVED. -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 728837] Review Request: python-xml2dict - Use attributes of dictionary to access xml elements.
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=728837
Parag AN(पराग) changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC||panem...@gmail.com
AssignedTo|nob...@fedoraproject.org|panem...@gmail.com
Flag||fedora-review?
--- Comment #11 from Parag AN(पराग) 2011-11-21 01:22:03
EST ---
suggestions:
1) use source URL as
http://xml2dict.googlecode.com/files/xml2dict-%{alphatag}-tar.gz
and define
%global alphatag 2008.6
at top of your spec
2) use release tag as 0.%{1}.%{alphatag}
While looking into source code I realised that I have seen similar tarball
somewhere and found it at
https://nodeload.github.com/mcspring/XML2Dict/tarball/master
with its project page
https://github.com/mcspring/XML2Dict
I see above project is updated and using license ASL 2.0 while your one is BSD
--
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 755275] Review Request: CSSLint - Detecting potential problems in CSS code
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=755275 --- Comment #2 from Guillaume Kulakowski 2011-11-21 01:46:57 EST --- SRPM URL: http://llaumgui.fedorapeople.org/review/csslint/csslint-0.8.1-1.fc16.src.rpm -- 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
