[Bug 753677] Review Request: gnome-shell-extension-no11y - remove acceblity icon from status area
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=753677 --- Comment #2 from Amir Hedayaty 2011-11-15 02:40:06 EST --- Thanks for the feedback I hope the issue are fixed! Here are the new urls: Spec URL: www.cs.sfu.ca/~aha49/personal/fedora/gnome-shell-extension-noa11y.spec SRPM URL: www.cs.sfu.ca/~aha49/personal/fedora/gnome-shell-extension-noa11y-3.2.0-1.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 753676] Review Request: gnome-shell-extention-netspeed -an internet speed indicator
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=753676 --- Comment #2 from Amir Hedayaty 2011-11-15 02:42:24 EST --- Thanks for feedback here are the new urls: Spec URL: http://www.cs.sfu.ca/~aha49/personal/fedora/gnome-shell-extension-netspeed.spec SRPM URL: http://www.cs.sfu.ca/~aha49/personal/fedora/gnome-shell-extension-netspeed-3.2.0-1.src.rpm This extension was something which was missing from gnome 3.0, I wrote this a while ago, now it is in a better shape I guess now I share share it with other users. I am looking for getting feedback/patches from fedora users. -- 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 710386] Review Request: gnome-shell-extension-system-monitor-applet - A Gnome shell system monitor extension
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=710386 Rahul Sundaram changed: What|Removed |Added CC||methe...@gmail.com --- Comment #34 from Rahul Sundaram 2011-11-15 02:36:24 EST --- No. Go ahead -- 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 752836] Review Request: perl-Class-Field - Class Field Accessor Generator
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=752836 Parag AN(पराग) changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com AssignedTo|nob...@fedoraproject.org|panem...@gmail.com Flag||fedora-review+ --- Comment #1 from Parag AN(पराग) 2011-11-15 02:23:40 EST --- Review:- + koji build ->http://koji.fedoraproject.org/koji/taskinfo?taskID=3514609 + rpmlint on package gave perl-Class-Field.src: W: spelling-error %description -l en_US const -> cons, cont, cost perl-Class-Field.noarch: W: spelling-error %description -l en_US const -> cons, cont, cost + Source match with upstream as (sha1sum) 908a42ded2f16a5066313a8cd20ed41a6de599b8 Class-Field-0.15.tar.gz 908a42ded2f16a5066313a8cd20ed41a6de599b8 ../SOURCES/Class-Field-0.15.tar.gz + Follows packaging guidelines. + make test shows All tests successful. Files=3, Tests=14, 0 wallclock secs ( 0.02 usr 0.01 sys + 0.08 cusr 0.01 csys = 0.12 CPU) + Package perl-Class-Field-0.15-1.fc17.noarch Provides: perl(Class::Field) = 0.15 Requires: perl >= 0:5.006001 perl(Encode) perl(Exporter) perl(base) perl(strict) perl(utf8) perl(warnings) Suggestions: 1) you don't need to specify following BuildRequires: perl >= 1:5.6.1 2) For the active fedora releases you can remove following from spec a) buildroot b) cleaning of buildroot in %install c) %clean section d) %defattr(-,root,root,-) -- 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 754023] New: Review Request: Mumpot - GTK mapping application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: Mumpot - GTK mapping application https://bugzilla.redhat.com/show_bug.cgi?id=754023 Summary: Review Request: Mumpot - GTK mapping application Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: volke...@gmx.at QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: http://www.geofrogger.net/review/mumpot.spec SRPM URL: http://www.geofrogger.net/review/mumpot-0.6-1.fc16.src.rpm Description: Mumpot is a GTK mapping application with simple editing features (for editing OpenStreetMap data) mainly (but not only) for use with mobile Linux equipped devices, tested on a GPE installation on an iPAQ2200 and on the Openmoko platform. Mumpot can connect to a GPS, which provides NMEA-Data. It can down- and upload OSM data and also search for shortest paths. -- 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 753911] Review Request: django-profiles - A fairly simple user-profile management application for Django
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=753911 --- Comment #2 from Matthias Runge 2011-11-15 02:13:31 EST --- Parag, thank you for your review. I'd need this for epel 6, too, so I keep both sections. Source URL: I was unable to create a something conveniant file name, since there has been a 0.2 release some time ago, followed by a patch, which is absolutely useful. So I fetched the latest version from repository (called default.tar.bz2). default.tar.bz2 is not a useful name of a file, I think. So I noted, how I retrieved the file, a md5sum to verify, the right file was retrieved and named the file how in a way more usable. -- 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 753911] Review Request: django-profiles - A fairly simple user-profile management application for Django
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=753911 Parag AN(पराग) changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com AssignedTo|nob...@fedoraproject.org|panem...@gmail.com Flag||fedora-review+ --- Comment #1 from Parag AN(पराग) 2011-11-15 01:59:46 EST --- Review:- + koji build ->http://koji.fedoraproject.org/koji/taskinfo?taskID=3514594 + rpmlint on package gave django-profiles.src: W: invalid-url Source0: django-profiles.tar.bz2 2 packages and 0 specfiles checked; 0 errors, 1 warnings. + Source matches upstream (sha1sum) as 5a94560b95f64dc3b786701647876484 django-profiles.tar.bz2 5a94560b95f64dc3b786701647876484 ../SOURCES/django-profiles.tar.bz2 + Follows packaging guidelines. suggestions: 1) you don't need now following in current active Fedora releases. %defattr (-,root,root,-) and in %install rm -rf $RPM_BUILD_ROOT -- 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 750997] Review Request: scilab - Scientific software package for numerical computations
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=750997 Clément DAVID 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 753676] Review Request: gnome-shell-extention-netspeed -an internet speed indicator
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=753676 Parag AN(पराग) changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com AssignedTo|nob...@fedoraproject.org|panem...@gmail.com Flag||fedora-review? --- Comment #1 from Parag AN(पराग) 2011-11-15 01:07:22 EST --- Review:- + koji build ->http://koji.fedoraproject.org/koji/taskinfo?taskID=3514555 + rpmlint on package gave gnome-shell-extension-netspeed.noarch: W: no-version-in-last-changelog gnome-shell-extension-netspeed.noarch: W: invalid-url URL Note gnome-shell-extension-netspeed.noarch: W: no-documentation gnome-shell-extension-netspeed.noarch: W: empty-%postun gnome-shell-extension-netspeed.src: W: no-version-in-last-changelog gnome-shell-extension-netspeed.src: W: invalid-url URL Note gnome-shell-extension-netspeed.src:2: W: mixed-use-of-spaces-and-tabs (spaces: line 2, tab: line 1) gnome-shell-extension-netspeed.src: W: invalid-url Source0: netsp...@hedayaty.gmail.com.tar.gz 2 packages and 0 specfiles checked; 0 errors, 8 warnings. suggestions: 1) you don't need now following in current active Fedora releases. %defattr (-,root,root,-) and in %install rm -rf %{buildroot} 2) add the version-release information in changelog like * Sun Nov 13 2011 Amir Hedayaty - 0-0.1 - Initial package for Fedora 3) you can remove warning mixed-use-of-spaces-and-tabs by using command sed -e 's|\t| |g' gnome-shell-extension-netspeed.spec 3) I see you have wrong spec file added in srpm. you added gnome-shell-extension-no11y.spec whereas it should be gnome-shell-extension-noa11y.spec 4) you don't need %postun in spec file. remove that. 5) Host your tarball on some server and use that download url in URL of spec. submit a new SPEC and SRPM by fixing above issues. Add some document files in upstream tarball that tells how to install this extension as well as license information and its text in a separate 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 753677] Review Request: gnome-shell-extension-no11y - remove acceblity icon from status area
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=753677 Parag AN(पराग) changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com AssignedTo|nob...@fedoraproject.org|panem...@gmail.com Flag||fedora-review? --- Comment #1 from Parag AN(पराग) 2011-11-15 01:03:18 EST --- Review:- + koji build ->http://koji.fedoraproject.org/koji/taskinfo?taskID=3514553 + rpmlint on package gave gnome-shell-extension-noa11y.noarch: W: no-version-in-last-changelog gnome-shell-extension-noa11y.noarch: W: invalid-url URL Note gnome-shell-extension-noa11y.noarch: W: no-documentation gnome-shell-extension-noa11y.noarch: W: empty-%postun gnome-shell-extension-noa11y.src: W: no-version-in-last-changelog gnome-shell-extension-noa11y.src: W: invalid-url URL Note gnome-shell-extension-noa11y.src: E: invalid-spec-name gnome-shell-extension-noa11y.src:2: W: mixed-use-of-spaces-and-tabs (spaces: line 2, tab: line 1) gnome-shell-extension-noa11y.src: W: invalid-url Source0: noa...@hedayaty.gmail.com.tar.gz 2 packages and 0 specfiles checked; 1 errors, 8 warnings. suggestions: 1) you don't need now following in current active Fedora releases. %defattr (-,root,root,-) and in %install rm -rf %{buildroot} 2) add the version-release information in changelog like * Sun Nov 13 2011 Amir Hedayaty - 0-0.1 - Initial package for Fedora 3) you can remove warning mixed-use-of-spaces-and-tabs by using command sed -e 's|\t| |g' gnome-shell-extension-noa11y.spec 3) I see you have wrong spec file added in srpm. you added gnome-shell-extension-no11y.spec whereas it should be gnome-shell-extension-noa11y.spec 4) you don't need %postun in spec file. remove that. 5) Host your tarball on some server and use that download url in URL of spec. submit a new SPEC and SRPM by fixing above issues. -- 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 754004] New: Review Request: lv2-abGate - an LV2 Noise Gate plugin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: lv2-abGate - an LV2 Noise Gate plugin https://bugzilla.redhat.com/show_bug.cgi?id=754004 Summary: Review Request: lv2-abGate - an LV2 Noise Gate plugin Product: Fedora Version: 16 Platform: Unspecified OS/Version: Unspecified Status: NEW Severity: unspecified Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: brendan.jones...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- A simple LV2 noise gate plugin for use in hosts that support LV2 SPEC: http://bsjones.fedorapeople.org/lv2-abGate.spec SRPM: http://bsjones.fedorapeople.org/lv2-abGate-1.1.3-1.fc16.src.rpm RPMLINT: rpmlint /home/bsjones/rpmbuild/SRPMS/lv2-abGate-1.1.3-1.fc16.src.rpm /home/bsjones/rpmbuild/RPMS/x86_64/lv2-abGate*.rpm 3 packages and 0 specfiles checked; 0 errors, 0 warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 737286] Review Request: salt - A parallel remote execution system
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=737286 --- Comment #3 from Clint Savage 2011-11-14 23:44:52 EST --- Indeed there is news. I am working on packaging up 0.9.3 this week. I've been a bit under water with the dayjob, but I think I've found some time to update this ticket. -- 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 750139] Review Request: lv2-mdala-plugins - LV2 port of the MDA VST plugins
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=750139 --- Comment #1 from Brendan Jones 2011-11-14 23:43:18 EST --- A minor update SPEC: http://bsjones.fedorapeople.org/lv2-mdala-plugins.spec SRPM: http://bsjones.fedorapeople.org/lv2-mdala-plugins-0-0.2.svn3580.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 748180] Review Request: dbus-sharp-glib - C# bindings for D-Bus glib main loop integration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=748180 Theodore Lee changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #10 from Theodore Lee 2011-11-14 20:34:55 EST --- Okay, I'll take over this review. MUST Items == OK - rpmlint must be run on all rpms $ rpmlint dbus-sharp-glib-0.5.0-1.fc16.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. $ rpmlint dbus-sharp-glib-0.5.0-1.fc16.x86_64.rpm dbus-sharp-glib-devel-0.5.0-1.fc16.x86_64.rpm dbus-sharp-glib.x86_64: E: no-binary dbus-sharp-glib.x86_64: W: only-non-binary-in-usr-lib dbus-sharp-glib-devel.x86_64: W: no-documentation 2 packages and 0 specfiles checked; 1 errors, 2 warnings. After install: $ rpmlint dbus-sharp-glib dbus-sharp-glib.x86_64: E: no-binary dbus-sharp-glib.x86_64: W: only-non-binary-in-usr-lib 1 packages and 0 specfiles checked; 1 errors, 1 warnings. These errors seem to be the usual ones seen with mono packages, nothing significant. OK - Package must meet naming guidelines OK - Spec file name must match base package name OK - Package must meet packaging guidelines OK - Package must meet licensing guidelines OK - License tag must match actual license OK - Any license files must be in %doc OK - Spec file must be in American English OK - Spec file must be legible OK - Sources must match upstream $ sha1sum dbus-sharp-glib-0.5.0.tar.gz dbus-sharp-glib-0.5.0.tar.gz.fedora bff1d3e8def9f5c7f956adffdef3a860a05e0e95 dbus-sharp-glib-0.5.0.tar.gz bff1d3e8def9f5c7f956adffdef3a860a05e0e95 dbus-sharp-glib-0.5.0.tar.gz.fedora OK - Package must build on at least one primary arch OK - Arches that the package doesn't build on must be excluded with a relevant bug In this case mono simply isn't available on some arches, so I don't think this is a blocking issue. OK - All necessary build dependencies must be in BuildRequires N/A - Locales must be handled properly N/A - Binary rpms containing libraries must call ldconfig OK - Package must not bundle system libraries N/A - Relocatable packages must have rationalization OK - Package must own all directories it creates OK - Package must not list a file more than once in %files OK - Files must have correct permissions OK - Macros must be consistent OK - Package must contain code or permissible content N/A - Large documentation files must be in a -doc subpackage OK - %doc files must not affect program operation N/A - Header files must be in a -devel subpackage N/A - Static libraries must be in a -static package N/A - Library files that end in .so must go in a -devel package OK - -devel packages must require the base package using a fully versioned dependency OK - Package must NOT contain any .la libtool archives N/A - Packages containing GUI applications must include a %{name}.desktop file OK - Packages must not own files or directories already owned by other packages OK - All filenames in rpm packages must be valid UTF-8 SHOULD Items N/A - If the package is missing license text in a separate file, the packager should query upstream for it N/A - Description and summary should contain translations if available OK - Package should build in mock OK - Package should build on all supported architectures Koji scratch build seems okay: https://koji.fedoraproject.org/koji/taskinfo?taskID=3511249 OK - Package should function as described Built and used banshee-2.2 against this, and it seems to be working well. N/A - Scriptlets should be sane N/A - Non-devel subpackages should require the base package with a full version OK - pkgconfig files should be placed appropriately N/A - File dependencies outside of /etc, /bin, /sbin, /usr/bin, or /usr/sbin should require package instead N/A - Binaries/scripts should have man pages Mono-specific Items === OK - DLLs must be registered with gacutil OK - .pc files must be in a -devel package OK - Empty -debuginfo packages must not be built OK - Package must NOT contain any pre-compiled .dll or .exe files OK - Package must NOT contain .dll files from other projects OK - Package should not redefine _libdir Issues == 1) There's no link to a tracking bug for the architectures that this package doesn't build on. However, this is something that covers pretty much all mono packages, so I don't think that's too important. 2) The main package description is missing a full stop, and the description for the -devel package should probably mention GLib at some point. None of these issues are blocking, so I think this package can be ACCEPTED. -- 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. __
[Bug 689056] Review Request: lmd - Linux Malware Detecter
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=689056 --- Comment #8 from Mark McKinstry 2011-11-14 20:35:26 EST --- ping? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 748180] Review Request: dbus-sharp-glib - C# bindings for D-Bus glib main loop integration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=748180 Theodore Lee changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|theo...@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 753513] Review Request: minetest - Multiplayer infinite-world block sandbox with survival mode
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=753513 --- Comment #1 from Aleksandra Bookwar 2011-11-14 17:50:34 EST --- I've fixed the %clean section and %defattr according to guidelines. New URLs: Spec URL: https://raw.github.com/RussianFedora/minetest/fedora/minetest.spec SRPM URL: http://koji.russianfedora.ru/koji/getfile?taskID=16122&name=minetest-0.3.1-4.gitbc0e5c0.fc16.src.rpm $ rpmlint minetest minetest.x86_64: W: spelling-error Summary(en_US) Multiplayer -> Multiplier, Multiplexer minetest.x86_64: W: spelling-error %description -l en_US multiplayer -> multiplier, multiplexer minetest.x86_64: W: no-documentation minetest.x86_64: W: no-manual-page-for-binary minetest 1 packages and 0 specfiles checked; 0 errors, 4 warnings. $ rpmlint minetest-server minetest-server.x86_64: W: spelling-error Summary(en_US) multiplayer -> multiplier, multiplexer minetest-server.x86_64: W: spelling-error %description -l en_US multiplayer -> multiplier, multiplexer minetest-server.x86_64: E: incoherent-logrotate-file /etc/logrotate.d/minetest minetest-server.x86_64: W: non-standard-uid /var/lib/minetest minetest minetest-server.x86_64: W: non-standard-gid /var/lib/minetest minetest minetest-server.x86_64: W: no-manual-page-for-binary minetestserver 1 packages and 0 specfiles checked; 1 errors, 5 warnings. $ rpmlint ~/rpmbuild/SRPMS/minetest-0.3.1-4.gitbc0e5c0.fc16.src.rpm minetest.src: W: spelling-error Summary(en_US) Multiplayer -> Multiplier, Multiplexer minetest.src: W: spelling-error %description -l en_US multiplayer -> multiplier, multiplexer 1 packages and 0 specfiles checked; 0 errors, 2 warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 753577] Review Request: gmsh - finite element grid generator
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=753577 --- Comment #3 from Alexey Vasyukov 2011-11-14 17:40:23 EST --- I wrote patches and made some changes to Spec. Regarding the issues mentioned: - ANN library - fixed by patch, using system ANN now - MPEG - disabled just as Tetgen and Metis Regarding contrib directory. It contains: - ANN - unmodified ANN - replaced with system copy of ANN - bamg - modified bamg - Chaco - modified Chaco - DiscreteIntegration - separate sub-module of Gmsh itself - Fl_Tree - modified Fl_Tree widget for FLTK UI - gmm - unmodified GMM - replaced with system copy of GMM - kbipack - separate sub-module of Gmsh itself - MathEx - unmodified MathEx, part of SSCILIB - Fedora does not ship SSCILIB - Metis - disabled in our build - mpeg_encode - disabled in our build - NativeFileChooser - modified NativeFileChooser widget for FLTK UI - Netgen - modified Netgen - Tetgen - disabled in our build Contrib dir summary: - ANN, gmm - replaced with system copies - Metis, mpeg_encode, Tetgen - disabled - DiscreteIntegration, kbipack - sub-modules of Gmsh itself - bamg, Chaco, Fl_Tree, NativeFileChooser, Netgen - modified, derived works - MathEx - unmodified, not in Fedora New spec: http://dl.dropbox.com/u/4532412/RPMS/gmsh.spec New SRPM: http://dl.dropbox.com/u/4532412/RPMS/gmsh-2.5.0-5.fc16.src.rpm Summary of changes: * Mon Nov 14 2011 Alexey Vasyukov - 2.5.0-5 - Disable MPEG support because of patent issues - Add patch to use system ANN instead of built-in one - Add patch to use system GMM instead of built-in one - Update CMake soname patch to avoid hardcoded numbers + rpmlint output (incorrect-fsf-address, spurious-executable-perm, shared-lib-calls-exit) reported upstream The questions: #1 There are 3 'bad' components - MPEG with patent issues plus Tetgen and Metis with non-commercial only clause. I disabled them now with CMake options. Is it enough or should I remove support for them completely from sources? #2 How to deal with contrib dir correctly? I guess, I should package MathEx (SSCILIB) separately and leave derived works (dirs bamg/, Chaco/, Fl_Tree/, NativeFileChooser/, Netgen/) in contribs. Is it so? Any comments and/or suggestions? -- 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 719103] Review Request: media-explorer - Media centre 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=719103 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||media-explorer-0.3.2-1.fc16 Resolution||ERRATA Last Closed||2011-11-14 17:28:29 --- Comment #27 from Fedora Update System 2011-11-14 17:28:29 EST --- media-explorer-0.3.2-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 737286] Review Request: salt - A parallel remote execution system
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=737286 --- Comment #2 from Thomas Spura 2011-11-14 17:38:45 EST --- Ping, any news 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 744349] Review Request: python-rtslib - python lib to configure TCM/LIO kernel target
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=744349 --- Comment #2 from Andy Grover 2011-11-14 17:32:37 EST --- Updated spec URL: http://fedorapeople.org/gitweb?p=grover/public_git/python-rtslib.git;a=blob;f=python-rtslib.spec;h=58ab9c5f68fec5b9b8e6b8295097714eecb31cdf;hb=3e5f448a6b0436a88f836be1f4a4765e71cdeff2 Updated SRPM URL: http://grover.fedorapeople.org/srpms/python-rtslib-1.99.1.git644eece-8.el6.src.rpm Changed checkout instructions to use gzip -n option, hopefully this will help create a tarball with identical bits: 758c519cad364f290320918719db71ea rtslib-1.99.1.git644eece.tar.gz Fixed other issues, thanks as always! -- 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 698692] Review Request: grilo-plugins - Plugins for the Grilo 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=698692 Fedora Update System changed: What|Removed |Added Fixed In Version||grilo-plugins-0.1.15-4.fc15 Resolution|NEXTRELEASE |ERRATA --- Comment #16 from Fedora Update System 2011-11-14 17:24:40 EST --- grilo-plugins-0.1.15-4.fc15, grilo-0.1.15-3.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 743615] Review Request: nagios-plugins-openmanage - Nagios plugin to monitor hardware health on Dell servers
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=743615 Thomas Spura changed: What|Removed |Added CC||toms...@fedoraproject.org --- Comment #3 from Thomas Spura 2011-11-14 16:50:01 EST --- (In reply to comment #2) > (In reply to comment #1) > > Would be nice if you could create a FAS account and do a koji scratch build. > > Yes, I will. Any news here? :) As you are searching for a sponsor and this is your only review request so far, have you done some informal review requests yet? For more information on how to get sponsored see: https://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group#Submitting_quality_new_packages -- 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 744339] Review Request: dieharder - Random number generator tester and timer
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=744339 Richard Shaw changed: What|Removed |Added Status|NEW |ASSIGNED -- 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 744339] Review Request: dieharder - Random number generator tester and timer
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=744339 Richard Shaw changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|hobbes1...@gmail.com Flag||fedora-review? --- Comment #11 from Richard Shaw 2011-11-14 16:50:50 EST --- Ok, I went ahead and built the package, which was successfuly on my Fedora 15 x86_64 system, so that's good. A couple of things: 1. Here's the rpmlint output of the installed package. There's things rpmlint can catch here that it can't from just checking the package: $ rpmlint dieharder dieharder.x86_64: I: enchant-dictionary-not-found en_US dieharder.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libdieharder.so.3.31.1 /usr/lib64/libgslcblas.so.0 dieharder.x86_64: W: shared-lib-calls-exit /usr/lib64/libdieharder.so.3.31.1 exit@GLIBC_2.2.5 1 packages and 0 specfiles checked; 0 errors, 2 warnings. In this case it caught an unused shared library dependency (see below). This isn't a showstopper but should be reported upstream. The configure script probably just needs to be tweaked. $ rpmlint -I unused-direct-shlib-dependency unused-direct-shlib-dependency: The binary contains unused direct shared library dependencies. This may indicate gratuitously bloated linkage; check that the binary has been linked with the intended shared libraries only. 2. You don't need the "*" glob in %files for %{_bindir}/%{name} since there's nothing else to glob. I'll start the full review when I get a chance. Perhaps tonight or tomorrow. -- 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 751925] Review Request: python-tables - Hierarchical datasets in Python
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=751925 --- Comment #12 from Thibault North 2011-11-14 16:36:07 EST --- New Package SCM Request === Package Name: python-tables Short Description: Hierarchical datasets in Python Owners: tnorth Branches: f15 f16 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 744339] Review Request: dieharder - Random number generator tester and timer
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=744339 --- Comment #10 from Jiri Hladky 2011-11-14 16:34:36 EST --- Hi Richard, thanks for your comments. I have cleaned-up a SPEC file a little bit, removing some unnecessary comments:-) > 1. The release tag should start with 1, not 0. I didn't know that, fixed. > 2. If you're not going to build for EL 5 you can remove the following from > your spec file: My intention was to provide rpm for EPEL-5, EPEL-6. The developer has stated that he has used RHEL and Fedora to develop the software. I think it woould be nice to provide packages for RHEL. > 3. The devel subpackage should be arch specific. Fixed. > 4. Leave the "*" off of. Fixed. I have uploaded new version of SPEC file and source rpm to this location: Spec URL: http://jhladky.fedorapeople.org/dieharder.spec SRPM URL: http://jhladky.fedorapeople.org/dieharder-3.31.1-1.fc16.src.rpm Thanks Jirka -- 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 751925] Review Request: python-tables - Hierarchical datasets in Python
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=751925 --- Comment #11 from Thibault North 2011-11-14 16:34:44 EST --- Thank you 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 749132] Review Request: dpm-dsi - Disk Pool Manager (DPM) plugin to GridFTP
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=749132 --- Comment #38 from Steve Traylen 2011-11-14 16:31:22 EST --- Did a bit more digging. So 32 bit packages make it into the 64 repository if they end in a '-devel'. Of course they pull in all their dependencies as well. So when globus-gridftp-server-devel.i686 is selected it pulls in globus-gridftp-server.i686 but not globus-gridftp-server-progs.i686. So that's why its missing but as mentioned why would you ever want this "progs" package at 32 bit on a 64 bit machine. Drop the %{_isa} tag and it should go way. why no epel6 error, simple autoqa is not running :-) and epel5 %{?_isa} is "" anyway. Steve. -- 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 744433] Review Request: libquvi - A cross-platform library for parsing flash media stream
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=744433 Nicoleau Fabien changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE Last Closed||2011-11-14 16:14:39 -- 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 683218] Review Request: drizzle7 - A Lightweight SQL Database for Cloud and Web
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=683218 --- Comment #6 from BJ Dierkes 2011-11-14 15:56:21 EST --- There has been a shortage of development resources upstream... meaning, development has slowed drastically as of several months ago. There have been no followup releases to the stable branch which has kept me from moving forward with this package in Fedora/EPEL. The next GA release is around the corner so it makes sense just to wait for that release. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 753911] New: Review Request: django-profiles - A fairly simple user-profile management application for Django
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: django-profiles - A fairly simple user-profile management application for Django https://bugzilla.redhat.com/show_bug.cgi?id=753911 Summary: Review Request: django-profiles - A fairly simple user-profile management application for Django Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: mru...@matthias-runge.de QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: http://www.matthias-runge.de/fedora/django-profiles.spec SRPM URL: http://www.matthias-runge.de/fedora/django-profiles-0.2-1.fc16.src.rpm Description: A fairly simple user-profile management application for Django, designed to make the management of site-specific user profiles as painless as possible. It requires a functional installation of Django 1.0 or newer and provides a useful complement to `django-registration`, but has no other dependencies. --- [mrunge@sofja SPECS]$ rpmlint ./django-profiles.spec ../SRPMS/django-profiles-0.2-1.fc16.src.rpm ../RPMS/noarch/django-profiles-0.2-1.fc16.noarch.rpm ./django-profiles.spec: W: invalid-url Source0: django-profiles.tar.bz2 django-profiles.src: W: invalid-url Source0: django-profiles.tar.bz2 2 packages and 1 specfiles checked; 0 errors, 2 warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 751925] Review Request: python-tables - Hierarchical datasets in Python
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=751925 Brendan Jones changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #10 from Brendan Jones 2011-11-14 15:28:26 EST --- All good Thibault. This package is 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 556128] Review Request: ff-utils - Utilities to test force feedback of input device
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=556128 Michal Ambroz changed: What|Removed |Added Status|ASSIGNED|ON_QA --- Comment #20 from Michal Ambroz 2011-11-14 15:22:23 EST --- (In reply to comment #19) > Michal, if you put BZ number to Bodhi: > Can you please update either the update or switch this BZ manually? Sure Package was tagged for updates-testing, but is still in status pending - I assume that within days it should be available from the repositories. -- 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 753900] New: Review Request: abi-compliance-checker - An ABI Compliance Checker
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: abi-compliance-checker - An ABI Compliance Checker https://bugzilla.redhat.com/show_bug.cgi?id=753900 Summary: Review Request: abi-compliance-checker - An ABI Compliance Checker Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: hobbes1...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: http://hobbes1069.fedorapeople.org/abi-compliance-checker/abi-compliance-checker.spec SRPM http://hobbes1069.fedorapeople.org/abi-compliance-checker/abi-compliance-checker-1.95.9-1.fc15.src.rpm Description: A tool for checking backward binary compatibility of a shared C/C++ library. It checks for changes in calling stack, changes in v-table, removed symbols, etc. rpmlint output: $ rpmlint SRPMS/abi-compliance-checker-1.95.9-1.fc15.src.rpm RPMS/noarch/abi-compliance-checker-1.95.9-1.fc15.noarch.rpm abi-compliance-checker.src:27: W: rpm-buildroot-usage %prep sed -i "s,\$PREFIX/share/\$TOOL_SNAME,%{buildroot}%{perl_vendorlib}/\$TOOL_SNAME,g" Makefile.pl abi-compliance-checker.noarch: E: zero-length /usr/share/perl5/vendor_perl/abi-compliance-checker/modules/RulesSrc.xml abi-compliance-checker.noarch: W: no-manual-page-for-binary abi-compliance-checker 2 packages and 0 specfiles checked; 1 errors, 2 warnings. rpm-buildroot-usage: Necessary because the makefile does not support DESTDIR. zero-length: I assume the file may not stay empty in the future. no-manual-page-for-binary: I will query upstream for a man page. -- 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 751925] Review Request: python-tables - Hierarchical datasets in Python
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=751925 --- Comment #9 from Thibault North 2011-11-14 14:46:41 EST --- Hi Brendan, According to upstream, Toshio's suggestion was good: remove lrucache.py and get rid of the AFL license. A patch was sent upstream. Here is an updated build. In the %prep section, that file is removed along with its license. Is that good enough for this version of PyTables? Or do we need to remove lrucache.py from the tarball ? Spec URL: http://tnorth.fedorapeople.org/python-tables.spec SRPM URL: http://tnorth.fedorapeople.org/python-tables-2.3.1-3.fc14.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 717502] Review Request: i4uc - IDE for developing micro-controllers firmware
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=717502 --- Comment #25 from Guillermo Gómez 2011-11-14 14:25:00 EST --- Specific: [!] : MUST - Package contains a properly installed %{name}.desktop using desktop-file-install file if it is a GUI application. Please review: http://fedoraproject.org/wiki/Packaging:Guidelines#Desktop_files It is not simply enough to just include the .desktop file in the package, one MUST run desktop-file-install OR desktop-file-validate in %install (and have BuildRequires: desktop-file-utils), to help ensure .desktop file safety and spec-compliance. -- 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 717502] Review Request: i4uc - IDE for developing micro-controllers firmware
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=717502 --- Comment #24 from Guillermo Gómez 2011-11-14 14:11:42 EST --- I know the files are there (spec/srpm), however, if for any reason rpmlint outuput goes removed from such host, then there will be no evidence here about it. So just as a habit, please put here explicit links to both spec and srpm file and include here the rmplint output :) -- 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 752993] gnome-dictionary - A dictionary application for GNOME
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752993 Matthias Clasen changed: What|Removed |Added Flag|needinfo?(mcla...@redhat.co | |m) | --- Comment #2 from Matthias Clasen 2011-11-14 13:38:15 EST --- Hey, I've uploaded new spec and srpm to the same place. Changes: - added Obsoletes - added %doc content - fixed up scriplets -- 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 638786] Review Request: guessencoding - Guess encoding of files and return configured reader
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=638786 --- Comment #5 from Orion Poplawski 2011-11-14 13:04:57 EST --- (In reply to comment #4) > > *** ISSUES *** > - Folder copied is apidocs, should apidocs/* Nope, this is correct. Others fixed: * Mon Nov 14 2011 Orion Poplawski 1.4-2 - Drop BuildRoot, defattr, clean - Add comment for tar generation -- 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 717502] Review Request: i4uc - IDE for developing micro-controllers firmware
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=717502 --- Comment #23 from Larry Letelier 2011-11-14 12:58:32 EST --- (In reply to comment #22) > (In reply to comment #20) > > (In reply to comment #18) > > > Any progress here? > > > > The last changes are here: 02-Nov > > http://lletelier.fedorapeople.org/i4uc/ > > > > i wait next steps. > > Please include links to srpm/spec plus rpmlint output (must be a habit) Dear Guillermo, The rpmlint/srpm file was uploaded here: http://lletelier.fedorapeople.org/i4uc/ Thanks, --LL -- 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 638786] Review Request: guessencoding - Guess encoding of files and return configured reader
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=638786 --- Comment #4 from Tomas Radej 2011-11-14 12:51:50 EST --- Package Review == Key: - = N/A x = Check ! = Problem ? = Not evaluated === REQUIRED ITEMS === [x] Rpmlint output: guessencoding.noarch: W: spelling-error %description -l en_US encodings -> encoding, encoding s, recordings guessencoding.noarch: W: no-documentation guessencoding.src: W: spelling-error %description -l en_US encodings -> encoding, encoding s, recordings guessencoding.src: W: invalid-url Source0: guessencoding-1.4.tar.gz guessencoding-javadoc.noarch: W: spelling-error Summary(en_US) Javadocs -> Java docs, Java-docs, Avocados 3 packages and 0 specfiles checked; 0 errors, 5 warnings. [x] Package is named according to the Package Naming Guidelines[1]. [x] Spec file name must match the base package name, in the format %{name}.spec. [x] Package meets the Packaging Guidelines[2]. [x] Package successfully compiles and builds into binary rpms. [!] Buildroot definition is not present << BuildRoot def is present [x] Package is licensed with an open-source compatible license and meets other legal requirements as defined in the legal section of Packaging Guidelines[3,4]. [x] License field in the package spec file matches the actual license. License type: ASL 2.0 [-] If (and only if) the source package includes the text of the license(s) in its own file, then that file, containing the text of the license(s) for the package is included in %doc. [x] All independent sub-packages have license of their own [x] Spec file is legible and written in American English. [x] Sources used to build the package matches the upstream source, as provided in the spec URL. [x] All build dependencies are listed in BuildRequires, except for any that are listed in the exceptions section of Packaging Guidelines[5]. [x] Package must own all directories that it creates. [x] Package requires other packages for directories it uses. [x] Package does not contain duplicates in %files. [!] File sections do not contain %defattr(-,root,root,-) unless changed with good reason [x] Permissions on files are set properly. [!] Package does NOT have a %clean section which contains rm -rf %{buildroot} (or $RPM_BUILD_ROOT). (not needed anymore) [x] Package consistently uses macros (no %{buildroot} and $RPM_BUILD_ROOT mixing) [x] Package contains code, or permissable content. [-] Fully versioned dependency in subpackages, if present. [-] Package contains a properly installed %{name}.desktop file if it is a GUI application. [x] Package does not own files or directories owned by other packages. [x] Javadoc documentation files are generated and included in -javadoc subpackage [!] Javadocs are placed in %{_javadocdir}/%{name} (no -%{version} symlinks) << Folder copied is apidocs, should apidocs/* [x] Packages have proper BuildRequires/Requires on jpackage-utils [x] Javadoc subpackages have Require: jpackage-utils [-] Package uses %global not %define [!] If package uses tarball from VCS include comment how to re-create that tarball (svn export URL, git clone URL, ...) << Should contain line # tar caf guessencoding-1.4.tar.gz guessencoding-1.4 [-] If source tarball includes bundled jar/class files these need to be removed prior to building [x] All filenames in rpm packages must be valid UTF-8. [x] Jar files are installed to %{_javadir}/%{name}.jar (see [6] for details) [x] If package contains pom.xml files install it (including depmaps) even when building with ant [x] pom files has correct add_maven_depmap === Maven === [x] Use %{_mavenpomdir} macro for placing pom files instead of %{_datadir}/maven2/poms [-] If package uses "-Dmaven.test.skip=true" explain why it was needed in a comment [-] If package uses custom depmap "-Dmaven.local.depmap.file=*" explain why it's needed in a comment [x] Package DOES NOT use %update_maven_depmap in %post/%postun [x] Packages DOES NOT have Requires(post) and Requires(postun) on jpackage-utils for %update_maven_depmap macro === Other suggestions === [x] If possible use upstream build method (maven/ant/javac) [x] Avoid having BuildRequires on exact NVR unless necessary [x] Package has BuildArch: noarch (if possible) [x] Latest version is packaged. *** ISSUES *** - BuildRoot def is present - DefAttr is present - Clean section is present - Folder copied is apidocs, should apidocs/* - Should contain line # tar caf guessencoding-1.4.tar.gz guessencoding-1.4 -- 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://ad
[Bug 750997] Review Request: scilab - Scientific software package for numerical computations
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=750997 Clément DAVID changed: What|Removed |Added Summary|Review Request: Scilab -|Review Request: scilab - |Numerical Analysis toolkit |Scientific software package ||for numerical computations Bug 750997 depends on bug 730970, which changed state. Bug 730970 Summary: Review Request: jhdf5 - Java HDF5 Object Package https://bugzilla.redhat.com/show_bug.cgi?id=730970 What|Old Value |New Value Resolution||ERRATA Status|ON_QA |CLOSED --- Comment #9 from Clément DAVID 2011-11-14 12:28:21 EST --- New Package SCM Request === Package Name: scilab Short Description: Scientific software package for numerical computations Owners: davidcl Branches: f15 f16 el6 InitialCC: Thanks for your remark, I prefer to keep the lowercase version and the description from the spec file (as in comment #7). -- 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 753855] New: Review Request: pslib - C-library to create PostScript files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: pslib - C-library to create PostScript files https://bugzilla.redhat.com/show_bug.cgi?id=753855 Summary: Review Request: pslib - C-library to create PostScript files Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: or...@cora.nwra.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Spec URL: http://www.cora.nwra.com/~orion/fedora/pslib.spec SRPM URL: http://www.cora.nwra.com/~orion/fedora/pslib-0.4.5-1.fc16.src.rpm Description: pslib is a C-library to create PostScript files on the fly. It offers many drawing primitives, inclusion of png and eps images and a very sophisticated text rendering including hyphenation, kerning and ligatures. It can read external Type1 fonts and embed them into the output file. It supports pdfmarks which makes it in combination with ghostscript's pdfwriter an alternative for libraries creating PDF. -- 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 739798] Review Request: pcp-gui - Visualization tools for the Performance Co-Pilot toolkit
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=739798 Frank Ch. Eigler changed: What|Removed |Added Status|NEW |ASSIGNED Flag||needinfo?(mgoodwin@redhat.c ||om) --- Comment #2 from Frank Ch. Eigler 2011-11-14 11:42:35 EST --- FAIL MUST: rpmlint, tested on F15 x86-64 build: [root@very]~/rpmbuild# rpmlint SRPMS/pcp-gui-1.5.1-1.fc15.src.rpm [1 packages and 0 specfiles checked; 0 errors, 0 warnings. [[root@very]~/rpmbuild# rpmlint RPMS/x86_64/pcp-* [pcp-gui.x86_64: E: standard-dir-owned-by-package /usr/share/man/man1 [pcp-gui.x86_64: E: standard-dir-owned-by-package /var/lib [pcp-gui.x86_64: E: incorrect-fsf-address /usr/libexec/pcp/bin/pmsnap [pcp-gui-debuginfo.x86_64: E: incorrect-fsf-address /usr/src/debug/pcp-gui-1.5.1/src/chart/main.cpp [pcp-gui-debuginfo.x86_64: E: incorrect-fsf-address /usr/src/debug/pcp-gui-1.5.1/src/chart/pmchart.cpp [pcp-gui-debuginfo.x86_64: E: incorrect-fsf-address /usr/src/debug/pcp-gui-1.5.1/src/chart/namespace.cpp [3 packages and 0 specfiles checked; 6 errors, 0 warnings. PASS MUST: The spec file name must match the base package. FAIL MUST: No gross violations of Packaging:Guidelines seen. Minor stuff: run desktop-file-install on pmchart.desktop. PASS MUST: The package must be licensed with a Fedora approved license PASS MUST: The License field in the package spec file must match the actual license. (COPYING file includes all licenses). PASS MUST: [License file included in %doc] PASS MUST: The spec file must be written in American English PASS MUST: The spec file for the package MUST be legible PASS MUST: The sources used to build the package must match the upstream source PASS MUST: The package MUST successfully compile and build into binary rpms PASS MUST: If the package does not successfully compile, build or work on an architecture -- none found PASS MUST: All build dependencies must be listed in BuildRequires, PASS MUST: The spec file MUST handle locales properly -- package not i18n'd PASS MUST: Every binary RPM package (or subpackage) which stores shared library files -- none PASS MUST: Packages must NOT bundle copies of system libraries. PASS MUST: If the package is designed to be relocatable, the packager must state this fact PASS MUST: A package must own all directories that it creates. PASS MUST: A Fedora package must not list a file more than once in the spec file's %files listings. PASS MUST: Permissions on files must be set properly -- no gross errors seen PASS MUST: MUST: Each package must consistently use macros -- minimal use PASS MUST: The package must contain code, or permissable content PASS MUST: Large documentation files must go in a -doc subpackage. NB: it's named pcp-doc rather than pcp-gui-doc. PASS MUST: If a package includes something as %doc, it must not affect the runtime of the application. -- apparently PASS MUST: Header files must be in a -devel package -- none PASS MUST: Static libraries must be in a -static package. PASS MUST: If a package contains library files with a suffix (e.g. libfoo.so.1.1) -> -devel -- none PASS MUST: In the vast majority of cases, devel packages must require [...] - none PASS MUST: Packages must NOT contain any .la libtool FAIL MUST: Packages containing GUI applications must include a %{name}.desktop file, Package's pmchart.desktop needs to be desktop-file-install'd FAIL MUST: Packages must not own files or directories already owned by other packages. Same as rpmlint errors at top. PASS MUST: All filenames in rpm packages must be valid UTF-8. OK SHOULD: If the source package does not include license text(s) [...] OK SHOULD: The description and summary sections in the package spec file should contain translations -- n/a KO SHOULD: The reviewer should test that the package builds in mock -- tested on f15 native system only KO SHOULD: The package should compile and build into binary rpms on all supported architectures OK SHOULD: The reviewer should test that the package functions as described. -- smoke-tested OK SHOULD: If scriptlets are used, those scriptlets must be sane -- no scriptlets OK SHOULD: Usually, subpackages other than devel should require the base package using a fully versioned dependency NB: pcp-gui & pcp-doc subpackage not mutually required OK SHOULD: The placement of pkgconfig(.pc) files depends -- none OK SHOULD: If the package has file dependencies outside -- no /file dependencies OK SHOULD: your package should contain man pages for binaries/scripts --
[Bug 745510] Review Request: vdsm - Virtual Desktop Server Manager
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=745510 --- Comment #15 from Federico Simoncelli 2011-11-14 11:40:14 EST --- Spec URL: http://people.redhat.com/fsimonce/vdsm/vdsm.spec SRPM URL: http://people.redhat.com/fsimonce/vdsm/vdsm-4.9.1-0.git31.039976c.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 719908] Review Request: rubygem-multi_json - A gem to provide swappable JSON backends
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=719908 Vít Ondruch changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE Last Closed||2011-11-14 11:21:45 -- 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 738744] Review Request: rubygem-execjs - A package for the execjs Ruby gem
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=738744 Bug 738744 depends on bug 719908, which changed state. Bug 719908 Summary: Review Request: rubygem-multi_json - A gem to provide swappable JSON backends https://bugzilla.redhat.com/show_bug.cgi?id=719908 What|Old Value |New Value Status|NEW |ASSIGNED Status|ASSIGNED|CLOSED Resolution||RAWHIDE -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 746754] Review request: pdfcrack - A Password Recovery Tool for PDF-files.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=746754 Jon Ciesla changed: What|Removed |Added CC||l...@jcomserv.net --- Comment #30 from Jon Ciesla 2011-11-14 10:55:14 EST --- Not sure what happened but it just wasn't there at all. It is now, sans f14 branch since we're not doing new f14 branches. -- 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 753188] Review Request: R-rtracklayer - R interface to genome browsers and their annotation tracks
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=753188 --- Comment #5 from Jon Ciesla 2011-11-14 10:54:42 EST --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 746754] Review request: pdfcrack - A Password Recovery Tool for PDF-files.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=746754 --- Comment #29 from Jon Ciesla 2011-11-14 10:54:10 EST --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 739798] Review Request: pcp-gui - Visualization tools for the Performance Co-Pilot toolkit
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=739798 Frank Ch. Eigler changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|f...@redhat.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 742166] Review Request: wmctrl - X Window Manager command-line tool
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=742166 --- Comment #5 from Jon Ciesla 2011-11-14 10:48:05 EST --- Git done (by process-git-requests). NP. -- 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 719908] Review Request: rubygem-multi_json - A gem to provide swappable JSON backends
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=719908 --- Comment #9 from Jon Ciesla 2011-11-14 10:46:47 EST --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 753188] Review Request: R-rtracklayer - R interface to genome browsers and their annotation tracks
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=753188 Tom "spot" Callaway changed: What|Removed |Added Flag||fedora-cvs? --- Comment #4 from Tom "spot" Callaway 2011-11-14 10:27:31 EST --- New Package SCM Request === Package Name: R-rtracklayer Short Description: R interface to genome browsers and their annotation tracks Owners: spot Branches: f15 f16 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 746754] Review request: pdfcrack - A Password Recovery Tool for PDF-files.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=746754 --- Comment #28 from Richard Shaw 2011-11-14 09:48:50 EST --- FYI, since the package has already been setup once you may need to use the "change" format instead of the "new package" format[1], but Jon will know for sure. Richard [1] http://fedoraproject.org/wiki/Package_SCM_admin_requests#Package_Change_Requests_for_existing_packages -- 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 753188] Review Request: R-rtracklayer - R interface to genome browsers and their annotation tracks
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=753188 Pierre-YvesChibon changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #3 from Pierre-YvesChibon 2011-11-14 09:43:21 EST --- Ok this package is then 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 753188] Review Request: R-rtracklayer - R interface to genome browsers and their annotation tracks
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=753188 --- Comment #2 from Tom "spot" Callaway 2011-11-14 09:34:09 EST --- Jim Kent, the upstream copyright holder for those files, has given permission via email to use/modify/distribute them under the terms of the BSD license. New SPEC: http://spot.fedorapeople.org/R-rtracklayer.spec New SRPM: http://spot.fedorapeople.org/R-rtracklayer-1.14.1-2.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 752829] Review Request: glue-validator - A validation framework for GLUE 2.0 information
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=752829 --- Comment #4 from laurence.fi...@cern.ch 2011-11-14 09:04:37 EST --- I have addressed the issue mentioned. Here is the new version Spec URL: http://lfield.web.cern.ch/lfield/glue-validator.spec SRPM URL: http://lfield.web.cern.ch/lfield/glue-validator-1.0.0-1.el5.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 751411] Review Request: bash-modules - Modules for bash
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=751411 --- Comment #4 from Volodymyr M. Lisivka 2011-11-14 08:24:31 EST --- Ping. -- 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 683218] Review Request: drizzle7 - A Lightweight SQL Database for Cloud and Web
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=683218 Felix Kaechele changed: What|Removed |Added CC||fe...@fetzig.org --- Comment #5 from Felix Kaechele 2011-11-14 06:06:31 EST --- Any news 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 717502] Review Request: i4uc - IDE for developing micro-controllers firmware
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=717502 --- Comment #22 from Guillermo Gómez 2011-11-14 05:17:14 EST --- (In reply to comment #20) > (In reply to comment #18) > > Any progress here? > > The last changes are here: 02-Nov > http://lletelier.fedorapeople.org/i4uc/ > > i wait next steps. Please include links to srpm/spec plus rpmlint output (must be a habit) -- 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 656892] Review Request: ghc-augeas - Haskell bindings for the augeas library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=656892 Jens Petersen changed: What|Removed |Added Status Whiteboard|notready cabal2spec | --- Comment #11 from Jens Petersen 2011-11-14 05:01:22 EST --- Thanks for updating I think I can review this for you as sponsor if you can clean up a little more: We usually put the summary and description in common_summary and common_description. Please remember to bump the release number and add a new changelog entry when you update the package. You should use _bindir and _datadir instead of explicitly writing /usr/bin and /usr/share. You can add (doc) files under the %file sections but you need to be careful since secondary archs don't have shared libraries (ie base subpackage). If you are serious about packaging it would also help if you looked at some other packages that need reviewing and posted comments on them - feel free to put links here to any reviews you have looked over. -- 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 719908] Review Request: rubygem-multi_json - A gem to provide swappable JSON backends
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=719908 Vít Ondruch changed: What|Removed |Added Flag||fedora-cvs? --- Comment #8 from Vít Ondruch 2011-11-14 04:35:12 EST --- New Package SCM Request === Package Name: rubygem-multi_json Short Description: A gem to provide swappable JSON backends Owners: vondruch Branches: 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 742166] Review Request: wmctrl - X Window Manager command-line tool
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=742166 Jens Petersen changed: What|Removed |Added Flag||fedora-cvs? --- Comment #4 from Jens Petersen 2011-11-14 04:16:48 EST --- Sorry, Jon Package Change Request == Package Name: wmctrl New Branches: f16 el6 Owners: petersen -- 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 753577] Review Request: gmsh - finite element grid generator
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=753577 --- Comment #2 from Alexey Vasyukov 2011-11-14 04:04:10 EST --- Dan, many thanks for comments. > - system copy of the ANN library (from the ann package) > must be used instead of the copy in contrib/ANN Ok. I'll work on it. > - sources contains mpeg encoder which is a patented thing Ok. Got the issue. I'll just remove mpeg - it does not affect core functionality. > - contrib/tetgen contains a non-commercial only clause Metis has also similar non-commercial only clause. I disabled them both in CMake configure. Is it not enough? Should I remove them completely from sources? > - the contrib dir contains stuff that could/should exists as separate packages Ok. I'll re-check it. Could you advice which packages require attention? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 748180] Review Request: dbus-sharp-glib - C# bindings for D-Bus glib main loop integration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=748180 --- Comment #9 from Theodore Lee 2011-11-14 03:46:00 EST --- If there are no objections I'd be happy to take 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 753577] Review Request: gmsh - finite element grid generator
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=753577 Dan Horák changed: What|Removed |Added CC||d...@danny.cz --- Comment #1 from Dan Horák 2011-11-14 03:49:23 EST --- few notes - system copy of the ANN library (from the ann package) must be used instead of the copy in contrib/ANN - sources contains mpeg encoder which is a patented thing - contrib/tetgen contains a non-commercial only clause - the contrib dir contains stuff that could/should exists as separate packages -- 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 556128] Review Request: ff-utils - Utilities to test force feedback of input device
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=556128 --- Comment #19 from Miroslav Suchý 2011-11-14 03:44:29 EST --- Michal, if you put BZ number to Bodhi: https://admin.fedoraproject.org/updates/ff-utils-2.4.21-4.fc16 then it will move this BZ automatically to ON_QA and when pushed to stable to CLOSED CURRENT RELEASE. If you omit it you have to switch this BZ manually. Can you please update either the update or switch this BZ manually? -- 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 749132] Review Request: dpm-dsi - Disk Pool Manager (DPM) plugin to GridFTP
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=749132 --- Comment #37 from Ricardo Rocha 2011-11-14 03:25:27 EST --- EL5/EL6 look ok, but there seems to be a dependency missing in FC16/64. https://admin.fedoraproject.org/updates/dpm-dsi-1.8.2-4.fc16 """ AutoQA: depcheck test FAILED on x86_64. Result log: http://autoqa.fedoraproject.org/results/231613-autotest/virt06.qa.fedoraproject.org/depcheck/results /dpm-dsi-1.8.2-4.fc16.html (results are informative only) """ Looking deeper: http://autoqa.fedoraproject.org/results/231613-autotest/virt06.qa.fedoraproject.org/depcheck/results/dpm-dsi-1.8.2-4.fc16.html """ SKIPBROKEN: dpm-dsi-1.8.2-4.fc16.i686 from pending has depsolving problems SKIPBROKEN: --> Package: dpm-dsi-1.8.2-4.fc16.i686 (pending) --> Requires: globus-gridftp-server-progs(x86-32) [view »] "" It seems to try to build both the 64 and 32 versions, meaning it should also depend on the globus(32) libraries. Probably removing the ${_isa} from the dependency will do the trick, i'll have a look. I wonder why it does not fail in EL5/EL6? Anything obvious changed? -- 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 730970] Review Request: jhdf5 - Java HDF5 Object Package
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=730970 Alexander Kurtakov 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