[Bug 809348] Review Request: darkclient - A command line tool for the darkroom service
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=809348 Fedora Update System upda...@fedoraproject.org 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 810926] Review Request: rubygem-qpid - Ruby bindings for the Qpid messaging framework
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=810926 --- Comment #3 from Vít Ondruch vondr...@redhat.com 2012-04-11 03:39:21 EDT --- So are you going to contact maintainers of ruby-qpid to retire the ruby-qpid and add Obsolete: ruby(qpid)? Than would be nice. -- 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 809395] Review Request: jboss-as - JBoss Application Server
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=809395 Bug 809395 depends on bug 809398, which changed state. Bug 809398 Summary: Please reserve uid/guid for jboss-as https://bugzilla.redhat.com/show_bug.cgi?id=809398 What|Old Value |New Value Resolution||RAWHIDE Status|NEW |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 809747] Review Request: glassfish-jsp - Glassfish J2EE JSP API implementation
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=809747 --- Comment #3 from Stanislav Ochotnicky socho...@redhat.com 2012-04-11 04:41:33 EDT --- Spec URL: http://sochotni.fedorapeople.org/packages/glassfish-jsp.spec SRPM URL: http://sochotni.fedorapeople.org/packages/glassfish-jsp-2.2.3-2.fc16.src.rpm I've fixed the BR/R on jpackage and the whitespace, but as far as I see I already had the add_maven_depmap call in the spec. I assume that was what you meant by I have not found add_maven_depmap in pom? Also the unversioned provides: version is in the name of the api -- 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 809747] Review Request: glassfish-jsp - Glassfish J2EE JSP API implementation
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=809747 --- Comment #4 from Krzysztof Daniel kdan...@redhat.com 2012-04-11 05:41:24 EDT --- (In reply to comment #3) Sorry for the unnecessary hassle - of course the maven depmap are there. One thing that is left is building 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 796465] Review Request: exif - Utility to show EXIF information hidden in JPEG 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=796465 --- Comment #6 from Matthieu Saulnier casper.le.fan...@gmail.com 2012-04-11 05:57:18 EDT --- (In reply to comment #5) (In reply to comment #4) I think this subfolder is a add-on of the libjpeg, but I'm not sure. Perhaps the best then would be to ask upstream to clarify? Hello, this is the upstream answer: http://sourceforge.net/mailarchive/forum.php?thread_name=20120411094912.GA18222%40blackbirdforum_name=libexif-devel -- 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 808195] Review Request: nml - NewGRF Meta Language compiler
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=808195 Marcus Moeller marcus.moel...@gmx.ch changed: What|Removed |Added CC||marcus.moel...@gmx.ch AssignedTo|nob...@fedoraproject.org|marcus.moel...@gmx.ch 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 787020] Review Request: trafficserver - Apache Traffic Server
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=787020 --- Comment #49 from Fedora Update System upda...@fedoraproject.org 2012-04-11 06:25:10 EDT --- trafficserver-3.0.4-4.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/trafficserver-3.0.4-4.fc17 -- 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 808195] Review Request: nml - NewGRF Meta Language compiler
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=808195
Marcus Moeller marcus.moel...@gmx.ch changed:
What|Removed |Added
Status|NEW |ASSIGNED
Flag|fedora-review? |fedora-review+
--- Comment #1 from Marcus Moeller marcus.moel...@gmx.ch 2012-04-11 06:58:22
EDT ---
rpmlint -v output:
nml.noarch: I: checking
nml.noarch: W: spelling-error %description -l en_US grf - gr, gr f
nml.noarch: W: spelling-error %description -l en_US nfo - no, info
nml.noarch: W: spelling-error %description -l en_US newgrf - Nerf
nml.noarch: I: checking-url http://dev.openttdcoop.org/projects/nml (timeout 10
seconds)
nml.src: I: checking
nml.src: W: spelling-error %description -l en_US grf - gr, gr f
nml.src: W: spelling-error %description -l en_US nfo - no, info
nml.src: W: spelling-error %description -l en_US newgrf - Nerf
nml.src: I: checking-url http://dev.openttdcoop.org/projects/nml (timeout 10
seconds)
nml.src: I: checking-url
http://bundles.openttdcoop.org/nml/releases/0.2.3/nml-0.2.3.src.tar.gz (timeout
10 seconds)
SPECS/nml.spec: I: checking-url
http://bundles.openttdcoop.org/nml/releases/0.2.3/nml-0.2.3.src.tar.gz (timeout
10 seconds)
2 packages and 1 specfiles checked; 0 errors, 6 warnings.
OK so far.
-
key:
[+] OK
[.] OK, not applicable
[X] needs work
-
[+] MUST: The package must be named according to the Package Naming Guidelines.
[+] MUST: The spec file name must match the base package %{name}.
[+] MUST: The package must meet the Packaging Guidelines.
[+] MUST: The package must be licensed with a Fedora approved license.
GPLv2+
[+] MUST: The License field in the package spec file must match the actual
license.
[+] MUST: The file containing the text of the license(s) for the package must
be included in %doc.
[+] MUST: The spec file must be written in American English.
[+] MUST: The spec file for the package MUST be legible.
[+] MUST: The sources used to build the package must match the upstream source.
$ md5sum *
7188dcfffa6037b1a0735a1be1cc7a30 nml-0.2.3.src.tar.gz
7188dcfffa6037b1a0735a1be1cc7a30 nml-0.2.3.src.tar.gz.packaged
[+] MUST: The package MUST successfully compile and build into binary rpms on
at least one primary architecture.
- Succesfully builds in mock.
- Koji Scratch build available:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3979285
[.] MUST: If the package does not successfully compile, build or work on an
architecture, ...
[+] MUST: All build dependencies must be listed in BuildRequires.
[.] MUST: The spec file MUST handle locales properly.
[.] MUST: If a package installs files below %{_datadir}/icons, the icon cache
must be updated.
[.] MUST: Packages storing shared library files (not just symlinks) must call
ldconfig in %post and %postun.
[.] MUST: Packages must NOT bundle copies of system libraries.
[.] MUST: If the package is designed to be relocatable, ...
[.] MUST: A package must own all directories that it creates.
[+] MUST: A Fedora package must not list a file more than once in %files.
[+] MUST: Permissions on files must be set properly.
[+] MUST: Each package must consistently use macros.
[+] MUST: The package must contain code, or permissable content.
[.] MUST: Large documentation files must go in a -doc subpackage.
[+] MUST: Files in %doc must not affect the runtime of the application.
[.] MUST: Header files must be in a -devel package.
[.] MUST: Static libraries must be in a -static package.
[.] MUST: Development files must be in a -devel package.
[.] MUST: devel packages must require the base package using a fully versioned
dependency.
[.] MUST: Packages must NOT contain any .la libtool archives.
[.] MUST: Packages containing GUI applications must include a %{name}.desktop
file
[.] MUST: .desktop files must be properly installed with desktop-file-install
in the %install section.
[+] MUST: Packages must not own files or directories already owned by other
packages.
[+] MUST: All filenames in rpm packages must be valid UTF-8.
[.] SHOULD: If the source package does not include license text(s) as a
separate file from upstream, the packager SHOULD query upstream...
[+] SHOULD: Timestamps of files should be preserved.
[+] SHOULD: The reviewer should test that the package builds in mock.
[+] SHOULD: The reviewer should test that the package functions as described.
I assume the packager has tested it.
[.] SHOULD: If scriptlets are used, those scriptlets must be sane.
[.] SHOULD: Usually, subpackages other than devel should require the base
package using a fully versioned dependency.
[.] SHOULD: pkgconfig(.pc) files should be placed in a -devel pkg.
[.] SHOULD: If the package has file dependencies
[Bug 808195] Review Request: nml - NewGRF Meta Language compiler
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=808195 Felix Kaechele fe...@fetzig.org changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Felix Kaechele fe...@fetzig.org 2012-04-11 07:09:07 EDT --- New Package SCM Request === Package Name: nml Short Description: NewGRF Meta Language compiler Owners: heffer Branches: f15 f16 f17 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 808195] Review Request: nml - NewGRF Meta Language compiler
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=808195 --- Comment #3 from Jon Ciesla limburg...@gmail.com 2012-04-11 07:59:31 EDT --- 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 808336] Review Request: dia-gnomeDIAicons - Beautiful icon set for dia diagram editor
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=808336 --- Comment #20 from Jon Ciesla limburg...@gmail.com 2012-04-11 08:00:44 EDT --- 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 806665] Review Request: mosh - Mobile shell that supports roaming and intelligent local echo
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=806665 --- Comment #14 from Jon Ciesla limburg...@gmail.com 2012-04-11 07:58:27 EDT --- 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 624705] Review Request: pam_script - execute scripts from within pam
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=624705 --- Comment #13 from Marcus Moeller marcus.moel...@gmx.ch 2012-04-11 08:47:19 EDT --- Prepared pam_script 1.1.5 http://www.marcusmoeller.ch/share/build/pam_script/1.1.5-1/pam_script.spec SRPM URL: http://www.marcusmoeller.ch/share/build/pam_script/1.1.5-1/pam_script-1.1.5-1.fc17.src.rpm Meant to be built on Fedora 17+ -- 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 771137] Review Request: decibel-audio-player - straightforward music player
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=771137
--- Comment #5 from Lukáš Zapletal l...@redhat.com 2012-04-11 09:05:57 EDT ---
Christoph and Volker,
thanks for your help here. I was waiting for Debrashi to confirm me I am free
to take this package. I am now :-)
All your remarks incorporated, except:
- Was vendor used in desktop files in older versions of this package? If not,
it should not be used.
I did not change this, its from original version of the package. I'd rather
keep it if this is not an issue.
- I'm not sure about those shell scripts that act as starters. Are they really
necessary?
I dont follow here, you mean
%{_bindir}/%{name}
%{_bindir}/%{name}-remote
right? They both work and I use them quite often. Please elaborate if you mean
something different.
- When using sed, you are hardcoding /usr/bin. Use the %{_bindir} macro
instead.
Do you mean hardcoding of /usb/bin/sed invocation, or hardcoding the path in
the script itself?
Regarding harmonizing - I think its correct, I need to do the first sed
in-place, while 2nd and 3rd are used when copying the shell script to the
correct place.
I have also added BuildRequires: sed.
I am overwriting original files, use the same links to see it.
[lzap@lzapx noarch]$ rpmlint decibel-audio-player-1.08-1.fc16.noarch.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
[lzap@lzapx SRPMS]$ rpmlint decibel-audio-player-1.08-1.f15.src.rpm
decibel-audio-player.src:10: W: macro-in-comment %{name}
decibel-audio-player.src:10: W: macro-in-comment %{version}
1 packages and 0 specfiles checked; 0 errors, 3 warnings
Thanks!
--
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 811601] New: Review Request: openstack-utils - Helper utilities for OpenStack services
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: openstack-utils - Helper utilities for OpenStack services https://bugzilla.redhat.com/show_bug.cgi?id=811601 Summary: Review Request: openstack-utils - Helper utilities for OpenStack services Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: p...@draigbrady.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://pbrady.fedorapeople.org/openstack-utils/openstack-utils.spec SRPM URL: http://pbrady.fedorapeople.org/openstack-utils/openstack-utils-2012.1-1.fc15.src.rpm Description: Utilities to aid the setup and configuration of OpenStack packages These are to support the Essex release of OpenStack being shipped with Fedora 17 -- 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 807328] Review Request: drupal6-og - Organic Group Module for Drupal6
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=807328 Brett Lentz brett.le...@gmail.com changed: What|Removed |Added CC||brett.le...@gmail.com AssignedTo|nob...@fedoraproject.org|brett.le...@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 807331] Review Request: drupal6-addthis - AddThis module for Drupal6.
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=807331 Brett Lentz brett.le...@gmail.com changed: What|Removed |Added CC||brett.le...@gmail.com AssignedTo|nob...@fedoraproject.org|brett.le...@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 807331] Review Request: drupal6-addthis - AddThis module for Drupal6.
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=807331 --- Comment #1 from Brett Lentz brett.le...@gmail.com 2012-04-11 10:39:22 EDT --- $ rpmlint drupal6-addthis-3.0-3.beta2.fc16.src.rpm drupal6-addthis.src: W: summary-ended-with-dot C AddThis module for Drupal6. 1 packages and 0 specfiles checked; 0 errors, 1 warnings. $ rpmlint ~/rpmbuild/RPMS/noarch/drupal6-addthis-3.0-3.beta2.fc16.noarch.rpm drupal6-addthis.noarch: W: summary-ended-with-dot C AddThis module for Drupal6. drupal6-addthis.noarch: E: incorrect-fsf-address /usr/share/drupal6/sites/all/modules/addthis/LICENSE.txt 1 packages and 0 specfiles checked; 1 errors, 1 warnings. Issues needing correction before package can be approved: 1. Please remove the period from the end of the summary. 2. Please fix the location of the LICENSE.txt -- 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 807328] Review Request: drupal6-og - Organic Group Module for Drupal6
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=807328 Brett Lentz brett.le...@gmail.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #1 from Brett Lentz brett.le...@gmail.com 2012-04-11 10:39:28 EDT --- $ rpmlint drupal6-og-2.2-3.fc16.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. $ rpmlint ~/rpmbuild/RPMS/noarch/drupal6-og-2.2-3.fc16.noarch.rpm drupal6-og.noarch: W: incoherent-version-in-changelog 2.3-3 ['2.2-3.fc16', '2.2-3'] 1 packages and 0 specfiles checked; 0 errors, 1 warnings. Overall, I don't see anything critical that should prevent this package from being approved. All of the MUST requirements from the Review Guidelines are covered. Please correct the changelog entry prior to building the package in Koji. PACKAGE 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 226650] Merge Review: xorg-x11-xdm
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=226650 Bug 226650 depends on bug 237621, which changed state. Bug 237621 Summary: xdm needs to use consolekit https://bugzilla.redhat.com/show_bug.cgi?id=237621 What|Old Value |New Value Status|ASSIGNED|NEW Status|NEW |CLOSED Resolution||NOTABUG -- 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 810926] Review Request: rubygem-qpid - Ruby bindings for the Qpid messaging framework
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=810926 --- Comment #4 from Darryl L. Pierce dpie...@redhat.com 2012-04-11 11:20:08 EDT --- (In reply to comment #3) So are you going to contact maintainers of ruby-qpid to retire the ruby-qpid and add Obsolete: ruby(qpid)? Than would be nice. The package maintainer (nsantos) will retire it once rubygem-qpid is approved and generally available as a replacement. -- 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 809114] Review Request: novacomd - Utility to connect to WebOS devices
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=809114 --- Comment #8 from Mohamed El Morabity pikachu.2...@gmail.com 2012-04-11 11:17:09 EDT --- (In reply to comment #7) Ok, which brings me back to the original question. As the most common scenario is to have both server and client on the same machine, should we have novacom-server, novacom-client and novacom which brings in both? Sorry for this late answer. The scheme you suggest is OK for me: renaming your current novacomd package to novacom-server, the novacom package to novacom-client; the empty novacom package can be set as a subpackage of novacom-client. -- 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 810926] Review Request: rubygem-qpid - Ruby bindings for the Qpid messaging framework
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=810926 Nuno Santos nsan...@redhat.com changed: What|Removed |Added CC||nsan...@redhat.com --- Comment #5 from Nuno Santos nsan...@redhat.com 2012-04-11 11:24:06 EDT --- I will coordinate with Darryl to obsolete ruby-qpid. -- 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 811286] Review Request: rubygem-rhc-rest - Ruby bindings for OpenShift REST API
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=811286
Alec Leamas leamas.a...@gmail.com changed:
What|Removed |Added
CC||leamas.a...@gmail.com
AssignedTo|nob...@fedoraproject.org|leamas.a...@gmail.com
--- Comment #1 from Alec Leamas leamas.a...@gmail.com 2012-04-11 11:28:33 EDT
---
Hi!
Here we go! Some *very preliminary* thoughts after just having skimmed through
the spec files:
- Have you missed http://fedoraproject.org/wiki/Packaging:Ruby ? In particular,
the empty %prep and %build together with how the gem install is performed in
%install?
- The only difference between the two spec files is the ruby version. Such a
case is better handled with something like %if 0%{?fedora} 16 ... in a
common spec file.
- The long paths in the %install can be handled by doing a cd
%{buildroot}%{gemdir}/gems/%{gemname}-%{version} before doing the dirty
chmod/rm work, improving readability.
- Likewise, you can use doc/images/*.png.
- Please reorder the tags according to /etc/rpmdevtools/spectemplate-ruby.spec
in rpmdevtools (e. g., move BuildArch: above the Requires:).
- Try to use space in a consistent way e. g. two blank lines between main
sections as %prep, %build, %install %check etc., and a single blank line as
appropriate in other cases.
- The Group tag is not used by Fedora, and can be skipped. If you want to have
it anyway (which is perfectly sane, it's required by the RPM spec) I guess
System Environment/Libraries or Development/Libraries is a better choice(?)
- The empty %check can be removed
Run rpmlint on all packages + spec file and submit output together with the new
spec versions.
--
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 808195] Review Request: nml - NewGRF Meta Language compiler
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=808195 Felix Kaechele fe...@fetzig.org changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE Last Closed||2012-04-11 11:32:41 -- 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 811286] Review Request: rubygem-rhc-rest - Ruby bindings for OpenShift REST API
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=811286 --- Comment #2 from Wesley Hearn whe...@redhat.com 2012-04-11 12:08:40 EDT --- [0]whearn@Doom:/var/lib/mock/fedora-16-x86_64/result $ ls build.log rubygem-rhc-rest-0.0.7-2.fc16.noarch.rpm rubygem-rhc-rest-doc-0.0.7-2.fc16.noarch.rpm root.log rubygem-rhc-rest-0.0.7-2.fc16.src.rpm state.log [12:05 PM] [0]whearn@Doom:/var/lib/mock/fedora-16-x86_64/result $ rpmlint *.rpm rubygem-rhc-rest.src:46: W: macro-in-comment %setup 3 packages and 0 specfiles checked; 0 errors, 1 warnings. [12:05 PM] t.spec rn@Doom:/var/lib/mock/fedora-16-x86_64/result $ rpmlint ~/rpmbuild/SPECS/rubygem-rhc-rest /home/whearn/rpmbuild/SPECS/rubygem-rhc-rest.spec:46: W: macro-in-comment %setup 0 packages and 1 specfiles checked; 0 errors, 1 warnings. [12:05 PM] SRPM: http://jknife.fedorapeople.org/F16/SRPMS/rubygem-rhc-rest-0.0.7-2.fc16.src.rpm SPEC: http://jknife.fedorapeople.org/F16/SPECS/rubygem-rhc-rest.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 811661] New: Review Request: scirenderer - A Java rendering library based on JoGL
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: scirenderer - A Java rendering library based on JoGL https://bugzilla.redhat.com/show_bug.cgi?id=811661 Summary: Review Request: scirenderer - A Java rendering library based on JoGL Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: c.davi...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://davidcl.fedorapeople.org/scirenderer.spec SRPM URL: http://davidcl.fedorapeople.org/scirenderer-0.3.5-1.fc16.src.rpm Description: This Java API allows 2-D or 3-D plotting from simple 2-D graph to complex scenes. Independent library and used within Scilab software but is available for other application and developments. Note that this will be a strong dependency for Scilab 5.4.0 - beta -- 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 811661] Review Request: scirenderer - A Java rendering library based on JoGL
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=811661 Clément DAVID c.davi...@gmail.com changed: What|Removed |Added Blocks||652183(FE-JAVASIG) -- 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 811286] Review Request: rubygem-rhc-rest - Ruby bindings for OpenShift REST API
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=811286 --- Comment #3 from Wesley Hearn whe...@redhat.com 2012-04-11 12:29:23 EDT --- Err, sorry wrong links: SPEC: http://jknife.fedorapeople.org/review/rubygem-rhc-rest.spec SRPM: http://jknife.fedorapeople.org/review/rubygem-rhc-rest-0.0.7-2.fc16.src.rpm [12:29 PM] [0]whearn@Doom:/var/lib/mock/fedora-16-x86_64/result $ ls build.log root.log rubygem-rhc-rest-0.0.7-2.fc16.noarch.rpm rubygem-rhc-rest-0.0.7-2.fc16.src.rpm rubygem-rhc-rest-doc-0.0.7-2.fc16.noarch.rpm state.log [12:29 PM] [0]whearn@Doom:/var/lib/mock/fedora-16-x86_64/result $ rpmlint rubygem-rhc* 3 packages and 0 specfiles checked; 0 errors, 0 warnings. [12:29 PM] [0]whearn@Doom:/var/lib/mock/fedora-16-x86_64/result $ rpmlint ~/rpmbuild/SPECS/rubygem-rhc-rest.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. [12:29 PM] [0]whearn@Doom:/var/lib/mock/fedora-16-x86_64/result $ -- 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 806641] Review Request: perl-Test-TinyMocker - A very simple tool to mock external modules
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=806641 --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2012-04-11 12:50:05 EDT --- perl-Test-TinyMocker-0.03-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 806654] Review Request: perl-IPC-Filter - Filter data through an external process
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=806654 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|perl-IPC-Filter-0.004-1.fc1 |perl-IPC-Filter-0.004-1.fc1 |6 |5 --- Comment #10 from Fedora Update System upda...@fedoraproject.org 2012-04-11 12:48:44 EDT --- perl-IPC-Filter-0.004-1.fc15 has been pushed to the Fedora 15 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 806646] Review Request: perl-Data-Pond - Perl-based open notation for data
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=806646 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|perl-Data-Pond-0.004-2.fc15 |perl-Data-Pond-0.004-2.fc16 --- Comment #12 from Fedora Update System upda...@fedoraproject.org 2012-04-11 12:49:45 EDT --- perl-Data-Pond-0.004-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 786213] Review Request: trac-agilo-plugin - A plugin for supporting the Scrum process in Trac
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=786213 --- Comment #13 from Fedora Update System upda...@fedoraproject.org 2012-04-11 12:52:11 EDT --- trac-agilo-plugin-0.9.7-2.fc16 has been pushed to the Fedora 16 stable repository. If problems still persist, please make note of it in this bug report. -- 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 808518] Review Request: perl-Time-OlsonTZ-Download - Olson timezone database from source
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=808518 --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2012-04-11 12:55:19 EDT --- perl-Time-OlsonTZ-Download-0.003-1.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 804625] Review Request: perl-MooseX-Has-Options - Succinct options for Moose
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=804625 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|perl-MooseX-Has-Options-0.0 |perl-MooseX-Has-Options-0.0 |02-1.fc15 |02-1.fc16 --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2012-04-11 12:54:35 EDT --- perl-MooseX-Has-Options-0.002-1.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 803082] Review Request: pogo - Probably the simplest and fastest audio player for Linux
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=803082 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||pogo-0.5-2.fc16 Resolution||ERRATA Last Closed||2012-04-11 12:52:26 --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2012-04-11 12:52:26 EDT --- pogo-0.5-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 771137] Review Request: decibel-audio-player - straightforward music player
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=771137 --- Comment #7 from Brendan Jones brendan.jones...@gmail.com 2012-04-11 13:01:45 EDT --- This was orphaned right? -- 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 801092] Review Request: sumwars - a hack and slash role playing game
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=801092 --- Comment #23 from Fedora Update System upda...@fedoraproject.org 2012-04-11 12:57:10 EDT --- sumwars-0.5.6-5.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 771137] Review Request: decibel-audio-player - straightforward music player
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=771137 Brendan Jones brendan.jones...@gmail.com changed: What|Removed |Added CC||brendan.jones...@gmail.com Blocks||805236(FedoraAudio) --- Comment #6 from Brendan Jones brendan.jones...@gmail.com 2012-04-11 13:01:14 EDT --- Added tracker for the audio spin. -- 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 806643] Review Request: perl-End - Generalized END blocks
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=806643 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|perl-End-2009110401-1.fc15 |perl-End-2009110401-1.fc16 --- Comment #9 from Fedora Update System upda...@fedoraproject.org 2012-04-11 12:58:34 EDT --- perl-End-2009110401-1.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 806643] Review Request: perl-End - Generalized END blocks
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=806643 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|perl-End-2009110401-1.fc16 |perl-End-2009110401-1.fc15 --- Comment #10 from Fedora Update System upda...@fedoraproject.org 2012-04-11 13:02:50 EDT --- perl-End-2009110401-1.fc15 has been pushed to the Fedora 15 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 806641] Review Request: perl-Test-TinyMocker - A very simple tool to mock external modules
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=806641 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|perl-Test-TinyMocker-0.03-2 |perl-Test-TinyMocker-0.03-2 |.fc16 |.fc15 --- Comment #12 from Fedora Update System upda...@fedoraproject.org 2012-04-11 13:03:31 EDT --- perl-Test-TinyMocker-0.03-2.fc15 has been pushed to the Fedora 15 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 806646] Review Request: perl-Data-Pond - Perl-based open notation for data
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=806646 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|perl-Data-Pond-0.004-2.fc16 |perl-Data-Pond-0.004-2.fc15 --- Comment #13 from Fedora Update System upda...@fedoraproject.org 2012-04-11 13:06:14 EDT --- perl-Data-Pond-0.004-2.fc15 has been pushed to the Fedora 15 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 806654] Review Request: perl-IPC-Filter - Filter data through an external process
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=806654 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|perl-IPC-Filter-0.004-1.fc1 |perl-IPC-Filter-0.004-1.fc1 |5 |6 --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2012-04-11 13:06:49 EDT --- perl-IPC-Filter-0.004-1.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 804625] Review Request: perl-MooseX-Has-Options - Succinct options for Moose
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=804625 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|perl-MooseX-Has-Options-0.0 |perl-MooseX-Has-Options-0.0 |02-1.fc16 |02-1.fc15 --- Comment #12 from Fedora Update System upda...@fedoraproject.org 2012-04-11 13:05:21 EDT --- perl-MooseX-Has-Options-0.002-1.fc15 has been pushed to the Fedora 15 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 808518] Review Request: perl-Time-OlsonTZ-Download - Olson timezone database from source
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=808518 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|perl-Time-OlsonTZ-Download- |perl-Time-OlsonTZ-Download- |0.003-1.fc16|0.003-1.fc15 --- Comment #12 from Fedora Update System upda...@fedoraproject.org 2012-04-11 13:03:53 EDT --- perl-Time-OlsonTZ-Download-0.003-1.fc15 has been pushed to the Fedora 15 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 808875] Review Request: perl-Role-Tiny - A nouvelle cuisine portion size slice of Moose
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=808875 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||perl-Moo-0.009014-1.fc16 Resolution||ERRATA Last Closed||2012-04-11 13:05:05 --- Comment #12 from Fedora Update System upda...@fedoraproject.org 2012-04-11 13:05:05 EDT --- perl-Moo-0.009014-1.fc16, perl-Module-Runtime-0.013-1.fc16, perl-Role-Tiny-1.00-3.fc16 has been pushed to the Fedora 16 stable repository. If problems still persist, please make note of it in this bug report. -- 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 805013] Review Request: ecasound - Music application for multitrack audio editing
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=805013 Brendan Jones brendan.jones...@gmail.com changed: What|Removed |Added Flag||needinfo?(callkalpa@gmail.c ||om) --- Comment #4 from Brendan Jones brendan.jones...@gmail.com 2012-04-11 13:18:44 EDT --- Hi Kalpa, just a reminder as this is being tracked for the audio spin. Brendan -- 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 694994] Review Request: yoshimi - Rewrite of ZynAddSubFx aiming for better JACK support
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=694994 --- Comment #29 from Brendan Jones brendan.jones...@gmail.com 2012-04-11 13:24:30 EDT --- Adam, this is as good as APPROVED, Can yoou please action this? This is being tracked by the Fedora Audio spin now. Brendan -- 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 720818] Review Request: python-tw2-core - Web widget creation toolkit based on TurboGears widgets
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=720818 --- Comment #5 from Ralph Bean rb...@redhat.com 2012-04-11 14:02:06 EDT --- Another update, now to version 2.0.3 which includes upstream changes that fix rpmlint complaints. This is documented in the .spec changelog. Spec URL: http://threebean.org/rpm/python-tw2-core.spec SRPM URL: http://threebean.org/rpm/python-tw2-core-2.0.3-1.fc17.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 713313] Review Request: msktutil - Program for interoperability with Active Directory
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=713313 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|msktutil-0.4.1-1.fc16 |msktutil-0.4.1-1.fc15 --- Comment #53 from Fedora Update System upda...@fedoraproject.org 2012-04-11 13:07:33 EDT --- msktutil-0.4.1-1.fc15 has been pushed to the Fedora 15 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 810926] Review Request: rubygem-qpid - Ruby bindings for the Qpid messaging framework
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=810926 --- Comment #6 from Darryl L. Pierce dpie...@redhat.com 2012-04-11 14:08:22 EDT --- I've updated the specfile to include that it provides the ruby-qpid functionality: Updated SPEC: http://mcpierce.fedorapeople.org/rpms/rubygem-qpid.spec Updated SRPM: http://mcpierce.fedorapeople.org/rpms/rubygem-qpid-0.16.0-2.fc16.src.rpm I've also been in touch with the person who owns the unrelated gem on rubygems.org named QPID. He's going to release the name to us so that, in future, we can also distributed the gem via RubyGems.org as well. -- 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 811689] New: Review Request: python-tw2-forms - Forms for ToscaWidgets2
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: python-tw2-forms - Forms for ToscaWidgets2 https://bugzilla.redhat.com/show_bug.cgi?id=811689 Summary: Review Request: python-tw2-forms - Forms for ToscaWidgets2 Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: rb...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://threebean.org/rpm/python-tw2-forms.spec SRPM URL: http://threebean.org/rpm/python-tw2-forms-2.0.2-1.fc17.src.rpm Description: ToscaWidgets is a web widget toolkit for Python to aid in the creation, packaging and distribution of common view elements normally used in the web. tw2.forms contains the basic form widgets. This package requires python-tw2-core which is currently in process at https://bugzilla.redhat.com/show_bug.cgi?id=720818 -- 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 571993] Review Request: non-sequencer - A powerful, real-time, pattern-based MIDI sequencer
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=571993 Adam Huffman verdu...@fedoraproject.org changed: What|Removed |Added Flag|needinfo?(verdurin@fedorapr | |oject.org) | --- Comment #20 from Adam Huffman verdu...@fedoraproject.org 2012-04-11 14:41:58 EDT --- Apologies for the delay - I'm in the middle of moving across the country and starting a new job. That said, I do intend to complete the review, so I'm happy to continue unless someone else is desperate to do it. I should have time to update to the new release at the weekend. -- 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 430603] Review Request: clex - A free file manager with a full-screen user interface
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=430603 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #36 from Fedora Update System upda...@fedoraproject.org 2012-04-11 12:58:53 EDT --- Package clex-4.6.4-1.fc15: * should fix your issue, * was pushed to the Fedora 15 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=updates-testing clex-4.6.4-1.fc15' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-2012-5674/clex-4.6.4-1.fc15 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 571993] Review Request: non-sequencer - A powerful, real-time, pattern-based MIDI sequencer
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=571993 Brendan Jones brendan.jones...@gmail.com changed: What|Removed |Added Flag||needinfo?(verdurin@fedorapr ||oject.org) --- Comment #19 from Brendan Jones brendan.jones...@gmail.com 2012-04-11 13:21:41 EDT --- Hi Adam, can I close this so someone else can resubmit? This is being tracked by the Fedora Audio spin now. regards, Brendan -- 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 713313] Review Request: msktutil - Program for interoperability with Active Directory
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=713313 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|msktutil-0.4.1-1.fc15 |msktutil-0.4.1-1.fc16 --- Comment #52 from Fedora Update System upda...@fedoraproject.org 2012-04-11 12:51:57 EDT --- msktutil-0.4.1-1.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 694994] Review Request: yoshimi - Rewrite of ZynAddSubFx aiming for better JACK support
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=694994 --- Comment #30 from Adam Huffman verdu...@fedoraproject.org 2012-04-11 15:03:45 EDT --- Will take a look this weekend. -- 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 807328] Review Request: drupal6-og - Organic Group Module for Drupal6
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=807328 --- Comment #2 from Anderson Silva ansi...@redhat.com 2012-04-11 15:46:37 EDT --- updated/fixed: Spec URL: http://people.redhat.com/~ansilva/drupal6-og.spec SRPM URL: http://people.redhat.com/~ansilva/drupal6-og-2.2-4.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 807328] Review Request: drupal6-og - Organic Group Module for Drupal6
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=807328 Anderson Silva ansi...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? -- 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 807331] Review Request: drupal6-addthis - AddThis module for Drupal6.
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=807331 --- Comment #2 from Anderson Silva ansi...@redhat.com 2012-04-11 15:57:20 EDT --- Spec URL: http://people.redhat.com/~ansilva/drupal6-addthis.spec SRPM URL: http://people.redhat.com/~ansilva/drupal6-addthis-3.0-4.beta2.fc16.src.rpm fixed. -- 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 807328] Review Request: drupal6-og - Organic Group Module for Drupal6
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=807328 --- Comment #3 from Jon Ciesla limburg...@gmail.com 2012-04-11 15:59:16 EDT --- Please include an SCM request, and then re-set the cvs flag. Thanks! http://fedoraproject.org/wiki/Package_SCM_admin_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 811286] Review Request: rubygem-rhc-rest - Ruby bindings for OpenShift REST API
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=811286
--- Comment #4 from Alec Leamas leamas.a...@gmail.com 2012-04-11 16:01:00 EDT
---
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.
[-]: MUST %build honors applicable compiler flags or justifies otherwise.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
that are listed in the exceptions section of Packaging Guidelines.
[x]: MUST Buildroot is not present
[x]: MUST Package contains no bundled libraries.
[!]: MUST Changelog in prescribed format.
Changes not documented
[x]: MUST Package has no %clean section with rm -rf %{buildroot} (or
$RPM_BUILD_ROOT)
[x]: MUST Sources contain only permissible code or content.
[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.
[!]: MUST Package does not contain duplicates in %files.
Note: warning: File listed twice:
/builddir/.gem/ruby/1.9.1/gems/rhc-rest-0.0.7/LICENSE
[x]: MUST Fully versioned dependency in subpackages, if present.
[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.
[x]: 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.
[!]: MUST License field in the package spec file matches the actual license.
./bin/sample-usage.rb is MIT
[x]: MUST License file installed when any subpackage combination is installed.
[!]: MUST Package consistently uses macros (instead of hard-coded directory
names).
geminstdir defined but not used.
[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.
[!]: MUST Package must own all directories that it creates.
Does not claim mandatory ruby directories
[!]: MUST Package does not own files or directories owned by other packages.
Claims ownership of %{gemdir}
[!]: MUST Package installs properly.
Installation fails on my not so clean F16 and a almost pristine F17
virtual host.
[?]: MUST Requires correct, justified where necessary.
Installation problems might be caused by bad requires.
[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/mk/tmp/811286/rhc-rest-0.0.7.gem :
MD5SUM this package : b3c3f7588bb2540cb690018f86322846
MD5SUM upstream package : b3c3f7588bb2540cb690018f86322846
[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.
[-]: MUST Useful -debuginfo package or justification otherwise.
[x]: SHOULD Reviewer should test that the package builds in mock.
[-]: SHOULD If the source package does not include license text(s) as a
separate file from upstream, the packager SHOULD query upstream to
include it.
[x]: SHOULD Dist tag is present.
[x]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin,
/usr/sbin.
[!]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
--requires)
Package requires itself(!)
$ rpm -q --provides -p \
~/rpmbuild/RPMS/noarch/rubygem-rhc-rest-0.0.7-2.fc16.noarch.rpm
rubygem(rhc-rest) = 0.0.7
rubygem-rhc-rest = 0.0.7-2.fc16
$ rpm -q --requires -p \
~/rpmbuild/RPMS/noarch/rubygem-rhc-rest-0.0.7-2.fc16.noarch.rpm
/usr/bin/env
rpmlib(CompressedFileNames) = 3.0.4-1
rpmlib(FileDigests) = 4.6.0-1
rpmlib(PayloadFilesHavePrefix) = 4.0-1
ruby(abi) = 1.8
rubygem(json)
rubygem(rest)
rubygems
rpmlib(PayloadIsXz) = 5.2-1
[?]: 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.
[-]: SHOULD Description and summary sections in the package spec file contains
translations for supported Non-English languages, if
[Bug 728302] Review Request: pjproject - Libraries written in C language for building embedded/non-embedded VoIP 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=728302 --- Comment #32 from Tom spot Callaway tcall...@redhat.com 2012-04-11 15:59:00 EDT --- Created attachment 576866 -- https://bugzilla.redhat.com/attachment.cgi?id=576866 Updated spec file -- 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 728302] Review Request: pjproject - Libraries written in C language for building embedded/non-embedded VoIP 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=728302 --- Comment #33 from Tom spot Callaway tcall...@redhat.com 2012-04-11 15:59:51 EDT --- Created attachment 576867 -- https://bugzilla.redhat.com/attachment.cgi?id=576867 patch to use system libsrtp -- 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 728302] Review Request: pjproject - Libraries written in C language for building embedded/non-embedded VoIP 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=728302 --- Comment #31 from Tom spot Callaway tcall...@redhat.com 2012-04-11 15:58:14 EDT --- (In reply to comment #30) I am waiting to understand how to compile the library with external official fedora packages. I report in comment 28 the two issues. Took a look at this. The resample they use is not the same libresample that Fedora has, ours has a completely different API. It looks like at some point, someone took the application resample and converted it into a simple library. I think the pjsip upstream knows about this too, see: http://lists.pjsip.org/pipermail/pjsip_lists.pjsip.org/2007-September/64.html Given that, I don't see how we can sanely replace the bundled resample with the system libresample. srtp is a different story, that was straightforward to do. I've attached a new spec and patch that uses the system libsrtp. (P.S. In the future, start Release at 1, not 0) -- 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 807328] Review Request: drupal6-og - Organic Group Module for Drupal6
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=807328 Anderson Silva ansi...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? -- 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 807328] Review Request: drupal6-og - Organic Group Module for Drupal6
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=807328 --- Comment #4 from Anderson Silva ansi...@redhat.com 2012-04-11 16:07:51 EDT --- sorry about that... skipped a step by accident: New Package SCM Request === Package Name: drupal6-og Short Description: Enable users to create and manage their own 'groups' Owners: afsilva Branches: f16 f17 el6 -- 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 811732] New: Review Request: python-tw2-jquery - jQuery for ToscaWidgets2
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: python-tw2-jquery - jQuery for ToscaWidgets2 https://bugzilla.redhat.com/show_bug.cgi?id=811732 Summary: Review Request: python-tw2-jquery - jQuery for ToscaWidgets2 Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: rb...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://threebean.org/rpm/python-tw2-jquery.spec SRPM URL: http://threebean.org/rpm/python-tw2-jquery-2.0.1-1.fc17.src.rpm Description: toscawidgets2 (tw2) aims to be a practical and useful widgets framework that helps people build interactive websites with compelling features, faster and easier. Widgets are re-usable web components that can include a template, server-side code and JavaScripts/CSS resources. The library aims to be: flexible, reliable, documented, performant, and as simple as possible. jQuery is a fast and concise JavaScript Library that simplifies HTML document traversing, event handling, animating, and Ajax interactions for rapid web development. jQuery is designed to change the way that you write JavaScript. This module, tw2.jquery, provides toscawidgets2 (tw2) access to the jQuery library, a namespace package for jQuery plugins, and convenience classes for creating these plugins. This package requires the following two packages which are currently in review: python-tw2-core https://bugzilla.redhat.com/show_bug.cgi?id=720818 python-tw2-forms https://bugzilla.redhat.com/show_bug.cgi?id=811689 -- 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 807328] Review Request: drupal6-og - Organic Group Module for Drupal6
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=807328 --- Comment #5 from Jon Ciesla limburg...@gmail.com 2012-04-11 16:11:49 EDT --- 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 811732] Review Request: python-tw2-jquery - jQuery for ToscaWidgets2
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=811732 --- Comment #1 from Ralph Bean rb...@redhat.com 2012-04-11 16:14:13 EDT --- I got the summary wrong in the initial spec. Here is a second release. Spec URL: http://threebean.org/rpm/python-tw2-jquery.spec SRPM URL: http://threebean.org/rpm/python-tw2-jquery-2.0.1-2.fc17.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 811286] Review Request: rubygem-rhc-rest - Ruby bindings for OpenShift REST API
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=811286 --- Comment #5 from Alec Leamas leamas.a...@gmail.com 2012-04-11 16:21:27 EDT --- Hm... the licensing is actually somewhat bad, to have more than one license in a gem makes life complicated for those using it. You might want to put the ./bin/sample-usage.rb in a separate sub-package (with a separate License: tag) to handle this. -- 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 811689] Review Request: python-tw2-forms - Forms for ToscaWidgets2
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=811689 Ralph Bean rb...@redhat.com changed: What|Removed |Added Blocks||811732 -- 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 811732] Review Request: python-tw2-jquery - jQuery for ToscaWidgets2
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=811732 Ralph Bean rb...@redhat.com changed: What|Removed |Added Blocks||811739 -- 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 811732] Review Request: python-tw2-jquery - jQuery for ToscaWidgets2
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=811732 Ralph Bean rb...@redhat.com changed: What|Removed |Added Depends on||811689 -- 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 811739] Review Request: python-tw2-jqplugins-ui - jQuery UI for ToscaWidgets2
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=811739 Ralph Bean rb...@redhat.com changed: What|Removed |Added Depends on||811732 -- 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 810010] Review Request: genders - file based database for cluster managment
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=810010
Tom spot Callaway tcall...@redhat.com changed:
What|Removed |Added
CC||tcall...@redhat.com
--- Comment #5 from Tom spot Callaway tcall...@redhat.com 2012-04-11
16:38:38 EDT ---
Just some drive-by comments:
* Your BuildRoot value is invalid, please see:
https://fedoraproject.org/wiki/EPEL/GuidelinesAndPolicies#BuildRoot_tag
* The . on its own line in each %description is unnecessary, please omit.
* When doing explicit Requires, you should be doing it differently, see:
https://fedoraproject.org/wiki/Packaging/Guidelines#Requires
As an example, you currently have:
Requires: libgenders-devel = %{version}, libgendersplusplus = %{version}
This should be:
Requires: libgenders-devel%{?_isa} = %{version}-%{release}
Requires: libgendersplusplus%{?_isa} = %{version}-%{release}
(splitting it across two lines is not required, i just did that for legibility)
* You are mixing macros, specifically, $RPM_BUILD_ROOT and %{buildroot}.
You need to choose one and use it consistently. I strongly recommend using
%{buildroot}.
* Your %defattr lines should be %defattr(-,root,root,-)
* To filter out those provides, see:
https://fedoraproject.org/wiki/Packaging:AutoProvidesAndRequiresFiltering
* When you make changes, please increment Release and add a new entry to the
%changelog, so we can see the progress over time. :)
--
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 720818] Review Request: python-tw2-core - Web widget creation toolkit based on TurboGears widgets
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=720818 Ralph Bean rb...@redhat.com changed: What|Removed |Added Blocks||811689 -- 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 811689] Review Request: python-tw2-forms - Forms for ToscaWidgets2
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=811689 Ralph Bean rb...@redhat.com changed: What|Removed |Added Depends on||720818 -- 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 811739] New: Review Request: python-tw2-jqplugins-ui - jQuery UI for ToscaWidgets2
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: python-tw2-jqplugins-ui - jQuery UI for ToscaWidgets2 https://bugzilla.redhat.com/show_bug.cgi?id=811739 Summary: Review Request: python-tw2-jqplugins-ui - jQuery UI for ToscaWidgets2 Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: rb...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://threebean.org/rpm/python-tw2-jqplugins-ui.spec SRPM URL: http://threebean.org/rpm/python-tw2-jqplugins-ui-2.0.0-1.fc17.src.rpm Description: toscawidgets2 (tw2) aims to be a practical and useful widgets framework that helps people build interactive websites with compelling features, faster and easier. Widgets are re-usable web components that can include a template, server-side code and JavaScripts/CSS resources. The library aims to be: flexible, reliable, documented, performant, and as simple as possible. jQuery is a fast and concise JavaScript Library that simplifies HTML document traversing, event handling, animating, and Ajax interactions for rapid web development. jQuery is designed to change the way that you write JavaScript. jQuery UI provides abstractions for low-level interaction and animation, advanced effects and high-level, themeable widgets, built on top of the jQuery JavaScript Library, that you can use to build highly interactive web applications. This module, tw2.jqplugins.ui, provides toscawidgets2 (tw2) access to jQuery UI widgets. This package requires the following three others: python-tw2-jquery - https://bugzilla.redhat.com/show_bug.cgi?id=811732 python-tw2-forms - https://bugzilla.redhat.com/show_bug.cgi?id=811689 python-tw2-core - https://bugzilla.redhat.com/show_bug.cgi?id=720818 -- 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 811286] Review Request: rubygem-rhc-rest - Ruby bindings for OpenShift REST API
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=811286 --- Comment #6 from Wesley Hearn whe...@redhat.com 2012-04-11 17:03:18 EDT --- Well I think I got everything. The failed install was a typo on my part :/ rubygem(rest) needed to be rubygem(rest-client). Though I have no clue why FedoraReview is saying it depended on itself. *shrugs* Anywho here is the updated SPEC/SRPM: SPEC: http://jknife.fedorapeople.org/review/rubygem-rhc-rest.spec SRPM: http://jknife.fedorapeople.org/review/rubygem-rhc-rest-0.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 811732] Review Request: python-tw2-jquery - jQuery for ToscaWidgets2
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=811732 --- Comment #2 from Ralph Bean rb...@redhat.com 2012-04-11 17:13:57 EDT --- And python-tw2-jquery was accidentally installing its test suite alongside itself which collided with python-tw2-jqplugins-ui's tests. Here's an upstream change that fixes that. Spec URL: http://threebean.org/rpm/python-tw2-jquery.spec SRPM URL: http://threebean.org/rpm/python-tw2-jquery-2.0.2-1.fc17.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 811739] Review Request: python-tw2-jqplugins-ui - jQuery UI for ToscaWidgets2
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=811739 --- Comment #1 from Ralph Bean rb...@redhat.com 2012-04-11 17:18:30 EDT --- An upstream update was required to fix a collision with tw2.jquery. Both tw2.jqplugins.ui and tw2.jquery were accidentally installing their test suites alongside themselves. The problem was only revealed once yum complained that files were being overwritten. Here's the new update that addresses this. Spec URL: http://threebean.org/rpm/python-tw2-jqplugins-ui.spec SRPM URL: http://threebean.org/rpm/python-tw2-jqplugins-ui-2.0.1-1.fc17.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 811750] New: Review Request: python-tw2-jqplugins-gritter - jQuery gritter (growl-like popups) for ToscaWidgets2
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: python-tw2-jqplugins-gritter - jQuery gritter (growl-like popups) for ToscaWidgets2 https://bugzilla.redhat.com/show_bug.cgi?id=811750 Summary: Review Request: python-tw2-jqplugins-gritter - jQuery gritter (growl-like popups) for ToscaWidgets2 Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: rb...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://threebean.org/rpm/python-tw2-jqplugins-gritter.spec SRPM URL: http://threebean.org/rpm/python-tw2-jqplugins-gritter-2.0.0-1.fc17.src.rpm Description: This module provides growl like popups with the gritter plugin for python-tw2-jqplugins-ui. This module depends on python-tw2-jqplugins-ui which is currently up for review - https://bugzilla.redhat.com/show_bug.cgi?id=811739 -- 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 811739] Review Request: python-tw2-jqplugins-ui - jQuery UI for ToscaWidgets2
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=811739 Ralph Bean rb...@redhat.com changed: What|Removed |Added Blocks||811750 -- 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 811750] Review Request: python-tw2-jqplugins-gritter - jQuery gritter (growl-like popups) for ToscaWidgets2
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=811750 Ralph Bean rb...@redhat.com changed: What|Removed |Added Depends on||811739 -- 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 811286] Review Request: rubygem-rhc-rest - Ruby bindings for OpenShift REST API
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=811286
--- Comment #7 from Alec Leamas leamas.a...@gmail.com 2012-04-11 17:48:43 EDT
---
Well, fedora-review is not the only one involved here;) I took 'rest' for
'rhc-rest', my bad, but the conclusion is the same.
- Suddenly %defattr and 'rm -rf %{buildroot}' is there. Please remove.
- rpmlint on installed package:
$ rpmlint rubygem-rest-client
rubygem-rest-client.noarch: W: spelling-error %description -l en_US
microframework - micro framework, micro-framework, framework
rubygem-rest-client.noarch: W: unexpanded-macro
/usr/lib/ruby/gems/1.8/doc/rest-client-1.6.1/ri/RestClient/log%3d-c.yaml %3d
rubygem-rest-client.noarch: W: wrong-file-end-of-line-encoding
/usr/lib/ruby/gems/1.8/gems/rest-client-1.6.1/spec/integration/certs/equifax.crt
rubygem-rest-client.noarch: W: unexpanded-macro
/usr/lib/ruby/gems/1.8/doc/rest-client-1.6.1/ri/RestClient/Resource/%5b%5d-i.yaml
%5b
rubygem-rest-client.noarch: W: unexpanded-macro
/usr/lib/ruby/gems/1.8/doc/rest-client-1.6.1/ri/RestClient/Resource/%5b%5d-i.yaml
%5d
rubygem-rest-client.noarch: W: wrong-file-end-of-line-encoding
/usr/lib/ruby/gems/1.8/gems/rest-client-1.6.1/spec/integration/certs/verisign.crt
rubygem-rest-client.noarch: W: unexpanded-macro
/usr/lib/ruby/gems/1.8/doc/rest-client-1.6.1/ri/RestClient/%3c%3c-i.yaml %3c
rubygem-rest-client.noarch: W: unexpanded-macro
/usr/lib/ruby/gems/1.8/doc/rest-client-1.6.1/ri/RestClient/%3c%3c-i.yaml %3c
rubygem-rest-client.noarch: W: unexpanded-macro
/usr/lib/ruby/gems/1.8/doc/rest-client-1.6.1/ri/RestClient/Payload/has_file%3f-i.yaml
%3f
rubygem-rest-client.noarch: W: unexpanded-macro
/usr/lib/ruby/gems/1.8/doc/rest-client-1.6.1/ri/RestClient/AbstractResponse/return%21-i.yaml
%21
rubygem-rest-client.noarch: W: no-manual-page-for-binary restclient
1 packages and 0 specfiles checked; 0 errors, 11 warnings
- You need to take care of the eol warnings, the rest can be ignored (I think).
- For the spelling, what can a Swede say about 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 810010] Review Request: genders - file based database for cluster managment
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=810010
--- Comment #6 from David Brown david.br...@pnl.gov 2012-04-11 17:56:31 EDT
---
Updated the URLs above with the changes recommended. I also checked out EPEL
and that seems to work as well.
http://dmlb2000.homelinux.org/packages/genders-1.18-2.fc18.src.rpm
There's three issues rpmlint still seems to show up with.
1. Apparently libgenders.so.0.3.0 calls exit(). This in case flex can't parse
the primary genders config file. So I don't blame them for this since the
entire library is useless in the condition the config file is wrong.
2. There's a hardcoded library path in the spec %{_prefix}/lib/genders. This
is where the compat libraries live. The automake also has this same hard coded
library path and the files put there are old perl scripts. So I'm open for
suggestions on what the 'right' thing to do is with these files. Could go out
in %{_datadir}/%{name}?
3. EPEL build keeps complaining about having an unspecified group. However,
its not needed when I build it on Fedora. Should I care that much?
Also to the initial review I don't have any 'informal reviews' not sure what
they'd be and where I'd get a link for them.
--
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 772751] Review Request: git-review - Helper for Gerrit
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=772751 Pete Zaitcev zait...@redhat.com changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #12 from Pete Zaitcev zait...@redhat.com 2012-04-11 18:02:35 EDT --- Package Change Request == Package Name: git-review New Branches: f17 Owners: zaitcev Bodhi says: Creating a new update for git-review-1.16-1.fc17 AppError(PackageDBError, The package git-review is not in Collection Fedora 17., extras=None) Hopefuly adding the branch is the right answer here. -- 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 811759] New: Review Request: python-tw2-excanvas - Excanvas for ToscaWidgets2
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: python-tw2-excanvas - Excanvas for ToscaWidgets2 https://bugzilla.redhat.com/show_bug.cgi?id=811759 Summary: Review Request: python-tw2-excanvas - Excanvas for ToscaWidgets2 Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: rb...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://threebean.org/rpm/python-tw2-excanvas.spec SRPM URL: http://threebean.org/rpm/python-tw2-excanvas-2.0.0-1.fc17.src.rpm Description: Simple excanvas wrapper for ToscaWidgets2 This package depends on python-tw2-core which is currently up for review - https://bugzilla.redhat.com/show_bug.cgi?id=720818 -- 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 720818] Review Request: python-tw2-core - Web widget creation toolkit based on TurboGears widgets
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=720818 Ralph Bean rb...@redhat.com changed: What|Removed |Added Blocks||811759 -- 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 811759] Review Request: python-tw2-excanvas - Excanvas for ToscaWidgets2
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=811759 Ralph Bean rb...@redhat.com changed: What|Removed |Added Depends on||720818 -- 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 810928] Review Request: sticky-notes - Sticky notes is a free and open source paste-bin application
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=810928 --- Comment #5 from Kevin Fenzi ke...@scrye.com 2012-04-11 18:31:40 EDT --- Well, the problem with using the remote latest jquery is that if somehow they were compromised, you could well be as well. After talking things over with others, we currently don't require unbundling javascript... so, lets just stick with the bundled copy (Of course need to make sure and update it in the event of bugs). Or I suppose you could just have an option to use the bundled copy or the copy from the net? 1 and 2 above seem great. -- 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 810928] Review Request: sticky-notes - Sticky notes is a free and open source paste-bin application
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=810928 Athmane Madjoudj athma...@gmail.com changed: What|Removed |Added CC||athma...@gmail.com --- Comment #6 from Athmane Madjoudj athma...@gmail.com 2012-04-11 18:49:19 EDT --- Month ago I contacted the author about a vulnerability, still no response. I've created a merge request in gitorious if you're interested in the patch. -- 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 811689] Review Request: python-tw2-forms - Forms for ToscaWidgets2
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=811689 Ralph Bean rb...@redhat.com changed: What|Removed |Added Blocks||811769 -- 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 811769] Review Request: python-tw2-dynforms - Dynamic forms for ToscaWidgets2
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=811769 Ralph Bean rb...@redhat.com changed: What|Removed |Added Depends on||811689 -- 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 811769] New: Review Request: python-tw2-dynforms - Dynamic forms for ToscaWidgets2
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: python-tw2-dynforms - Dynamic forms for ToscaWidgets2 https://bugzilla.redhat.com/show_bug.cgi?id=811769 Summary: Review Request: python-tw2-dynforms - Dynamic forms for ToscaWidgets2 Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: rb...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://threebean.org/rpm/python-tw2-dynforms.spec SRPM URL: http://threebean.org/rpm/python-tw2-dynforms-2.0.0-1.fc17.src.rpm Description: ToscaWidgets is a web widget toolkit for Python to aid in the creation, packaging and distribution of common view elements normally used in the web. tw2.dynforms includes dynamic form building widgets that use JavaScript. This package depends on python-tw2-forms which is currently up for review - https://bugzilla.redhat.com/show_bug.cgi?id=811689 -- 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 810676] Review Request: aws - Ada Web Server (Web framework for Ada)
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=810676 --- Comment #4 from Björn Persson bj...@xn--rombobjrn-67a.se 2012-04-11 19:04:23 EDT --- Without knowing in detail how you produced those files I can't say for certain what the copyright notices should look like. Your files are dated 2003-2009. There are only two project files with that combination of years in aws-gpl-2.10.0-src.tgz, and those are very different from yours, so it doesn't look like your files are modified versions of Adacore's. Debian's AWS package has an aws.gpr that looks fairly similar to yours, as if you had taken Ludovic's project file, adapted it for Fedora, and replaced Ludovic's copyright notice with Adacore's. Debian's templates_parser.gpr, on the other hand, looks quite different from your template_parser.gpr. If major parts of the project file code stem from Ludovic's project files, then you should include Ludovic's copyright notice. If you have edited a file so much that you have essentially written it all yourself, then you can write something like GNAT project file for Ada Web Server, Copyright 2012 Pavel Zhukov, or you can choose to omit the copyright notice. That's at least how I understand it, but IANAL and so on. I suppose you can ask on the legal mailing list if you want other opinions. If you do, you'll probably have to describe in detail what you have done with the files. -- 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 799651] Review Request: smb4k - The SMB/CIFS Share Browser for KDE
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=799651 --- Comment #41 from Sergio Monteiro Basto ser...@serjux.com 2012-04-11 19:08:49 EDT --- smb4k-1.0.1-5.fc16 and smb4k-1.0.1-5.fc15 also has been pushed to testing you may do: yum install --enablerepo=updates-testing smb4k (as root) or yum update --enablerepo=updates-testing smb4k (as root, if you are in F15 with smb4k installed) . -- 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
