[Bug 220759] Review Request: pdfcube - PDF viewer with a spinning cube
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: pdfcube - PDF viewer with a spinning cube https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220759 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 03:00 EST --- 1. It is stated on the website (and in the packaged documentation) that pdfcube is alpha software, so yes it is probably a bit buggy. However since it is specifically made only for showing presentations it is a design feature that it only shows the top half of a page. PDF files should be made using LaTeX, Beamer, etc. 2. pdfcube is (for the time being at least) a commandline application, so therefore I didn't think to add a .desktop file. I haven't been able to find a Fedora Extras policy for whether or not the create .desktop files for commandline applications. I guess it could be done using a .desktop file that only sets the mime type for pdf, and is otherwise hidden. So in short: pdfcube might be a bit buggy, but it is a specialized pdf-viewer for presentations. And it is a commandline application. Maybe I should change the short description to: PDF presentation viewer with a spinning cube? -- 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 220860] Review Request: galternatives - Alternatives Configurator
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: galternatives - Alternatives Configurator https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220860 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 03:03 EST --- Well, also I * launch galternatives * select one item on alternatimes column (in the left) * select one options * push the button Properties (then a new window titled Details pops up) * Break the windows which poped up newly * Then push again the button Properties --- Then the new window poped up is totally gray.. -- 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 220860] Review Request: galternatives - Alternatives Configurator
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: galternatives - Alternatives Configurator https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220860 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 03:14 EST --- (In reply to comment #3) I get the following backtrace. - [EMAIL PROTECTED] ~]# LANG=C alternatives --display print print - status is auto. link currently points to /usr/bin/lpr.cups /usr/bin/lpr.cups - priority 40 slave print-cancel: /usr/bin/cancel.cups slave print-lp: /usr/bin/lp.cups slave print-lpq: /usr/bin/lpq.cups slave print-lprm: /usr/bin/lprm.cups slave print-lpstat: /usr/bin/lpstat.cups slave print-lpc: /usr/sbin/lpc.cups slave print-cancelman: /usr/share/man/man1/cancel-cups.1.gz slave print-lpman: /usr/share/man/man1/lp-cups.1.gz slave print-lpqman: /usr/share/man/man1/lpq-cups.1.gz slave print-lprman: /usr/share/man/man1/lpr-cups.1.gz slave print-lprmman: /usr/share/man/man1/lprm-cups.1.gz slave print-lpstatman: /usr/share/man/man1/lpstat-cups.1.gz slave print-lpcman: /usr/share/man/man8/lpc-cups.8.gz Current `best' version is /usr/bin/lpr.cups. [EMAIL PROTECTED] ~]# galternatives == (then select the item of print) == Traceback (most recent call last): File /usr/lib/python2.5/site-packages/galternatives/main.py, line 364, in alternative_selected_cb self.update_options_tree () File /usr/lib/python2.5/site-packages/galternatives/main.py, line 400, in update_options_tree self.PRIORITY, int(option['priority']), ValueError: invalid literal for int() with base 10: '40 cups' Would you know why? Ah... I found because /var/lib/alternatives/print says: -- auto /usr/bin/lpr print-cancel snip /usr/bin/lpr.cups 40 cups - THIS LINE /usr/bin/cancel.cups snip -- And -- [EMAIL PROTECTED] ~]$ rpm -q --scripts cups postinstall scriptlet (using /bin/sh): snip /usr/sbin/alternatives --install /usr/bin/lpr print /usr/bin/lpr.cups 40 \ --slave /usr/bin/lp print-lp /usr/bin/lp.cups \ snip --initscript cups -- and --initscripts option is a Red Hat Linux specific option according to man alternatives Then a patch is needed * to treat initscripts options correctly * or to ignore initscripts options for now. Then a documentation (like README.fedora) which explains that this package ignores initscripts option currently. -- 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 220860] Review Request: galternatives - Alternatives Configurator
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: galternatives - Alternatives Configurator https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220860 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 03:32 EST --- (In reply to comment #4) Well, also I * launch galternatives * select one item on alternatimes column (in the left) * select one options * push the button Properties (then a new window titled Details pops up) * Break the windows which poped up newly * Then push again the button Properties --- Then the new window poped up is totally gray.. This is a longstanding upstream bug, its not resolved yet. I was just about point out the --initscript option issue, but you already figured it out, I see what I can do about it tomorrow (well, later today ;). -- 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 220759] Review Request: pdfcube - PDF viewer with a spinning cube
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: pdfcube - PDF viewer with a spinning cube https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220759 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 03:39 EST --- Just an additional comment: There does some to be a bug in compiz (or maybe en pdfcube - but it works fine with metacity) that gives some weird shadow effects. But that shouldn't prevent packaging. -- 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 209214] Review Request: libprelude - Prelude library collection
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: libprelude - Prelude library collection https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209214 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 04:21 EST --- * new docs (NEWS, COPYING) added to the package, removed INSTALL * corrected permissions on /usr/lib/perl5/vendor_perl/5.8.8/i386-linux-thread-multi/Prelude.pm to 0644 * gnutls-devel is added to BuildRequires, MockBuild works now again * when using parallel make, package building fails. thats the reason why I avoid using parallel make with this package. * no other issues found new package is available here: http://people.redhat.com/tscherf/fedora-extra/libprelude-0.9.12-4.src.rpm http://people.redhat.com/tscherf/fedora-extra/libprelude.spec -- 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 207839] Review Request: lush - An object-oriented Lisp interpreter and compiler
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: lush - An object-oriented Lisp interpreter and compiler https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=207839 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 05:57 EST --- Build for FC-6 and FC-7. Added entries in owners.list and comps files. Thanks for the review! -- 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 220943] New: Review Request: po4a - A tool maintaining translations anywhere
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=220943 Summary: Review Request: po4a - A tool maintaining translations anywhere Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: fedora-package-review@redhat.com Spec URL: http://dl.atrpms.net/all/po4a.spec SRPM URL: http://dl.atrpms.net/all/po4a-0.29-3.at.src.rpm Description: The po4a (po for anything) project goal is to ease translations (and more interestingly, the maintenance of translations) using gettext tools on areas where they were not expected like documentation. -- 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 220944] New: Review Request: perl-Text-WrapI18N - Line wrapping with support for several locale setups
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=220944 Summary: Review Request: perl-Text-WrapI18N - Line wrapping with support for several locale setups Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: fedora-package-review@redhat.com Spec URL: http://dl.atrpms.net/all/perl-Text-WrapI18N.spec SRPM URL: http://dl.atrpms.net/all/perl-Text-WrapI18N-0.06-1.at.src.rpm Description: This is a module which intends to substitute Text::Wrap, which supports internationalized texts including: - multibyte encodings such as UTF-8, EUC-JP, EUC-KR, GB2312, and Big5, - fullwidth characters like east Asian characters which appear in UTF-8, EUC-JP, EUC-KR, GB2312, Big5, and so on, - combining characters like diacritical marks which appear in UTF-8, ISO-8859-11 (aka TIS-620), and so on, and - languages which don't use whitespaces between words, like Chinese and Japanese. -- 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 220945] New: Review Request: perl-Text-CharWidth - Get number of occupied columns of a string on terminal
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=220945 Summary: Review Request: perl-Text-CharWidth - Get number of occupied columns of a string on terminal Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: fedora-package-review@redhat.com Spec URL: http://dl.atrpms.net/all/perl-Text-CharWidth.spec SRPM URL: http://dl.atrpms.net/all/perl-Text-CharWidth-0.04-1.at.src.rpm Description: This is a module to provide equivalent feature as wcwidth(3) and wcswidth(3). This also provides mblen(3) equivalent subroutine. -- 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 220943] Review Request: po4a - A tool maintaining translations anywhere
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: po4a - A tool maintaining translations anywhere https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220943 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||220888 nThis|| -- 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 220888] Review Request: fakeroot - Gives a fake root environment
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: fakeroot - Gives a fake root environment https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220888 [EMAIL PROTECTED] changed: What|Removed |Added BugsThisDependsOn||220943 -- 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 220944] Review Request: perl-Text-WrapI18N - Line wrapping with support for several locale setups
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-Text-WrapI18N - Line wrapping with support for several locale setups https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220944 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||220943 nThis|| -- 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 220943] Review Request: po4a - A tool maintaining translations anywhere
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: po4a - A tool maintaining translations anywhere https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220943 [EMAIL PROTECTED] changed: What|Removed |Added BugsThisDependsOn||220944 -- 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 220945] Review Request: perl-Text-CharWidth - Get number of occupied columns of a string on terminal
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-Text-CharWidth - Get number of occupied columns of a string on terminal https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220945 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||220944 nThis|| -- 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 220944] Review Request: perl-Text-WrapI18N - Line wrapping with support for several locale setups
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-Text-WrapI18N - Line wrapping with support for several locale setups https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220944 [EMAIL PROTECTED] changed: What|Removed |Added BugsThisDependsOn||220945 -- 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 220789] Review Request: fail2ban - Ban IPs that make too many password failures
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: fail2ban - Ban IPs that make too many password failures
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220789
--- Additional Comments From [EMAIL PROTECTED] 2006-12-29 07:22 EST ---
Spec URL: http://dl.atrpms.net/all/fail2ban.spec
SRPM URL: http://dl.atrpms.net/all/fail2ban-0.6.2-2.at.src.rpm
* Fri Dec 29 2006 Axel Thimm [EMAIL PROTECTED] - 0.6.2-2
- Move /usr/lib/fail2ban to %%{_datadir}/fail2ban.
- Don't default chkconfig to enabled.
- Add dependencies on service/chkconfig.
- Use example iptables/ssh config as default config.
--
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 220888] Review Request: fakeroot - Gives a fake root environment
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: fakeroot - Gives a fake root environment https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220888 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 07:25 EST --- OK, the missing packages are now submitted as well - note that the dependency depth is three more packages found in bug #220943, bug #220944 and bug #220945. Spec URL: http://dl.atrpms.net/all/fakeroot.spec SRPM URL: http://dl.atrpms.net/all/fakeroot-1.5.10-10.at.src.rpm * Thu Dec 28 2006 Axel Thimm [EMAIL PROTECTED] - 1.5.10-10 - Don't build static lib. - Exclude libtool lib. - %%makeinstall to make install DESTDIR=%%buildroot. -- 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 220889] Review Request: fakechroot - Gives a fake chroot environment
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: fakechroot - Gives a fake chroot environment https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220889 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 07:25 EST --- Spec URL: http://dl.atrpms.net/all/fakechroot.spec SRPM URL: http://dl.atrpms.net/all/fakechroot-2.5-9.at.src.rpm * Thu Dec 28 2006 Axel Thimm [EMAIL PROTECTED] - 2.5-9 - Don't build static lib. - Exclude libtool lib. -- 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 209222] Review Request: prelude-lml - Prelude log analyzer
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: prelude-lml - Prelude log analyzer
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209222
[EMAIL PROTECTED] changed:
What|Removed |Added
Status|NEW |ASSIGNED
AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
OtherBugsDependingO|163776 |163778
nThis||
--- Additional Comments From [EMAIL PROTECTED] 2006-12-29 09:34 EST ---
I am still checking libprelude.
Well, you requested a sponsorship, however according to
http://fedoraproject.org/wiki/Extras/HowToGetSponsored
a person who want to get sponsored has to either
* submit (other) quality packages
* assist with package reviews (i.e. do a pre-review of other submitters'
review requests and have your pre-review checked)
So... would you rewrite the spec file of this package before
I sponsor you?
I have not checked this package yet, however as far as I glanced at
the spec file of this package, your spec file has a issue at least
on directory ownership. Perhaps the attribute of the files under
%{_sysconfdir}, the requirement for -devel package are also problems
(Again, I only glanced at the 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 209222] Review Request: prelude-lml - Prelude log analyzer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: prelude-lml - Prelude log analyzer https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209222 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 10:20 EST --- up to now I just corrected libprelude (#209214) and libpreludedb (#209215). Maybe we can work on libpreludedb before we move over to prelude-lml, altought I will work on this package as well. Thanks. -- 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 209215] Review Request: libpreludedb - Provides an abstraction layer for the libprelude
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: libpreludedb - Provides an abstraction layer for the libprelude https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209215 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 11:09 EST --- new package is available here: http://people.redhat.com/tscherf/fedora-extra/libpreludedb-0.9.11-1.src.rpm http://people.redhat.com/tscherf/fedora-extra/libpreludedb.spec the only rpmlint complain is on the devel-package which is not completely clear to me: [EMAIL PROTECTED] SPECS]$ rpmlint /home/tscherf/redhat/RPMS/i386/libpreludedb-devel-0.9.11-1.i386.rpm W: libpreludedb-devel no-documentation E: libpreludedb-devel library-without-ldconfig-postin /usr/lib/libpreludedb.so.0.4.2 E: libpreludedb-devel library-without-ldconfig-postun /usr/lib/libpreludedb.so.0.4.2 in the spec I have this: %post -p /sbin/ldconfig %postun -p /sbin/ldconfig what is rpmlint complaining about? don't think there are other issues available. -- 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 209215] Review Request: libpreludedb - Provides an abstraction layer for the libprelude
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: libpreludedb - Provides an abstraction layer for the libprelude https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209215 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 11:36 EST --- Umm?? Why is /usr/lib/libpreludedb.so.0.4.2 in -devel package? -devel package does not call ldconfig, so rpmlint complaint is reasonable (I just glanced at your 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 209214] Review Request: libprelude - Prelude library collection
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: libprelude - Prelude library collection https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209214 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 11:51 EST --- ( Some comments mentioned on bug 209215 ) -- 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 220890] Review Request: libcdaudio - Control operation of a CD-ROM when playing audio CDs
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: libcdaudio - Control operation of a CD-ROM when
playing audio CDs
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220890
--- Additional Comments From [EMAIL PROTECTED] 2006-12-29 12:20 EST ---
* /usr/bin/libcdaudio-config needs a patch for option --libs, because
it contains a hardcoded ${exec_prefix}/lib path.
* RPM Group better is changed to: System Environment/Libraries
--
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 220862] Review Request: php-channel-phpunit - Add phpunit channel to pear
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: php-channel-phpunit - Add phpunit channel to pear Alias: channel-phpunit https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220862 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 12:40 EST --- After IRC meeting, all is OK. Build fine, and allow the build of new php-pear-PHPUnit in mock. REVIEW * no source files * package meets naming and packaging guidelines (no special guideline for channels). * specfile is properly named, is cleanly written and uses macros consistently. * dist tag is present. * build root is correct. * license field matches the actual license. * license is open source-compatible (BSD). * BuildRequires are proper. * %clean is present. * package builds in mock (development). * package installs properly * rpmlint : W: php-channel-phpunit no-documentation * final provides are sane: php-channel-phpunit = 1.0-1.fc6 * %check is not present; no test suite provide. * doesn't own any directories it shouldn't. * no duplicates in %files. * file permissions are appropriate. * scriptlets are OK (pear channel..) * no documentation No version for the .xml file, but i think using 1.0 is OK. 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 220862] Review Request: php-channel-phpunit - Add phpunit channel to pear
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: php-channel-phpunit - Add phpunit channel to pear Alias: channel-phpunit https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220862 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] OtherBugsDependingO|163776 |163779 nThis|| -- 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 220888] Review Request: fakeroot - Gives a fake root environment
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: fakeroot - Gives a fake root environment https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220888 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 12:46 EST --- Is it possible to expand upon the %description a bit? I can't discern the function of this package from reading what's currently there. Even the summary from the web page: fakeroot provides a fake root environment by means of LD_PRELOAD and SYSV IPC or TCP trickery. is better, although it's still not all that explanatory. -- 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 208064] Review Request: courier-authlib - Courier authentication 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: courier-authlib - Courier authentication library https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=208064 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 12:53 EST --- Michael, any reply? -- 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 220862] Review Request: php-channel-phpunit - Add phpunit channel to pear
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: php-channel-phpunit - Add phpunit channel to pear Alias: channel-phpunit https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220862 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |CLOSED Resolution||NEXTRELEASE --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 13:00 EST --- - src rpm checked into cvs - Virtual provides added in -2 release - added entry to owners.list - built on devel - requested branch on fc5/6 Thanks for the review! -- 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 196837] Review Request: php-pear-PHPUnit3 - PEAR: Regression testing framework for unit tests
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: php-pear-PHPUnit3 - PEAR: Regression testing framework for unit tests Alias: pear-PHPUnit3 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=196837 Bug 196837 depends on bug 220862, which changed state. Bug 220862 Summary: Review Request: php-channel-phpunit - Add phpunit channel to pear https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220862 What|Old Value |New Value Resolution||NEXTRELEASE Status|NEW |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 220890] Review Request: libcdaudio - Control operation of a CD-ROM when playing audio CDs
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: libcdaudio - Control operation of a CD-ROM when playing audio CDs https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220890 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 13:05 EST --- God catch, thanks! -- 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 220888] Review Request: fakeroot - Gives a fake root environment
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: fakeroot - Gives a fake root environment https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220888 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 13:05 EST --- Indeed, maybe something along the lines of the following (assembled from the manpage): fakeroot runs a command in an environment wherein it appears to have root privileges for file manipulation. fakeroot works by replacing the file manipulation library functions (chmod(2), stat(2) etc.) by ones that simulate the effect the real library functions would have had, had the user really been root. But I won't yet respin a package with only this change, e.g. whoover reviews this, please assume the description will be the one above. -- 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 209215] Review Request: libpreludedb - Provides an abstraction layer for the libprelude
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: libpreludedb - Provides an abstraction layer for the libprelude https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209215 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 13:10 EST --- ok, fixed the problem with /usr/lib/libpreludedb.so.0.4.2 and ldconfig. rpmlint isn't complaining anymore. find the new packages here: http://people.redhat.com/tscherf/fedora-extra/libpreludedb-0.9.11-1.src.rpm http://people.redhat.com/tscherf/fedora-extra/libpreludedb.spec -- 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 220889] Review Request: fakechroot - Gives a fake chroot environment
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: fakechroot - Gives a fake chroot environment https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220889 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 13:12 EST --- %description is a bit terse, please assume the %description reads: fakechroot runs a command in an environment were is additionally possible to use the chroot(8) call without root privileges. This is useful for allowing users to create their own chrooted environment with possibility to install another packages without need for root privileges. E.g. this and fakeroot are useful a) for mock and friends for students w/o root access, as well as b) for hardening buildsystems (impossible to gain root privileges by injecting malicious packages into build hosts) There is no technical dependency between fakeroot and fakechroot, but it makes most sense to use both together for faking chroot calls and setting up faked chroots. -- 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 209222] Review Request: prelude-lml - Prelude log analyzer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: prelude-lml - Prelude log analyzer https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209222 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 13:13 EST --- updated package is online: http://people.redhat.com/tscherf/fedora-extra/prelude-lml-0.9.8.1-1.src.rpm http://people.redhat.com/tscherf/fedora-extra/prelude-lml.spec -- 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 220862] Review Request: php-channel-phpunit - Add phpunit channel to pear
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: php-channel-phpunit - Add phpunit channel to pear Alias: channel-phpunit https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220862 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 13:18 EST --- oops removed branch request for fc5/6, forgot that this is for fc7+ only. -- 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 218678] Review Request: pybluez - python API for the bluez bluetooth stack
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: pybluez - python API for the bluez bluetooth stack https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=218678 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 13:28 EST --- Thanks very much for the help! I don't currently have any other packages ready to review, but if you absolutely need to see some example specfiles I could dig some up from some internal packages I've made. As for the issues you mentioned: 1) Heh! I just wasn't sure what to do with the BuildArch line, so I put the three arches that Fedora currently supports. I'll remove it so this package can be all-arch. 2) This package is *not* noarch, hence the CFLAGS part *is* needed. I've removed the comment to avoid confusion. 3) Whoops, there should have been a space between that initial dash and the version. Fixed. Also, bluezchat.py is now chmod'd a-x before installation, which makes that rpmlint warning go away. 4) Why is that ugly? Isn't it normal for python modules which contain binary extensions to provide the .so file for that extension? Here are the updated files: Spec URL: http://homepage.mac.com/wgwoods/fedora/pybluez-0.9.1-3.spec SRPM URL: http://homepage.mac.com/wgwoods/fedora/pybluez-0.9.1-3.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 220967] New: Review Request: libscigraphica - A library of gtk+ widgets for SciGraphica
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=220967 Summary: Review Request: libscigraphica - A library of gtk+ widgets for SciGraphica Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: fedora-package-review@redhat.com Spec URL: ftp://czar.eas.yorku.ca/pub/scigraphica/libscigraphica.spec SRPM URL: ftp://czar.eas.yorku.ca/pub/scigraphica/libscigraphica-2.1.1-1.src.rpm Description: A library of gtk+ widgets for SciGraphica. SciGraphica is a free (GPL) scientific application for data analysis and technical graphics. -- 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 220967] Review Request: libscigraphica - A library of gtk+ widgets for SciGraphica
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: libscigraphica - A library of gtk+ widgets for SciGraphica https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220967 [EMAIL PROTECTED] changed: What|Removed |Added Summary|Review Request: |Review Request: |libscigraphica - A library |libscigraphica - A library |of gtk+ widgets for |of gtk+ widgets for |SciGraphica |SciGraphica Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] OtherBugsDependingO|163776 |163778 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 13:43 EST --- I'll give this a shot tonight -- 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 220967] Review Request: libscigraphica - A library of gtk+ widgets for SciGraphica
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: libscigraphica - A library of gtk+ widgets for
SciGraphica
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220967
--- Additional Comments From [EMAIL PROTECTED] 2006-12-29 13:49 EST ---
Requires: %name = %{version}
needs to be
Requires: %{name} = %{version}-%{release} pkgconfig
BuildRequires: gettext perl(XML::Parser) pkgconfig
should really be
BuildRequires: gettext-devel perl(XML::Parser)
you don't need pkgconfig as a main package BR as it's not used, but is in the
devel 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 220968] New: Review Request: scigraphica - Scientific application for data analysis and technical graphics
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=220968 Summary: Review Request: scigraphica - Scientific application for data analysis and technical graphics Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: fedora-package-review@redhat.com Spec URL: ftp://czar.eas.yorku.ca/pub/scigraphica/scigraphica.spec SRPM URL: ftp://czar.eas.yorku.ca/pub/scigraphica/scigraphica-2.1.0-1.src.rpm Description: SciGraphica is a free (GPL) scientific application for data analysis and technical graphics. It pretends to be a clone of the popular commercial (and expensive) application Microcal Origin. It fully supplies plotting features for 2D, 3D and polar charts. Please note that this will not build in mock at the moment, as it depends on libscigraphica, which I've just submitted for reviews. -- 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 220968] Review Request: scigraphica - Scientific application for data analysis and technical graphics
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: scigraphica - Scientific application for data analysis and technical graphics https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220968 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] BugsThisDependsOn||220967 OtherBugsDependingO|163776 |163778 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 13:52 EST --- I've got libscigraphica so I shall do this as well :-) -- 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 220967] Review Request: libscigraphica - A library of gtk+ widgets for SciGraphica
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: libscigraphica - A library of gtk+ widgets for SciGraphica https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220967 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||220968 nThis|| -- 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 220967] Review Request: libscigraphica - A library of gtk+ widgets for SciGraphica
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: libscigraphica - A library of gtk+ widgets for SciGraphica https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220967 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 13:57 EST --- It is, I've built it (libscigraphica) in mock. In response to the other comment, I believe pkgconfig is used in configure process. I'll put the corrected files up in a moment at Spec URL: ftp://czar.eas.yorku.ca/pub/scigraphica/libscigraphica.spec SRPM URL: ftp://czar.eas.yorku.ca/pub/scigraphica/libscigraphica-2.1.1-2.src.rpm Thanks. -- 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 220967] Review Request: libscigraphica - A library of gtk+ widgets for SciGraphica
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: libscigraphica - A library of gtk+ widgets for
SciGraphica
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220967
--- Additional Comments From [EMAIL PROTECTED] 2006-12-29 14:27 EST ---
(In reply to comment #2)
Requires: %name = %{version}
needs to be
Requires: %{name} = %{version}-%{release} pkgconfig
BuildRequires:gettext perl(XML::Parser) pkgconfig
should really be
BuildRequires: gettext-devel perl(XML::Parser)
you don't need pkgconfig as a main package BR as it's not used, but is in the
The build actually only relies on binaries in gettext itself, it doesn't compile
against gettext libs or use anything fron gettext-devel. Also, as mentioned
earlier, libscigraphica checks for pkgconfig when running the configure script.
devel package.
I've added it. Thanks
--
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 220969] New: Review Request: isomaster - GUI CD image editor
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=220969 Summary: Review Request: isomaster - GUI CD image editor Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: fedora-package-review@redhat.com Spec URL: http://timeoff.wsisiz.edu.pl/zrzut/isomaster.spec SRPM URL: http://timeoff.wsisiz.edu.pl/zrzut/isomaster-0.6-2.src.rpm Description: Iso Master is an open-source, easy to use, GUI CD image editor. It allows to extract files from an ISO, add files to an ISO, and create bootable ISOs - all in a graphical user interface. This is my first package for Fedora Extras and I'm seeking a 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 220969] Review Request: isomaster - GUI CD image editor
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: isomaster - GUI CD image editor https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220969 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|163776 |177841 nThis|| -- 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 220969] Review Request: isomaster - GUI CD image editor
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: isomaster - GUI CD image editor
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220969
[EMAIL PROTECTED] changed:
What|Removed |Added
OtherBugsDependingO||163776
nThis||
--- Additional Comments From [EMAIL PROTECTED] 2006-12-29 15:12 EST ---
Hi! Nice to see that another Pole wants to put his package in Extras! :-)
However there's a lot to do in your spec file.
* First of all mock build of your package fails due to a simple mistake. You
put desktop-file-utils as Requires but it should be BuildRequires.
* You don't have to explicitly set a version of gtk2 in requires. RPM should
build fine without it.
* Your %files section doesn't look good. Package owns files in
%{_datadir}/%{name}/icons/ but doesn't own the parent one. It means that if you
remove RPM, all files within %{_datadir}/%{name}/icons/ will be deleted but the
dir remains. In order to fix it you ought to simply remove all
%{_datadir}/%{name}/icons/ lines and replace them with a simple
%{_datadir}/%{name}
* I don't see it as a blocker but in my opinion much better solution would be
if you move a desktop file to another Source instead of creating it in spec.
That should be a lot more legible.
And the last thing: SPEC files are different at the URL you passed here and
inside SRPM. I hope you'll get rid of that issue in next release ;-)
PS. I also added FE-NEW blocker as it were missing.
--
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 219973] Review Request: pdns - A modern, advanced and high performance authoritative-only nameserver
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: pdns - A modern, advanced and high performance authoritative-only nameserver Alias: powerdns https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=219973 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 15:18 EST --- Hi Kevin, - Fixed the provides - I tried patching the Makefile.in for the backends to remove the -rpath options, and tried patching the included libtool as well, but the build failed in the install phase because it couldn't find a number of .lai files. I've now used chrpath to remove the rpath information. New versions: http://rubenkerkhof.com/packages/powerdns/pdns.spec http://rubenkerkhof.com/packages/powerdns/pdns-2.9.20-4.src.rpm Thanks, Ruben -- 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 220736] Review Request: doclifter - lifts man pages and other troff-based markups to XML-DocBook
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: doclifter - lifts man pages and other troff-based markups to XML-DocBook https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220736 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] OtherBugsDependingO||163776 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 15:20 EST --- Hey Eric. I'm re-adding the FE-NEW blocker bug here. That needs to be there in addition to the FE-NEEDSPONSOR so people can see that the review request is new and waiting for a reviewer. If no one else reviews this in the next few days I can try and do so... Can you spin a new version addressing the items in comment #2? Remember to increase the version and add a changelog so we can keep track of what was addressed when. -- 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 220969] Review Request: isomaster - GUI CD image editor
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: isomaster - GUI CD image editor
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220969
--- Additional Comments From [EMAIL PROTECTED] 2006-12-29 16:44 EST ---
(In reply to comment #1)
Hi! Nice to see that another Pole wants to put his package in Extras! :-)
However there's a lot to do in your spec file.
No-one is perfect ;)
* First of all mock build of your package fails due to a simple mistake. You
put desktop-file-utils as Requires but it should be BuildRequires.
I moved it there due to:
desktop-file-install --vendor fedora \
--dir %{buildroot}%{_datadir}/applications \
%{name}.desktop
in %install section. But If mock doesn't like it I moved it back to
BuildRequires.
* Your %files section doesn't look good. Package owns files in
%{_datadir}/%{name}/icons/ but doesn't own the parent one. It means that if
you remove RPM, all files within %{_datadir}/%{name}/icons/ will be deleted
but the dir remains. In order to fix it you ought to simply remove all
%{_datadir}/%{name}/icons/ lines and replace them with a simple
%{_datadir}/%{name}
Indeed. I missed it bacause I usually update my packages rather than remove :)
* I don't see it as a blocker but in my opinion much better solution would be
if you move a desktop file to another Source instead of creating it in spec.
That should be a lot more legible.
Ok, I moved it.
And the last thing: SPEC files are different at the URL you passed here and
inside SRPM. I hope you'll get rid of that issue in next release ;-)
This time I did my best to not update descriptions at the last moment :)
Thanks for you suggestions. 0.6-3 is ready for a review:
http://timeoff.wsisiz.edu.pl/zrzut/isomaster.spec
http://timeoff.wsisiz.edu.pl/zrzut/isomaster-0.6-3.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 220979] New: Review Request: tesseract - Raw OCR Engine
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=220979 Summary: Review Request: tesseract - Raw OCR Engine Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: fedora-package-review@redhat.com Spec URL: http://mutebox.net/~karlik/tesseract.spec SRPM URL: http://mutebox.net/~karlik/tesseract-1.02-1.src.rpm Description: A commercial quality OCR engine originally developed at HP between 1985 and 1995. In 1995, this engine was among the top 3 evaluated by UNLV. It was open-sourced by HP and UNLV in 2005. It is my first rpm and I am looking for sponsor. I want to add this rpm (in future I hope add more), because in extras is not any ocr and this program is on wishlist. -- 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 220922] Review Request: geany - a lightweight gtk2 based IDE
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: geany - a lightweight gtk2 based IDE Alias: geany https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220922 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 17:05 EST --- hmm, for some reason I nuked all of the other BR's that are supposed to be there. Fixed that and added vte-devel for the virtual-terminal stuff that comes with geany. Same url as above for updated src.rpm and 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 220979] Review Request: tesseract - Raw OCR Engine
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: tesseract - Raw OCR Engine https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220979 [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 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 220969] Review Request: isomaster - GUI CD image editor
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: isomaster - GUI CD image editor
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220969
--- Additional Comments From [EMAIL PROTECTED] 2006-12-29 17:14 EST ---
(In reply to comment #2)
I moved it there due to:
desktop-file-install --vendor fedora \
--dir %{buildroot}%{_datadir}/applications \
%{name}.desktop
in %install section. But If mock doesn't like it I moved it back to
BuildRequires.
Remember that %install section is executed at building an SRPM, not at
installing the binary one. That's why you need to put it into BR.
* I don't see it as a blocker but in my opinion much better solution would
be
if you move a desktop file to another Source instead of creating it in spec.
That should be a lot more legible.
Ok, I moved it.
Desktop files aren't as large to put them into tarball. You can remove a tar
compression and get rid of second %setup macro. Now, you can put %{SOURCE1}
macro into desktop-file-install:
desktop-file-install --vendor fedora \
--dir %{buildroot}%{_datadir}/applications \
%{SOURCE1}
and it looks much better :)
--
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 220979] Review Request: tesseract - Raw OCR Engine
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: tesseract - Raw OCR Engine https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220979 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 17:27 EST --- This has been discussed before IIRC. tesseract might give good results, but it is hopeless to compile with 64bit. IMHO, this indicates very poor code quality. I wouldn't want to volunteer to maintain something like 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 219973] Review Request: pdns - A modern, advanced and high performance authoritative-only nameserver
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: pdns - A modern, advanced and high performance authoritative-only nameserver Alias: powerdns https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=219973 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 18:17 EST --- The package in comment #6 still has some rpath issues. You fixed up the binaries, but some of the shared libraries still have rpath. rpmlint says: E: pdns-backend-geo binary-or-shlib-defines-rpath /usr/lib64/pdns/libgeobackend.so ['/usr/lib64'] E: pdns-backend-ldap binary-or-shlib-defines-rpath /usr/lib64/pdns/libldapbackend.so ['/usr/lib64'] E: pdns-backend-mysql binary-or-shlib-defines-rpath /usr/lib64/pdns/libgmysqlbackend.so ['/usr/lib64'] E: pdns-backend-pipe binary-or-shlib-defines-rpath /usr/lib64/pdns/libpipebackend.so ['/usr/lib64'] E: pdns-backend-postgresql binary-or-shlib-defines-rpath /usr/lib64/pdns/libgpgsqlbackend.so ['/usr/lib64'] Fix those up and I think we are reaching approval... ;) -- 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 218678] Review Request: pybluez - python API for the bluez bluetooth stack
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: pybluez - python API for the bluez bluetooth stack https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=218678 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|163778, 177841 |163779 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 18:30 EST --- 1, 2, 3 all look fixed up in the package in comment #5. On item 4: rpm happily looks for .so files anywhere in the package and then automagically Provides them in the rpm namespace. I think that rpm should instead only do this with .so files when they are in the standard ldconfig directories. The only thing that can really use the _bluetooth.so thats provided here is this package, so the provides just adds to the rpm provides namespace. It's not a blocker though, since it doesn't hurt anything. If the .so here was bluetooth.so that would be a problem... since bluez-libs-devel already provides that. Hopefully that makes some sense to someone aside from me. ;) I don't see any other blockers here... I will go ahead and APPROVE this package and sponsor you. You can continue the process from here: http://fedoraproject.org/wiki/Extras/Contributors#head-a89c07b5b8abe7748b6b39f0f89768d595234907 Don't forget to close this review request NEXTRELASE once everything has been imported and built. -- 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 211807] Review Request: firefox2 - Mozilla Firefox 2.0 Web browser for FC6
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: firefox2 - Mozilla Firefox 2.0 Web browser for FC6 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=211807 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 18:54 EST --- Hey Kevin, I have been away and unable to get to this earlier. The good news is that I am working on an updated rpm now. The bad news is that initial discussion with caillon was that he was not in favour of having this happen[1]. However if we could find a way to appease his concerns there might be hope. 1. https://www.redhat.com/archives/fedora-devel-list/2006-October/msg00624.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 211807] Review Request: firefox2 - Mozilla Firefox 2.0 Web browser for FC6
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: firefox2 - Mozilla Firefox 2.0 Web browser for FC6 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=211807 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 19:14 EST --- Yeah, I thought that was the case, but I couldn't find the discussion when I looked for it a while ago. ;( I have mailed Christopher and asked him to comment here on this package. Perhaps something has changed since October. Hopefully he will have time to do so soon. -- 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 219972] Review Request: poker-network - A poker server, client and abstract user interface 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: poker-network - A poker server, client and abstract user interface library Alias: poker-network https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=219972 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 20:20 EST --- Spec URL: http://tkmame.retrogames.com/fedora-extras/poker-network.spec SRPM URL: http://tkmame.retrogames.com/fedora-extras/poker-network-1.0.32-3.src.rpm %changelog * Fri Dec 29 2006 Christopher Stone [EMAIL PROTECTED] 1.0.32-3 - Update BR/R to new python-twisted stack -- 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 220860] Review Request: galternatives - Alternatives Configurator
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: galternatives - Alternatives Configurator https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220860 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 20:24 EST --- -- and --initscripts option is a Red Hat Linux specific option according to man alternatives Then a patch is needed * to treat initscripts options correctly * or to ignore initscripts options for now. Then a documentation (like README.fedora) which explains that this package ignores initscripts option currently. I've added a patch to treat the initscripts appropriately, and thus fixing the traceback. Updated spec and srpm below; Spec URL: ftp://czar.eas.yorku.ca/pub/galternatives/galternatives.spec SRPM URL: ftp://czar.eas.yorku.ca/pub/galternatives/galternatives-0.13.4-3.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 209215] Review Request: libpreludedb - Provides an abstraction layer for the libprelude
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: libpreludedb - Provides an abstraction layer for the libprelude https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209215 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] OtherBugsDependingO|163776 |163778 nThis|| -- 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 220919] Review Request: kronolith - The Horde calendar application
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: kronolith - The Horde calendar application https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220919 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|163778 |163779 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 21:11 EST --- I needed to remove tarballs/ from the Source URL in order to grab the tarball. Other than that, this package is fine so you can fix it when you check in. As with the other horde-using packages, these rpmlint warnings are expected and necessary: E: kronolith htaccess-file /usr/share/horde/kronolith/scripts/.htaccess E: kronolith non-readable /etc/horde/kronolith/conf.xml 0660 [six more] E: kronolith non-standard-dir-perm /etc/horde/kronolith 0770 E: kronolith non-standard-gid /etc/horde/kronolith apache [seven more] E: kronolith non-standard-uid /etc/horde/kronolith apache [seven more] W: kronolith conffile-without-noreplace-flag /etc/horde/kronolith/conf.xml [three more] Review: * source files match upstream: df6d6fc99012865b18b089212c7544ad kronolith-h3-2.1.4.tar.gz * package meets naming and versioning guidelines. * specfile is properly named, is cleanly written and uses macros consistently. * dist tag is present. * build root is correct. * license field matches the actual license. * license is open source-compatible. License text included in package. * latest version is being packaged. * BuildRequires are proper. * %clean is present. * package builds in mock (development, x86_64). * package installs properly * rpmlint has only ignoreable complaints. * final provides and requires are sane: config(kronolith) = 2.1.4-1.fc7 kronolith = 2.1.4-1.fc7 = /usr/bin/php config(kronolith) = 2.1.4-1.fc7 horde = 3 php = 4.3.0 * %check is not present; no test suite upstream. Basic manual testing shows things work OK * owns the directories it creates. * doesn't own any directories it shouldn't. * no duplicates in %files. * file permissions are appropriate. * locales are handled properly. * no scriptlets present. * code, not content. * documentation is small, so no -docs subpackage is necessary. * %docs are not necessary for the proper functioning of the package. 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 220926] Review Request: ingo - The Horde email filter 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: ingo - The Horde email filter manager https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220926 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] OtherBugsDependingO|163776 |163778 nThis|| -- 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 220926] Review Request: ingo - The Horde email filter 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: ingo - The Horde email filter manager https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220926 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 22:12 EST --- As with the Kronolith package, I needed to remove tarballs/ from the Source URL in order to download the source. Also like the Turba package, this package has an incorrect license. I believe this one should also be Apache Software License v1. It doesn't look like anyone has packaged up the Net_Sieve module, which limits the utility of Ingo a bit. As usual for these Horde-based packages, there are a number of acceptable rpmlint complaints: E: ingo htaccess-file /usr/share/horde/ingo/scripts/.htaccess E: ingo non-readable /etc/horde/ingo/backends.php 0660 [six more] E: ingo non-standard-dir-perm /etc/horde/ingo 0770 E: ingo non-standard-gid /etc/horde/ingo apache [seven more] E: ingo non-standard-uid /etc/horde/ingo apache [seven more] W: ingo conffile-without-noreplace-flag /etc/horde/ingo/backends.php.dist [three more] Review: * package meets naming and versioning guidelines. * specfile is properly named, is cleanly written and uses macros consistently. * dist tag is present. * build root is correct. X license field matches the actual license. * license is open source-compatible. License text included in package. * latest version is being packaged. * BuildRequires are proper. * %clean is present. * package builds in mock (development, x86_64). * package installs properly * debuginfo package looks complete. * rpmlint has only ignoreable complaints. * final provides and requires are sane: config(ingo) = 1.1.2-1.fc7 ingo = 1.1.2-1.fc7 = config(ingo) = 1.1.2-1.fc7 horde = 3 php = 4.3.0 * %check is not present; no test suite upstream. Things seem OK after manual testing. * owns the directories it creates. * doesn't own any directories it shouldn't. * no duplicates in %files. * file permissions are appropriate. * locales are handle appropriately. * no scriptlets present. * code, not content. * documentation is small, so no -docs subpackage is necessary. * %docs are not necessary for the proper functioning of the package. -- 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 209215] Review Request: libpreludedb - Provides an abstraction layer for the libprelude
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: libpreludedb - Provides an abstraction layer for the libprelude https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209215 --- Additional Comments From [EMAIL PROTECTED] 2006-12-29 23:55 EST --- Well, * For ownership issue: - Please check if directories/files are owned correctly. -- [EMAIL PROTECTED] mock]$ rpm -qlp libpreludedb-*i386*rpm | grep -v debug | sort -- says: -- /usr/lib/libpreludedb/plugins/sql/mysql.so /usr/lib/libpreludedb/plugins/sql/mysql.so /usr/lib/libpreludedb/plugins/sql/pgsql.so /usr/lib/libpreludedb/plugins/sql/pgsql.so /usr/lib/libpreludedb/plugins/sql/sqlite3.so /usr/lib/libpreludedb/plugins/sql/sqlite3.so snip /usr/share/libpreludedb/classic/mysql2sqlite.sh /usr/share/libpreludedb/classic/pgsql-update-14-1.sql /usr/share/libpreludedb/classic/pgsql-update-14-1.sql /usr/share/libpreludedb/classic/pgsql-update-14-2.sql /usr/share/libpreludedb/classic/pgsql-update-14-2.sql /usr/share/libpreludedb/classic/pgsql-update-14-3.sql /usr/share/libpreludedb/classic/pgsql-update-14-3.sql /usr/share/libpreludedb/classic/pgsql-update-14-4.sql /usr/share/libpreludedb/classic/pgsql-update-14-4.sql /usr/share/libpreludedb/classic/pgsql-update-14-5.sql /usr/share/libpreludedb/classic/pgsql-update-14-5.sql /usr/share/libpreludedb/classic/pgsql.sql /usr/share/libpreludedb/classic/pgsql.sql snip /usr/share/libpreludedb/classic/sqlite.sql /usr/share/libpreludedb/classic/sqlite.sql -- ... There are duplicate entries. -- /usr/share/doc/libpreludedb-0.9.11/html/libpreludedb/style.css /usr/share/doc/libpreludedb-0.9.11/html/libpreludedb/up.png /usr/share/libpreludedb/classic/mysql-update-14-1.sql /usr/share/libpreludedb/classic/mysql-update-14-2.sql /usr/share/libpreludedb/classic/mysql-update-14-3.sql -- /usr/share/libpreludedb /usr/share/libpreludedb/classic are not owned by any packages. * For -devel package: - Why is the dependency for main package with =, not equal? (also for other subpackages) - And again check the dependency against other package. --- [EMAIL PROTECTED] mock]# /usr/bin/libpreludedb-config --libs -L/usr/lib -lpreludedb -L/usr/lib -lprelude -lgnutls -lgcrypt -lgpg-error -lrt -ldl --- This means that this package requires other -devel packages (like bug 209214) (libprelude-devel) Also /usr/share/aclocal/libpreludedb.m4 requires Requires: automake. -- 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 209214] Review Request: libprelude - Prelude library collection
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: libprelude - Prelude library collection https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209214 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|163778 |163779 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2006-12-30 00:02 EST --- Well, now this package meets http://fedoraproject.org/wiki/Packaging/Guidelines http://fedoraproject.org/wiki/Packaging/ReviewGuidelines This package (libprelude) is APPROVED by me. Please step forward according to http://fedoraproject.org/wiki/Extras/Contributors -- 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 209214] Review Request: libprelude - Prelude library collection
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: libprelude - Prelude library collection https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209214 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|177841 | nThis|| --- Additional Comments From [EMAIL PROTECTED] 2006-12-30 00:08 EST --- Umm... It seems that Paul Johnson has already sponsored you (I don't know why, however, it is okay..) -- 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 209215] Review Request: libpreludedb - Provides an abstraction layer for the libprelude
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: libpreludedb - Provides an abstraction layer for the libprelude https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209215 [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 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 209215] Review Request: libpreludedb - Provides an abstraction layer for the libprelude
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: libpreludedb - Provides an abstraction layer for the libprelude https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209215 [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 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 209222] Review Request: prelude-lml - Prelude log analyzer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: prelude-lml - Prelude log analyzer https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209222 [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 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 209217] Review Request: libpreludedb-mysql - mysql plugin for libpreludedb
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: libpreludedb-mysql - mysql plugin for libpreludedb https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209217 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|177841 | nThis|| --- Additional Comments From [EMAIL PROTECTED] 2006-12-30 00:10 EST --- (removing NEEDSPONSOR) -- 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 209219] Review Request: libpreludedb-sqlite - Sqlite plugin for libprelude
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: libpreludedb-sqlite - Sqlite plugin for libprelude https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209219 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|177841 | nThis|| --- Additional Comments From [EMAIL PROTECTED] 2006-12-30 00:13 EST --- (removing NEEDSPONSOR) -- 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 209220] Review Request: libpreludedb-pgsql - pgsql plugin for libprelude
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: libpreludedb-pgsql - pgsql plugin for libprelude https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209220 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|177841 | nThis|| --- Additional Comments From [EMAIL PROTECTED] 2006-12-30 00:14 EST --- (removing NEEDSPONSOR) -- 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 209224] Review Request: prelude-manager - Prelude Management server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: prelude-manager - Prelude Management server https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209224 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|177841 | nThis|| --- Additional Comments From [EMAIL PROTECTED] 2006-12-30 00:15 EST --- (removing NEEDSPONSOR) -- 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 220381] Review Request: flex-old - Legacy version of flex, a tool for creating scanners
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: flex-old - Legacy version of flex, a tool for creating
scanners
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220381
--- Additional Comments From [EMAIL PROTECTED] 2006-12-30 00:31 EST ---
(In reply to comment #6)
This builds fine for me; a few comments:
W: compat-flex devel-file-in-non-devel-package /usr/lib64/libfl-compat.a
Static libraries are generally frowned upon, but I don't know enough about
just how flex is used to say whether this is completely unacceptable.
Well, libfl basically provides a single function. One, most real world
lex-scanners normally implement themselves (i.e. link against statically) and
won't use anyway.
However, what is an issue, is the name of this library: libfl-compat.a.
This will break all packages because they expect to link against libfl. IMO, it
must remain libfl.
Similar considerations apply to the location of FlexScanner.h.
I.e. instead of playing games with directories named compat and renaming libs,
I'd prefer to see the flex's headers and link-libraries files (*.h,*.a) to be
installed into a versioned directory, e.g.
%{libdir}/flex-version
%{includedir}/flex-version
--
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 220945] Review Request: perl-Text-CharWidth - Get number of occupied columns of a string on terminal
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-Text-CharWidth - Get number of occupied columns of a string on terminal https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220945 --- Additional Comments From [EMAIL PROTECTED] 2006-12-30 00:45 EST --- Missing directory ownerships: The package must own these dirs: /usr/lib/perl5/vendor_perl/5.8.8/i386-linux-thread-multi/Text /usr/lib/perl5/vendor_perl/5.8.8/i386-linux-thread-multi/auto/Text -- 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 220999] New: Review Request: php-pear-Net-Sieve - Communication with timsieved
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=220999 Summary: Review Request: php-pear-Net-Sieve - Communication with timsieved Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: normal Priority: normal Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: fedora-package-review@redhat.com Spec URL: http://theholbrooks.org/RPMS/php-pear-Net-Sieve.spec SRPM URL: http://theholbrooks.org/RPMS/php-pear-Net-Sieve-1.1.5-1.src.rpm Description: Provides an API to talk to the timsieved server that comes with Cyrus IMAPd. Can be used to install, remove, mark active, etc sieve scripts. -- 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 220926] Review Request: ingo - The Horde email filter 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: ingo - The Horde email filter manager https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220926 [EMAIL PROTECTED] changed: What|Removed |Added BugsThisDependsOn||220999 --- Additional Comments From [EMAIL PROTECTED] 2006-12-30 01:21 EST --- Spec URL: http://theholbrooks.org/RPMS/ingo.spec SRPM URL: http://theholbrooks.org/RPMS/ingo-1.1.2-2.src.rpm Added a requirement on php-pear(Net_Sieve) (bug 220999), fixed the download URL, and changed the license. -- 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 220999] Review Request: php-pear-Net-Sieve - Communication with timsieved
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: php-pear-Net-Sieve - Communication with timsieved https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220999 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||220926 nThis|| -- 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 220919] Review Request: kronolith - The Horde calendar application
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: kronolith - The Horde calendar application https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220919 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Additional Comments From [EMAIL PROTECTED] 2006-12-30 01:25 EST --- Changed download URL, Imported, Built, Branches requested -- 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 220999] Review Request: php-pear-Net-Sieve - Communication with timsieved
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: php-pear-Net-Sieve - Communication with timsieved Alias: pear-Net-Sieve https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220999 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] OtherBugsDependingO|163776 |163778 nThis|| Alias||pear-Net-Sieve --- Additional Comments From [EMAIL PROTECTED] 2006-12-30 01:42 EST --- I'll review this (probably tomorrow). Two second glance at spec looks like you have the wrong license. -- 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 220999] Review Request: php-pear-Net-Sieve - Communication with timsieved
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: php-pear-Net-Sieve - Communication with timsieved Alias: pear-Net-Sieve https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220999 --- Additional Comments From [EMAIL PROTECTED] 2006-12-30 01:47 EST --- Brandon, I'm seeing a couple little minor things that need fixing. Could you please redo this spec using the fedora-newrpmspec command? Just run fedora-newrpmsepc php-pear-Net-Sieve and it will create a default template for you to use. This will be much easier for me to review. Thanks. -- 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 220999] Review Request: php-pear-Net-Sieve - Communication with timsieved
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: php-pear-Net-Sieve - Communication with timsieved Alias: pear-Net-Sieve https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220999 --- Additional Comments From [EMAIL PROTECTED] 2006-12-30 01:54 EST --- I don't mind regenerating a new spec file, but this one is almost verbatim the pear template found at: http://fedoraproject.org/wiki/Extras/SIGs/PHP/PearSpecTemplate If something is wrong with the template I'd rather point out specifics so the wiki can be updated as well as this 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 220860] Review Request: galternatives - Alternatives Configurator
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: galternatives - Alternatives Configurator https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220860 --- Additional Comments From [EMAIL PROTECTED] 2006-12-30 02:36 EST --- Your patch works well, thanks. Then, first full review for galternatives * General notes for python related packages - Unlike shared libraries' dependencies, python related dependencies are not checked automatically by rpmbuild and these have to be looked into manually. Normally, this can be done by checking what modules this package has to import. For this package [EMAIL PROTECTED] ~]$ grep import /usr/sbin/galternatives `rpm -ql galternatives | grep py$` /usr/sbin/galternatives:import os, sys /usr/sbin/galternatives:import galternatives /usr/sbin/galternatives:from galternatives import gtk /usr/sbin/galternatives:import gettext /usr/sbin/galternatives:from galternatives.common import PACKAGE /usr/lib/python2.5/site-packages/galternatives/__init__.py:from main import * /usr/lib/python2.5/site-packages/galternatives/alternative.py:from common import PACKAGE /usr/lib/python2.5/site-packages/galternatives/alternative.py:import os, gettext /usr/lib/python2.5/site-packages/galternatives/alternative.py:from gadebug import print_debug /usr/lib/python2.5/site-packages/galternatives/main.py:import pygtk /usr/lib/python2.5/site-packages/galternatives/main.py: import gtk, gobject /usr/lib/python2.5/site-packages/galternatives/main.py: from gtk import glade /usr/lib/python2.5/site-packages/galternatives/main.py:from common import PACKAGE /usr/lib/python2.5/site-packages/galternatives/main.py:import sys, os, gettext /usr/lib/python2.5/site-packages/galternatives/main.py:from alternative import Alternative This means that this package needs Requires: pygtk2 Then A. http://fedoraproject.org/wiki/Packaging/Guidelines * Licensing - Please include the license document. For this package, debian/copyright seems the best - Also, adding debian/changelog seems useful and should be included in the package. B. http://fedoraproject.org/wiki/Packaging/ReviewGuidelines = This is okay, except for issues on A C. Other notes: - For upstream URL: Maybe http://packages.qa.debian.org/g/galternatives.html is more useful? - I think this package to be useful, however, how do you think of http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=365365 ? -- 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
