[Bug 613857] Review Request: uvcdynctrl - Command line interface to libwebcam
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=613857 Randall Berry changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR) -- 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 473046] Review Request: miniupnpc - command line tool to control NAT in UPnP-enabled routers as Linksys, D-Link etc
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=473046 Avi Alkalay changed: What|Removed |Added Keywords||Reopened Status|CLOSED |ASSIGNED Resolution|NOTABUG | --- Comment #28 from Avi Alkalay 2010-07-13 01:26:20 EDT --- I have made updates to this package based on comments here and now a working x86_64 version is available. Please check here: http://avi.alkalay.net/software/miniupnpc/ http://avi.alkalay.net/software/miniupnpc/miniupnpc-1.4-5.fc13.src.rpm http://avi.alkalay.net/software/miniupnpc/miniupnpc-1.4-5.fc13.x86_64.rpm http://avi.alkalay.net/software/miniupnpc/miniupnpc-devel-1.4-5.fc13.x86_64.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 613881] New: Review Request: contextkit - Contextual information collection framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: contextkit - Contextual information collection framework https://bugzilla.redhat.com/show_bug.cgi?id=613881 Summary: Review Request: contextkit - Contextual information collection framework Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: supercyp...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Description: This is ContextKit, a framework for collecting contextual information from the bowels of the system, cleaning them up and offering them through a simple API. The ContextKit consists of: - libcontextprovider, a convenience library to export contextual properties to the rest of the system. - user documentation including a list of standard context properties - contextd, daemon for combining and refining contextual information - libcontextsubscriber, a library implementing the simple API for accessing the contextual information. SPEC:http://dl.dropbox.com/u/1338197/1/contextkit.spec SRPM:http://dl.dropbox.com/u/1338197/1/contextkit-0.5.8-1.fc13.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 612023] Review Request: tinycdb - Utility and library for manipulating constant databases
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=612023 Chen Lei changed: What|Removed |Added Blocks||613881 -- 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 613881] Review Request: contextkit - Contextual information collection 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=613881 Chen Lei changed: What|Removed |Added Depends on||612023 -- 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 613857] Review Request: uvcdynctrl - Command line interface to libwebcam
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=613857 Randall Berry changed: What|Removed |Added CC||randyn3...@gmail.com Depends on||576023 -- 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 576023] Review Request: libwebcam - user-space configuration of the uvcvideo driver
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=576023 Randall Berry changed: What|Removed |Added Blocks||613857 -- 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 613879] New: fillmore - audio and video editors for GNOME
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: fillmore - audio and video editors for GNOME https://bugzilla.redhat.com/show_bug.cgi?id=613879 Summary: fillmore - audio and video editors for GNOME Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: mcla...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Target Release: --- http://mclasen.fedorapeople.org/fillmore.spec http://mclasen.fedorapeople.org/fillmore-0.1.0-1.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 612533] Review Request: qrencode - Generate QR 2D barcodes
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=612533
--- Comment #7 from Parag AN(पराग) 2010-07-13 00:43:43 EDT
---
1)rpmlint when run on above package gives
a)qrencode.x86_64: E: binary-or-shlib-defines-rpath /usr/bin/qrencode
['/usr/lib64']
The binary or shared library defines `RPATH'. Usually this is a bad thing
because it hardcodes the path to search libraries and so makes it difficult to
move libraries around. Most likely you will find a Makefile with a line like:
gcc test.o -o test -Wl,--rpath. Also, sometimes configure scripts provide a
--disable-rpath flag to avoid this.
=> sed commands should be run after %configure and then make command. But this
way tests are failing. So, I think we can use last approach given at
http://fedoraproject.org/wiki/Packaging/Guidelines#Removing_Rpath and instead
of adding sed commands in %build, you can add following at the end of %install
commands.
chrpath --delete $RPM_BUILD_ROOT%{_bindir}/qrencode
b)qrencode.x86_64: E: library-without-ldconfig-postin
/usr/lib64/libqrencode.so.3.1.1
This package contains a library and provides no %post scriptlet containing a
call to ldconfig.
qrencode.x86_64: E: library-without-ldconfig-postun
/usr/lib64/libqrencode.so.3.1.1
This package contains a library and provides no %postun scriptlet containing a
call to ldconfig.
==> When package contains libraries you need to use
http://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Shared_libraries
Use following
%post -p /sbin/ldconfig
%postun -p /sbin/ldconfig
2) If you have plan to build this only for devel branch then you can remove
BuildRoot tag, %clean section and cleaning of buildroot in %install
See http://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag
http://fedoraproject.org/wiki/Packaging/Guidelines#.25clean
--
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 456353] Review Request: libffado - Free firewire audio driver 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=456353 --- Comment #36 from Orcan 'oget' Ogetbil 2010-07-13 00:37:10 EDT --- Here is an update: SPEC: http://oget.fedorapeople.org/review/libffado.spec SRPM: http://oget.fedorapeople.org/review/libffado-2.0.1-2.20100706.svn1864.fc13.src.rpm I am working with the Debian maintainer to patch out the external libraries. The discussion is on the upstream devel ML. Changelog: 2.0.1-2.20100706.svn1864 - Add ENABLE_ALL flag to support more devices - Don't bundle tests - Include some preliminary documentation for the tools until the manpages arrive - Patch out bundled libraries. Also fixes some rpmlints - Improve the instructions how to create the tarball I hope I got everything. -- 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 609506] Review Request: gtk2hs-buildtools - Buildtools for gtk2hs
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=609506 Shakthi Kannan changed: What|Removed |Added Flag||fedora-review? --- Comment #4 from Shakthi Kannan 2010-07-12 23:58:24 EDT --- Sorry! Done now. Forgot 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 609506] Review Request: gtk2hs-buildtools - Buildtools for gtk2hs
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=609506 Jens Petersen changed: What|Removed |Added Status|NEW |ASSIGNED --- Comment #3 from Jens Petersen 2010-07-12 23:41:59 EDT --- Shakthi, if you are taking the review then please set the "fedora‑review" Flag to '?', 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 608447] Review Request: perl-Data-Serializer - Modules that serialize data structures
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=608447 Iain Arnell changed: What|Removed |Added Status|ASSIGNED|MODIFIED --- Comment #6 from Iain Arnell 2010-07-12 23:18:08 EDT --- Thanks for the review. Permissions fixed and committed. -- 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 608447] Review Request: perl-Data-Serializer - Modules that serialize data structures
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=608447 --- Comment #8 from Fedora Update System 2010-07-12 23:18:59 EDT --- perl-Data-Serializer-0.49-2.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/perl-Data-Serializer-0.49-2.fc12 -- 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 608447] Review Request: perl-Data-Serializer - Modules that serialize data structures
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=608447 --- Comment #7 from Fedora Update System 2010-07-12 23:18:54 EDT --- perl-Data-Serializer-0.49-2.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/perl-Data-Serializer-0.49-2.fc13 -- 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 595697] Review Request: ghc-regex-tdfa - Haskell "tagged" DFA regex engine
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=595697 Jens Petersen changed: What|Removed |Added Blocks|177841(FE-NEEDSPONSOR) | Depends on||595603 --- Comment #5 from Jens Petersen 2010-07-12 23:19:07 EDT --- (In reply to comment #3) > Yes, I am doing some packaging for Leksah IDE and this is a requirement. OK cool. So hope you will submit leksah later too then. > regex-tdfa,binary_shared,ltk,hslogger. BTW ltk needs gtk, which is available for F13 but not yet repackaged for F14, though there are dependent package reviews in the queue... I am sponsoring Lakshmi in bug 595603, so making that bug block this for now instead. -- 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 612023] Review Request: tinycdb - Utility and library for manipulating constant databases
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=612023
--- Comment #6 from Chen Lei 2010-07-12 23:18:25 EDT ---
(In reply to comment #5)
> Your package looks almost fine to me. Just a couple of minor remarks:
>
> - Add file debian/copyright as %doc. Even if "public domain" isn't a real
> license, the corresponding copyright information should be added to the binary
> package.
>
> - The tarball contains a pkgconfig file in folder debian/. It's probably a
> good
> idea to add it to the package. It must be slightly adapted, though.
>
> - I suggest to replace the second sentence of the devel %description with:
> This package contains library and header files for developing applications
> that
> use %{name}.
All fixed except license file which is not needed for this package.
>From licensing guideline:
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
must be included in %doc. If the source package does not include the text of
the license(s), the packager should contact upstream and encourage them to
correct this mistake.
SRPM: http://dl.dropbox.com/u/1338197/1/tinycdb-0.77-2.fc13.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 595603] Review Request: ghc-hslogger - Haskell logging 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=595603 Jens Petersen changed: What|Removed |Added Blocks||595697 -- 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 595603] Review Request: ghc-hslogger - Haskell logging 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=595603 Jens Petersen changed: What|Removed |Added Status|NEW |ASSIGNED Blocks|177841(FE-NEEDSPONSOR) | AssignedTo|nob...@fedoraproject.org|peter...@redhat.com Flag||fedora-review? --- Comment #12 from Jens Petersen 2010-07-12 23:10:19 EDT --- I am offering to sponsor Lakshmi for this review. I took a quick look at the .spec file and basically looks ok. Can you add a changelog entry to document the changes you made relative to cabal2spec (you can also use cabal2spec-diff for that). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 613646] Review Request: twlog - A simple day to day logging program for ham radio
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=613646 --- Comment #4 from Randall Berry 2010-07-12 22:18:04 EDT --- No MimeType key in the .desktop file so I guess update-desktop-database is not needed. -- 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 583672] Review Request: tomtom - A CLI interface to Tomboy or Gnote via dbus
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=583672 --- Comment #14 from Tom "spot" Callaway 2010-07-12 21:20:15 EDT --- Red Hat Legal advises that they can't give out legal advise to independent projects, so whether that name is free of any possible legal claim is something you'd need to talk to your own lawyer about. With that said, they're okay with Fedora including this code under the name "Scout". -- 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 613646] Review Request: twlog - A simple day to day logging program for ham radio
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=613646 --- Comment #3 from Randall Berry 2010-07-12 21:07:57 EDT --- Originally Ham Radio apps .desktop files were solely in the "HamRadio;" category which put them in "Other" on the desktop. Someone came along and filed bugs against all ham radio apps that the categories were against FreeDesktop.org's standards so they were all changed to "Network;HamRadio;" to conform. Which put them in the "Internet" category on the desktop. Doesn't make sense I know, but I guess logic was not in the mix when the standard was made. Documentation is here: http://standards.freedesktop.org/menu-spec/latest/apa.html I'll check into adding update-desktop-database 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 613857] Review Request: uvcdynctrl - Command line interface to libwebcam
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=613857 --- Comment #2 from Michael Cronenworth 2010-07-12 21:08:03 EDT --- And... I was missing license files. :( Same spec link. SRPM: http://michael.cronenworth.com/RPMS/uvcdynctrl-0.2.0-0.2.20100322svn.fc13.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 613857] Review Request: uvcdynctrl - Command line interface to libwebcam
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=613857 --- Comment #1 from Michael Cronenworth 2010-07-12 21:03:51 EDT --- Ah, I clicked commit too quickly. Had older spec with wrong SVN release convention. Fixed. Spec: http://michael.cronenworth.com/RPMS/uvcdynctrl.spec SRPM: http://michael.cronenworth.com/RPMS/uvcdynctrl-0.2.0-0.1.20100322svn.fc13.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 576023] Review Request: libwebcam - user-space configuration of the uvcvideo driver
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=576023 --- Comment #11 from Michael Cronenworth 2010-07-12 20:58:29 EDT --- (In reply to comment #10) > Hi Michael, Hi Hans, wow big reply! I will try to digest it. > Oh and since circa 2 years I work for RedHat now, but that is of little > relevance in this case, this is me with my Fedora hat firmly on :) Yes, I am familiar with who you are. Nice to see you here. > We want the libwebcam package to install udev rules to automatically run > uvcdynctrl to tell the driver about these controls when a relevant webcam gets > plugged in (or is plugged in wen the system boots up). I see that you're > current package is missing uvcdynctrl and the udev rules, so that will be a > first thing to fix. It is probably best to base your work for this on top of > the work Paulo has done for Debian, instead of: > # svn export -r 90 http://svn.quickcamteam.net/svn/qct/webcam-tools/trunk/ > libwebcam-0.2.0 > do: > # svn export -r 90 > http://svn.quickcamteam.net/svn/qct/webcam-tools/branches/deb libwebcam-0.2.0 I was waiting[1] on upstream to merge in the patches for this, but it looks like it won't happen any time soon. I'll add it to my spec. > And I have matching patches for libwebcam / uvcdynctrl for this. I've send > these to libwebcam upstream, but unfortunately libwebcam upstream is not > really > fast with processing these, so we will need to add these to the Fedora > libwebcam package for now. Are these in the kernel already? > 1) You should add uvcdynctrl + udev rules to the libwebcam package, it is best > to base your work on the Debian branch for this, to get the Debian branch do: > svn export -r 90 http://svn.quickcamteam.net/svn/qct/webcam-tools/branches/deb > libwebcam-0.2.0 New spec: http://michael.cronenworth.com/RPMS/libwebcam.spec SRPM: http://michael.cronenworth.com/RPMS/libwebcam-0.2.0-0.3.20100322svn.fc13.src.rpm > > 2) I would like to see some patches added to the Fedora package which add > support for menu and button controls, but lets first just tackle 1). If you > can > provide an updated spec file and srpm with 1) fixed I'll do a more thorough > review of that. I'll be more than happy to work with you on those patches. > An important part of the getting sponsored process is showing that you've got > the necessary packaging skills and knowledge of the packaging guidelines as > mentioned already in comment #8 one way of doing this is doing reviews of > other > packages which are awaiting review (when you do this please add a note you are > not doing an official review). I haven't gotten to do some unofficial reviews yet. > > An other option is to show your skills by creating a couple of packages. Given > that your current libwebcam package only packages the library itself, I'm > guessing that you are packaging libwebcam because it is needed by something > else you plan to package. You could for example also submit that for review > (it > is ok to have multiple review requests outstanding while waiting for > sponsoring). Or if I may suggest something you could package v4l-utils, the > successor of libv4l: > http://www.linuxtv.org/downloads/v4l-utils/ > http://cvs.fedoraproject.org/viewvc/devel/libv4l/libv4l.spec?view=co I'll let you handle that, if you wish. I am going to package uvcdynctrl plus my own app, uvcdynctrl-gtk (pretty obvious what that is). I wanted to start with libwebcam first, but if having those two packages in review as well will help this process along, I'll add them. I'll start with uvcdynctrl. My app needs a .desktop and icon file, which I have not made yet. Bug 613857 > > Regards, > > Hans Thanks for the pointers! -- 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 613857] New: Review Request: uvcdynctrl - Command line interface to libwebcam
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: uvcdynctrl - Command line interface to libwebcam https://bugzilla.redhat.com/show_bug.cgi?id=613857 Summary: Review Request: uvcdynctrl - Command line interface to libwebcam Product: Fedora Version: 13 Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: m...@cchtml.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://michael.cronenworth.com/RPMS/uvcdynctrl.spec SRPM URL: http://michael.cronenworth.com/RPMS/uvcdynctrl-0.2.0-0.1.svn90.fc13.src.rpm Description: Uvcdynctrl is a command line interface for manipulating settings in UVC-type webcams. It uses the libwebcam library for webcam access. This is my second package up for review. libwebcam[1] being my first. [1] https://bugzilla.redhat.com/show_bug.cgi?id=576023 -- 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 613646] Review Request: twlog - A simple day to day logging program for ham radio
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=613646 John J. McDonough changed: What|Removed |Added CC||wb8...@arrl.net --- Comment #2 from John J. McDonough 2010-07-12 20:02:53 EDT --- Looks pretty good to me, and it works. The only delta I see is that it would be better for it to appear on the "Other" menu rather than "Internet", but it isn't the only amateur radio app with that behavior. Should there be an update-desktop-database? I never got that whole business clear. -- 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 606498] Review Request: hwloc - portable abstraction of hierarchical architectures
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=606498 --- Comment #17 from Jiri Hladky 2010-07-12 19:05:27 EDT --- Thanks Brice! Mamoru, I had discussion with Brice regarding creating a patch file to hwloc 1.0.1. However, there is quite a lot of changes and it includes some binaries files (.png, .pdf) which has been added to hwloc source tree. Nevertheless I have tried to create a patch file but it does not work: svn diff http://svn.open-mpi.org/svn/hwloc/tags/hwloc-1.0.1 http://svn.open-mpi.org/svn/hwloc/branches/v1.0 > my.patch tar zxvf ~/tests/performance/hwloc/hwloc-1.0.1.tar.gz cd hwloc-1.0.1/ $patch -p0 -i ../my.patch patching file include/private/private.h ..stripped. up-to this error message: patching file utils/Makefile.am can't find file to patch at input line 822 Perhaps you used the wrong -p or --strip option? The text leading up to this was: -- |Index: tests/embedded/Makefile.am |=== |--- tests/embedded/Makefile.am (.../tags/hwloc-1.0.1) (revision 2305) |+++ tests/embedded/Makefile.am (.../branches/v1.0) (revision 2305) = Since we are close to hwloc 1.0.2 release we have agreed to post here 1.0.2 alpha version. Brice will prepare a source tree tarball for me. Would it be OK with you? 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 613646] Review Request: twlog - A simple day to day logging program for ham radio
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=613646 --- Comment #1 from Randall Berry 2010-07-12 18:48:03 EDT --- Mon Jul 12 2010 Randall "Randy" Berry - 2.6-2 - Tweek install wrapper Spec URL: http://dp67.fedorapeople.org/pkgs/SPECS/twlog.spec SRPM URL: http://dp67.fedorapeople.org/pkgs/SRPMS/twlog-2.6-2.fc13.src.rpm Koji Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2314723 -- 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 225243] Merge Review: amanda
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=225243 Orion Poplawski changed: What|Removed |Added CC||jgo...@redhat.com --- Comment #19 from Orion Poplawski 2010-07-12 18:06:42 EDT --- We should get this into EPEL for EL-6. I'm willing to maintain there if needed. -- 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 592074] Review Request: R-SparseM - Sparse linear algebra
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=592074 --- Comment #5 from Sandro Mathys 2010-07-12 18:08:17 EDT --- Paul Betten stated that PCx is commercially licensed and he can't carve out a routine (cholesky.f). -- 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 611648] Review Request: mtdev - Multitouch Protocol Translation 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=611648 Peter Hutterer changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #9 from Peter Hutterer 2010-07-12 17:46:47 EDT --- unrelated, this is the base package for future true multitouch support that's currently being worked on upstream. package is imported, closing as NEXTRELEASE -- 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 200760] Review Request: dogtail
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=200760 Zack Cerza changed: What|Removed |Added Flag||fedora-cvs? --- Comment #12 from Zack Cerza 2010-07-12 17:03:36 EDT --- Package Change Request == Package Name: dogtail New Branches: EL-6 Owners: zmc -- 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 606498] Review Request: hwloc - portable abstraction of hierarchical architectures
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=606498 --- Comment #16 from Brice Goglin 2010-07-12 16:41:53 EDT --- FYI, upstream patch backported to the 1.0 branch in commit r2304 https://svn.open-mpi.org/trac/hwloc/changeset/2304 -- 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 613200] Review Request: rmic-maven-plugin - RMI Compiler Maven Plugin
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=613200 chris.sp...@arcor.de changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE -- 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 613766] Review Request: python-xlwt - Library to generate files compatible with Microsoft Excel
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=613766 Athmane Madjoudj changed: What|Removed |Added CC||athma...@gmail.com --- Comment #1 from Athmane Madjoudj 2010-07-12 15:38:42 EDT --- Build: [OK] rpmlint output: $ rpmlint python-xlwt-0.7.2-1.fc13.src.rpm python-xlwt.src: I: enchant-dictionary-not-found en_US 1 packages and 0 specfiles checked; 0 errors, 0 warnings. $ rpmlint python-xlwt.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. $ rpmlint python-xlwt-0.7.2-1.fc13.noarch.rpm python-xlwt.noarch: I: enchant-dictionary-not-found en_US python-xlwt.noarch: W: file-not-utf8 /usr/share/doc/python-xlwt-0.7.2/licences.py 1 packages and 0 specfiles checked; 0 errors, 1 warnings. Notes: - Tip to correct file-not-utf8 warning: https://fedoraproject.org/wiki/Common_Rpmlint_issues#file-not-utf8 -- 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 592074] Review Request: R-SparseM - Sparse linear algebra
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=592074 --- Comment #4 from Sandro Mathys 2010-07-12 15:36:11 EDT --- I wrote Paul Betten a mail, maybe a second poke after the one of spot helps - you never know. No idea what else we could do 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 241279] Review Request: avr-libc - C library for use with GCC on Atmel AVR microcontrollers
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=241279 Thibault North changed: What|Removed |Added CC||thibault.no...@gmail.com Flag|fedora-cvs+ |fedora-cvs? --- Comment #24 from Thibault North 2010-07-12 15:31:16 EDT --- Package Change Request == Package Name: avr-libc New Branches: EL-6 Owners: tnorth trondd We would like to have FEL-related packages available for EL-6. Thanks ! -- 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 234750] Review Request: avr-binutils - Cross Compiling GNU binutils targeted at avr
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=234750 Thibault North changed: What|Removed |Added CC||thibault.no...@gmail.com Flag|fedora-cvs+ |fedora-cvs? --- Comment #30 from Thibault North 2010-07-12 15:31:09 EDT --- Package Change Request == Package Name: avr-binutils New Branches: EL-6 Owners: tnorth trondd We would like to have FEL-related packages available for EL-6. Thanks ! -- 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 476056] Review Request: gnustep-back - The GNUstep backend 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=476056 --- Comment #15 from Michel Alexandre Salim 2010-07-12 15:25:14 EDT --- PS please request at least EL-6, and possibly EL-5 branches 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 591737] Review Request: R-ROC - Utilities for ROC
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=591737 --- Comment #6 from Sandro Mathys 2010-07-12 15:27:18 EDT --- Answer from upstream: "because some day i may check and introduce the code." - so I guess it's not old but new code but not in use anyway. I could cease to install the file but I fear that I'll forget about it later and it'll give me some headache some day when the code would be used. So if you have no objection I'd leave it as-is. -- 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 591737] Review Request: R-ROC - Utilities for ROC
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=591737 --- Comment #5 from Sandro Mathys 2010-07-12 15:21:53 EDT --- Spec URL: http://red.fedorapeople.org/SRPMS/R-ROC.spec SRPM URL: http://red.fedorapeople.org/SRPMS/R-ROC-1.24.1-1.fc12.src.rpm Okay, there's been a change to the license, COPYING was removed and the license changed to Artistic 2.0. Asked upstream about ROC.hyndman.R now but I think we don't need to wait for the answer here as this is non-critical. I guess it's simply old code that upstream didn't want to throw away yet. -- 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 476056] Review Request: gnustep-back - The GNUstep backend 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=476056 Michel Alexandre Salim changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #14 from Michel Alexandre Salim 2010-07-12 15:24:40 EDT --- Looks good. 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 608069] Tango standard library for D language of d1 specification
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=608069 --- Comment #6 from MERCIER Jonathan 2010-07-12 15:23:38 EDT --- thanks for this macro (i did not know) spec anr src.rpm are updated: - http://bioinfornatics.fedorapeople.org/tango.spec - http://bioinfornatics.fedorapeople.org/tango-0.99.9-1.3.20102406svn5487.fc13.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 613766] New: Review Request: python-xlwt - Library to generate files compatible with Microsoft Excel
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-xlwt - Library to generate files compatible with Microsoft Excel https://bugzilla.redhat.com/show_bug.cgi?id=613766 Summary: Review Request: python-xlwt - Library to generate files compatible with Microsoft Excel Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: jspal...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://people.fedoraproject.org/~jspaleta/python-xlwt/python-xlwt.spec SRPM URL: http://people.fedoraproject.org/~jspaleta/python-xlwt/python-xlwt-0.7.2-1.fc13.src.rpm Description: A Library to create spreadsheet files compatible with MS Excel 97/2000/XP/2003 XLS files. This is a python module required by helper tools included in matplotlib's mpl_toolkit collection. -- 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 608419] Review Request: perl-CGI-Application-FastCGI - For using CGI::Application under FastCGI
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=608419 --- Comment #5 from Fedora Update System 2010-07-12 15:02:29 EDT --- perl-CGI-Application-FastCGI-0.02-2.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/perl-CGI-Application-FastCGI-0.02-2.fc12 -- 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 608419] Review Request: perl-CGI-Application-FastCGI - For using CGI::Application under FastCGI
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=608419 --- Comment #4 from Fedora Update System 2010-07-12 15:02:24 EDT --- perl-CGI-Application-FastCGI-0.02-2.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/perl-CGI-Application-FastCGI-0.02-2.fc13 -- 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 609142] Review Request: felix-main - Apache Felix Main Classes
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=609142 Bug 609142 depends on bug 607389, which changed state. Bug 607389 Summary: Review Request: felix-osgi-compendium - Felix OSGi R4 Compendium Bundle https://bugzilla.redhat.com/show_bug.cgi?id=607389 What|Old Value |New Value Status|NEW |ASSIGNED Resolution||NEXTRELEASE Status|ASSIGNED|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 607389] Review Request: felix-osgi-compendium - Felix OSGi R4 Compendium Bundle
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=607389 Victor G. Vasilyev changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #9 from Victor G. Vasilyev 2010-07-12 14:44:59 EDT --- Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2313872 -- 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 609130] Review Request: felix-framework - Felix Framework Interfaces and Classes
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=609130 Bug 609130 depends on bug 607389, which changed state. Bug 607389 Summary: Review Request: felix-osgi-compendium - Felix OSGi R4 Compendium Bundle https://bugzilla.redhat.com/show_bug.cgi?id=607389 What|Old Value |New Value Status|NEW |ASSIGNED Resolution||NEXTRELEASE Status|ASSIGNED|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 591415] Review Request: R-timeDate - Rmetrics - chronological and calendarical objects
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=591415 --- Comment #7 from Sandro Mathys 2010-07-12 14:23:45 EDT --- There's been a new release recently but I don't think this issue has been addressed :( Also I never got a reply to my last email :( But I don't give up yet, wrote yet another message. If that doesn't work I'll think about going to their office, they should be in reachable distance. -- 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 612943] Review Request: spice-protocol - Spice protocol header 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=612943 --- Comment #7 from Kevin Fenzi 2010-07-12 13:25:49 EDT --- CVS done (by process-cvs-requests.py). -- 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 613067] Review Request:spice - Implements the SPICE protocol
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=613067 --- Comment #9 from Kevin Fenzi 2010-07-12 13:26:53 EDT --- CVS done (by process-cvs-requests.py). -- 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 608925] Review Request: python-iso8601 - Simple module to parse ISO 8601 dates
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=608925 --- Comment #3 from Kevin Fenzi 2010-07-12 13:22:52 EDT --- CVS done (by process-cvs-requests.py). -- 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 613200] Review Request: rmic-maven-plugin - RMI Compiler Maven Plugin
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=613200 --- Comment #8 from Kevin Fenzi 2010-07-12 13:29:07 EDT --- CVS done (by process-cvs-requests.py). -- 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 609164] Review Request: xbean - Java plugin based server analogous to Eclipse being a plugin based IDE
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=609164 --- Comment #5 from Kevin Fenzi 2010-07-12 13:23:46 EDT --- CVS done (by process-cvs-requests.py). -- 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 568315] Review Request: xapply - Parallel Execution tool ala xargs/apply
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=568315 --- Comment #10 from Paul P Komkoff Jr 2010-07-12 13:25:57 EDT --- Very sorry for the delay. Will try to send an update this week for real. -- 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 604331] Review Request: redshift - Adjusts the color temperature of your screen according to time of day
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=604331
Mamoru Tasaka changed:
What|Removed |Added
CC||mtas...@ioa.s.u-tokyo.ac.jp
--- Comment #3 from Mamoru Tasaka 2010-07-12
13:43:13 EDT ---
Some notes:
* License
- As far as I checked the whole source code, the license tag should
be "GPLv3+".
* BuildRoot
- BuildRoot tag is no longer needed on Fedora:
https://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag
! Note that on EPEL5 and below BuildRoot tag is still needed.
* Naming of gtk subpackage
- Normally Fedora names gui related subpackage as "%{name}-gui" or
"%{name}-gtk", and the former (-gui) is more general if no other
gui backend (such as qt or so) is provided.
* Python related (Build)Requires
- Currently your srpm won't build on Fedora 12 because of lacking
%python_sitelib definition. Please refer to:
https://fedoraproject.org/wiki/Packaging/Python#Macros
- Please use "python2" or "python3" for (Build)Requires, c.f.
https://fedoraproject.org/wiki/Packaging/Python#BuildRequires
* Dependencies between subpackages
- Usually dependencies between binary rpms rebuilt from the same
srpm must be EVR(Epoch-Version-Release) specific:
https://fedoraproject.org/wiki/Packaging/Guidelines#Requiring_Base_Package
* Make build.log more verbose
- Currently build.log does not show how linkage on the created binary
is done:
--
183CCLD redshift
184 make[3]: Leaving directory `/builddir/build/BUILD/redshift-1.4.1/src'
185 make[2]: Leaving directory `/builddir/build/BUILD/redshift-1.4.1/src'
--
Please add "V=1" to "make %{?_smp_mflags}".
* Timestamp
- Please consider to use
--
make DESTDIR=%{buildroot} install INSTALL="install -p"
--
to keep timestamps on the installed files as much as possible.
This method usually works for Makefiles generated by recent
autotools.
* Desktop file
- When gui program is installed, the corresponding desktop file must be
properly installed:
https://fedoraproject.org/wiki/Packaging/Guidelines#Desktop_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 554243] Review Request: moovida - Media Center
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=554243 --- Comment #31 from Alex Lancaster 2010-07-12 13:41:38 EDT --- (In reply to comment #30) > (In reply to comment #29) > > > I posted a request for status update over a week ago (2010-06-23), therefore > > according to policy: > > a week , if the guy is on holidays ? , IMHO you should wait a year . There's nothing to stop the submitter re-opening the bug and continuing the review. We waited for over 3 months for an update, and still nothing, as per policy: https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews#Submitter_not_responding Closing it will allow somebody else to open a new review without thinking that somebody else is working on it, otherwise it might stay in this limbo state forever. -- 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 607389] Review Request: felix-osgi-compendium - Felix OSGi R4 Compendium Bundle
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=607389 --- Comment #8 from Kevin Fenzi 2010-07-12 13:20:16 EDT --- CVS done (by process-cvs-requests.py). -- 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 608419] Review Request: perl-CGI-Application-FastCGI - For using CGI::Application under FastCGI
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=608419 --- Comment #3 from Kevin Fenzi 2010-07-12 13:22:06 EDT --- CVS done (by process-cvs-requests.py). -- 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 608447] Review Request: perl-Data-Serializer - Modules that serialize data structures
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=608447 --- Comment #5 from Kevin Fenzi 2010-07-12 13:22:24 EDT --- CVS done (by process-cvs-requests.py). -- 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 607875] Review Request: perl-Net-IMAP-Simple-SSL - Simple IMAP account handling with SSL
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=607875 --- Comment #5 from Kevin Fenzi 2010-07-12 13:22:33 EDT --- You may also want to consider a EL-6 branch. ;) -- 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 607875] Review Request: perl-Net-IMAP-Simple-SSL - Simple IMAP account handling with SSL
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=607875 --- Comment #4 from Kevin Fenzi 2010-07-12 13:21:46 EDT --- CVS done (by process-cvs-requests.py). -- 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 561456] Review Request: jnr-x86asm - Pure-java port of asmjit
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=561456 Mohammed Morsi changed: What|Removed |Added Flag||fedora-cvs? --- Comment #7 from Mohammed Morsi 2010-07-12 13:20:09 EDT --- Sorry for the delay. New Package CVS Request === Package Name: jnr-x86asm Short Description: Pure-java port of asmjit (http://code.google.com/p/asmjit/) Owners: mmorsi Branches: F-13 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 613725] New: Review Request: mediawiki-SyntaxHighlight_GeSHi - This extension adds source code highlighting for mediawiki
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: mediawiki-SyntaxHighlight_GeSHi - This extension adds source code highlighting for mediawiki https://bugzilla.redhat.com/show_bug.cgi?id=613725 Summary: Review Request: mediawiki-SyntaxHighlight_GeSHi - This extension adds source code highlighting for mediawiki Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: mastah...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://stahnma.fedorapeople.org/reviews/mediawiki-SyntaxHighlight_GeSHi.spec SRPM URL: http://stahnma.fedorapeople.org/reviews/mediawiki-SyntaxHighlight_GeSHi-1.14_r45462-1.el5.src.rpm Description: This extension adds source code highlighting for mediawiki via the tag. -- 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 606720] Review Request: maven-shade-plugin - This plugin provides the capability to package the artifact in an uber-jar
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=606720 --- Comment #11 from Kevin Fenzi 2010-07-12 13:19:42 EDT --- CVS done (by process-cvs-requests.py). -- 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 581104] Review Request: lv2-EQ10Q-plugins - Parametric equalizer lv2 plugin
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=581104 --- Comment #18 from Kevin Fenzi 2010-07-12 13:15:43 EDT --- CVS done (by process-cvs-requests.py). -- 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 543689] Package review: libipathverbs - Driver library for libibverbs
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=543689 --- Comment #7 from Kevin Fenzi 2010-07-12 13:15:02 EDT --- CVS done (by process-cvs-requests.py). -- 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 543685] Package review: libnes - Driver library for libibverbs
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=543685 --- Comment #8 from Kevin Fenzi 2010-07-12 13:14:36 EDT --- CVS done (by process-cvs-requests.py). -- 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 532382] Review Request: fvkbd - Free Virtual Keyboard
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=532382 --- Comment #13 from Kevin Fenzi 2010-07-12 13:13:11 EDT --- CVS done (by process-cvs-requests.py). -- 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 473583] Review Request: wordnet - A lexical database for the english language
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=473583 --- Comment #37 from Kevin Fenzi 2010-07-12 13:12:01 EDT --- CVS done (by process-cvs-requests.py). -- 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 433497] Review Request: swing-layout - Natural layout for Swing panels
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=433497 Kevin Fenzi changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #16 from Kevin Fenzi 2010-07-12 13:10:45 EDT --- cvs done. -- 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 465897] Review Request: Judy - General purpose dynamic array 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=465897 --- Comment #20 from Kevin Fenzi 2010-07-12 13:11:26 EDT --- CVS done (by process-cvs-requests.py). -- 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 238705] Review Request: avr-gcc - Cross Compiling GNU GCC targeted at avr
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=238705 --- Comment #23 from Kevin Fenzi 2010-07-12 13:08:51 EDT --- CVS done (by process-cvs-requests.py). -- 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 608973] Review Request: gtk-equinox-engine - Equinox theme engine for GTK+ 2.x
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=608973 Germán Racca changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE -- 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 226239] Merge Review: perl-Archive-Tar
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=226239 --- Comment #15 from Kevin Fenzi 2010-07-12 13:07:25 EDT --- I see a devel branch... just import the src.rpm and file a rel-eng ticket to get it unblocked. -- 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 225683] Merge Review: dev86
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=225683 --- Comment #8 from Kevin Fenzi 2010-07-12 13:00:52 EDT --- CVS done (by process-cvs-requests.py). -- 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 226142] Merge Review: mikmod
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=226142 --- Comment #6 from Kevin Fenzi 2010-07-12 13:01:30 EDT --- CVS done (by process-cvs-requests.py). -- 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 608973] Review Request: gtk-equinox-engine - Equinox theme engine for GTK+ 2.x
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=608973 --- Comment #19 from Fedora Update System 2010-07-12 12:55:23 EDT --- gtk-equinox-engine-1.20-3.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/gtk-equinox-engine-1.20-3.fc13 -- 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 613200] Review Request: rmic-maven-plugin - RMI Compiler Maven Plugin
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=613200 chris.sp...@arcor.de 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 554243] Review Request: moovida - Media Center
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=554243 --- Comment #30 from Sergio Monteiro Basto 2010-07-12 12:41:17 EDT --- (In reply to comment #29) > I posted a request for status update over a week ago (2010-06-23), therefore > according to policy: a week , if the guy is on holidays ? , IMHO you should wait a year . -- 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 608973] Review Request: gtk-equinox-engine - Equinox theme engine for GTK+ 2.x
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=608973 --- Comment #18 from Fedora Update System 2010-07-12 12:43:46 EDT --- gtk-equinox-engine-1.20-3.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/gtk-equinox-engine-1.20-3.fc12 -- 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 179916] Review Request: docbook2X - Convert docbook into man and Texinfo
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=179916 --- Comment #11 from Kevin Fenzi 2010-07-12 12:41:08 EDT --- Should be there now. -- 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 613697] Review Request: mingw32-libogg - The Ogg bitstream file format 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=613697
--- Comment #2 from Richard W.M. Jones 2010-07-12 12:22:10
EDT ---
+ rpmlint output
+ package name satisfies the packaging naming guidelines
+ specfile name matches the package base name
+ package should satisfy packaging guidelines
+ license meets guidelines and is acceptable to Fedora
BSD, and the native package is already in Fedora
+ license matches the actual package license
+ %doc includes license file
+ spec file written in American English
+ spec file is legible
+ upstream sources match sources in the srpm
+ package successfully builds on at least one architecture
tried it on x86-64, cross-compiled to i386
n/a ExcludeArch bugs filed
+ BuildRequires list all build dependencies
+ %find_lang instead of %{_datadir}/locale/*
n/a binary RPM with shared library files must call ldconfig in %post and
%postun
+ does not use Prefix: /usr
+ package owns all directories it creates
- no duplicate files in %files
%{_mingw32_docdir}/libogg-%{version} is not needed, or if
you want to include it, it must be marked %doc
+ %defattr line
n/a %clean contains rm -rf $RPM_BUILD_ROOT
+ consistent use of macros
+ package must contain code or permissible content
n/a large documentation files should go in -doc subpackage
+ files marked %doc should not affect package
+ header files should be in -devel
this rule is not applicable for mingw32-* packages
+ static libraries should be in -static
this rule is not applicable for mingw32-* packages
n/a libfoo.so must go in -devel
n/a -devel must require the fully versioned base
+ packages should not contain libtool .la files
this rule is not applicable for mingw32-* packages
n/a packages containing GUI apps must include %{name}.desktop file
+ packages must not own files or directories owned by other packages
+ filenames must be valid UTF-8
+ use %global instead of %define
Optional:
n/a if there is no license file, packager should query upstream
n/a translations of description and summary for non-English languages, if
available
- reviewer should build the package in mock
- the package should build into binary RPMs on all supported architectures
- review should test the package functions as described
+ scriptlets should be sane
n/a pkgconfig files should go in -devel
+ shouldn't have file dependencies outside /etc /bin /sbin /usr/bin or
/usr/sbin
Please fix the docdir thing mentioned above.
--
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 613697] Review Request: mingw32-libogg - The Ogg bitstream file format 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=613697 Richard W.M. Jones changed: What|Removed |Added Flag||needinfo?(kra...@redhat.com ||) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 613697] Review Request: mingw32-libogg - The Ogg bitstream file format 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=613697 Richard W.M. Jones changed: What|Removed |Added Depends on||454410(mingw32-gcc) -- 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 454410] Review Request: mingw32-gcc - MinGW Windows cross-compiler (GCC) for C and C++
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=454410 Richard W.M. Jones changed: What|Removed |Added Blocks||613697 -- 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 613697] Review Request: mingw32-libogg - The Ogg bitstream file format 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=613697 --- Comment #1 from Richard W.M. Jones 2010-07-12 12:16:03 EDT --- That file is a binary RPM, not the SRPM. Nevertheless I reconstructed the SRPM from the spec file. rpmlint output: mingw32-libogg.src: W: spelling-error Summary(en_US) Ogg -> Egg, Org, Gog mingw32-libogg.src: W: spelling-error Summary(en_US) bitstream -> bit stream, bit-stream, midstream mingw32-libogg.src: W: spelling-error %description -l en_US Ogg -> Egg, Org, Gog mingw32-libogg.src: W: spelling-error %description -l en_US bitstream -> bit stream, bit-stream, midstream mingw32-libogg.src: W: spelling-error %description -l en_US bitstreams -> bit streams, bit-streams, bloodstreams Bogus as usual. mingw32-libogg.src: W: no-cleaning-of-buildroot %install mingw32-libogg.src: W: no-cleaning-of-buildroot %clean mingw32-libogg.src: W: no-buildroot-tag mingw32-libogg.src: W: no-%clean-section I think these are all no longer required. mingw32-libogg.noarch: W: spelling-error Summary(en_US) Ogg -> Egg, Org, Gog mingw32-libogg.noarch: W: spelling-error Summary(en_US) bitstream -> bit stream, bit-stream, midstream mingw32-libogg.noarch: W: spelling-error %description -l en_US Ogg -> Egg, Org, Gog mingw32-libogg.noarch: W: spelling-error %description -l en_US bitstream -> bit stream, bit-stream, midstream mingw32-libogg.noarch: W: spelling-error %description -l en_US bitstreams -> bit streams, bit-streams, bloodstreams Bogus as above. 2 packages and 0 specfiles checked; 0 errors, 14 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 606430] Review Request: DMitry - Deepmagic Information Gathering 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=606430 Mamoru Tasaka changed: What|Removed |Added CC||mtas...@ioa.s.u-tokyo.ac.jp --- Comment #2 from Mamoru Tasaka 2010-07-12 12:17:41 EDT --- (In reply to comment #0) > Additionally, upstream appears to have > died so I have taken over upstream maintenance and released a new version that > fixes a few compilation warnings. I think it is better that you rename the upstream tarball you maintain (and the project name) to DMitry-ng (new generation) or so, name srpm as such and make DMitry-ng binary rpm obsolete/provide old DMitry. How do you think? -- 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 613697] Review Request: mingw32-libogg - The Ogg bitstream file format 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=613697 Richard W.M. Jones changed: What|Removed |Added Status|NEW |ASSIGNED 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 613697] Review Request: mingw32-libogg - The Ogg bitstream file format 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=613697 Richard W.M. Jones changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|rjo...@redhat.com -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 613697] New: Review Request: mingw32-libogg - The Ogg bitstream file format library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: mingw32-libogg - The Ogg bitstream file format library https://bugzilla.redhat.com/show_bug.cgi?id=613697 Summary: Review Request: mingw32-libogg - The Ogg bitstream file format library Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: kra...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Target Release: --- Spec URL: http://kraxel.fedorapeople.org/review/mingw32-libogg/mingw32-libogg.spec SRPM URL: http://kraxel.fedorapeople.org/review/mingw32-libogg/mingw32-libogg-1.1.4-1.fc14.noarch.rpm Description: Libogg is a library for manipulating Ogg bitstream file formats. Libogg supports both making Ogg bitstreams and getting packets from Ogg bitstreams. -- 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 597315] Review Request: bwa - Burrows-Wheeler Alignment 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=597315 --- Comment #12 from Adam Huffman 2010-07-12 11:20:05 EDT --- Indeed - I had neglected to push my builds to Bodhi. Once I've updated the package, is it best to remove the old update from Bodhi? -- 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 578269] Review Request: xgospel - An X11 client for Internet Go 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=578269 --- Comment #27 from Mamoru Tasaka 2010-07-12 11:18:09 EDT --- Again 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 612943] Review Request: spice-protocol - Spice protocol header 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=612943 Gerd Hoffmann changed: What|Removed |Added Flag||fedora-cvs? --- Comment #6 from Gerd Hoffmann 2010-07-12 11:05:02 EDT --- New Package CVS Request === Package Name: spice-protocol Short Description: Spice protocol header files Owners: kraxel alexl 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 613067] Review Request:spice - Implements the SPICE protocol
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=613067 Gerd Hoffmann changed: What|Removed |Added Flag||fedora-cvs? --- Comment #8 from Gerd Hoffmann 2010-07-12 11:07:47 EDT --- New Package CVS Request === Package Name: spice Short Description: Implements the SPICE protocol Owners: kraxel alexl 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 612023] Review Request: tinycdb - Utility and library for manipulating constant databases
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=612023
--- Comment #5 from Martin Gieseking 2010-07-12
11:08:12 EDT ---
Your package looks almost fine to me. Just a couple of minor remarks:
- Add file debian/copyright as %doc. Even if "public domain" isn't a real
license, the corresponding copyright information should be added to the binary
package.
- The tarball contains a pkgconfig file in folder debian/. It's probably a good
idea to add it to the package. It must be slightly adapted, though.
- I suggest to replace the second sentence of the devel %description with:
This package contains library and header files for developing applications that
use %{name}.
--
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 597315] Review Request: bwa - Burrows-Wheeler Alignment 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=597315 --- Comment #11 from Chen Lei 2010-07-12 10:42:59 EDT --- It seems bwa 0.5.8a is out now. -- 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 613067] Review Request:spice - Implements the SPICE protocol
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=613067
Matthias Clasen changed:
What|Removed |Added
Flag||fedora-review+
--- Comment #7 from Matthias Clasen 2010-07-12 10:37:39
EDT ---
Setting CFLAGS as you do now is probably a nop, looking at the definition of
%configure:
%configure \
CFLAGS="${CFLAGS:-%optflags}" ; export CFLAGS ; \
CXXFLAGS="${CXXFLAGS:-%optflags}" ; export CXXFLAGS ; \
...
but that is not a big deal. The rest looks fine now. 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
