[Bug 222521] Review Request: IceWM - Lightweight Window Manager.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: IceWM - Lightweight Window Manager. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222521 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 03:40 EST --- OK. Thanks. I misunderstood the how-to-join HOWTO. - Gilboa -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 222087] Review Request: pcmanx-gtk2 - Telnet client designed for BBS browsing
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: pcmanx-gtk2 - Telnet client designed for BBS browsing https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222087 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 03:41 EST --- (In reply to comment #5) (In reply to comment #4) No it's not missing. See line: Icon=pcmanx Should I change it to pcmanx.png? Ah, okay. You may leave this as Icon=pcmanx * Still fails on mockbuild Where can I use this? or do I have to setup my own? Well, first install mock rpm (from Fedora Extras) and please check the manual of mock (if you use the default configuration, you can do the mockbuild by mock ?.src.rpm) -- + /usr/lib/rpm/redhat/find-lang.sh /var/tmp/pcmanx-gtk2-0.3.5-2.fc7-root-mockbuild pcmanx No translations found for pcmanx in /var/tmp/pcmanx-gtk2-0.3.5-2.fc7-root-mockbuild error: Bad exit status from /var/tmp/rpm-tmp.40758 (%install) -- Any ideas why it failed? AFAIK, there are thse files: /usr/share/locale/zh_CN/LC_MESSAGES/pcmanx.mo /usr/share/locale/zh_TW/LC_MESSAGES/pcmanx.mo For this package, creating these gettext .mo files needs gettext rpm. Please add gettext to BuildRequires. --add-category X-Fedora --add-category X-Red-Hat-Extra --add-category Applications All these are not deprecated and should be removed I can't understand this. Should be removed or not? Oops.. What I meant was All these are now deprecated Please remove these.. Seems all fixed. http://www.srcf.ucam.org/~sl392/fedora/SRPMs/pcmanx-gtk2-0.3.5-4.fc6.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 222523] Review Request:gmrun - A lightweight Run program window with TAB completion
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:gmrun - A lightweight Run program window with TAB completion https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523 [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 222523] Review Request:gmrun - A lightweight Run program window with TAB completion
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:gmrun - A lightweight Run program window with TAB completion https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222523 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 03:42 EST --- Changed NEEDREVIEW to 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 215224] Review Request: gtk-murrine-engine - Murrine GTK2 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: gtk-murrine-engine - Murrine GTK2 engine https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215224 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 03:50 EST --- Tried to 'make build' gtk-murrine-engine but failed. -- /usr/bin/plague-client build gtk-murrine-engine gtk-murrine-engine-0_41-1_fc7 devel Server returned an error: Insufficient privileges. -- Any ideas? -- 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 - an easy to use 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 - an easy to use GUI CD image editor
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220969
--- Additional Comments From [EMAIL PROTECTED] 2007-01-13 06:56 EST ---
Version 0.7 is out. Locale was added and few other changes in a build process
occured.
* Fri Jan 12 2007 Marcin Zajaczkowski mszpak ATT wp DOTT pl - 0.7-1
- updated to 0.7
- added locale files
- added manual page
- adjusted %%{optflags} patch to a new Makefile
- added patch to correct wrong dependencies which broke parallel build
- removed redundant deletion of builddir (--clean option in rpmbuild does the
same)
Spec: timeoff.wsisiz.edu.pl/zrzut/isomaster.spec
SRPM: timeoff.wsisiz.edu.pl/zrzut/isomaster-0.7-1.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 220969] Review Request: isomaster - an easy to use 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 - an easy to use GUI CD image editor https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220969 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 06:58 EST --- Ehh, again with clickable links... Spec: http://timeoff.wsisiz.edu.pl/zrzut/isomaster.spec SRPM: http://timeoff.wsisiz.edu.pl/zrzut/isomaster-0.7-1.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 222326] Review Request: gxine - GTK frontend for the xine multimedia 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: gxine - GTK frontend for the xine multimedia library https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222326 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 07:30 EST --- the current SRPM builds successfully in mock for Fedora Development/i386 and rpmlint is silent on all rpms -- 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 222220] Review Request: polyester - KDE style and window decoration
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: polyester - KDE style and window decoration https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=20 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 08:05 EST --- Hello, sorry about the dealy, I have some material problem at my place. I'm trying to mock right now. (In reply to comment #13) Just a note to the conversation about the *.desktop-files: I've tried to avoid packages which requires desktop-file-install in the past because I cannot say I've really understanded this (usage/guidelines/syntax/etc.). :) In this case, desktop-install-file isn't required since this is for kwin. But (for another package) however they should be used for GUI. The package looks good. Josef Whiter can you try rebuilding the package on x86_64 arch please? -- 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 221027] Review Request: LabPlot - Data Analysis and Visualization
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: LabPlot - Data Analysis and Visualization
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221027
--- Additional Comments From [EMAIL PROTECTED] 2007-01-13 08:22 EST ---
(In reply to comment #9)
2. subpkgs, particularly -devel, should
Requires: %{name} = %{version}-%{devel}
Hmm how can a %{name}-%{version}-%{devel} be autodependent ?
%{name}-%{version}-%{devel} depends on %{name}-%{version}-%{devel} doesn't make
sense.
4. instead of
sed -i -e \
s'|Categories=Qt;KDE;Education;Science;Physics;Math;|Categories=Science;|' \
%{buildroot}%{_datadir}/applications/kde/%{name}.desktop
It's preferable to use
desktop-file-install --remove-category=...
(and I'd argue you shouldn't be removing Qt;KDE;Education categories
Labplot will be part of the Scientific Tools at Fedora at the same time a KDE
package.
Since Labplot will be installed at kmenuEdutainementMathematics and other
scientific tools such as Qalculate, the next release of the geda packages, kicad
and xcircuit are being installed at kmenuScienceMath, that is why I prefer
that important tool, Labplot, to be installed at ScienceMath instead of having
multiple copies/items of Labplot in the kmenu (which makes the kde menu huge)
--
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 222522] Review Request: aqbanking - A library for online banking functions and financial data import/export
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: aqbanking - A library for online banking functions and
financial data import/export
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222522
--- Additional Comments From [EMAIL PROTECTED] 2007-01-13 09:02 EST ---
Created an attachment (id=145526)
-- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=145526action=view)
build failure log
Not a full review, just an early bird's picks:
* %defattr missing in several sub-packages
* Obsoletes ought to specify max.versions (using LT, or LE
inequations)
* main package must not own %{_libdir}/gwenhywfar since that belongs
into the gwenhywfar pkg already
* Excludes are not symmetric. That's dangerous since you can
lose files:
main package:
%exclude %{_libdir}/aqbanking/plugins/*/debugger
%exclude %{_libdir}/aqbanking/plugins/*/frontends/*
%exclude %{_libdir}/aqbanking/plugins/*/wizards
qbanking package:
%{_libdir}/aqbanking/plugins/*/debugger
%{_libdir}/aqbanking/plugins/*/frontends/qbanking
%{_libdir}/aqbanking/plugins/*/wizards
You exclude everything in
%{_libdir}/aqbanking/plugins/*/frontends/*
but only
%{_libdir}/aqbanking/plugins/*/frontends/qbanking
is included explicitly. Other content below
%{_libdir}/aqbanking/plugins/*/frontends/
would be skipped/excluded silently.
* the -devel packages ought to Requires: automake, so the
%{_datadir}/aclocal/ directory is not orphaned - and because smart
applications use automake anyway - it's kinda difficult to determine
the config values without using the foo-config scripts/pkgconfig/automake
* [ $RPM_BUILD_ROOT != / ] rm -rf $RPM_BUILD_ROOT
Useless check. Nowadays buildroot cannot be / anymore.
* Requires: pkgconfig in the frontends' -devel packages is redundant,
since they all need aqbanking-devel which in turn requires pkgconfig
for all its *-config queries (due to the applied Patch2)
--
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 222533] New: Review Request: grig - a Ham Radio Control graphical user interface
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=222533 Summary: Review Request: grig - a Ham Radio Control graphical user interface 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://www.poolshark.org/src/grig.spec SRPM URL: http://www.poolshark.org/src/grig-0.7.2-1.fc7.src.rpm Description: Grig is a graphical user interface for the Ham Radio Control Libraries. It is intended to be simple and generic, presenting the user to the same interface regardless of which radio he or she uses. -- 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 222522] Review Request: aqbanking - A library for online banking functions and financial data import/export
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: aqbanking - A library for online banking functions and
financial data import/export
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222522
--- Additional Comments From [EMAIL PROTECTED] 2007-01-13 09:16 EST ---
Doesn't build in mock (rawhide x86_64), build.log ends with;
DIE_RPATH_DIE=/usr/lib64:$DIE_RPATH_DIE gcc -O2 -g -pipe -Wall
-Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4
-m64 -mtune=generic -g -Wall -g -o .libs/testlib testlib.o
./.libs/libcbanking.so
/usr/bin/ld: warning: libaqbanking.so.16, needed by ./.libs/libcbanking.so, not
found (try using -rpath or -rpath-link)
testlib.o: In function `main':
/builddir/build/BUILD/aqbanking-2.1.0/src/frontends/cbanking/testlib.c:6:
undefined reference to `AB_Banking_free'
./.libs/libcbanking.so: undefined reference to `AB_Banking_SetProgressEndFn'
./.libs/libcbanking.so: undefined reference to `AB_Banking_SetHideBoxFn'
./.libs/libcbanking.so: undefined reference to `AB_Banking_new'
./.libs/libcbanking.so: undefined reference to `AB_Banking_SetProgressLogFn'
./.libs/libcbanking.so: undefined reference to `AB_Banking_InputBox'
./.libs/libcbanking.so: undefined reference to `AB_Banking_SetShowBoxFn'
./.libs/libcbanking.so: undefined reference to `AB_Banking_SetMessageBoxFn'
./.libs/libcbanking.so: undefined reference to `AB_Banking_SetGetPinFn'
./.libs/libcbanking.so: undefined reference to `AB_BANKING__INHERIT_GETLIST'
./.libs/libcbanking.so: undefined reference to `AB_BANKING__INHERIT_SETDATA'
./.libs/libcbanking.so: undefined reference to `AB_Banking_SetProgressAdvanceFn'
./.libs/libcbanking.so: undefined reference to `AB_Banking_SetInputBoxFn'
./.libs/libcbanking.so: undefined reference to `AB_Banking_MessageBox'
./.libs/libcbanking.so: undefined reference to `AB_Banking_SetProgressStartFn'
collect2: ld returned 1 exit status
make[4]: *** [testlib] Error 1
make[4]: Leaving directory
`/builddir/build/BUILD/aqbanking-2.1.0/src/frontends/cbanking'
make[3]: *** [all-recursive] Error 1
make[3]: Leaving directory `/builddir/build/BUILD/aqbanking-2.1.0/src/frontends'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory `/builddir/build/BUILD/aqbanking-2.1.0/src'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/builddir/build/BUILD/aqbanking-2.1.0'
make: *** [all] Error 2
error: Bad exit status from /var/tmp/rpm-tmp.37465 (%build)
Other minor issues;
Shouldn't Buildrequires on python be python-devel instead?
Also the python packaging guildeline mandates defining python_sitelib at the top
of your specfile, so the files under -python-%{name} subpackages would just go
into %{python_sitelib}/%{name}/
--
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 222534] New: Review Request: libsnack - Snack Sound Toolkit provides audio handling to tlc/tk or python
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=222534 Summary: Review Request: libsnack - Snack Sound Toolkit provides audio handling to tlc/tk or python 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://fedora.hoentjen.eu/libsnack/libsnack.spec SRPM URL: http://fedora.hoentjen.eu/libsnack/libsnack-2.2.10-1.src.rpm Description: The Snack Sound Toolkit is designed to be used with a scripting language such as Tcl/Tk or Python. Using Snack you can create powerful multi-platform audio applications with just a few lines of code. Snack has commands for basic sound handling, e.g. sound card and disk I/O. Snack also has primitives for sound visualization, e.g. waveforms and spectrograms. It was developed mainly to handle digital recordings of speech, but is just as useful for general audio. Snack has also successfully been applied to other one-dimensional signals. This packages includes Ogg library. -- 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 199681] Review Request: slab
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: slab https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=199681 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 09:42 EST --- Chris, are you still up to do the review ? A quick note first: in the devel package section: those should be Requires:, not BuildRequires:, and i don't think automake should be in there. -- 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 222175] Review Request: six - Hex playing program
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: six - Hex playing program https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222175 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 222534] Review Request: libsnack - Snack Sound Toolkit provides audio handling to tlc/tk or python
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: libsnack - Snack Sound Toolkit provides audio handling to tlc/tk or python https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222534 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 10:57 EST --- added dist tag Spec URL: http://fedora.hoentjen.eu/libsnack/libsnack.spec SRPM URL: http://fedora.hoentjen.eu/libsnack/libsnack-2.2.10-2.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 222456] Review Request: bibletime - BibleTime is a frontend for the SWORD Bible Framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: bibletime - BibleTime is a frontend for the SWORD Bible Framework https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222456 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 11:19 EST --- New versions: Spec URL: http://david.dw-perspective.org.uk/tmp/bibletime.spec SRPM URL: http://david.dw-perspective.org.uk/tmp/bibletime-1.6.2-2.src.rpm Thanks for the review. I adopted all your suggestions. With the find_lang... I think again this is probably a no-op, as I believe that the i18n stuff is in a separate tarball and not in the main BT distribution. I was thinking of adding this at a later stage once I've got it into Extras. Almost there - I hope! -- 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 219873] Review Request: kio_sword - lightweight Sword front-end for KDE
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: kio_sword - lightweight Sword front-end for KDE https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=219873 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 11:19 EST --- Thanks again, Deji. Patch added. New files: Spec URL: http://david.dw-perspective.org.uk/tmp/kio_sword.spec SRPM URL: http://david.dw-perspective.org.uk/tmp/kio_sword-0.3-3.src.rpm Personally I'm not sure of whether silencing trivial rpmlint warnings is any better than cluttering up the specfile, but seeing as you've helped me so much I'll go with you! -- 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 220703] Review Request: libgtksourceviewmm - A C++ wrapper for the gtksourceview widget 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: libgtksourceviewmm - A C++ wrapper for the gtksourceview widget library https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220703 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|163778 |163779 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 11:36 EST --- Okay. All issues are now fixed correctly. One issue. * Timestamps - -devel package contains many header files and keeping timestamps on these files are recommended. Please use --- make INSTALL=install -c -p DESTDIR=$RPM_BUILD_ROOT install Other things are okay. - This package (libgtksoureviewmm) is APPROVED by me. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 221698] Review Request: compat-guile-16 - Guile 1.6 compatibility package
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: compat-guile-16 - Guile 1.6 compatibility package https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221698 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 222534] Review Request: libsnack - Snack Sound Toolkit provides audio handling to tlc/tk or python
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: libsnack - Snack Sound Toolkit provides audio handling to tlc/tk or python https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222534 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 11:54 EST --- * doesn't build with RPM %optflags * Python docs are included, python module is NOT. Package description also mentions Python support. * snacksphere docs are included, snacksphere is NOT. Trying to load it fails. pkgIndex also lists snacksphere, although it's missing. * licence is not included * the make test suite fails here * Tcl SIG - What are the usual naming guidelines for this? The snack - libsnack name is questionable. -- 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 219873] Review Request: kio_sword - lightweight Sword front-end for KDE
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: kio_sword - lightweight Sword front-end for KDE https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=219873 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|163778 |163779 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 11:54 EST --- (In reply to comment #14) Personally I'm not sure of whether silencing trivial rpmlint warnings is any better than cluttering up the specfile, I agree with you, but the packaging guildeline says the rpmlint must be satisfied as much as possible, for thus it becometh us to fulfill all righteousness ;). GOOD: * rmplint silent mock built binary * rpmlint warning on srpm can be ignored; (W: kio_sword dangling-relative-symlink /usr/share/doc/HTML/en/kio_sword/common ../common) * source tarball matches upstream's md5sum: 957c563737d47900f67661086732ee12 kio_sword-0.3.tar.gz * package name meets the Package Naming Guidelines. * Satifies the Packaging Guidelines. * Builds in mock (x86_64 development) * Installs OK * License: GPL * Spec file generally OK. It includes a .la, but that's OK for kde packages for now (coments #6 #7) 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 222220] Review Request: polyester - KDE style and window decoration
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: polyester - KDE style and window decoration https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=20 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 11:57 EST --- I 've just rebuilt the release 2 on a x86_64 rpmlint is clean... There is no rpath issues I should have used theses solutions: http://fedoraproject.org/wiki/Extras/Schedule/RpathCheckBuildsys But anyway this one needs also to be documented: # Don't use rpath! sed -i 's|^hardcode_libdir_flag_spec=.*|hardcode_libdir_flag_spec=|g' libtool sed -i 's|^runpath_var=LD_RUN_PATH|runpath_var=DIE_RPATH_DIE|g' libtool -- 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 222220] Review Request: polyester - KDE style and window decoration
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: polyester - KDE style and window decoration https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=20 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 12:01 EST --- http://kwizart.free.fr/fedora/6/testing/polyester/ -- 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 222456] Review Request: bibletime - BibleTime is a frontend for the SWORD Bible Framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: bibletime - BibleTime is a frontend for the SWORD Bible Framework https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222456 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|163778 |163779 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 12:25 EST --- (In reply to comment #4) With the find_lang... I think again this is probably a no-op, as I believe that the i18n stuff is in a separate tarball and not in the main BT distribution. Now thinking about it again, I believe you're right. Considering that the review guildelines says locale files must not be explicitly listed, and since it doesn't hurt, I think it can stay. Almost there - I hope! Yes. GOOD: * rmplint silent on mock srpm * rpmlint warning on mock built binary can be ignored (targets really exist); (W: bibletime dangling-relative-symlink /usr/share/doc/HTML/en/bibletime/handbook/common ../../common W: bibletime devel-file-in-non-devel-package /usr/include/bibletimeinterface.h - A separate -devel subpackage is not needed for this, I believe. Moreso BT doesn't ship any library W: bibletime dangling-relative-symlink) /usr/share/doc/HTML/en/bibletime/howto/common ../../common * source tarball matches upstream's md5sum: b2b8b624d21d397201aec742d43501e5 bibletime-1.6.2.tar.bz2 * package name meets the Package Naming Guidelines. * Satifies the Packaging Guidelines. * Builds in mock (x86_64 development) * Installs and works OK (for me at least) * License: GPL * Spec file generally OK. APPROVED. It'll be nice to include the 'Changelog' with the README and License in the %doc section before/after you eventually import it into CVS. Also please don't forget the bibletime-i18n you promised. -- 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 222456] Review Request: bibletime - BibleTime is a frontend for the SWORD Bible Framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: bibletime - BibleTime is a frontend for the SWORD Bible Framework https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222456 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 12:40 EST --- Thanks! Imported into CVS. I dislike changelogs in packages - useless bloat, IMO. People who want them know where to get them. I have 31Mb on my Fedora installation and never wanted any of them! (I once filed a bug against wine-core to get 8Mb of them removed from the package). The gcc rpm ships with 10 years of changelogs - I have no idea why! -- 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 222220] Review Request: polyester - KDE style and window decoration
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: polyester - KDE style and window decoration
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=20
--- Additional Comments From [EMAIL PROTECTED] 2007-01-13 12:45 EST ---
Thanks Nicolas Chauvet,
MUST Items:
- MUST: rpmlint's output is clean
- MUST: The package is named according to the Package Naming Guidelines.
- MUST: The spec file name matches the base package %{name}
- MUST: The package meets the Packaging Guidelines.
- MUST: The package is licensed (GPL) with an open-source compatible license
and meet other legal requirements as defined in the legal section of Packaging
Guidelines.
- MUST: The License field in the package spec file matches the actual license.
- MUST: the source package includes the text of the license(s) in its own file,
then that file, containing the text of the license(s) for the package is
included in %doc.
- MUST: The spec file must be written in American English.
- MUST: The spec file for the package is be legible.
- MUST: The sources used to build the package must matches the upstream source,
as provided in the spec URL.
- MUST: The package successfully compiles and builds into binary rpms on at
least i386.
- MUST: All build dependencies is listed in BuildRequires.
- MUST: If the package does not contain shared library files located in the
dynamic linker's default paths
- MUST: the package is not designed to be relocatable
- MUST: the package owns all directories that it creates.
- MUST: the package does not contain any duplicate files in the %files listing.
- MUST: Permissions on files are set properly.
- MUST: The package has a %clean section, which contains rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
- MUST: The package consistently uses macros, as described in the macros section
of Packaging Guidelines.
- MUST: The package contains code, or permissable content. This is described in
detail in the code vs. content section of Packaging Guidelines.
- MUST: There are no Large documentation files
- MUST: %doc does not affect the runtime of the application. To summarize: If it
is in %doc, the program must run properly if it is not present.
- MUST: There are no Header files or static libraries
- MUST: The package does not contain library files with a suffix
- MUST: Package does not own files or directories already owned by other
packages.
SHOULD Items:
- SHOULD: mock builds succcessfully in i386.
- SHOULD: The reviewer tested that the package functions as described.
- SHOULD: No scriptlets were used
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 222220] Review Request: polyester - KDE style and window decoration
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: polyester - KDE style and window decoration https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=20 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|163778 |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 222087] Review Request: pcmanx-gtk2 - Telnet client designed for BBS browsing
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: pcmanx-gtk2 - Telnet client designed for BBS browsing https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222087 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 12:47 EST --- Created an attachment (id=145531) -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=145531action=view) gdb log of pcmanx Well, one issue and one _critical_ problem * Catetogory X-Fedora is not used now and please remove it. * On FC7 (rawhide 20070113), pcmanx always cause SEGV on startup. I attached a gdb log at the situation. I looked at the log, however, currently I cannot find out the cause. Would you contact with upstream? Note: On FC5, there is no segv. -- 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 222388] Review Request: gnucash - personal finance management
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: gnucash - personal finance management https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222388 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] OtherBugsDependingO|163776 |163778 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 12:58 EST --- ok, I would be happy to review this package... Look for a full review in a bit (after I can figure out why doing a mockbuild of this on my test box causes it to lockup). -- 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 222043] Review Request: gnomescan - Gnome Scanner Utility
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: gnomescan - Gnome Scanner Utility https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222043 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 219025] Review Request: ntop - A network traffic probe similar to the UNIX top command
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: ntop - A network traffic probe similar to the UNIX top command https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=219025 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 13:14 EST --- Well, I want to check CVS accroding to your comment 32 and as I can see many changes between 3.2 - cvs 07. Would you package cvs version? However, I am afraid that as long as I check the diff of util.c, the bug I pointed out on comment 30 doesn't seem to be fixed.. -- 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 222534] Review Request: libsnack - Snack Sound Toolkit provides audio handling to tlc/tk or python
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: libsnack - Snack Sound Toolkit provides audio handling to tlc/tk or python https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222534 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 13:15 EST --- (In reply to comment #3) * Tcl SIG - What are the usual naming guidelines for this? The snack - libsnack name is questionable. I don't think there is a Tcl SIG, but it might be nice to have one. 'snack' would be a more appropriate name, since that's what upstream uses, and that's what Tcl developers would likely expect it to be called. I had considered packaging snack a few months ago, but noticed that it comes with a mp3 decoder (generic/mkFormatMP3.c), which is unacceptible for Fedora. You'll have to regenerate the tarball to omit the mp3 decoding bits before it can be accepted. -- 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 222087] Review Request: pcmanx-gtk2 - Telnet client designed for BBS browsing
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: pcmanx-gtk2 - Telnet client designed for BBS browsing https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222087 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 13:20 EST --- (In reply to comment #7) Created an attachment (id=145531) -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=145531action=view) [edit] gdb log of pcmanx Well, one issue and one _critical_ problem * Catetogory X-Fedora is not used now and please remove it. * On FC7 (rawhide 20070113), pcmanx always cause SEGV on startup. I attached a gdb log at the situation. I looked at the log, however, currently I cannot find out the cause. Would you contact with upstream? Note: On FC5, there is no segv. Email sent out. I will put out a new src rpm once the seq fault fixed along with fixing the category problem. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 222534] Review Request: libsnack - Snack Sound Toolkit provides audio handling to tlc/tk or python
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: libsnack - Snack Sound Toolkit provides audio handling to tlc/tk or python https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222534 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 13:28 EST --- Yes, browsing the rather large tarball is another thing to do. I only had a first brief look because the package description as quoted at the top of the ticket says This packages includes Ogg library. which is confusing enough to suggest that possibly it ships an included libogg or something like that. That made it worth taking a look. The description (s/packages/package/) refers to libsnackogg.so which wasn't obvious. ;) -- 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 218844] Review Request: python-yadis - Relying party support for the Yadis service discovery protocol
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-yadis - Relying party support for the Yadis service discovery protocol https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=218844 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 13:36 EST --- Spec URL: http://repo.ocjtech.us/misc/fedora/6/SRPMS/python-yadis-1.1.0-1.fc6.spec SRPM URL: http://repo.ocjtech.us/misc/fedora/6/SRPMS/python-yadis-1.1.0-1.fc6.src.rpm Updated to 1.1.0 -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 222534] Review Request: libsnack - Snack Sound Toolkit provides audio handling to tlc/tk or python
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: libsnack - Snack Sound Toolkit provides audio handling to tlc/tk or python https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222534 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 13:36 EST --- This also doesn't build on x86_64. The included patch to configure fixes the search path for tclConfig.sh on 64-bit systems, but doesn't do the for tkConfig.sh. (In reply to comment #5) which is confusing enough to suggest that possibly it ships an included libogg or something like that. That made it worth taking a look. The description (s/packages/package/) refers to libsnackogg.so which wasn't obvious. ;) It does include 'SnackOgg.c', which has a suspicious header that reads: * THIS FILE IS PART OF THE OggVorbis SOFTWARE CODEC SOURCE CODE. * ...and 2000 lines of Ogg manipulation code. You might want to see if you can omit this source file (rm it during %prep) and build against libogg/libvorbis instead. -- 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 222070] Review Request: prewikka - Prelude IDS Management Console
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: prewikka - Prelude IDS Management Console
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222070
--- Additional Comments From [EMAIL PROTECTED] 2007-01-13 13:39 EST ---
And.. where is -5 srpm? -4 srpm does not have README.fedora
and I cannot create -5 srpm by rpmbuild -bs prewikka.spec
(In reply to comment #4)
when I use this:
%files
%{python_sitearch}/%{name}/
As commented in comment 3, Writing just
%files
some_directory
without marking as %dir is interpreted as the directory
some_directory and all files/directories under the
directory.
%doc AUTHORS README NEWS HACKING.README README.fedora doc/apache.conf
when written as this,
* rpmbuild makes the directory
%{buildroot}/%{_datadir}/doc/%{name}-%{version}
* rpmbuild just copies all the listed files/directories into
the directory
* rpmbuild automatically adds automatically the directory
/%{_datadir}/doc/%{name}-%{version} and all files/directories
under there into file list and mark all files as %doc.
--
sed -i -e '/^#!\/usr\/bin\/env python$/d'
%{buildroot}/%{python_sitearch}/%{name}/templates/*
---
* After these, please make sure that all files under
%{python_sitearch}/%{name}/templates/ do _not_ have
executable permission.
--
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 222534] Review Request: libsnack - Snack Sound Toolkit provides audio handling to tlc/tk or python
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: libsnack - Snack Sound Toolkit provides audio handling to tlc/tk or python https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222534 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 13:42 EST --- (In reply to comment #6) This also doesn't build on x86_64. The included patch to configure fixes the search path for tclConfig.sh on 64-bit systems, but doesn't do the for tkConfig.sh. ...it also needs 'BuildRequires: tk-devel'. -- 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 211728] Review Request: stklos - Scheme Interpreter/Compiler System
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: stklos - Scheme Interpreter/Compiler System https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=211728 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 13:47 EST --- Here is the new version, with Provides: stklos-devel: http://math.ifi.unizh.ch/fedora/6/i386/SRPMS.gemi/stklos-0.82-1.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 218018] Review Request: spampd - Transparent SMTP/LMTP proxy filter using spamassassin
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: spampd - Transparent SMTP/LMTP proxy filter using spamassassin https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=218018 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |NEEDINFO Flag||needinfo?([EMAIL PROTECTED] ||net) --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 13:55 EST --- ping -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 215569] Review Request: beryl-vidcap - Beryl OpenGL window and compositing manager video capture utility
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: beryl-vidcap - Beryl OpenGL window and compositing manager video capture utility https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215569 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 13:58 EST --- Created an attachment (id=145532) -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=145532action=view) objdump log of libseom.so.0.0.0 Well, I tried 0.1.4-1.fc7 (on FC-devel i386), however, libseom.so.0.0.0 still contains non-pic code. -- 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 222534] Review Request: libsnack - Snack Sound Toolkit provides audio handling to tlc/tk or python
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: libsnack - Snack Sound Toolkit provides audio handling to tlc/tk or python https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222534 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 13:59 EST --- hmm, sorry for the bad shape.. i actually took this from somewhere else some time ago, built it for my own use and *thought* i had checked it back then for fedora like quality. Let me work on it a bit and you will hear from me again. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 215569] Review Request: beryl-vidcap - Beryl OpenGL window and compositing manager video capture utility
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: beryl-vidcap - Beryl OpenGL window and compositing manager video capture utility https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215569 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 14:00 EST --- Created an attachment (id=145533) -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=145533action=view) Mock build log of beryl-vidcap-0.1.4-1.fc7 I attach a mockbuild log of 0.1.4-1 on FC-devel i386. Any ideas, anyone? -- 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 219930] Review Request: lxpanel - A lightweight X11 desktop panel
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: lxpanel - A lightweight X11 desktop panel https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=219930 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 14:49 EST --- (In reply to comment #10) One thine you may need to know: the lxde author says he will no more develop and maintain these programs So there is no NEXT version of lxde. That is one reason why I don't want to maintain any of lxde packages. Mhh. Not good. Didn't know that. I've written to the maintainer of lxpanel and asked him about that. After I get an answer I will decide what to do with lxpanel in extras. -- 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 222521] Review Request: IceWM - Lightweight Window Manager.
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: IceWM - Lightweight Window Manager.
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222521
[EMAIL PROTECTED] changed:
What|Removed |Added
CC||[EMAIL PROTECTED]
--- Additional Comments From [EMAIL PROTECTED] 2007-01-13 14:54 EST ---
Comments:
* for me, tabs result in bad formatting
* For man pages, I prefer something along
%{_mandir}/man1/icewm.1*
to catch different compression or no compression at all.
* the themes are not packaged. Is it on purpose?
* CXXFLAGS add -fpermissive -Wall -Wpointer-arith -Wconversion -Wwrite-strings
-W
inline -Woverloaded-virtual -W -fno-exceptions -fno-rtti
is it right?
* compilation warnings don't seem to be that harmfull to me, but may be
worth reporting upstream (some system return flag may not be checked)
* ldd -u -r on executables leads to many Unused direct dependencies. It is
not very problematic, but it leads to unusefull dependencies on other
libs sonames, which will lead you to unusefull rebuild when the soname
change. Most of the time this is caused by pkgfonfig files not using
corrrectly .private, or project not using pkgconfig it is this case for
unusefull link on X libs in icewm, since icewm don't use pkgconfig for X
libs.
Needswork:
* patch should be applied to lib/menu.in, not lib/menu
* To have the prefix set correctly, configure and configure.in
should be patched to set CONFIG_GNOME2_MENU_DIR instead of lib/menu(.in)
CONFIG_GNOME2_MENU_DIR=${GNOME2_PREFIX}/share/gnome/vfolders/
should be
CONFIG_GNOME2_MENU_DIR=${GNOME2_PREFIX}/share/gnome/desktop-directories/
Then lib/menu.in could be patched to remove
menuprog Gnome folder icewm-menu-gnome1 --list
and use icewm-menu-gnome2 for kde folder too.
* BR (BuildRequires) for kde-config is needed for CONFIG_KDE_MENU_DIR
* things in the default menu should be removed or be Requires. In my
opinion xterm could stay, but the remaining should go (especially mozilla
which isn't in fedora). Mozilla could be replaced by htmlview,
alternatively.
* files in %{datadir}/icewm are not %config files, these are defaults.
What is in --with-cfgdir are config file and there are none.
My opinion is that %configure should have
--with-cfgdir=%{_sysconfdir}/icewm otherwise /etc is hardcoded.
* %{_sysconfdir}/icewm should be created and owned by the package
* locales aren't handled right, it is covered in the guidelines
http://fedoraproject.org/wiki/Packaging/Guidelines#head-8c605ebf8330f6d505f384e671986fa99a8f72ee
* KDEDIR is taken from the environment, so
/etc/profile.d/kde.sh
should be sourced
* gettext BuildRequires is missing
* VERSION PLATFORMS in %doc seem unuseful to me
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 218844] Review Request: python-yadis - Relying party support for the Yadis service discovery protocol
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: python-yadis - Relying party support for the Yadis
service discovery protocol
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=218844
[EMAIL PROTECTED] changed:
What|Removed |Added
OtherBugsDependingO|163778 |163779
nThis||
--- Additional Comments From [EMAIL PROTECTED] 2007-01-13 15:35 EST ---
Packaging note from the README:
This is the last release of python-yadis as an independently
distributed package. Future versions will come bundled with the
python-openid library.
Review:
+ License (LGPL)
+ noarch package, installs to %{python_sitelib}
+ follows python template
+ source matches upstream
+ %check passed
+ use of macros
+ follows naming guidelines
+ rpmlint clean
+ builds on x86_64
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 222388] Review Request: gnucash - personal finance management
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: gnucash - personal finance management
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222388
--- Additional Comments From [EMAIL PROTECTED] 2007-01-13 16:08 EST ---
OK - Package meets naming and packaging guidelines
OK - Spec file matches base package name.
OK - Spec has consistant macro usage.
OK - Meets Packaging Guidelines.
OK - License (GPL/GFDL)
OK - License field in spec matches
OK - License file included in package
OK - Spec in American English
OK - Spec is legible.
OK - Sources match upstream md5sum:
5755b05a3eaebab392fe9ad49073beb2 gnucash-2.0.4.tar.bz2
5755b05a3eaebab392fe9ad49073beb2 gnucash-2.0.4.tar.bz2.1
ffc058efd0283a4b43ca31980c40db49 gnucash-docs-2.0.1.tar.bz2
ffc058efd0283a4b43ca31980c40db49 gnucash-docs-2.0.1.tar.bz2.1
afa10712d00b6a90aef0dc7fbb116ff30ded91cb gnucash-2.0.4.tar.bz2
afa10712d00b6a90aef0dc7fbb116ff30ded91cb gnucash-2.0.4.tar.bz2.1
ce04f51e8eeb8324b7abca6bf84ddb18562cf6b4 gnucash-docs-2.0.1.tar.bz2
ce04f51e8eeb8324b7abca6bf84ddb18562cf6b4 gnucash-docs-2.0.1.tar.bz2.1
See below - BuildRequires correct
OK - Spec handles locales/find_lang
See below - Package has %defattr and permissions on files is good.
OK - Package has a correct %clean section.
OK - Package has correct buildroot
OK - Package is code or permissible content.
OK - Packages %doc files don't affect runtime.
OK - Package is a GUI app and has a .desktop file
OK - Package compiles and builds on at least one arch.
OK - Package has no duplicate files in %files.
OK - Package doesn't own any directories other packages own.
OK - Package owns all the directories it creates.
See below - No rpmlint output.
OK - final provides and requires are sane
SHOULD Items:
See below - Should build in mock.
OK - Should build on all supported archs
OK - Should have dist tag
OK - Should package latest version
Issues:
1. Is this package built often from svn snapshots?
ie, are the %if's for svn building needed anymore?
2. Is the
%defattr(-,root,root,755)
needed? Or will
%defattr(-,root,root,-)
work?
3. Does rpm fail at finding the perl requires?
Is the '%define __perl_requires %{nil}' still needed?
4. Doesn't seem to build here in mock/devel.
The build.log has at the end:
checking for libgsf-1 = 1.12.2 libgsf-gnome-1 = 1.12.2... Package libgsf-
gnome-1 was not found in the pkg-config search path. Perhaps you should add the
directory containing `libgsf-gnome-1.pc' to the PKG_CONFIG_PATH environment
variable No package 'libgsf-gnome-1' found
configure: error: Library requirements (libgsf-1 = 1.12.2 libgsf-gnome-1 =
1.12.2) not met; consider adjusting the PKG_CONFIG_PATH environment variable if
your libraries are in a nonstandard prefix so pkg-config can find them.
error: Bad exit status from /var/tmp/rpm-tmp.55382 (%build)
Looks like missing 'Buildrequires: libgsf-gnome-devel'
Adding that gets it building on devel here.
5. The --disable-sql seems to have been added back in fc4.
Is it worth re-enabling now?
6. Should the --with-cairo be commented in or out?
7. Our friend rpmlint says:
E: gnucash obsolete-not-provided gnucash-backend-postgres
I don't know how long ago the gnucash-backend-postgres was removed,
but it might be good to provide gnucash-backend-postgres as long
as the obsolete is still there.
E: gnucash invalid-soname /usr/lib/libgncqof-backend-qsf.so libgncqof-backend-
qsf.so
E: gnucash invalid-soname /usr/lib/libgnc-backend-file.so libgnc-backend-file.so
Can be ignored. I think rpmlint can't handle things with - in the filename
when it's not a major version number.
W: gnucash non-conffile-in-etc /etc/gconf/schemas/apps_gnucash_warnings.schemas
W: gnucash non-conffile-in-etc /etc/gconf/schemas/apps_gnucash_history.schemas
W: gnucash non-conffile-in-etc /etc/gconf/schemas/
apps_gnucash_dialog_prices.schemas
W: gnucash non-conffile-in-etc /etc/gconf/schemas/
apps_gnucash_window_pages_register.schemas
W: gnucash non-conffile-in-etc /etc/gconf/schemas/
apps_gnucash_dialog_reconcile.schemas
W: gnucash non-conffile-in-etc /etc/gconf/schemas/
apps_gnucash_dialog_hbci.schemas
W: gnucash non-conffile-in-etc /etc/gconf/schemas/
apps_gnucash_dialog_common.schemas
W: gnucash non-conffile-in-etc /etc/gconf/schemas/apps_gnucash_general.schemas
W: gnucash non-conffile-in-etc /etc/gconf/schemas/
apps_gnucash_dialog_business_common.schemas
W: gnucash non-conffile-in-etc /etc/gconf/schemas/
apps_gnucash_dialog_print_checks.schemas
W: gnucash non-conffile-in-etc /etc/gconf/schemas/
apps_gnucash_import_generic_matcher.schemas
W: gnucash non-conffile-in-etc /etc/gconf/schemas/
apps_gnucash_dialog_totd.schemas
W: gnucash non-conffile-in-etc /etc/gconf/schemas/
apps_gnucash_window_pages_account_tree.schemas
W: gnucash non-conffile-in-etc /etc/gconf/schemas/
apps_gnucash_dialog_scheduled_transctions.schemas
W: gnucash
[Bug 221672] Review Request: php-pear-Net-POP3 - Provides a POP3 class to access POP3 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: php-pear-Net-POP3 - Provides a POP3 class to access POP3 server Alias: pear-Net-POP3 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221672 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 16:11 EST --- (In reply to comment #1) Issues: 1. I don't think that: Requires(hint): php-pear(Net_Auth_SASL) = 1.0 is usable with any rpm shipped by Fedora. I suppose it just makes it a hard Requires. Perhaps just change that to a Requires? Also, I can't seem to find that requirement in yet, should that package be added? Even though (hint) does not do anything right now, I use this for optional packages. I added it because the Summary on the web page: ...using optional Auth_SASL package 3. The /usr/share/pear/Net directory doesn't seem to be owned by this package, it seems owned by php-pear-Net-FTP. Should that package be required? or should this package also own that directory? This package Requires php-pear(Net_Socket) = 1.0 which should own the Net directory. -- 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 221672] Review Request: php-pear-Net-POP3 - Provides a POP3 class to access POP3 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: php-pear-Net-POP3 - Provides a POP3 class to access POP3 server Alias: pear-Net-POP3 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221672 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 16:13 EST --- (In reply to comment #2) (In reply to comment #1) Issues: 1. I don't think that: Requires(hint): php-pear(Net_Auth_SASL) = 1.0 is usable with any rpm shipped by Fedora. I suppose it just makes it a hard Requires. Perhaps just change that to a Requires? Also, I can't seem to find that requirement in yet, should that package be added? Even though (hint) does not do anything right now, I use this for optional packages. I added it because the Summary on the web page: ...using optional Auth_SASL package Although this should read Auth_SASL not Net_Auth_SASL. I will fix this. Nice catch. -- 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 221672] Review Request: php-pear-Net-POP3 - Provides a POP3 class to access POP3 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: php-pear-Net-POP3 - Provides a POP3 class to access POP3 server Alias: pear-Net-POP3 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221672 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 16:16 EST --- Spec URL: http://tkmame.retrogames.com/fedora-extras/php-pear-Net-POP3.spec SRPM URL: http://tkmame.retrogames.com/fedora-extras/php-pear-Net-POP3-1.3.6-2.src.rpm %changelog * Sat Jan 13 2007 Christopher Stone [EMAIL PROTECTED] 1.3.6-2 - Fix Requires for Auth_SASL -- 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 222534] Review Request: libsnack - Snack Sound Toolkit provides audio handling to tlc/tk or python
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: libsnack - Snack Sound Toolkit provides audio handling to tlc/tk or python https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222534 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 16:25 EST --- SnackOgg.c *urks* Additionally, it links against libogg.so.0, libvorbis.so.0, and libvorbisenc.so.2,... generic/ Included are several files licenced by Microsoft with the terms in the BSD.txt file. It requires that the file is included and that a licence change (e.g. to the GPL) is indicated at the top of each file to which the new licence applies. That is not the case, hence the BSD.txt licence applies to the whole product. In the README they write that the GPL is only for the included mp3 decoder, however that one may not be sold without prior written consent by its author. Plus, other files, such as jkSynthesis.c and sound.c, have a GPL header, so the entire release must be GPL. Licence mess. -- 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 221672] Review Request: php-pear-Net-POP3 - Provides a POP3 class to access POP3 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: php-pear-Net-POP3 - Provides a POP3 class to access POP3 server Alias: pear-Net-POP3 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221672 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|163778 |163779 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 16:30 EST --- ok, that appears to fix up all the blockers I see. This package is APPROVED. Don't forget to close this review request NEXTRELEASE once it's 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 222070] Review Request: prewikka - Prelude IDS Management Console
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: prewikka - Prelude IDS Management Console https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222070 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 16:33 EST --- sorry, it agree it would be helpful to upload the package I talk about. :) find it here: http://people.redhat.com/tscherf/fedora-extra/prewikka-0.9.7.1-5.src.rpm http://people.redhat.com/tscherf/fedora-extra/prewikka.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 221672] Review Request: php-pear-Net-POP3 - Provides a POP3 class to access POP3 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: php-pear-Net-POP3 - Provides a POP3 class to access POP3 server Alias: pear-Net-POP3 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221672 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 17:27 EST --- - Imported into CVS - Entry added to owners.list - built successfully for devel - cvs sync request added for 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 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] 2007-01-13 17:39 EST --- I have uploaded a new version which adds COPYING and AUTHORS files, and adds URL to Source0 (as per review request). New SRPM: http://krakoa.dk/fedora/pdfcube-0.0.2-3.src.rpm The new spec file has the same url as the old one. -- 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 221670] Review Request: php-pear-Net-UserAgent-Detect - Extract information from an HTTP user agent
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-UserAgent-Detect - Extract information from an HTTP user agent Alias: Net-UserAgent-Detect https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221670 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 17:40 EST --- Spec URL: http://tkmame.retrogames.com/fedora-extras/php-pear-Net-UserAgent-Detect.spec SRPM URL: http://tkmame.retrogames.com/fedora-extras/php-pear-Net-UserAgent-Detect-2.2.0-2.src.rpm %changelog * Sat Jan 13 2007 Christopher Stone [EMAIL PROTECTED] 2.2.0-2 - Use PHP License version 2.02 2. This package owns the /usr/share/pear/Net directory. Should it? Or should it require some other package that does? There are many pear packages that share ownership of the same directories. The general rule of thumb in this case is that if the owner of a directory is unclear then the package should own the directory. -- 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 221045] Review Request: gtk-rezlooks-engine - Rezlooks GTK2 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: gtk-rezlooks-engine - Rezlooks GTK2 engine https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221045 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 17:41 EST --- I have uploaded a new version which adds COPYING file. New SRPM: http://krakoa.dk/fedora/gtk-rezlooks-engine-0.6-3.src.rpm The new spec file has the same url as the old one. -- 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 222547] Review Request: blobby - Blobby Volley 2, a volley-ball game
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: blobby - Blobby Volley 2, a volley-ball game https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222547 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] OtherBugsDependingO|163776 |163778 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 17:45 EST --- I'll review it. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 222547] Review Request: blobby - Blobby Volley 2, a volley-ball game
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: blobby - Blobby Volley 2, a volley-ball game
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222547
--- Additional Comments From [EMAIL PROTECTED] 2007-01-13 18:24 EST ---
REVIEW:
* sources match upstream (md5: fdef3e59f0870d890be8dffaccd773ca)
* package meets all guidelines it should
* correct build root: %{_tmppath}/%{name}-%{version}-%{release}-
root-%(%{__id_u} -n)
* package is licensed under a GPL license and license text is included in
package
* the latest version is being packaged
* no shared libraries
* no duplicates in %files
* %clean is present
* proper scriptlets
* no need to any subpackages
* rpmlint is silent
* final provides and requires are sane
!* desktop file:
Category Application is deprecated and should be removed.
https://www.redhat.com/archives/fedora-extras-list/2006-October/msg00723.html
* BRs listed good
* mock builds fine (fc6/x86_64)
* all directories are owned well
!* a lack of Requires:
you should add hicolor-icon-theme dependency:
https://www.redhat.com/archives/fedora-extras-list/2006-September/msg00282.html
THINGS to do:
- get rid of an Application category
- add hicolor-icon-theme dependency
--
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 215569] Review Request: beryl-vidcap - Beryl OpenGL window and compositing manager video capture utility
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: beryl-vidcap - Beryl OpenGL window and compositing manager video capture utility https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=215569 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 18:28 EST --- (In reply to comment #28) Well, I tried 0.1.4-1.fc7 (on FC-devel i386), however, libseom.so.0.0.0 still contains non-pic code. How do I find out which code (symbols?) is non-PIC? -- 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 199682] Review Request: dbi-link - Partial implementation of the SQL/MED portion of the SQL:2003 specification
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: dbi-link - Partial implementation of the SQL/MED portion of the SQL:2003 specification https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=199682 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 18:47 EST --- Hi, Thanks for the review. I have a newer spec file already, for 2.0beta1. I applied some fixes to it per this review. Per author, it will go stable next week. Until then we can push this. Here are the spec file and SRPM: http://developer.postgresql.org/~devrim/rpms/other/dbi-link/postgresql-dbi-link.spec http://developer.postgresql.org/~devrim/rpms/other/dbi-link/postgresql-dbi-link-2.0beta1-1.src.rpm Regards, Devrim -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 222388] Review Request: gnucash - personal finance management
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: gnucash - personal finance management https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222388 --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 18:55 EST --- FYI, I just did a test build here... the smp_flags is a no go. It blows up... scratch item 10 off. On item 2: The %defattr(-,root,root,-) seems to generate ok permissions from a quick look. -- 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 221670] Review Request: php-pear-Net-UserAgent-Detect - Extract information from an HTTP user agent
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-UserAgent-Detect - Extract information from an HTTP user agent Alias: Net-UserAgent-Detect https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221670 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|163778 |163779 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 19:15 EST --- 1. Ok, looks good now. 2. Fair enough, just wanted to make sure you were aware of the issue. I see no further blockers here, so this package is APPROVED. Don't forget to close this review request NEXTRELEASE once it's 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 199682] Review Request: postgresql-dbi-link - Partial implementation of the SQL/MED portion of the SQL:2003 specification
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: postgresql-dbi-link - Partial implementation of the SQL/MED portion of the SQL:2003 specification https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=199682 [EMAIL PROTECTED] changed: What|Removed |Added Summary|Review Request: dbi-link - |Review Request: postgresql- |Partial implementation of |dbi-link - Partial |the SQL/MED portion of the |implementation of the |SQL:2003 specification |SQL/MED portion of the ||SQL:2003 specification --- Additional Comments From [EMAIL PROTECTED] 2007-01-13 19:59 EST --- ok, I see you changed the package name, and postgresql-dbi-link might make more sense... I am changing the summary of this review to match for tracking purposes. 1. ok. dist tag now present. 2. ok. license is correctly BSD. 3. ok. The source link looks good now. 4. ok. No longer an issue with this version. 5. ok. empty build section added. 6. ok. subpackage now requires the version-release of the main package. New issue: The naming doesn't seem to match what it should be for pre-release packages. See: http://fedoraproject.org/wiki/Packaging/NamingGuidelines#head- d97a3f40b6dd9d2288206ac9bd8f1bf9b791b22a I think it should be: postgresql-dbi-link-2.0-0.1.beta1 -- 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 222552] New: Review Request: mailgraph - A RRDtool frontend for Mail statistics
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=222552 Summary: Review Request: mailgraph - A RRDtool frontend for Mail statistics 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://www.symetrix.com/~bjohnson/projects/Fedora-Extras/mailgraph.spec SRPM URL: http://www.symetrix.com/~bjohnson/projects/Fedora-Extras/mailgraph-1.12-1.fc6.src.rpm Description: Mailgraph is a very simple mail statistics RRDtool frontend for Postfix and Sendmail that produces daily, weekly, monthly and yearly graphs of received/sent and bounced/rejected mail. -- 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 222552] Review Request: mailgraph - A RRDtool frontend for Mail statistics
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: mailgraph - A RRDtool frontend for Mail statistics https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222552 [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 222555] New: Review Request: tango-icon-theme-extras - Extra icons from the Tango Project
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=222555 Summary: Review Request: tango-icon-theme-extras - Extra icons from the Tango Project 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://thecodergeek.com/downloads/fedora/tango-icon-theme-extras.spec SRPM URL: http://thecodergeek.com/downloads/fedora/tango-icon-theme-extras-0.1.0-1.src.rpm Description: Contains extra icons for from the Tango Project. Currently this includes Tango icons for iPod Digital Audio Player (DAP) devices and the Dell Pocket DJ DAP. rpmlint is silent except for one warning: W: tango-icon-theme-extras invalid-license Creative Commons Attribution Share-Alike This is a a false positive, as it is simply not in the list in rpmlint's TagsCheck.py file; and the tango-icon-theme in Extras uses this exact License tag. 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 221665] Review Request: php-pear-Numbers-Roman - Provides methods for converting to and from Roman Numerals
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-Numbers-Roman - Provides methods for converting to and from Roman Numerals Alias: pear-Numbers-Roman https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221665 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] -- 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 221670] Review Request: php-pear-Net-UserAgent-Detect - Extract information from an HTTP user agent
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-UserAgent-Detect - Extract information from an HTTP user agent Alias: Net-UserAgent-Detect https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221670 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Additional Comments From [EMAIL PROTECTED] 2007-01-14 02:36 EST --- - imported into cvs - updated owners.list - tagged and built for devel - cvs sync requested for 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 222326] Review Request: gxine - GTK frontend for the xine multimedia 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: gxine - GTK frontend for the xine multimedia library https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222326 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2007-01-14 02:38 EST --- Another pre-review (I do not have sponsoring privileges) - Source tarball downloadable and matches upstream - Builds fine under mock for fedora-6-x86_64 - Works fine (note: logo won't appear unless xine-lib-extras is installed. Would be nice to have this once Fedora's RPM supports the 'Suggest' tag) Note to intended sponsor: this is the continuation of the packaging process Martin and I did in bz #213511 -- 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 221667] Review Request: php-pear-Numbers-Words - Methods for spelling numerals in words
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-Numbers-Words - Methods for spelling numerals in words Alias: pear-Numbers-Words https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221667 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Additional Comments From [EMAIL PROTECTED] 2007-01-14 02:53 EST --- - imported into cvs - updated owners.list - tagged and built for devel - branch requested for 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
