[Bug 456429] Review Request: chm2pdf - A tool to convert CHM files to PDF files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: chm2pdf - A tool to convert CHM files to PDF files https://bugzilla.redhat.com/show_bug.cgi?id=456429 [EMAIL PROTECTED] 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456828] Review Request: libasyncns - Asynchronous Name Service Library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: libasyncns - Asynchronous Name Service Library https://bugzilla.redhat.com/show_bug.cgi?id=456828 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 02:25 EST --- OK source files match upstream: 047dd4c748c6061fa107c1877affc0a34c0847b3 libasyncns-0.4.tar.gz OK package meets naming and versioning guidelines. OK specfile is properly named, is cleanly written and uses macros consistently. OK dist tag is present. OK build root is correct. OK license field matches the actual license. OK license is open source-compatible (LGPLv2+). License text included in package. OK latest version is being packaged. OK BuildRequires are proper. OK compiler flags are appropriate. OK %clean is present. OK package builds in mock (Rawhide/x86_64). OK debuginfo package looks complete. OK rpmlint is silent. OK final provides and requires look sane. N/A %check is present and all tests pass. OK shared libraries are added to the regular linker search paths. OK owns the directories it creates. OK doesn't own any directories it shouldn't. OK no duplicates in %files. OK file permissions are appropriate. OK correct scriptlets present. OK code, not content. OK documentation is small, so no -docs subpackage is necessary. OK %docs are not necessary for the proper functioning of the package. OK headers in devel. OK pkgconfig files in devel. OK no libtool .la droppings. OK not a GUI app. this package is APPROVED you could remove --disable-rpath when calling %configure as it is not understood ... configure: WARNING: Unrecognized options: --disable-rpath ... -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456684] Review Request: pathfinder - X.509 Path Discovery and Validation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: pathfinder - X.509 Path Discovery and Validation https://bugzilla.redhat.com/show_bug.cgi?id=456684 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Status|NEW |ASSIGNED Summary|Review Request: pathfinder -|Review Request: pathfinder - |X.509 Path Discovery and|X.509 Path Discovery and |Validation |Validation 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456684] Review Request: pathfinder - X.509 Path Discovery and Validation
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: pathfinder - X.509 Path Discovery and Validation
https://bugzilla.redhat.com/show_bug.cgi?id=456684
--- Additional Comments From [EMAIL PROTECTED] 2008-07-28 03:12 EST ---
first part of review is here, but it must build in Rawhide before continuing :-)
OK source files match upstream:
cdad78a08f7fc614da104459714da2f94093fae9
pathfinder-0.2.6-Source.tar.gz
OK package meets naming and versioning guidelines.
OK specfile is properly named, is cleanly written and uses macros
consistently.
OK dist tag is present.
OK build root is correct.
OK license field matches the actual license.
OK license is open source-compatible (LGPLv2+). License text included in
package.
OK latest version is being packaged.
OK* BuildRequires are proper.
OK compiler flags are appropriate.
OK %clean is present.
BAD package builds in mock (Rawhide/x86_64).
?? debuginfo package looks complete.
?? rpmlint is silent.
?? final provides and requires look sane.
?? %check is present and all tests pass.
OK shared libraries are added to the regular linker search paths.
BAD owns the directories it creates.
OK doesn't own any directories it shouldn't.
OK no duplicates in %files.
OK file permissions are appropriate.
OK correct scriptlets present (shared libs, new user, service).
OK code, not content.
OK documentation is small, so no -docs subpackage is necessary.
OK %docs are not necessary for the proper functioning of the package.
OK headers in -devel.
OK pkgconfig files in -devel.
OK no libtool .la droppings.
OK not a GUI app.
- pkgconfig can be removed from BR: as it is taken in by the *-devels
- doesn't build in Rawhide
/usr/lib64/ccache/c++-O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions
-fstack-protector --param=ssp-buffer-size=4
-m6/builddir/build/BUILD/pathfinder-0.2.6/dbus/wvdbuslistener.h: In member
function 'void WvDBusListenerP1, P2, E,
E::dispatch(DB/builddir/build/BUILD/pathfinder-0.2.6/pathclient.cc:114:
instantiated from here
/builddir/build/BUILD/pathfinder-0.2.6/dbus/wvdbuslistener.h:115: error: cast
from 'DBusMessageIter*' to 'int' loses precision
make[2]: Leaving directory `/builddir/build/BUILD/pathfinder-0.2.6'
make[2]: *** [CMakeFiles/pathclient.dir/pathclient.o] Error 1
- -devel should own the whole %{_includedir}/pathfinder-1/ dir not only the file
inside it
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456603] Review Request: oggvideotools - Toolbox for manipulating Ogg video files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: oggvideotools - Toolbox for manipulating Ogg video files https://bugzilla.redhat.com/show_bug.cgi?id=456603 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Status|NEW |ASSIGNED Flag||fedora-review? --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 03:37 EST --- For the C*FLAGS I think that you should use CXXFLAGS=$CXXFLAGS -Wall -std=c++0x instead of only CXXFLAGS=-Wall -std=c++0x. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456138] Review Request: edb - Debugger based on the ptrace API and QT4
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: edb - Debugger based on the ptrace API and QT4 https://bugzilla.redhat.com/show_bug.cgi?id=456138 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 03:37 EST --- Update for 0.9.1 : Spec URL: http://nicoleau.fabien.free.fr/rpms/SPECS/edb.spec SRPM URL: http://nicoleau.fabien.free.fr/rpms/srpms.fc9/edb-0.9.1-1.fc9.src.rpm rpmlint output : [EMAIL PROTECTED] tmp]$ rpmlint edb-0.9.1-1.fc9.i386.rpm edb-0.9.1-1.fc9.src.rpm edb-debuginfo-0.9.1-1.fc9.i386.rpm edb-plugins-0.9.1-1.fc9.i386.rpm 4 packages and 0 specfiles checked; 0 errors, 0 warnings. [EMAIL PROTECTED] tmp]$ As it's an x86 debugger, ppc and ppc64 are excluded. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 453569] Review Request: libmirage - library to provide access to different image formats
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: libmirage - library to provide access to different
image formats
https://bugzilla.redhat.com/show_bug.cgi?id=453569
--- Additional Comments From [EMAIL PROTECTED] 2008-07-28 04:15 EST ---
Hi,
sorry for the delay,I was on vacation. The following issues you pointed out
should be fixed:
+ http://cdemu.sourceforge.net/pkg_libmirage.php looks to be a more
-fixed
+ Did you try to get the patch accepted upstream? It might affect SPARC too.
- accepted (should be applied in the latest released version)
+ The versioned dependencies on pkgconfig, flex and glib2-devel are not
- fixed
+ No need to delete %{_libdir}/libmirage/*.a in %install since
-fixed
+ Remove the empty ChangeLog and NEWS from %doc.
- fixed
+ The gzipped tarball is no longer available at the given Source location.
- a new version has been released during this package review, should I update
to it?
+ In order to build the documentation 'BuildRequires: gtk-doc' is needed.
- fixed
+ The -devel subpackage should have 'Requires: gtk-doc' since it puts files
- fixed
+ Should be GPLv2+ instead of LGPLv2+
- fixed
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 441899] Review Request: inadyn-mt - Dynamic DNS Client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: inadyn-mt - Dynamic DNS Client https://bugzilla.redhat.com/show_bug.cgi?id=441899 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 04:38 EST --- Hello, I see no problem with upgrading the license to version 3. I welcome feedback though. That is, do you, or anyone who happens to be listening in, know of any reason simply upgrading to version 3 could be a problem? My reading is that version 3 license is less restrictive than version 2. To clarify, I've changed the relevant source files' license notice to read, either version 3 of the License, or (at your option) any later version. I'll await your feedback before putting up another version. Regards, Bryan PS The program expects the language file is either in /etc/inadyn-mt/lang, or the program will search for directory, lang, from current working directory, up to /inadyn-mt. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456603] Review Request: oggvideotools - Toolbox for manipulating Ogg video files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: oggvideotools - Toolbox for manipulating Ogg video files https://bugzilla.redhat.com/show_bug.cgi?id=456603 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 04:49 EST --- OK source files match upstream: ae804223c15ac2ed39988bd82bd0996bb8c566f2 oggvideotools-0.4.tar.gz OK package meets naming and versioning guidelines. OK specfile is properly named, is cleanly written and uses macros consistently. OK dist tag is present. OK build root is correct. OK license field matches the actual license. OK license is open source-compatible (GPLv2+). License text included in package. OK latest version is being packaged. BAD BuildRequires are proper. BAD compiler flags are appropriate. OK %clean is present. OK package builds in mock (Rawhide/x86_64). BAD debuginfo package looks complete. OK* rpmlint is silent. OK final provides and requires look sane. N/A %check is present and all tests pass. OK no shared libraries are added to the regular linker search paths. OK owns the directories it creates. OK doesn't own any directories it shouldn't. OK no duplicates in %files. OK file permissions are appropriate. OK no scriptlets present. OK code, not content. OK documentation is small, so no -docs subpackage is necessary. OK %docs are not necessary for the proper functioning of the package. OK no headers. OK no pkgconfig files. OK no libtool .la droppings. OK not a GUI app. - you should add BR: libogg-devel, libtheora-devel, SDL-devel or is this intentional ??? - see my comment #4 for the flags, correct debuginfo depends on them too - the Patch: tags are using tabs instead of spaces -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456772] Review Request: libv4l - Collection of video4linux support libraries
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: libv4l - Collection of video4linux support libraries https://bugzilla.redhat.com/show_bug.cgi?id=456772 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 05:31 EST --- Imported and build for rawhide, closing. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 247402] Review Request: GspiceUI - A GUI to freely available Spice Electronic circuit similators
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: GspiceUI - A GUI to freely available Spice Electronic circuit similators https://bugzilla.redhat.com/show_bug.cgi?id=247402 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 05:47 EST --- Thanks Kevin. Imported built. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456756] Review Request: alien - A program that converts between the rpm, dpkg, stampede slp, and slackware tgz
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: alien - A program that converts between the rpm, dpkg, stampede slp, and slackware tgz https://bugzilla.redhat.com/show_bug.cgi?id=456756 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 05:54 EST --- Unless I am wrong, alien depends on dpkg which is not in fedora. I packaged it in the past, it is in http://www.environnement.ens.fr/perso/dumas/fc-srpms/debian/dpkg-1.14.5-1.fc8.src.rpm but needs certainly to be updated. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456527] Review Request: gentium-basic-fonts - Gentium Basic Font Family
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: gentium-basic-fonts - Gentium Basic Font Family https://bugzilla.redhat.com/show_bug.cgi?id=456527 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEEDINFO|NEW Flag|needinfo?([EMAIL PROTECTED]| |m) | --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 06:41 EST --- (In reply to comment #1) I'll start the review process now but since I'll be unavailable till september in a few days someone else will probably have to finish it (or you'll have to wait or be very reactive) Thanks. 2. Gentium Basic is OFL, not GPL Oops. My fault. http://scripts.sil.org/cms/scripts/render_download.php?site_id=nrsiformat=filemedia_id=GentiumBasic_110_difffilename=ttf-sil-gentium-basic_1.1.diff.gz 8. Since the font is effectively a limited Gentium with more faces, you need to teach fontconfig to substitute it to Gentium (and the Gentium packager will need to do it the other way). Look how it's done for dejavu and dejavu lgc, that's just a little fontconfig file to add I am little unsure about what to substitute for faces. The families are generically serif for all the fonts in this package. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456527] Review Request: gentium-basic-fonts - Gentium Basic Font Family
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: gentium-basic-fonts - Gentium Basic Font Family https://bugzilla.redhat.com/show_bug.cgi?id=456527 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 06:45 EST --- Nicolas, The archive name in the %setup section has to be written with '\' to escape the spaces on F-8 machines but does not work on F-9. %setup -q -n Gentium\ Basic\ 1.1 Could you please confirm which is the correct way? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 441899] Review Request: inadyn-mt - Dynamic DNS Client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: inadyn-mt - Dynamic DNS Client https://bugzilla.redhat.com/show_bug.cgi?id=441899 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 06:54 EST --- For the moment the lang file is not packaged at all. Jochen, could you add it, please ? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456828] Review Request: libasyncns - Asynchronous Name Service Library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: libasyncns - Asynchronous Name Service Library https://bugzilla.redhat.com/show_bug.cgi?id=456828 [EMAIL PROTECTED] 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456828] Review Request: libasyncns - Asynchronous Name Service Library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: libasyncns - Asynchronous Name Service Library https://bugzilla.redhat.com/show_bug.cgi?id=456828 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 07:32 EST --- New Package CVS Request === Package Name: libasyncns Short Description: Asynchronous Name Service Library Owners: lennart Branches: InitialCC: Cvsextras Commits: yes -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 452749] Review Request: ocp - Open Cubic Player for MOD/S3M/XM/IT/SID/MIDI music files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: ocp - Open Cubic Player for MOD/S3M/XM/IT/SID/MIDI music files https://bugzilla.redhat.com/show_bug.cgi?id=452749 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|NEEDINFO Flag||needinfo?([EMAIL PROTECTED]) --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 07:58 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456756] Review Request: alien - A program that converts between the rpm, dpkg, stampede slp, and slackware tgz
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: alien - A program that converts between the rpm, dpkg, stampede slp, and slackware tgz https://bugzilla.redhat.com/show_bug.cgi?id=456756 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 09:28 EST --- You are correct, I think including dpkg support would be difficult due to it's need for devhelper. [EMAIL PROTECTED] Desktop]# alien -d '/home/leigh05/Desktop/compizconfig-backend-gconf-0.7.5+git20080423.shame-1.i386.rpm' Package build failed. Here's the log: dh_testdir make: dh_testdir: Command not found make: *** [build] Error 127 find: compizconfig-backend-gconf-0.7.5+git20080423.shame: No such file or directory [EMAIL PROTECTED] Desktop]# -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456756] Review Request: alien - A program that converts between the rpm, dpkg, stampede slp, and slackware tgz
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: alien - A program that converts between the rpm, dpkg, stampede slp, and slackware tgz https://bugzilla.redhat.com/show_bug.cgi?id=456756 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 09:33 EST --- I also packaged it, together with other debian packages, also on: http://www.environnement.ens.fr/perso/dumas/fc-srpms/debian/ -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226319] Merge Review: procps
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: procps https://bugzilla.redhat.com/show_bug.cgi?id=226319 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 09:56 EST --- You can list the GFDL if you want to. We're not requiring that the License tag reflect packaged documentation unless it is the only thing in a package/subpackage. As to the code license, it is up to you, but I would say that the combined work as generated from the procps source is GPLv2 (the most restrictive of the several GPL versions listed in the code bits). If you wanted to list GPLv2 and LGPLv2 as the license, that would also be fine. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 453224] Review Request: prelude-correlator - Correlates IDMEF events for prelude-manager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: prelude-correlator - Correlates IDMEF events for prelude-manager https://bugzilla.redhat.com/show_bug.cgi?id=453224 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 09:57 EST --- Package is shipped in rawhide...closing. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456892] New: Review Request: aget - multi-threaded download accelerator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/show_bug.cgi?id=456892 Summary: Review Request: aget - multi-threaded download accelerator Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: [EMAIL PROTECTED] CC: fedora-package-review@redhat.com,[EMAIL PROTECTED] Spec URL: http://goessinger.eu/fedora/aget.spec SRPM URL: http://goessinger.eu/fedora/aget-0.4-3.fc9.src.rpm Description: Aget is a multi-threaded download accelerator like Flashget. It supports HTTP downloads and can be run from the console. *first ticket, looking for sponsor* -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 192436] Review Request: xorg-x11-server-Xgl
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: xorg-x11-server-Xgl https://bugzilla.redhat.com/show_bug.cgi?id=192436 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 10:27 EST --- FYI: [[ http://gitweb.freedesktop.org/?p=xorg/xserver.git;a=commit;h=d15b3790307053587df8daed1936ff6923881b63 ]] Remove xgl. This code hasn't been updated with anything even resembling what anyone is shipping in nearly thirty months. It hasn't built out of the box since 7.1. Most of its features over AIGLX are accomplished with DRI2 and friends. Is this review worth trying? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456892] Review Request: aget - multi-threaded download accelerator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: aget - multi-threaded download accelerator https://bugzilla.redhat.com/show_bug.cgi?id=456892 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] Summary|Review Request: aget - |Review Request: aget - |multi-threaded download |multi-threaded download |accelerator |accelerator --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 10:29 EST --- So this one is back. Good to see. I can approve this package but I can't sponsor you. Refer http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 452427] Review Request: awesome - Extremely fast, small, dynamic and awesome floating and tiling window manager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: awesome - Extremely fast, small, dynamic and awesome floating and tiling window manager Alias: awesome https://bugzilla.redhat.com/show_bug.cgi?id=452427 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 10:34 EST --- * Thu Jul 28 2008 Michal Nowak [EMAIL PROTECTED] 2.3.3-1 - version bump fixes two bugs - give floating dialogs of maximized windows focus - awesomerc: fix xterm -e in case of others terms * http://mnowak.fedorapeople.org/awesome/awesome.spec * http://mnowak.fedorapeople.org/awesome/awesome-2.3.3-1.fc9.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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 452749] Review Request: ocp - Open Cubic Player for MOD/S3M/XM/IT/SID/MIDI music files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: ocp - Open Cubic Player for MOD/S3M/XM/IT/SID/MIDI music files https://bugzilla.redhat.com/show_bug.cgi?id=452749 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEEDINFO|ASSIGNED Flag|needinfo?([EMAIL PROTECTED]) | --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 10:41 EST --- I've asked upstream to add copyright information to all source files, and they agreed that it was a good idea. So now I'm just waiting to get a snapshot or release with those changes. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456723] Review Request: libatasmart - ATA S.M.A.R.T. Disk Health Monitoring Library
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: libatasmart - ATA S.M.A.R.T. Disk Health Monitoring
Library
https://bugzilla.redhat.com/show_bug.cgi?id=456723
[EMAIL PROTECTED] changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Additional Comments From [EMAIL PROTECTED] 2008-07-28 10:41 EST ---
(In reply to comment #3)
(In reply to comment #1)
[-] source files match upstream:
it doesn't fbffdc4551dd63368babf6fdd659a4bef8e0e647 vs
23116bb3a88fb9ccfbe229ac5a209beb8c80535f
Fixed. Since I am both upstream and the packager I got a bit confused I guess.
OK both are the same now (fbffdc4551dd63368babf6fdd659a4bef8e0e647).
I know that you are upstream, hence the comment about bumping the version when
generating new tarballs ;)
[-] license field matches the actual license:
The spec says License: LGPLv2 while the source files say or
(at your
option) any later version.
Please change to LGPLv2+
Fixed.
OK
(In reply to comment #2)
One more thing pass %{?_smp_mflags} to the make command.
Fixed.
OK
I uploaded the spec file and the srpm to the same location. I didn't bump the
rpm revision.
You should (even during the review process).
As for the package now its fine.
= 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, or are watching someone who is.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 359961] Review Request: drupal-service_links - Enables admins to add links to a number of sites
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: drupal-service_links - Enables admins to add links to a number of sites https://bugzilla.redhat.com/show_bug.cgi?id=359961 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 10:49 EST --- Pretty good! Thanks for the review! New Package CVS Request === Package Name: drupal-service_links Short Description: Enables admins to add links to a number of sites Owners: limb Branches: F-9 InitialCC: Cvsextras Commits: yes -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 455032] Review Request: iwl5000-firmwa re - Firmware for Intel® PRO/Wireless 5000 A/G /N network adaptors
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: iwl5000-firmware - Firmware for Intel® PRO/Wireless 5000 A/G/N network adaptors https://bugzilla.redhat.com/show_bug.cgi?id=455032 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 10:56 EST --- Spec URL: http://kwizart.fedorapeople.org/SPECS/iwl5000-firmware.spec SRPM URL: http://kwizart.fedorapeople.org/SRPMS/iwl5000-firmware-5.4.A.11-2.src.rpm Description: Firmware for Intel® PRO/Wireless 5000 A/G/N network adaptors Changelog - Add LICENSE.iwlwifi-5000-ucode README.iwlwifi-5000-ucode as %%doc I think that previously, packaging License within /lib/firmware was required if packages are installed with --excludedocs , so the License could still be installed. But indeed, nothing still mention this from our packaging guideline. https://fedoraproject.org/wiki/Packaging/LicensingGuidelines#Binary_Firmware Usually nodist package are aimed to be built in the older release (F-8) and then inherited in lastest (Rawhide). Maybe there will be a problem if users expect support from iwl5000 from F-8 as soon as the related firmware is in the F-8 repository (F-8 last kernel doesn't seems to have iwl5000.ko). @Linville Do you think that such support can be provided within the F-8 kernel ? If not, I will built it since F-9 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456723] Review Request: libatasmart - ATA S.M.A.R.T. Disk Health Monitoring Library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: libatasmart - ATA S.M.A.R.T. Disk Health Monitoring Library https://bugzilla.redhat.com/show_bug.cgi?id=456723 [EMAIL PROTECTED] 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456723] Review Request: libatasmart - ATA S.M.A.R.T. Disk Health Monitoring Library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: libatasmart - ATA S.M.A.R.T. Disk Health Monitoring Library https://bugzilla.redhat.com/show_bug.cgi?id=456723 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 10:55 EST --- New Package CVS Request === Package Name: libatasmart Short Description: ATA S.M.A.R.T. Disk Health Monitoring Library Owners: lennart Branches: InitialCC: Cvsextras Commits: yes -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450816] Review Request: alevt - Teletext decoder/browser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: alevt - Teletext decoder/browser https://bugzilla.redhat.com/show_bug.cgi?id=450816 [EMAIL PROTECTED] changed: What|Removed |Added 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 192436] Review Request: xorg-x11-server-Xgl
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: xorg-x11-server-Xgl https://bugzilla.redhat.com/show_bug.cgi?id=192436 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||WONTFIX --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 11:01 EST --- Thanks Michal, I don't had see that... We can now happily close this review. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 429435] Review Request: gnubversion - Gnome interface to Subversion
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: gnubversion - Gnome interface to Subversion
https://bugzilla.redhat.com/show_bug.cgi?id=429435
--- Additional Comments From [EMAIL PROTECTED] 2008-07-28 11:16 EST ---
Thanks for the package, Xavier.
Three things, all of them fairly trivial (and optional -- you may want to choose
to address them before commiting), so they won't block the approval. I'll
approve this once this builds in mock for me and inc case no more issues arise.
1.) Vendor tag
desktop-file-install\
--vendor \
You probably want to set Vendor to fedora as per [1] (given this is a new
package it won't break the updates).
[1] http://fedoraproject.org/wiki/Packaging/Guidelines#Desktop_files
2.) Encoding key
#Fix .desktop file
echo Encoding=UTF-8
$RPM_BUILD_ROOT%{_datadir}/applications/gvn-checkout.desktop
Actually the comment here is misleading. The Encoding key is in fact deprecated
[2]. This seems useless -- older desktop-file-utils implementations just issue a
warning here, not breaking the build (unless I am mistaken...).
[2]
http://standards.freedesktop.org/desktop-entry-spec/desktop-entry-spec-latest.html
3.) A typo in changelog
* Sun Jul 27 2008 Xavier Lamien lxtnow[at]gmail.com - 0.5-3
You may want a character after your e-mail address.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 429435] Review Request: gnubversion - Gnome interface to Subversion
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: gnubversion - Gnome interface to Subversion https://bugzilla.redhat.com/show_bug.cgi?id=429435 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 11:37 EST --- Does not build in el5. Trying fc10. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 454917] Review Request: figlet - a program to print stdin to stdout as ASCIIart
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: figlet - a program to print stdin to stdout as ASCIIart https://bugzilla.redhat.com/show_bug.cgi?id=454917 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 11:54 EST --- I've fixed these problems and uploaded the changes. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 441899] Review Request: inadyn-mt - Dynamic DNS Client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: inadyn-mt - Dynamic DNS Client https://bugzilla.redhat.com/show_bug.cgi?id=441899 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 11:59 EST --- New Upload: Spec URL: http://www.herr-schmitt.de/pub/inadyn-mt/inadyn-mt.spec SRPM URL: http://www.herr-schmitt.de/pub/inadyn-mt/inydyn-mt-2.12.22-2.fc9.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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456603] Review Request: oggvideotools - Toolbox for manipulating Ogg video files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: oggvideotools - Toolbox for manipulating Ogg video files https://bugzilla.redhat.com/show_bug.cgi?id=456603 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 12:07 EST --- Thanks for the review. Regarding the missing BRs - the package needs some autoconf/automake love, which I haven't had time to provide, nor am I an expert in such. I omitted them, because right now if they are present, it builds one extra (and IMHO useless) application, and has the bad side-effect of unnecessarily linking those libs into all the other executables it creates. I've spoken with the author on these points, and he isn't an autoconf/automake master either, so I would like to simply omit the extra BRs until such a time this is fixed upstream. I've fixed the tabs-as-spaces and CXXFLAGS in the spec I had posted, but I guess I didn't post the new srpm. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433892] Review Request: xfce4-gsynaptics-mcs-plugin - GSynaptics icon for the Xfce Settings Manager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: xfce4-gsynaptics-mcs-plugin - GSynaptics icon for the Xfce Settings Manager https://bugzilla.redhat.com/show_bug.cgi?id=433892 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 12:11 EST --- ok, I will create the new package name, can you follow the package end of life procedures for the old name? http://fedoraproject.org/wiki/PackageMaintainers/PackageEndOfLife Did you just want devel here? Or also F8/F9? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 359961] Review Request: drupal-service_links - Enables admins to add links to a number of sites
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: drupal-service_links - Enables admins to add links to a number of sites https://bugzilla.redhat.com/show_bug.cgi?id=359961 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 12:15 EST --- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456828] Review Request: libasyncns - Asynchronous Name Service Library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: libasyncns - Asynchronous Name Service Library https://bugzilla.redhat.com/show_bug.cgi?id=456828 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 12:13 EST --- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456723] Review Request: libatasmart - ATA S.M.A.R.T. Disk Health Monitoring Library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: libatasmart - ATA S.M.A.R.T. Disk Health Monitoring Library https://bugzilla.redhat.com/show_bug.cgi?id=456723 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 12:16 EST --- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450816] Review Request: alevt - Teletext decoder/browser
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: alevt - Teletext decoder/browser
https://bugzilla.redhat.com/show_bug.cgi?id=450816
--- Additional Comments From [EMAIL PROTECTED] 2008-07-28 12:21 EST ---
[EMAIL PROTECTED] SPECS]# diff -pruN alevt.spec_bak alevt.spec
--- alevt.spec_bak 2008-07-28 17:43:35.364713214 +0200
+++ alevt.spec 2008-07-28 18:13:27.216716710 +0200
@@ -1,32 +1,36 @@
Name: alevt
Version: 1.6.2
Release: 4%{?dist}
-Summary: Teletext decoder/browser
+Summary: This program decodes and displays Videotext/Teletext from a /dev/vbi
device
Group: Applications/Multimedia
License: GPLv2
URL: http://goron.de/~froese
Source: http://goron.de/~froese/%{name}/%{name}-%{version}.tar.gz
Source1: alevt.desktop
Patch0: alevt-1.6.2-pixmap.patch
+Patch1: alevt-1.6.2-manpath.patch
BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
BuildRequires: libX11-devel
BuildRequires: libpng-devel
BuildRequires: desktop-file-utils
%description
-This program decodes and displays Videotext/Teletext from a /dev/vbi device.
+AleVT is a teletext/videotext decoder and browser for the
+bttv driver (/dev/vbi) and X11. It features multiple windows,
+a page cache, regexp searching, built-in manual, and more.
+There's also a program to get the time from teletext and
+one to capture teletext pages from scripts.
+
%prep
%setup -q
%patch0 -p1 -b .pixmap
+%patch1 -p1 -b .manpath
%build
-#overwrite $OPT to produce standard Fedora build with propper FLAGS
-
# alevt does not have standard build system, so we populate OPT,
-# which is internal build variable to accomodate Fedora opt flags
-
-# will produce lot of garbage on output
+# which is internal build variable to accommodate Fedora opt flags
+# This will produce lot of garbage on output.
make %{?_smp_mflags} -e OPT=%{optflags}
@@ -35,7 +39,7 @@ rm -rf %{buildroot}
mkdir -p %{buildroot}%{_bindir}
mkdir -p %{buildroot}%{_mandir}/man1
-make USR_X11R6=%{_prefix} MAN=share/man rpm-install
+make USR_X11R6=%{_prefix} MAN=%{_mandir} rpm-install
desktop-file-install \
--dir=%{buildroot}%{_datadir}/applications %{SOURCE1}
* It's mostly description, summary and comments changes. Feel free to tweak
them.
* There's bigger change according to hard-coded man path in MAN variable, now,
with patch, it's in Fedora-macro-way.
[EMAIL PROTECTED] BUILD]# cat ../SOURCES/alevt-1.6.2-manpath.patch
--- alevt-1.6.2/Makefile2008-07-28 18:08:04.778713433 +0200
+++ alevt-1.6.2/Makefile_manpath2008-07-28 18:09:28.572715966 +0200
@@ -66,9 +66,9 @@ rpm-install: all
install -m 0755 alevt${RPM_BUILD_ROOT}$(USR_X11R6)/bin
install -m 0755 alevt-date ${RPM_BUILD_ROOT}$(USR_X11R6)/bin
install -m 0755 alevt-cap${RPM_BUILD_ROOT}$(USR_X11R6)/bin
- install -m 0644 alevt.1x ${RPM_BUILD_ROOT}$(USR_X11R6)/$(MAN)/man1
- install -m 0644 alevt-date.1 ${RPM_BUILD_ROOT}$(USR_X11R6)/$(MAN)/man1
- install -m 0644 alevt-cap.1 ${RPM_BUILD_ROOT}$(USR_X11R6)/$(MAN)/man1
+ install -m 0644 alevt.1x ${RPM_BUILD_ROOT}$(MAN)/man1
+ install -m 0644 alevt-date.1 ${RPM_BUILD_ROOT}$(MAN)/man1
+ install -m 0644 alevt-cap.1 ${RPM_BUILD_ROOT}$(MAN)/man1
install -d 0755 $(RPM_BUILD_ROOT)$(USR_X11R6)/share/pixmaps
install -m 0644 contrib/mini-alevt.xpm
$(RPM_BUILD_ROOT)$(USR_X11R6)/share/pixmaps
* When is the last point done, I'll finish the 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, or are watching someone who is.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 238379] Package review: emesene
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Package review: emesene Alias: emesene https://bugzilla.redhat.com/show_bug.cgi?id=238379 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 12:41 EST --- Spec URL: http://rawen.greystorm.free.fr/Fedora/emesene.spec SRPM URL: http://rawen.greystorm.free.fr/Fedora/emesene-1.0.1-2.fc9.src.rpm I made a mistake, the .spec included in the .src.rpm was not the good one. So, I renamed emesene_old.spec to emesene.spec, I included the doc. An for the remove the dummy .pyc .pyo..., this was something I wanted to investigate, but I discovered that these .pyc and .pyo MUST be in the package, so, nothing has to be done for this. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 442714] Review Request: sat-solver - Satisfyability Solver library which can be used to compute inter-package dependencies.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: sat-solver - Satisfyability Solver library which can be used to compute inter-package dependencies. https://bugzilla.redhat.com/show_bug.cgi?id=442714 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 12:43 EST --- NEW SPEC URL: http://www.binaryhelix.net/sat-solver.spec NEW SRPM URL: http://www.binaryhelix.net/sat-solver-0.9.2-2.fc9.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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 447740] Review Request: zypper - easy to use command line package manager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: zypper - easy to use command line package manager https://bugzilla.redhat.com/show_bug.cgi?id=447740 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 12:45 EST --- NEW Spec URL: http://www.binaryhelix.net/zypper.spec NEW SRPM URL: http://www.binaryhelix.net/zypper-0.11.9-2.fc9.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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 454917] Review Request: figlet - a program to print stdin to stdout as ASCIIart
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: figlet - a program to print stdin to stdout as ASCIIart https://bugzilla.redhat.com/show_bug.cgi?id=454917 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 12:50 EST --- It is very required that for changes you should update URLs to new srpm and spec, increment in release count. So update changelog and release field and post newly build srpm and spec file URLs -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 454917] Review Request: figlet - a program to print stdin to stdout as ASCIIart
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: figlet - a program to print stdin to stdout as ASCIIart https://bugzilla.redhat.com/show_bug.cgi?id=454917 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 12:54 EST --- As soon as you do so I can just help with a detailed pre-review, to help you :-) I am not a sponsor so cannot review officially until you are sponsored. You may like to check wiki entry for 'HowToGetSponsored' -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 429435] Review Request: gnubversion - Gnome interface to Subversion
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: gnubversion - Gnome interface to Subversion https://bugzilla.redhat.com/show_bug.cgi?id=429435 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 12:53 EST --- Built fine. Rpmlint is silent. SPEC file is clean and legible. Filelist is sane. Provides and requires are ok. Proper compiler flags are used. 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 455500] Need OpenGTL package
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Need OpenGTL package https://bugzilla.redhat.com/show_bug.cgi?id=455500 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 13:04 EST --- Created an attachment (id=312796) -- (https://bugzilla.redhat.com/attachment.cgi?id=312796action=view) .spec for OpenGTL package Ok, looks like I'll be trying out the maintainer hat. I don't have an account set up yet (will get to it as soon as I have time), but meanwhile here's the .spec. No patches currently, so it should be trivial to build after wget'ting the upstream source tarball, but note that bug 455502 is still blocking (so expect it to fail on x86_64 unless using CyrilleB's 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 454917] Review Request: figlet - a program to print stdin to stdout as ASCIIart
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: figlet - a program to print stdin to stdout as ASCIIart https://bugzilla.redhat.com/show_bug.cgi?id=454917 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 13:06 EST --- Alright. The spec file is the same url. the srpm is now: http://jesusfreak91.googlepages.com/figlet-2.2.2-2.fc9.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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 455500] Review Request: OpenGTL -- graphics transformation languages (used by krita2)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: OpenGTL -- graphics transformation languages (used by krita2) https://bugzilla.redhat.com/show_bug.cgi?id=455500 [EMAIL PROTECTED] changed: What|Removed |Added Summary|Need OpenGTL package|Review Request: OpenGTL -- ||graphics transformation ||languages (used by krita2) --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 13:11 EST --- Packaging note: last I checked, the non-OpenCTL implementation of painterly colorspaces in krita2 (koffice 2.0+) was buggy (read: it crashed if you tried to use it). I haven't checked if the situation is improved, but IIRC there was talk about making OpenGTL a hard requirement for painterly spaces. Thus, we (Fedora) need OpenGTL to prevent feature loss and/or stability issues. Additionally, since the OpenGTL developer (Cyrille Berger) is also one of the main krita developers, the level of dependency is likely to increase. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456927] Review Request: python-tw-forms - Web Widgets for building and validating forms in ToscaWidgets
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-tw-forms - Web Widgets for building and validating forms in ToscaWidgets https://bugzilla.redhat.com/show_bug.cgi?id=456927 [EMAIL PROTECTED] changed: What|Removed |Added Summary|Review Request: python-tw- |Review Request: python-tw- |forms - short summary here|forms - Web Widgets for ||building and validating ||forms in ToscaWidgets -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456927] New: Review Request: python-tw-forms - short summary here
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/show_bug.cgi?id=456927 Summary: Review Request: python-tw-forms - short summary here Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: [EMAIL PROTECTED] CC: fedora-package-review@redhat.com,[EMAIL PROTECTED] Spec URL: http://toshio.fedorapeople.org/packages/python-tw-forms.spec SRPM URL: http://toshio.fedorapeople.org/packages/python-tw-forms-0.9.1-1.fc9.noarch.rpm Description: tw.forms is a set of html and javascript widgets for the ToscaWidgets framework. They give authors basic html form elements as building blocks for their own web pages and widgets. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 450816] Review Request: alevt - Teletext decoder/browser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: alevt - Teletext decoder/browser https://bugzilla.redhat.com/show_bug.cgi?id=450816 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 13:58 EST --- I've added your patch (thanks). I altered the description: AleVT is a teletext/videotext decoder and browser for the bttv driver (/dev/vbi) and X11. to AleVT is a teletext/videotext decoder and browser for the vbi (/dev/vbi) device and X11. because alevt is not driver dependant, it works with any vbi device and 'bttv driver' seems confusing. ... and bumped version http://lucilanga.fedorapeople.org/alevt.spec http://lucilanga.fedorapeople.org/alevt-1.6.2-5.fc9.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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 359961] Review Request: drupal-service_links - Enables admins to add links to a number of sites
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: drupal-service_links - Enables admins to add links to a number of sites https://bugzilla.redhat.com/show_bug.cgi?id=359961 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 14:04 EST --- drupal-service_links-6.x.1.0-1.fc9 has been submitted as an update for Fedora 9 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456542] Review Request: hotssh - An interface to Secure Shell, for GNOME and OpenSSH
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: hotssh - An interface to Secure Shell, for GNOME and OpenSSH https://bugzilla.redhat.com/show_bug.cgi?id=456542 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 14:09 EST --- Thanks for the review. http://cdn.verbum.org/hotssh-0.2-2.fc9.src.rpm http://cdn.verbum.org/hotssh.spec -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433892] Review Request: xfce4-gsynaptics-mcs-plugin - GSynaptics icon for the Xfce Settings Manager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: xfce4-gsynaptics-mcs-plugin - GSynaptics icon for the Xfce Settings Manager https://bugzilla.redhat.com/show_bug.cgi?id=433892 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 14:38 EST --- Sorry for not being more precise. devel is enough, I don't want to make this change in a stable release. TIA! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456542] Review Request: hotssh - An interface to Secure Shell, for GNOME and OpenSSH
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: hotssh - An interface to Secure Shell, for GNOME and OpenSSH https://bugzilla.redhat.com/show_bug.cgi?id=456542 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 14:56 EST --- More fixes: http://cdn.verbum.org/hotssh-0.2.1-1.fc9.src.rpm http://cdn.verbum.org/hotssh.spec -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456603] Review Request: oggvideotools - Toolbox for manipulating Ogg video files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: oggvideotools - Toolbox for manipulating Ogg video files https://bugzilla.redhat.com/show_bug.cgi?id=456603 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 15:19 EST --- updated spec and SRPM posted at the above URLs. I wound up fixing up upstream's autotools breakage, and sent them the patches to include in the next version. Please complete review. Thanks, Matt -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456941] New: Review Request: thrift - Multi-language RPC and serialization framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/show_bug.cgi?id=456941 Summary: Review Request: thrift - Multi-language RPC and serialization framework Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: [EMAIL PROTECTED] CC: fedora-package-review@redhat.com,[EMAIL PROTECTED] Spec URL: http://dgoodwin.fedorapeople.org/thrift/thrift.spec SRPM URL: http://dgoodwin.fedorapeople.org/thrift/thrift-20080728-0.1.svn680338.fc9.src.rpm Description: Thrift is a software framework for scalable cross-language services development. It combines a powerful software stack with a code generation engine to build services that work efficiently and seamlessly between C++, Java, C#, Python, Ruby, Perl, PHP, Objective C/Cocoa, Smalltalk, Erlang, Objective Caml, and Haskell. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456941] Review Request: thrift - Multi-language RPC and serialization framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: thrift - Multi-language RPC and serialization framework https://bugzilla.redhat.com/show_bug.cgi?id=456941 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 15:22 EST --- ([EMAIL PROTECTED])[~/rpmbuild/SPECS] % rpmlint ~/rpmbuild/SRPMS/thrift-20080728-0.1.svn680338.fc9.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. ([EMAIL PROTECTED])[~/rpmbuild/SPECS] % rpmlint ~/rpmbuild/RPMS/x86_64/*thrift* libthrift-cpp-devel.x86_64: W: no-documentation 7 packages and 0 specfiles checked; 0 errors, 1 warnings. There is a README in the lib/cpp directory but I chose to include it with the libthrift-cpp package instead of -devel. I considered including it with both but saw the package guidelines state that duplicate files shouldn't be included, assuming this applies to sub-packages 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456941] Review Request: thrift - Multi-language RPC and serialization framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: thrift - Multi-language RPC and serialization framework https://bugzilla.redhat.com/show_bug.cgi?id=456941 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 15:29 EST --- Mistakenly submitted this from my work Bugzilla account, this is my Fedora account/email, will it be possible to have this as the package owner? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 448397] Review Request: ntop - A network traffic probe similar to the UNIX top command
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: ntop - A network traffic probe similar to the UNIX top command Alias: ntop_package https://bugzilla.redhat.com/show_bug.cgi?id=448397 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 15:51 EST --- Created an attachment (id=312813) -- (https://bugzilla.redhat.com/attachment.cgi?id=312813action=view) A modified patch A modified ntop-am.patch that applies even with --fuzz=0. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 448397] Review Request: ntop - A network traffic probe similar to the UNIX top command
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: ntop - A network traffic probe similar to the UNIX top command Alias: ntop_package https://bugzilla.redhat.com/show_bug.cgi?id=448397 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 15:54 EST --- Doh, I haven't sent the comment describing the attachment, sorry. Here's the thing - the ntop-am.patch did not apply due to --fuzz=0 being used in rawhide and thus ntop did not build cleanly. I've modified the patch to make things work again. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456603] Review Request: oggvideotools - Toolbox for manipulating Ogg video files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: oggvideotools - Toolbox for manipulating Ogg video files https://bugzilla.redhat.com/show_bug.cgi?id=456603 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 16:14 EST --- All issues are fixed, so this package is APPROVED. But when you publish an updated package, you should increase the release too. It makes easier for the reviewer to track the changes. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456542] Review Request: hotssh - An interface to Secure Shell, for GNOME and OpenSSH
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: hotssh - An interface to Secure Shell, for GNOME and OpenSSH https://bugzilla.redhat.com/show_bug.cgi?id=456542 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 16:24 EST --- http://cdn.verbum.org/hotssh-0.2.1-2.fc9.src.rpm http://cdn.verbum.org/hotssh.spec -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 448397] Review Request: ntop - A network traffic probe similar to the UNIX top command
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: ntop - A network traffic probe similar to the UNIX top command Alias: ntop_package https://bugzilla.redhat.com/show_bug.cgi?id=448397 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 16:25 EST --- Thanks for your work ;-) I will apply it and rebuild soon. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 444745] Review Request: perl-Tk-TableMatrix - Perl module for creating and manipulating tables
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: perl-Tk-TableMatrix - Perl module for creating and manipulating tables https://bugzilla.redhat.com/show_bug.cgi?id=444745 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Status|NEW |ASSIGNED Flag||fedora-review? --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 16:52 EST --- + source files match upstream : 6b7653d129bf1a8327054a88b58d6364 + package meets naming and versioning guidelines. + specfile is properly named, is cleanly written and uses macros consistently. + dist tag is present. + build root is correct. + license field matches the actual license. + license is open source-compatible. License text is included. + latest version is being packaged. + BuildRequires are proper. + compiler flags are appropriate. + %clean is present. + package builds in mock. + package installs properly. - rpmlint is silent : perl-Tk-TableMatrix.i386: W: devel-file-in-non-devel-package /usr/lib/perl5/vendor_perl/5.10.0/i386-linux-thread-multi/Tk/pTk/mm.h perl-Tk-TableMatrix.i386: W: devel-file-in-non-devel-package /usr/lib/perl5/vendor_perl/5.10.0/i386-linux-thread-multi/Tk/pTk/tkTable.h perl-Tk-TableMatrix.i386: W: devel-file-in-non-devel-package /usr/lib/perl5/vendor_perl/5.10.0/i386-linux-thread-multi/Tk/pTk/tkTableversion.h perl-Tk-TableMatrix.i386: W: devel-file-in-non-devel-package /usr/lib/perl5/vendor_perl/5.10.0/i386-linux-thread-multi/Tk/pTk/version.h These ones are ok. perl-Tk-TableMatrix.i386: E: wrong-script-end-of-line-encoding /usr/share/doc/perl-Tk-TableMatrix-1.23/demos/edit_styles.pl perl-Tk-TableMatrix.i386: W: spurious-executable-perm /usr/share/doc/perl-Tk-TableMatrix-1.23/demos/basic perl-Tk-TableMatrix.i386: W: spurious-executable-perm /usr/share/doc/perl-Tk-TableMatrix-1.23/demos/buttons perl-Tk-TableMatrix.i386: W: spurious-executable-perm /usr/share/doc/perl-Tk-TableMatrix-1.23/demos/command perl-Tk-TableMatrix.i386: W: spurious-executable-perm /usr/share/doc/perl-Tk-TableMatrix-1.23/demos/debug perl-Tk-TableMatrix.i386: W: spurious-executable-perm /usr/share/doc/perl-Tk-TableMatrix-1.23/demos/dynarows perl-Tk-TableMatrix.i386: W: spurious-executable-perm /usr/share/doc/perl-Tk-TableMatrix-1.23/demos/edit_styles.pl perl-Tk-TableMatrix.i386: W: spurious-executable-perm /usr/share/doc/perl-Tk-TableMatrix-1.23/demos/maxsize perl-Tk-TableMatrix.i386: W: spurious-executable-perm /usr/share/doc/perl-Tk-TableMatrix-1.23/demos/spreadsheet perl-Tk-TableMatrix.i386: W: spurious-executable-perm /usr/share/doc/perl-Tk-TableMatrix-1.23/demos/SpreadsheetHideRows perl-Tk-TableMatrix.i386: W: spurious-executable-perm /usr/share/doc/perl-Tk-TableMatrix-1.23/demos/TableMatrixSpreadsheetTest perl-Tk-TableMatrix.i386: W: spurious-executable-perm /usr/share/doc/perl-Tk-TableMatrix-1.23/demos/TableMatrixTest Please fix these ones. + final provides and requires are sane: Provides: TableMatrix.so perl(Tk::TableMatrix) = 1.23 perl(Tk::TableMatrix::Spreadsheet) = 1.23 perl(Tk::TableMatrix::SpreadsheetHideRows) = 1.23 Requires: libX11.so.6 libc.so.6 libc.so.6(GLIBC_2.0) libc.so.6(GLIBC_2.1.3) libc.so.6(GLIBC_2.3) libc.so.6(GLIBC_2.3.4) libc.so.6(GLIBC_2.4) libm.so.6 libnsl.so.1 perl(AutoLoader) perl(Carp) perl(Tk) perl(Tk::Derived) perl(Tk::Submethods) perl(Tk::TableMatrix) perl(Tk::TableMatrix::Spreadsheet) perl(base) perl(strict) perl(vars) rtld(GNU_HASH) + %check is present and all tests pass check is disabled by default (needs a display) and a compile switch is provided. + owns the directories it creates. + doesn't own any directories it shouldn't. + no duplicates in %files. + file permissions are appropriate. + no scriptlets present. + code, not content. + documentation is small, so no -docs subpackage is necessary. + %docs are not necessary for the proper functioning of the package. Please fix the rpmlint issues. Also, please reference the perl-Tk bug (RHBZ#456019). Once this is done, I'll approve the package. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456953] Review Request: ocaml-mikmatch - OCaml extension for pattern matching with regexps
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: ocaml-mikmatch - OCaml extension for pattern matching with regexps https://bugzilla.redhat.com/show_bug.cgi?id=456953 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||456796 nThis|| URL||http://martin.jambon.free.fr ||/micmatch.html -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456953] New: Review Request: ocaml-mikmatch - OCaml extension for pattern matching with regexps
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/show_bug.cgi?id=456953 Summary: Review Request: ocaml-mikmatch - OCaml extension for pattern matching with regexps Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: [EMAIL PROTECTED] CC: fedora-package-review@redhat.com,[EMAIL PROTECTED] Spec URL: http://www.annexia.org/tmp/ocaml/ocaml-mikmatch.spec SRPM URL: http://www.annexia.org/tmp/ocaml/ocaml-mikmatch-1.0.0-1.fc10.src.rpm Description: OCaml extension for pattern matching with regexps Mikmatch (with a 'k') is the OCaml = 3.10 version of Micmatch, an extension for adding pattern matching with regular expressions to the language. The goal of Micmatch/Mikmatch is to make text-oriented programs even easier to write, read and run without losing the unique and powerful features of Objective Caml (OCaml). Micmatch/Mikmatch provides a concise and highly readable syntax for regular expressions, and integrates it into the syntax of OCaml thanks to Camlp4. Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=744162 This package is rpmlint CLEAN. $ rpm -q --requires -p ocaml-mikmatch-1.0.0-1.fc10.i386.rpm rpmlib(PayloadFilesHavePrefix) = 4.0-1 rpmlib(CompressedFileNames) = 3.0.4-1 rpmlib(VersionedDependencies) = 3.0.3-1 ocaml(Arg) = 03e86a4154064ea900dc32c05f53e364 ocaml(Array) = aa8e3cd5824f9bb40b93fcd38d0c95b5 ocaml(Buffer) = f6cef633ea14963b84b79c4095c63dc3 ocaml(CamlinternalOO) = 6d0d5b328d6db88f403ca4393b4abd38 ocaml(Camlp4) = 1e46a133b8062d1571640f7fa36f32c4 ocaml(Camlp4_config) = cb716b4361f43326c6ad695c7a1bb5c0 ocaml(Camlp4_import) = 0134ca95282ef6821081c0c11802cea0 ocaml(Char) = e98bc9c9e918a84b3c1a5a122d42fac1 ocaml(Filename) = 633a1e7f590ff5e95124293dbef3b476 ocaml(Format) = 35fe566f7a37d8991a5c822bd1463949 ocaml(Hashtbl) = 083f2c94b44ff4e0b3220aaea6a783b4 ocaml(Int32) = 711321870c949bd3bbdd092d9bae92e4 ocaml(Int64) = f8f7e2e4c0667ead94596040b12e732d ocaml(Lazy) = 8a4b5e7f0bdc6316df9264fd73cde981 ocaml(Lexing) = b1793496643444d3762dd42bebe2cfe3 ocaml(List) = da1ce9168f0408ff26158af757456948 ocaml(Map) = dedde7683d54ae7db1eb97cc868dd047 ocaml(Nativeint) = e79cdc4d3575c2ed044955cb7ef49aca ocaml(Obj) = 5cfae708052c692ea39d23ed930fd64d ocaml(Parsing) = 62cca107e4e88af303516459a87c3e9a ocaml(Pcre) = 9cf03a45728e3cf29272c957775befee ocaml(Pervasives) = 8ba3d1faa24d659525c9025f41fd0c57 ocaml(Printf) = 5dbbf45a03b54e6dbfcf39178d0d6341 ocaml(Queue) = caa3a209bfc63d23a30f573541a88fec ocaml(Set) = 7da14e671a035f12386ace3890018ef3 ocaml(Str) = 56bb7ee61b2da83d42394686e3558fe4 ocaml(Stream) = 21a833e12efd34ea0c87d8d9da959809 ocaml(String) = 2c162ab314b2f0a2cfd22d471b2e21ab ocaml(Sys) = 0da495f5a80f31899139359805318f28 ocaml(Unix) = 9a46a8db115947409e54686ada118599 ocaml(runtime) = 3.10.2 $ rpm -q --provides -p ocaml-mikmatch-1.0.0-1.fc10.i386.rpm ocaml(Mikmatch) = 07d182b26c0f6d948b038a3af1b41469 ocaml(Run_mikmatch_pcre) = 44f39ac3e6ac55f6fbf917bddf8072cb ocaml(Run_mikmatch_str) = 360adfb0137da51ec56cce62f91b7b6f ocaml-mikmatch = 1.0.0-1.fc10 Upstream URL / more about micmatch/mikmatch here: http://martin.jambon.free.fr/micmatch.html -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456955] Review-Request: New package - libzdb
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review-Request: New package - libzdb https://bugzilla.redhat.com/show_bug.cgi?id=456955 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 17:20 EST --- Created an attachment (id=312822) -- (https://bugzilla.redhat.com/attachment.cgi?id=312822action=view) SPEC file for libzdb. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456955] New: Review-Request: New package - libzdb
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/show_bug.cgi?id=456955 Summary: Review-Request: New package - libzdb Product: Fedora Version: rawhide Platform: All URL: http://www.tildeslash.com/libzdb/dist/libzdb- 2.2.2.tar.gz OS/Version: Linux Status: NEW Severity: low Priority: low Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: [EMAIL PROTECTED] CC: fedora-package-review@redhat.com,[EMAIL PROTECTED] Description of problem: New package: libzdb Dependency for: dbmail 2.3.3 (uploaded in another report) Version-Release number of selected component (if applicable): 2.2.2 How reproducible: SRPM and SPEC included. Steps to Reproduce: 1. 2. 3. Actual results: Expected results: Additional info: --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 17:19 EST --- Created an attachment (id=312821) -- (https://bugzilla.redhat.com/attachment.cgi?id=312821action=view) SRPM for libzdb - builds on fedora-rawhide (10) - amd64 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456603] Review Request: oggvideotools - Toolbox for manipulating Ogg video files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: oggvideotools - Toolbox for manipulating Ogg video files https://bugzilla.redhat.com/show_bug.cgi?id=456603 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456603] Review Request: oggvideotools - Toolbox for manipulating Ogg video files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: oggvideotools - Toolbox for manipulating Ogg video files https://bugzilla.redhat.com/show_bug.cgi?id=456603 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 17:38 EST --- New Package CVS Request === Package Name: oggvideotools Short Description: Toolbox for manipulating Ogg video files Owners: mdomsch Branches: F-8 F-9 InitialCC: Cvsextras Commits: yes -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456542] Review Request: hotssh - An interface to Secure Shell, for GNOME and OpenSSH
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: hotssh - An interface to Secure Shell, for GNOME and
OpenSSH
https://bugzilla.redhat.com/show_bug.cgi?id=456542
[EMAIL PROTECTED] changed:
What|Removed |Added
AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
Flag||fedora-review?
--- Additional Comments From [EMAIL PROTECTED] 2008-07-28 18:00 EST ---
Review
==
[-] source files match upstream:
no upstream source, as you are upstream please upload it.
[+] package meets naming and versioning guidelines.
[-] specfile is properly named, is cleanly written and uses macros
consistently:
one minior issue: changelog does not match version
[+] dist tag is present.
[+] build root is correct.
(%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) is the
recommended value, but not the only one)
[+] license field matches the actual license.
[+] license is open source-compatible.
(include one of the below)
license text not included upstream.
[+] license text included in package.
[?] latest version is being packaged.
well I am sure it is its even newer than what is on the
upstream server ;)
[-] BuildRequires are proper:
BR: desktop-file-utils is missing for desktop-file-install
[+] %clean is present.
[-] package builds in koji:
no due to missing BR (desktop-file-install)
[+] package installs properly.
NOTE: tested with local build
[-] rpmlint is silent.
hotssh.noarch: W: non-conffile-in-etc /etc/profile.d/hotssh.csh
hotssh.noarch: W: non-conffile-in-etc /etc/profile.d/hotssh.sh
hotssh.noarch: W: incoherent-version-in-changelog 0.2.1-1
0.2.1-2.fc9
First two warings can be ignored, last one is already noted above
please fix.
[+] owns the directories it creates.
[+] doesn't own any directories it shouldn't.
[+] no duplicates in %files.
[+] file permissions are appropriate.
[+] code, not content.
[+] documentation is small, so no -docs subpackage is necessary.
[+] %docs are not necessary for the proper functioning of the package.
[+] desktop files valid and installed properly.
Comments:
Please do the following fixes:
1) Upload the source tarball.
2) Fix the changelog
3) add the missing BR
After those are done I will approve the package.
There are easy to fix so it should be your last round of fixups ;)
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 454008] Review Request: iax - Implementation of Inter-Asterisk eXchange protocol
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: iax - Implementation of Inter-Asterisk eXchange
protocol
https://bugzilla.redhat.com/show_bug.cgi?id=454008
--- Additional Comments From [EMAIL PROTECTED] 2008-07-28 18:04 EST ---
1. iax.spec.in still refers to the license as LGPL and breaks when I install and
build from the source RPM
2. FYI, my rpmbuild failed on Ubuntu (it could not find any .so packages to
install). It works on Fedora.
Comments
1. The package owns the directories and files it creates
2. ldconfig is run
3. %doc is used correctly
4. Header files and .so are in -devel
5. Main package contains .so.version files
6. -devel package depends on main
7. Package does not contain any .la archives
8. Package does rm -rf %{buildroot} at the beginning of install
9. Package installs to DESTDIR
Balbir
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 455953] Review Request: rakarrack - Audio effects processing rack for guitar
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: rakarrack - Audio effects processing rack for guitar https://bugzilla.redhat.com/show_bug.cgi?id=455953 [EMAIL PROTECTED] changed: What|Removed |Added Summary|Review Request: rakarrack - |Review Request: rakarrack - |Audio effects processing|Audio effects processing |rack for guitar |rack for guitar --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 18:25 EST --- Oops. I noticed that I am ending up placing two desktop items in the Audio Video submenu, one without an icon. I think I noticed newer rakarrak includes a .desktop internally, need to check it out. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 238379] Package review: emesene
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Package review: emesene Alias: emesene https://bugzilla.redhat.com/show_bug.cgi?id=238379 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 19:45 EST --- Checked files in comment #41 and it looks fine. Well done, Florent. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 255721] Review Request: ruby-rrdtool - RRDtool module for Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: ruby-rrdtool - RRDtool module for Ruby https://bugzilla.redhat.com/show_bug.cgi?id=255721 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|201449 | nThis|| Status|CLOSED |ASSIGNED Keywords||Reopened Resolution|DEFERRED| Flag||fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 20:00 EST --- I've found some time and addressed the issues stated above. New SRPM and SPEC are at: SPEC: http://www.kanarip.com/custom/SPECS/ruby-RRDtool.spec SRPM: http://www.kanarip.com/custom/el5/SRPMS/ruby-RRDtool-0.6.0-3.el5.src.rpm Builds in koji (and logs): dist-f8-updates-candidate: http://koji.fedoraproject.org/koji/taskinfo?taskID=744434 dist-f9-updates-candidate: http://koji.fedoraproject.org/koji/taskinfo?taskID=744439 dist-f10: http://koji.fedoraproject.org/koji/taskinfo?taskID=744453 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456972] New: Review Request: eclipse-nls - Babel translations for Eclipse
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/show_bug.cgi?id=456972 Summary: Review Request: eclipse-nls - Babel translations for Eclipse Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: [EMAIL PROTECTED] CC: fedora-package-review@redhat.com,[EMAIL PROTECTED] Spec URL: http://seanf.fedorapeople.org/eclipse-nls/0.2.0-0.1/eclipse-nls.spec SRPM URL: http://seanf.fedorapeople.org/eclipse-nls/0.2.0-0.1/eclipse-nls-0.2.0-0.1.20080720snap.fc10.src.rpm Description: Babel language packs include translations for the Eclipse platform and other Eclipse-related packages. This project obsoletes eclipse-sdk-nls 3.2.1. The Babel project is essentially the successor to the old Eclipse language packs from IBM. About the source: the Babel project hasn't made any releases yet, just an Eclipse update site with nightly snapshots. I have written a script http://seanf.fedorapeople.org/eclipse-nls/0.2.0-0.1/fetch-babel.sh that fetches the latest version from the update site. The source is 40MB, and will only be updated when I fetch a new snapshot, so you might not want to download every new SRPM during the review process. rpmlint complains about my Obsoletes: line, but it looks like a bug to me: https://bugzilla.redhat.com/show_bug.cgi?id=456843 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456972] Review Request: eclipse-nls - Babel translations for Eclipse
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: eclipse-nls - Babel translations for Eclipse https://bugzilla.redhat.com/show_bug.cgi?id=456972 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 20:22 EST --- Just to note that this is Sean's first package and I am sponsoring him. :) But if someone with more experience on eclipse packaging wants to review that is ok by me. Andrew, if you have any initial comments or input on the packaging that would be useful. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 441899] Review Request: inadyn-mt - Dynamic DNS Client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: inadyn-mt - Dynamic DNS Client https://bugzilla.redhat.com/show_bug.cgi?id=441899 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 21:21 EST --- Technically, now I'd say that there are no problems with the new spec. Unfortunately I get Error 404 - Not found Die angegebene Seite konnte nicht gefunden werden. when trying to download the src.rpm. I'll try later today to build locally the src.rpm using upstream's sources The only problem is that I have looked over all the src/* files from v02.12.22 and they either have no license info at all, or claim GPLv2+. According to our guidelines what's in the individual source files wins, so the mere presence of the gpl.txt corresponding to GPL v3 does not make the project GPLv3. Therefore, unless I am very mistaken, the license field must be GPLv2+. I would appreciate a second opinion here. Ref comment #12: AFAIK, you cannot relicence the older sources (which were not written by you) from v2+ to v3 without the approval of the previous devs. Of course, this does not apply to the bits written by yourself. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456892] Review Request: aget - multi-threaded download accelerator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: aget - multi-threaded download accelerator https://bugzilla.redhat.com/show_bug.cgi?id=456892 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||177841 nThis|| -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 453395] Review Request: libmapi - OpenChange: Microsoft Exchange access with native protocols
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: libmapi - OpenChange: Microsoft Exchange access with native protocols https://bugzilla.redhat.com/show_bug.cgi?id=453395 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||177841 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 21:32 EST --- I should have mentioned, I will need to be sponsored, as this is one of my first packages (ie, with the others in this chain). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 453083] Review Request: Samba4 - Samba4 CIFS and AD server and client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: Samba4 - Samba4 CIFS and AD server and client https://bugzilla.redhat.com/show_bug.cgi?id=453083 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 21:37 EST --- BTW, the reason I took and used the tdb_version and talloc_version stuff from the Samba3 spec is that I do expect we might update this requirement in future, as both are fairly core Samba 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 453083] Review Request: Samba4 - Samba4 CIFS and AD server and client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: Samba4 - Samba4 CIFS and AD server and client https://bugzilla.redhat.com/show_bug.cgi?id=453083 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||177841 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 21:43 EST --- And I need to be sponsored -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 454029] Review Request: perl-Catalyst-Model-DBIC-Schema - DBIx::Class::Schema Model Class
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: perl-Catalyst-Model-DBIC-Schema - DBIx::Class::Schema Model Class https://bugzilla.redhat.com/show_bug.cgi?id=454029 [EMAIL PROTECTED] changed: What|Removed |Added BugsThisDependsOn||455151 --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 21:47 EST --- Marking this bug as depending on bug 455151. As soon as we get a push, we should be good to go... -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 452212] Review Request: heimdal - Heimdal Kerberos libraries and KDC
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: heimdal - Heimdal Kerberos libraries and KDC https://bugzilla.redhat.com/show_bug.cgi?id=452212 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||177841 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2008-07-28 21:50 EST --- I'll need to be sponsored for this package. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456603] Review Request: oggvideotools - Toolbox for manipulating Ogg video files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: oggvideotools - Toolbox for manipulating Ogg video files https://bugzilla.redhat.com/show_bug.cgi?id=456603 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Additional Comments From [EMAIL PROTECTED] 2008-07-29 00:02 EST --- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 456603] Review Request: oggvideotools - Toolbox for manipulating Ogg video files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: oggvideotools - Toolbox for manipulating Ogg video files https://bugzilla.redhat.com/show_bug.cgi?id=456603 --- Additional Comments From [EMAIL PROTECTED] 2008-07-29 01:22 EST --- oggvideotools-0.4-1.fc9 has been submitted as an update for Fedora 9 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
