[Bug 221769] Review Request: libcgi - CGI easy as C
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: libcgi - CGI easy as C https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221769 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 02:10 EST --- approved. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245431] Review Request: libtommath - portable number theoretic multiple-precision integer 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: libtommath - portable number theoretic multiple-precision integer library https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245431 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 02:38 EST --- (In reply to comment #3) Some remarks. MUSTFIX: - Package doesn't acknowledge RPM_OPT_FLAGS. makefile.shared overrides CFLAGS causing this package to be compiled with non-standard CFLAGS: E.g.: ... libtool --mode=compile --tag=CC gcc -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m32 -march=i386 -mtune=generic -fasynchronous-unwind-tables -I./ -Wall -W -Wshadow -Wsign-compare -O3 -funroll-loops -fomit-frame-pointer -c -o bn_mp_exch.o bn_mp_exch.c ... The -W flags are harmless, the usefulness of -funroll-loops is arguable, but -O3 and -fomit-frame-pointer should not be used in rpms. I would suggest you to remove all -f* flags from makefile.shared or to change your spec file in such a way these -f*-flags are not being used. I'll look into different ways to fix this. CONSIDER: - The package's name doesn't match the tarball name. Please read: http://fedoraproject.org/wiki/Packaging/NamingGuidelines Yes, the tarball is ltm, but the project name is libtommath, and the tarball creates a libtommath directory. The Guidelines say use the tarball name or the project name. Additionnally Gentoo, Debian, PLD distros along with OpenBSD and FreeBSD all package libtommath as 'libtommath'. There are other libtom* libraries, libtomcrypt among them which I also have up for review and will use the feedback here to help with that package. - The devel package installs its headers to /usr/include. A better design would be it to install them to a per package header subdirectory (e.g. /usr/include/tommath or similar). I'll look into fixing this and seeing how that affects packages that depend on libtommath too. I'll have a new spec and srpm up in a couple of days. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 231267] Review Request: stardict-dic - dictionaries for StarDict
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: stardict-dic - dictionaries for StarDict https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=231267 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 03:07 EST --- yes, these rpm packages will be generated: stardict-dic-en-2.4.2-2.fc7.noarch.rpm stardict-dic-ja-2.4.2-2.fc7.noarch.rpm stardict-dic-ru-2.4.2-2.fc7.noarch.rpm stardict-dic-zh_CN-2.4.2-2.fc7.noarch.rpm stardict-dic-zh_TW-2.4.2-2.fc7.noarch.rpm I think make them as one spec file and one cvs module is more convenient and simple to maintain. Yeah, I make one rpm package contain more than one dictionary, although to upgrade is slower, but the installation is simpler as only need to install one package. Anyway, the update should not be very frequent. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 231267] Review Request: stardict-dic - dictionaries for StarDict
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: stardict-dic - dictionaries for StarDict https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=231267 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 03:34 EST --- (In reply to comment #20) I think make them as one spec file and one cvs module is more convenient and simple to maintain. I strongly disagree; I took the issue to the packaging mailing list to get some more opinions; see: https://www.redhat.com/archives/fedora-packaging/2007-June/msg00134.html -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 242310] Review Request: moreutils - Additional unix utilities
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: moreutils - Additional unix utilities https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=242310 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 03:24 EST --- I appreciate your comments. moreutils has been in debian for some time and is now also in the ubuntu repo and as such the names already have some history behind them. On the one hand there is the issue of the likely to clash names but IMO it would be more important in this case to keep the names consistant with both upstream and with the equivalent package in the other distros. ImageMagick has not included the combine binary for a significant number of versions. There is a tcl package with a LaTeX component which uses /usr/bin/ts for SUSE (ref: http://www.novell.com/products/linuxpackages/suselinux/ts.html). I would appreciate some further opinions on this issue. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 231114] Review Request: keepassx - Cross-platform password 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: keepassx - Cross-platform password manager https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=231114 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 242310] Review Request: moreutils - Additional unix utilities
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: moreutils - Additional unix utilities https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=242310 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 04:16 EST --- (In reply to comment #7) I appreciate your comments. moreutils has been in debian for some time and is now also in the ubuntu repo and as such the names already have some history behind them. This only means their QA has failed to recognize the potential dangers and has slipped through a package they better should not ship. On the one hand there is the issue of the likely to clash names but IMO it would be more important in this case to keep the names consistant with both upstream and with the equivalent package in the other distros. I think it would be appropriate to notify upstream to have their choices revisited and not to include this package until upstream has fixed them. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 218258] Review Request: audacious-docklet - a docklet plugin for Audacious
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: audacious-docklet - a docklet plugin for Audacious https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=218258 [EMAIL PROTECTED] changed: What|Removed |Added Product|Fedora Extras |Fedora --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 04:41 EST --- Audacious upstream hereby requests that you do not include this plugin because: (a) we ship our own statusicon plugin in 1.3+ (b) this plugin is known to cause state corruption in GTK and Xlib (c) this plugin is known to use our (Audacious) APIs improperly -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245708] New: Review Request: scsi-target-utils - SCSI target daemon and tools
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/bugzilla/show_bug.cgi?id=245708 Summary: Review Request: scsi-target-utils - SCSI target daemon and tools Product: Fedora Version: devel 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://people.redhat.com/mchristi/target/FC7/rpm/scsi-target-utils-0.1-1.fc7/scsi-target-utils.spec SRPM URL: http://people.redhat.com/mchristi/target/FC7/rpm/scsi-target-utils-0.1-1.fc7/scsi-target-utils-0.1-1.fc7.src.rpm Description: The SCSI target package contains the daemon and tools to setup a SCSI targets. Currently, software iSCSI targets are supported. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 222998] Review Request: iDesk - Desktop icons and background for minimal WMs
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: iDesk - Desktop icons and background for minimal WMs https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222998 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 05:24 EST --- ... Sadly enough, I cannot seem to get in touch with the up-stream maintainer. I'll give it another run before closing this bug :( - Gilboa -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 197867] Review Request: iscsitarget (includes kernel module)
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: iscsitarget (includes kernel module) https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=197867 [EMAIL PROTECTED] changed: What|Removed |Added Severity|normal |medium Priority|normal |medium Product|Fedora Extras |Fedora --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 05:27 EST --- Hey, I made a package for stgt in this bugzilla https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245708 It is currently very basic. For example you have to run tgtadm by hand to create targets. I am working on a tool, that will read iet config files and translate it to tgtadm calls for a user. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245708] Review Request: scsi-target-utils - SCSI target daemon and tools
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: scsi-target-utils - SCSI target daemon and tools https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245708 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 05:29 EST --- Note that this is related to https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=197867 stgt mentioned in that bugzilla is what I am packaging here. This is the project that is replacing iet/iscsitarget, because the upstream kernel would not accept iet/iscsitarget. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 225622] Merge Review: boost
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: boost https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225622 [EMAIL PROTECTED] changed: What|Removed |Added Product|Fedora Extras |Fedora --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 05:35 EST --- There is a new release, 1.34.0 -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245708] Review Request: scsi-target-utils - SCSI target daemon and tools
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: scsi-target-utils - SCSI target daemon and tools https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245708 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 05:38 EST --- Oh yeah, one question on the review. I ran rpmlint and it did not report any errors. On the iscsi-initiator-utils review, it was requested that I should hook that into rc.sysinit. For scsi-target-utils, I was not sure if I should do the same. Is there any guidelines about what should or should not be hooked into rc.sysinit? Does this qualify? I did a init script, but let me know if you want me to hook this into rc.sysinit and I will send a patch. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 225915] Merge Review: iscsi-initiator-utils
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: iscsi-initiator-utils https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225915 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 05:48 EST --- Hey, one question about this package, For iscsi it was requested that we create a library that would read special iscsi boot values. For example IBM/intel want to read the value to use for boot from the sysfs acpi tree, or for IBM and ppc from the open firmware tree. iscsistart would use this library, and anaconda could then also use this library if you wanted to do something like when you hit the advanced storage button, anaconda could check for those special values being set and could then use them for install and boot. For the library, should I create another package/rpm and submit it for review or can I just include it in this one? I thought I read that it must go in a new devel package, but I cannot find where I read that. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 226407] Merge Review: sendmail
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: sendmail
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226407
[EMAIL PROTECTED] changed:
What|Removed |Added
Product|Fedora Extras |Fedora
--- Additional Comments From [EMAIL PROTECTED] 2007-06-26 05:58 EST ---
Created an attachment (id=157873)
-- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=157873action=view)
Patch fixing build issue in current rawhide
The current package in CVS does not build in rawhide. This is because m4 in
rawhide has recently been upgraded to version 1.4.9, and versions of m4 later
than 1.4.8 return a non-zero exit code if a file to be include-d is not found.
One of the patches applied in the Fedora package to the default submit.mc
triggers this problem during the early part of %install and causes the package
build to fail.
The attached patch treats submit.{mc,cf} similarly to sendmail.{mc,cf} and
fixes this problem.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245708] Review Request: scsi-target-utils - SCSI target daemon and tools
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: scsi-target-utils - SCSI target daemon and tools https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245708 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 06:32 EST --- (In reply to comment #2) Oh yeah, one question on the review. I ran rpmlint and it did not report any errors. One correction. I got this: W: scsi-target-utils strange-permission tgtd.init 0755 But the packaging guidelines said I should be using 0755, so I am not sure what I am messing up and if I am supposed to be using rc.sysinit instead of the init script. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245720] New: Review Request: prcsys - Parallel rc with LSB dependency resolver
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/bugzilla/show_bug.cgi?id=245720 Summary: Review Request: prcsys - Parallel rc with LSB dependency resolver Product: Fedora Version: devel 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://people.redhat.com/harald/downloads/initscripts/parallel/prcsys.spec SRPM URL: http://people.redhat.com/harald/downloads/initscripts/parallel/prcsys-0-0.1.20070622svn.src.rpm Description: Parallel rc with LSB dependency resolver Needed for http://fedoraproject.org/wiki/FCNewInit/RC -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 230762] Review Request: xawtv - TV applications for video4linux compliant devices
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: xawtv - TV applications for video4linux compliant devices https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=230762 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 08:12 EST --- Hmm... It seems you managed to convince me. :) -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 241099] Review Request: lcdtest - utility to display monitor test patterns
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: lcdtest - utility to display monitor test patterns https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=241099 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 08:56 EST --- ping again? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245760] New: Review Request: engine_pkcs11 - A PKCS11 engine for use with OpenSSL
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/bugzilla/show_bug.cgi?id=245760 Summary: Review Request: engine_pkcs11 - A PKCS11 engine for use with OpenSSL Product: Fedora Version: devel 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://free.linux.hp.com/~mra/rpms/engine_pkcs11/engine_pkcs11.spec SRPM URL: http://free.linux.hp.com/~mra/rpms/engine_pkcs11/engine_pkcs11-0.1.3-1.fc7.src.rpm Description: Engine_pkcs11 is an implementation of an engine for OpenSSL. It is built upon libp11. Engine_pkcs11 is meant to be used with smart cards and software for using smart cards in PKCS#11 format, such as OpenSC. Originaly this engine was a part of OpenSC, until OpenSC was split into several small projects for improved flexibility. engine_pkcs11 has a underscore in its upstream name so it meets with the Naming Guidelines. engine_pkcs11 claims to be licensed under the New BSD license. http://www.opensc-project.org/engine_pkcs11/wiki/AuthorsAndCredits although it is missing the endorsement clause. For this initial release I have BSD listed in the spec file, but if there is a better keyword to use instead I will change it. This rpm was built successfully on i386 and x86_64 and passes rpmlint with no errors or warnings. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 197488] Review Request: uread - Utilities for unformatted fortran 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: uread - Utilities for unformatted fortran files https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=197488 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 11:24 EST --- Sorry, you are right. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245597] Review Request: pygtksourceview - Python bindings for gtksourceview
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: pygtksourceview - Python bindings for gtksourceview https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245597 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245597] Review Request: pygtksourceview - Python bindings for gtksourceview
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: pygtksourceview - Python bindings for gtksourceview https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245597 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 11:39 EST --- You missspelted pgkconfig -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 241081] Review Request: R-widgetTools-1.12.0-2 - Tools to support the construction of tcltk widgets
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: R-widgetTools-1.12.0-2 - Tools to support the
construction of tcltk widgets
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=241081
--- Additional Comments From [EMAIL PROTECTED] 2007-06-26 12:11 EST ---
A few comments:
It is possible to put noarch R packages into %{_datadir}/R/library, instead of
in %{_libdir}/R/library (where arch-specific R packages should go). I had
forgotten that this was possible, which is shameful, since I'm the person who
enabled it. :P
I'd argue that the DESCRIPTION is a useful %doc item.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 241081] Review Request: R-widgetTools-1.12.0-2 - Tools to support the construction of tcltk widgets
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: R-widgetTools-1.12.0-2 - Tools to support the
construction of tcltk widgets
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=241081
--- Additional Comments From [EMAIL PROTECTED] 2007-06-26 12:22 EST ---
(In reply to comment #3, Xavier)
Also, i see no license text or file in this package.
Are you under the impression that this is mandatory? If there's a license file
in the upstream tarball then it has to be included in the package. If upstream
includes no license file then the package maintainer certainly doesn't have to
manufacture one.
(In reply to comment #4, Spot)
It is possible to put noarch R packages into %{_datadir}/R/library, instead of
in %{_libdir}/R/library (where arch-specific R packages should go).
That's great; do you just do the usual R CMD INSTALL and give the other
directory? Does this work for FC6 and F7?
I'd argue that the DESCRIPTION is a useful %doc item.
I certainly agree; rpm -q --info certainly does not give the same information as
is present in the DESCRIPTION file. I really don't understand why Xavier would
make such a comment unless he didn't actually look at what's in the file.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245708] Review Request: scsi-target-utils - SCSI target daemon and tools
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: scsi-target-utils - SCSI target daemon and tools https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245708 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 12:29 EST --- The initscript seems reasonable. Is tgtd configured only by tgtadm? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245597] Review Request: pygtksourceview - Python bindings for gtksourceview
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: pygtksourceview - Python bindings for gtksourceview https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245597 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 12:33 EST --- The -devel description says file where it should say files -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239435] Review Request: Gnome Network Monitor - A Network Monitor for the GNOME Desktop
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: Gnome Network Monitor - A Network Monitor for the GNOME Desktop https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239435 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 12:35 EST --- Yes, thanks for spotting that..the desktop file was broken wrt latest desktop-file-utils. I fixed the desktop file, new packages are at: http://gnetworkmonitor.sourceforge.net/rpm/gnome-network-monitor.spec http://gnetworkmonitor.sourceforge.net/rpm/gnome-network-monitor-0.9.1-4.fc8.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245597] Review Request: pygtksourceview - Python bindings for gtksourceview
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: pygtksourceview - Python bindings for gtksourceview https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245597 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 12:35 EST --- you ship .la files in the package. We don't do them here do we ? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 226294] Merge Review: php
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: php Alias: php https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226294 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 12:44 EST --- Well, see new sub-packages in devel. But what about the current F7 release? Depending of your plans, I should either still update my php-extras, or wait (how long?) for an upcoming update of the main php package... -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221769] Review Request: libcgi - CGI easy as C
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: libcgi - CGI easy as C https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221769 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 13:17 EST --- cvs done. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 244752] Review Request: mod_dnssd - An Apache HTTPD module which adds Zeroconf support
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: mod_dnssd - An Apache HTTPD module which adds Zeroconf support https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244752 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 13:09 EST --- cvs done. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245597] Review Request: pygtksourceview - Python bindings for gtksourceview
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: pygtksourceview - Python bindings for gtksourceview https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245597 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED Flag||fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 13:45 EST --- Other than those small issues, seems to look pretty good. Ship it! -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239435] Review Request: Gnome Network Monitor - A Network Monitor for the GNOME Desktop
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: Gnome Network Monitor - A Network Monitor for the
GNOME Desktop
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239435
--- Additional Comments From [EMAIL PROTECTED] 2007-06-26 13:48 EST ---
Good:
+ Package meat naming guideline
+ Name of SPEC file meat base package name
* Package contains %{?dist} tag
+ Consistent usage of RPM macros
+ Package meets packaging guideline
+ License of the package is GPL
+ License field in SPEC file matches
+ SPEC is written in englisch
+ SPEC look legible
+ Tar ball in source RPM metaches with upstream
md5sum: 5e9eb652e4e24e2057f6eb1b4794a683
+ Package has correct buildroot
+ BuildRequires are not redundant
+ Package contains no sub packages
+ Local build works fine
+ SPEC file contains proper %defattr and %attr permission
+ Package has a correct %clean section
+ $RPM_BUILD_ROOT will be cleaned on start of %install section
+ Packages %doc section doesn't affect runtime.
+ %files section contains no duplicate files
+ Package contains no file or directories own by other packages.
+ Changelog section is correct.
+ Rpmlint is quite of source and binary rpm.
+ Mock build wors fine on Devel and F-7 (x86_64)
+ Package should contains the most recent version.
+ Startup of the package works without crash.
Bad:
- License file should be includes from tar ball.
- BR on gettext-devel may be better, because gettext-devel comtains
development tools which are not in the gettext package.
- Wrong Requires:
You use:
Requires(post): /usr/bin/update-desktop-database
Requires(postun): /usr/bin/update-desktop-database
because you use desktop-file-install in your scriplets, you shoud
have the following in your Requires:
Requires(post): /usr/bin/desktop-file-install
Requires(postun): /usr/bin/desktop-file-install
- Package does not sure the ownership of the directory
%{python_sitelib}/%{name}/
The best way to make sure, that the directory and all files belong to the
package is to write
%{python_sitelib}/%{name}/
into the SPEC file, so you can remove the line
%{python_sitelib}/%{name}/*.py*
from the SPEC file.
- Desktop entry contains no icon.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 221769] Review Request: libcgi - CGI easy as C
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: libcgi - CGI easy as C https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221769 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 13:50 EST --- Hi Jose, are you willing to maintain this for EPEL as well? I have libcgi running on a lot of servers at work, and could use a decent rpm ;-) -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 197488] Review Request: uread - Utilities for unformatted fortran 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: uread - Utilities for unformatted fortran files https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=197488 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 13:55 EST --- I changed the version and release accordingly: - use 0 as version and put the timestamp in the release http://www.environnement.ens.fr/perso/dumas/fc-srpms/uread.spec http://www.environnement.ens.fr/perso/dumas/fc-srpms/uread-0-0.19981218.fc8.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245597] Review Request: pygtksourceview - Python bindings for gtksourceview
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: pygtksourceview - Python bindings for gtksourceview https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245597 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 14:05 EST --- New Package CVS Request === Package Name: pygtksourceview Short Description: Python bindings for gtksourceview Owners: [EMAIL PROTECTED], [EMAIL PROTECTED] Branches: InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 209615] Review Request: wmcpuload - WindowMaker dockapp
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: wmcpuload - WindowMaker dockapp https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209615 [EMAIL PROTECTED] changed: What|Removed |Added Product|Fedora Extras |Fedora [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|163778 | nThis|| --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 14:16 EST --- I packaged libdockapp some time ago for wmacpi, you can use it. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 197488] Review Request: uread - Utilities for unformatted fortran 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: uread - Utilities for unformatted fortran files
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=197488
[EMAIL PROTECTED] changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Additional Comments From [EMAIL PROTECTED] 2007-06-26 14:35 EST ---
Good:
+ Package meets the naming guideline
+ Package versioning schema idicates, that a unversion source was used
+ Package source was renamed to contains a timestamp, becaused it was
unversioned
+ License of the package is GPL
+ Package contains a mail from upstream with explicit GPL permission
+ License field metches with mail.
+ Spec is written in english.
+ RPM macros are used consistently
+ Tar ball matches with upsream
(md5sum: 656a18c6e731f5ff405d69279c897b1f)
+ Package has correct buildroot
+ Package has no BuildRequires
+ Package has no Requires
+ Package has no subpackages
+ Package use optflags
+ Package use %{_smp_mflags}
+ Local build works
+ Package contains proper %defattr
+ Package has a %clean section and buildroot will clean at the start of %install
+ Package filelist has no duplicates
+ Package contains no files own by an other package
+ Application does not crash on startup
+ Changelog section look ok.
+ Rpmlint is quite on source and binary rpm.
+ Debuginfo package contains source files
* Mock build works fine for Devel and F-7 (x86_64)
Bad:
- Package doesn't contains a file with the verbatim license text
*** APPROVED ***
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 222475] Review Request: sofia-sip - Sofia SIP User-Agent 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: sofia-sip - Sofia SIP User-Agent library https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222475 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 14:36 EST --- Spec URL: http://repo.ocjtech.us/misc/fedora/7/SRPMS/sofia-sip-1.12.6-2.fc7.spec SRPM URL: http://repo.ocjtech.us/misc/fedora/7/SRPMS/sofia-sip-1.12.6-2.fc7.src.rpm %changelog * Tue Jun 26 2007 Jeffrey C. Ollie [EMAIL PROTECTED] - 1.12.6-2 - Re-run libtoolize and auto* to fix rpath issues. - Add --disable-rpath to the configure line. - The devel packages need to BR pkgconfig. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 226489] Merge Review: tftp
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: tftp https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226489 [EMAIL PROTECTED] changed: What|Removed |Added Priority|normal |medium Product|Fedora Extras |Fedora [EMAIL PROTECTED] changed: What|Removed |Added Status|CLOSED |ASSIGNED Keywords||Reopened Resolution|RAWHIDE | CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 14:48 EST --- Is there a reason this bug is closed? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245597] Review Request: pygtksourceview - Python bindings for gtksourceview
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: pygtksourceview - Python bindings for gtksourceview
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245597
[EMAIL PROTECTED] changed:
What|Removed |Added
Flag|fedora-cvs? |fedora-cvs+
--- Additional Comments From [EMAIL PROTECTED] 2007-06-26 14:56 EST ---
cvs done.
Aside from the .la issue, Any reason you aren't using %{?_smp_mflags} with your
make?
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 220743] Review Request: postgis - Geographic Information Systems Extensions to PostgreSQL
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: postgis - Geographic Information Systems Extensions to PostgreSQL https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220743 [EMAIL PROTECTED] changed: What|Removed |Added Severity|normal |medium Priority|normal |medium Product|Fedora Extras |Fedora [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 220743] Review Request: postgis - Geographic Information Systems Extensions to PostgreSQL
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: postgis - Geographic Information Systems Extensions to PostgreSQL https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220743 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 15:20 EST --- cvs done. Sorry this got missed, but you didn't set the fedora-cvs flag, so cvsadmins didn't see it until now. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245597] Review Request: pygtksourceview - Python bindings for gtksourceview
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: pygtksourceview - Python bindings for gtksourceview https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245597 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 220743] Review Request: postgis - Geographic Information Systems Extensions to PostgreSQL
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: postgis - Geographic Information Systems Extensions to PostgreSQL https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220743 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 15:26 EST --- AFAIR this package was approved before we added these flags to bugzilla... Also there is CVS directory for Postgis already :) But thanks anyway. Regards, Devrim -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 220743] Review Request: postgis - Geographic Information Systems Extensions to PostgreSQL
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: postgis - Geographic Information Systems Extensions to PostgreSQL https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220743 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 15:30 EST --- Yeah, could be... however, it didn't have a owners.list entry, so no bugzilla component and all sorts of other issues. Also, it doesn't appear to have a F-7 branch. Feel free to request one if you need it. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 241081] Review Request: R-widgetTools-1.12.0-2 - Tools to support the construction of tcltk widgets
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: R-widgetTools-1.12.0-2 - Tools to support the
construction of tcltk widgets
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=241081
--- Additional Comments From [EMAIL PROTECTED] 2007-06-26 15:43 EST ---
(In reply to comment #5)
(In reply to comment #3, Xavier)
Also, i see no license text or file in this package.
Are you under the impression that this is mandatory? If there's a license
file
in the upstream tarball then it has to be included in the package. If
upstream
includes no license file then the package maintainer certainly doesn't have to
manufacture one.
surely not, i just pasted a note.
However, there're some packaging draft that talk about the why a license text is
usefull.
(In reply to comment #4, Spot)
It is possible to put noarch R packages into %{_datadir}/R/library, instead
of
in %{_libdir}/R/library (where arch-specific R packages should go).
That's great; do you just do the usual R CMD INSTALL and give the other
directory? Does this work for FC6 and F7?
I'd argue that the DESCRIPTION is a useful %doc item.
I certainly agree; rpm -q --info certainly does not give the same information
as
is present in the DESCRIPTION file. I really don't understand why Xavier
would
make such a comment unless he didn't actually look at what's in the file.
i looked the file, and it's why I commented about that, if you look close, sure
there's some things that dismatch between rpm -q [package-name] --info and
DESCRIPTION are the maintainers, the date of the packaged and the packager,
etc.
AFAIK the maintainer of the package (rpm) which should be mentione is the
fedoraproject (just my guess).
Note that some end-users could be confused who is who ? or what is what ?
here is a exmeple output of rpm -q [package_name] --info command:
---
$ rpm -q glade2 --info
Name: glade2 Relocations: (not relocatable)
Version : 2.12.1Vendor: Red Hat, Inc.
Release : 5.fc6 Build Date: Fri 08 Sep 2006 01:03:55
PM AST
Install Date: Thu 05 Jul 2007 12:56:02 PM AST Build Host:
hs20-bc1-7.build.redhat.com
Group : Development/Tools Source RPM:
glade2-2.12.1-5.fc6.src.rpm
Size: 5171445 License: GPL
Signature : DSA/SHA1, Tue 03 Oct 2006 09:49:46 PM AST, Key ID b44269d04f2a6fd2
Packager: Red Hat, Inc. http://bugzilla.redhat.com/bugzilla
URL : http://glade.gnome.org/
Summary : A GTK+ GUI builder.
Description :
Glade is a free user interface builder for GTK+ and the GNOME GUI
desktop. Glade can produce C source code. Support for C++, Ada95,
Python, and Perl is also available, via external tools which process
the XML interface description files output by GLADE.
--
Also, i'm wondering if a lil' packaging guideline for R package could be usefull
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 236539] Review Request: perl-Math-Vec - Perl Math::Vec module
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-Math-Vec - Perl Math::Vec module Alias: perl-Math-Vec https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=236539 [EMAIL PROTECTED] changed: What|Removed |Added Product|Fedora Extras |Fedora [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |NEEDINFO Flag||needinfo?([EMAIL PROTECTED] ||nfo) --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 16:30 EST --- ping -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 243573] Review Request: python-memcached - A Python memcached client 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: python-memcached - A Python memcached client library. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=243573 [EMAIL PROTECTED] changed: What|Removed |Added Product|Fedora Extras |Fedora --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 14:44 EST --- Sean, you closed the ticket but you haven't imported python-memcached in cvs yet. Do you need any help? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245826] New: Review Request: pida - A Python IDE written in Python and GTK
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/bugzilla/show_bug.cgi?id=245826 Summary: Review Request: pida - A Python IDE written in Python and GTK Product: Fedora Version: devel 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: https://fedoraproject.org/wiki/DebarshiRay?action=AttachFiledo=gettarget=pida.spec SRPM URL: https://fedoraproject.org/wiki/DebarshiRay?action=AttachFiledo=gettarget=pida-0.4.4-1.fc8.src.rpm Description: PIDA is an IDE. PIDA is different from other IDEs in that it will use the tools you already have available rather than attempting to reinvent each one. PIDA is written in Python with the PyGTK toolkit, and although is designed to be used to program in any language, PIDA has fancy Python IDE features. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245151] Review Request: crystal-project - KDE Icon set
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: crystal-project - KDE Icon set
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245151
--- Additional Comments From [EMAIL PROTECTED] 2007-06-26 17:30 EST ---
Review :
MUST : name : OK
MUST : macros (%{name}, %{version}) : OK
MUST : licence : (LGPL) OK (chitlesh told me he asked the upstream to include
the text in the source)
MUST : source match the upstream source (MD5 sum match)
MUST : dependencies : OK
MUST : owns directory : OK
MUST : %defattr : OK
MUST : rm -rf %{buildroot} in %install and %clean section: OK
MUST : RpmLint is NOT quiet :
W: crystal-project dangling-relative-symlink
/usr/share/icons/crystal_project/16x16/apps/kmenu.png
../../../Bluecurve/16x16/apps/gnome-main-menu.png
W: crystal-project dangling-relative-symlink
/usr/share/icons/crystal_project/22x22/apps/kmenu.png
../../../Bluecurve/24x24/apps/gnome-main-menu.png
W: crystal-project dangling-relative-symlink
/usr/share/icons/crystal_project/32x32/apps/kmenu.png
../../../Bluecurve/32x32/apps/gnome-main-menu.png
W: crystal-project dangling-relative-symlink
/usr/share/icons/crystal_project/48x48/apps/kmenu.png
../../../Bluecurve/48x48/apps/gnome-main-menu.png
=That's OK since fedora-logos is a dependency.
This is OK for me : approved
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245151] Review Request: crystal-project - KDE Icon set
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: crystal-project - KDE Icon set https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245151 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review? |fedora-review+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245151] Review Request: crystal-project - KDE Icon set
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: crystal-project - KDE Icon set https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245151 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 17:46 EST --- Just one more thing : - you should report to the upstream that file permissions (as all files are icons) should be set to 644. - and if possible to avoid space in names (perhaps this is due to KDE, but i don't think so) -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 241081] Review Request: R-widgetTools-1.12.0-2 - Tools to support the construction of tcltk widgets
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: R-widgetTools-1.12.0-2 - Tools to support the construction of tcltk widgets https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=241081 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 18:19 EST --- Based on the comments made by spot and tibbs there is the new spec file and the new srpm. I took me some time but I checked the set up and removal and both are working the folder are in /usr/share/R/library I have also checked with a second package (R-DynDoc) to see whatever the update of the help html interface is working (and it does :-)) The rpmlint is clean now and mock too. There are the lastest versions: SPEC : http://pingoured.dyndns.org/public/RPM/R-widgetTools/R-widgetTools.spec SRPM : http://pingoured.dyndns.org/public/RPM/R-widgetTools/R-widgetTools-1.12.0-4.fc6.src.rpm I do agree with Xavier on the fact that a R packaging guideline can be useful, especially because R package are not difficult to do but if we want them in constant quality we will need it. I have already made a spectemplate-R that I can changed to take into account these remarks and send to anyone who wants it. Regards P. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 239162] Review Request: perl-Net-Packet - A framework to easily send and receive frames from layer 2 to la yer 7
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-Net-Packet - A framework to easily send and receive frames from layer 2 to la yer 7 Alias: perl-Net-Packet https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239162 [EMAIL PROTECTED] changed: What|Removed |Added Product|Fedora Extras |Fedora --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 18:24 EST --- Any reason this hasn't been imported yet? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245151] Review Request: crystal-project - KDE Icon set
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: crystal-project - KDE Icon set https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245151 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 18:29 EST --- New Package CVS Request === Package Name: crystal-project Short Description: KDE Icon set Owners: [EMAIL PROTECTED] Branches: FC-6 F-7 Devel -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245151] Review Request: crystal-project - KDE Icon set
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: crystal-project - KDE Icon set https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245151 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245151] Review Request: crystal-project - KDE Icon set
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: crystal-project - KDE Icon set https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245151 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 18:32 EST --- (In reply to comment #5) Just one more thing : - you should report to the upstream that file permissions (as all files are icons) should be set to 644. - and if possible to avoid space in names (perhaps this is due to KDE, but i don't think so) Upstream works for Linspire. He is already aware of KDE related issues. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 193712] Review Request: sos
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: sos https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193712 [EMAIL PROTECTED] changed: What|Removed |Added Product|Fedora Extras |Fedora --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 18:34 EST --- Anything happening here? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 240323] Review Request: telepathy-salut - Link-local XMPP telepathy connection 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: telepathy-salut - Link-local XMPP telepathy connection manager https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=240323 [EMAIL PROTECTED] changed: What|Removed |Added Product|Fedora Extras |Fedora [EMAIL PROTECTED] changed: What|Removed |Added Summary|Review Request: telepathy- |Review Request: telepathy- |salut - Link-local XMPP |salut - Link-local XMPP |telepathy connection manager|telepathy connection manager Priority|medium |high --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 18:42 EST --- telepathy-glib has been added to fedora. Upping the priority of this. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 240323] Review Request: telepathy-salut - Link-local XMPP telepathy connection 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: telepathy-salut - Link-local XMPP telepathy connection manager https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=240323 [EMAIL PROTECTED] changed: What|Removed |Added Attachment #154828|0 |1 is obsolete|| --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 19:13 EST --- Created an attachment (id=157965) -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=157965action=view) Build log for latest spec file. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 241081] Review Request: R-widgetTools-1.12.0-2 - Tools to support the construction of tcltk widgets
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: R-widgetTools-1.12.0-2 - Tools to support the
construction of tcltk widgets
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=241081
--- Additional Comments From [EMAIL PROTECTED] 2007-06-26 19:40 EST ---
just a comment,
The DESCRIPTION file is installed twice.
in %{_docdir}/%{name}-%{version}/ and %{_datadir}/R/library/%{name}/
Sounds like a lining thing.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 243573] Review Request: python-memcached - A Python memcached client 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: python-memcached - A Python memcached client library. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=243573 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 22:55 EST --- Oops, thanks for catching that. I must have gotten distracted and didn't get it done. I believe it's completed now, please let me know if you see any other problems. Thanks, Sean -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 244623] Review Request: OggConvert - OggConvert is a small GNOME utility that uses GStreamer to convert media files to the licence-free Theora and Vorbis 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: OggConvert - OggConvert is a small GNOME utility that uses GStreamer to convert media files to the licence-free Theora and Vorbis formats. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244623 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 23:10 EST --- I should have some time to do the review tomorrow, and would be willing to be your sponsor. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245863] New: Review Request: gdesklets-quote-of-the-day - A quote of the day display for gdesklets
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/bugzilla/show_bug.cgi?id=245863 Summary: Review Request: gdesklets-quote-of-the-day - A quote of the day display for gdesklets Product: Fedora Version: devel 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://scurn.net/fedora/gdesklets-quote-of-the-day.spec SRPM URL: http://scurn.net/fedora/gdesklets-quote-of-the-day-0.61-1.fc7.src.rpm Description: A quote of the day display for gdesklets Note I am still seeking a sponsor. I have also submitted the follwing packages for review: proxychains - https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244018 gdesklets-goodweather - https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245527 gdesklets-calendar - https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245571 I have submitted a review for the following packages as well: https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244086 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244171 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244411 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244623 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244704 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244936 I can also be reached on Freenode under the nick owentl if there are any questions/comments -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245863] Review Request: gdesklets-quote-of-the-day - A quote of the day display for gdesklets
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: gdesklets-quote-of-the-day - A quote of the day display for gdesklets https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245863 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||177841 nThis|| -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245491] Review Request: ndesk-dbus - Managed DBus implementation
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: ndesk-dbus - Managed DBus implementation https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245491 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 23:43 EST --- Review in progress; will be posted shortly. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 231114] Review Request: keepassx - Cross-platform password 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: keepassx - Cross-platform password manager https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=231114 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 23:51 EST --- keepassx-0.2.2-1.fc7 has been pushed to the Fedora 7 testing repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 243076] Review Request: reciteword - Recite Word Easily
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: reciteword - Recite Word Easily https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=243076 --- Additional Comments From [EMAIL PROTECTED] 2007-06-26 23:54 EST --- reciteword-0.8.3-3.fc7 has been pushed to the Fedora 7 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 243076] Review Request: reciteword - Recite Word Easily
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: reciteword - Recite Word Easily https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=243076 [EMAIL PROTECTED] changed: What|Removed |Added Status|CLOSED |CLOSED Resolution|NEXTRELEASE |ERRATA Fixed In Version||0.8.3-3.fc7 -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 240557] Review Request: ocaml-findlib - Objective CAML package manager and build helper
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-findlib - Objective CAML package manager and
build helper
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=240557
--- Additional Comments From [EMAIL PROTECTED] 2007-06-26 23:56 EST ---
http://annexia.org/tmp/ocaml/ocaml-findlib-1.1.2pl1-6.src.rpm
http://annexia.org/tmp/ocaml/ocaml-findlib.spec
Mock failed to build on F-7_x86_64:
This is the first time I've tried mock. It turned out there was
a missing BuildRequires (ncurses-devel), which I fixed. However I
wasn't able to get mock to install ocaml 3.10.0 (which is required)
because that RPM isn't in Fedora yet. What I'd like it to do is
to create the chroot environment, get all the packages, then install
my ocaml 3.10.0 package before starting the build.
The updated ocaml 3.10.0 package is here: http://math.ifi.unizh.ch/fedora/tmp/
or see bug 239004.
Anyhow, if you can work out how to do that, then the updated
SRPM should work.
Your release tag doesn't quite good.
As your package contains pre-release it should be more like this:
1.1.2-0.1.pl1 instead of 1.1.2pl1-1
I don't understand what this means. The upstream package is
called 1.1.2pl1, so I just added a release number.
typo: from %build stage
-bindir %{_bindir} instead of -bindir %{_mandir}
Fixed. This didn't seem to have any effect, but it's obviously
wrong.
About -config %{_sysconfdir}/ocamlfind.conf, its provided by this package
The configure argument apparently tells the package where to
create/install that file, not where to find it.
I also made the changes to %doc.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 240557] Review Request: ocaml-findlib - Objective CAML package manager and build helper
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-findlib - Objective CAML package manager and build helper https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=240557 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2007-06-27 00:20 EST --- Anyhow, if you can work out how to do that, then the updated SRPM should work. well, currently i can do this by uploaded ocaml-3.10.0 to my personnal repository and run mock with my .cfg modified to match with my repository. So, i will catch it. I don't understand what this means. The upstream package is called 1.1.2pl1, so I just added a release number. see this naming guidelines for more details: http://fedoraproject.org/wiki/Packaging/NamingGuidelines?action=showredirect=PackageNamingGuidelines#head-d97a3f40b6dd9d2288206ac9bd8f1bf9b791b22a -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245878] New: Review Request: stardict-dic-en - English(en) dictionaries for StarDict
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/bugzilla/show_bug.cgi?id=245878 Summary: Review Request: stardict-dic-en - English(en) dictionaries for StarDict Product: Fedora Version: devel 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://reciteword.cosoft.org.cn/redhat/stardict-dic-en.spec SRPM URL: http://reciteword.cosoft.org.cn/redhat/stardict-dic-en-2.4.2-3.fc7.src.rpm Description: English(en) dictionaries for StarDict -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245879] New: Review Request: stardict-dic-ja - Japanese(ja) dictionaries for StarDict
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/bugzilla/show_bug.cgi?id=245879 Summary: Review Request: stardict-dic-ja - Japanese(ja) dictionaries for StarDict Product: Fedora Version: devel 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://reciteword.cosoft.org.cn/redhat/stardict-dic-ja.spec SRPM URL: http://reciteword.cosoft.org.cn/redhat/stardict-dic-ja-2.4.2-3.fc7.src.rpm Description: Japanese(ja) dictionaries for StarDict -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245880] New: Review Request: stardict-dic-ru - Russian(ru) dictionaries for StarDict
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/bugzilla/show_bug.cgi?id=245880 Summary: Review Request: stardict-dic-ru - Russian(ru) dictionaries for StarDict Product: Fedora Version: devel 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://reciteword.cosoft.org.cn/redhat/stardict-dic-ru.spec SRPM URL: http://reciteword.cosoft.org.cn/redhat/stardict-dic-ru-2.4.2-3.fc7.src.rpm Description: Russian(ru) dictionaries for StarDict -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245882] New: Review Request: stardict-dic-zh_CN - Simplified Chinese(zh_CN) dictionaries for StarDict
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/bugzilla/show_bug.cgi?id=245882 Summary: Review Request: stardict-dic-zh_CN - Simplified Chinese(zh_CN) dictionaries for StarDict Product: Fedora Version: devel 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://reciteword.cosoft.org.cn/redhat/stardict-dic-zh_CN.spec SRPM URL: http://reciteword.cosoft.org.cn/redhat/stardict-dic-zh_CN-2.4.2-3.fc7.src.rpm Description: Simplified Chinese(zh_CN) dictionaries for StarDict -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245883] New: Review Request: stardict-dic-zh_TW - Traditional Chinese(zh_TW) dictionaries for StarDict
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/bugzilla/show_bug.cgi?id=245883 Summary: Review Request: stardict-dic-zh_TW - Traditional Chinese(zh_TW) dictionaries for StarDict Product: Fedora Version: devel 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://reciteword.cosoft.org.cn/redhat/stardict-dic-zh_TW.spec SRPM URL: http://reciteword.cosoft.org.cn/redhat/stardict-dic-zh_TW-2.4.2-3.fc7.src.rpm Description: Traditional Chinese(zh_TW) dictionaries for StarDict -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245867] Review Request: stardict-dic-{zh_CN, zh_TW, en, ja, ru} - Dictionaries for StarDict
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: stardict-dic-{zh_CN, zh_TW, en, ja, ru} - Dictionaries
for StarDict
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245867
[EMAIL PROTECTED] changed:
What|Removed |Added
Status|ASSIGNED|CLOSED
Resolution||NOTABUG
--- Additional Comments From [EMAIL PROTECTED] 2007-06-27 01:47 EST ---
%{_datadir}/stardict/dic/*
Isn't this right?
Request separated:
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245878
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245879
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245880
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245882
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245883
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245879] Review Request: stardict-dic-ja - Japanese(ja) dictionaries for StarDict
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: stardict-dic-ja - Japanese(ja) dictionaries for
StarDict
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245879
--- Additional Comments From [EMAIL PROTECTED] 2007-06-27 01:50 EST ---
MUSTFIX:
Use %{_datadir} instead of /usr/share on these occations:
mkdir -p ${RPM_BUILD_ROOT}/usr/share/stardict/dic
cp -rf stardict-* ${RPM_BUILD_ROOT}/usr/share/stardict/dic/
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245878] Review Request: stardict-dic-en - English(en) dictionaries for StarDict
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: stardict-dic-en - English(en) dictionaries for StarDict https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245878 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245879] Review Request: stardict-dic-ja - Japanese(ja) dictionaries for StarDict
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: stardict-dic-ja - Japanese(ja) dictionaries for StarDict https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245879 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245883] Review Request: stardict-dic-zh_TW - Traditional Chinese(zh_TW) dictionaries for StarDict
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: stardict-dic-zh_TW - Traditional Chinese(zh_TW) dictionaries for StarDict https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245883 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245879] Review Request: stardict-dic-ja - Japanese(ja) dictionaries for StarDict
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: stardict-dic-ja - Japanese(ja) dictionaries for StarDict https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245879 --- Additional Comments From [EMAIL PROTECTED] 2007-06-27 01:58 EST --- OK. Fixed as the same file. Thank you! :) -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245880] Review Request: stardict-dic-ru - Russian(ru) dictionaries for StarDict
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: stardict-dic-ru - Russian(ru) dictionaries for StarDict https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245880 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 245882] Review Request: stardict-dic-zh_CN - Simplified Chinese(zh_CN) dictionaries for StarDict
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: stardict-dic-zh_CN - Simplified Chinese(zh_CN) dictionaries for StarDict https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=245882 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/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
