[Bug 219972] Review Request: poker-network - A poker server, client and abstract user interface library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: poker-network - A poker server, client and abstract user interface library Alias: poker-network https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=219972 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 04:09 EST --- Is it really worth having separate selinux subpackages for poker-server and poker-bot? What's the advantage of this approach rather than a combined package that handles both? -- 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 195645] Review Request: rasqal - RDF query 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: rasqal - RDF query library https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=195645 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 06:03 EST --- pushed, closing -- 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 195647] Review Request: redland
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: redland https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=195647 Bug 195647 depends on bug 195645, which changed state. Bug 195645 Summary: Review Request: rasqal - RDF query library https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=195645 What|Old Value |New Value Resolution||NEXTRELEASE Status|ASSIGNED|CLOSED -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 224245] Review Request: squirrelmail
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: squirrelmail https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=224245 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 07:21 EST --- that's funny... I've heard very good things about http://www.roundcube.net/ 1. Licensed under GPL and bundles code incompatible with GPL. 2. implements features that are not implemented in SquirrelMail because they are unstable and require too many safety checks. Roundcube implementation of output character set does not include required safety checks. 3. Less stable than SquirrelMail. 4. Spell check depends on external third party API. 5. PHP array based translations. See horror stories of translation maintainers. 6. Heavy dependency on JavaScript. -- 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 222960] Review Request: XenMan - Graphical management tool for Xen
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: XenMan - Graphical management tool for Xen
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222960
--- Additional Comments From [EMAIL PROTECTED] 2007-01-27 07:41 EST ---
(In reply to comment #7)
maybe it would make sense to also touch an empty logfile and get it owned by
the package?
Add it in %post ands as %ghost file, what do you think about it ?
Doesn't sound good. Just create it in %install and simply add
%{_sysconfdir}/log/%{name} to %files.
--
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 225010] New: Review Request: glob2 - An innovative RTS game
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=225010 Summary: Review Request: glob2 - An innovative RTS game 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://rafalzaq.nonlogic.org/fedora/glob2/glob2.spec SRPM URL: http://rafalzaq.nonlogic.org/fedora/glob2/glob2-0.8.21-1.src.rpm Description: Globulation 2 brings a new type of gameplay to RTS games. The player chooses the number of units to assign to various tasks, and the units do their best to satisfy the requests. This allows players to manage more units and focus on strategy rather than individual unit's jobs. Globulation 2 also features AI allowing single-player games or any possible combination of human-computer teams. Also included is a scripting language for versatile gameplay or tutorials and an integrated map editor. Globulation2 can be played in single player mode, through your local network, or over the Internet with Ysagoon Online Gaming (or YOG for short). -- 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 191005] Review Request: glob2 - Realtime Strategy 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: glob2 - Realtime Strategy game https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=191005 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||DUPLICATE --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 08:07 EST --- *** This bug has been marked as a duplicate of 225010 *** -- 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 225010] Review Request: glob2 - An innovative RTS 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: glob2 - An innovative RTS game https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225010 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 08:07 EST --- *** Bug 191005 has been marked as a duplicate of this bug. *** -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 220979] Review Request: tesseract - Raw OCR Engine
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: tesseract - Raw OCR Engine https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220979 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|179259 | nThis|| --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 09:52 EST --- No, it was not intentional. New release: ChangeLog: - Update BRs - Fix x86_64 compile The patch x86_64-fix is not written by me. I found it in debian package. New URLs: http://karlik.nonlogic.org/tesseract/tesseract.spec http://karlik.nonlogic.org/tesseract/tesseract-1.02-3.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 191005] Review Request: glob2 - Realtime Strategy 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: glob2 - Realtime Strategy game https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=191005 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|163778, 177841 |201449 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 222589] Review Request: dolphin - A file manager for KDE focusing on usability
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: dolphin - A file manager for KDE focusing on usability
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222589
--- Additional Comments From [EMAIL PROTECTED] 2007-01-27 10:11 EST ---
Thanks for the review. I'll be uploading it once I'm back from holidays.
http://fedoraproject.org/wiki/Vacation
(In reply to comment #2)
APPROVED.
Any reason behind not installing translation files?
What do you mean?
In the spec file, I have already stated:
## File lists
# locale's
%find_lang %{name} || touch %{name}.lang
# HTML (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 220967] Review Request: libscigraphica - A library of gtk+ widgets for SciGraphica
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: libscigraphica - A library of gtk+ widgets for SciGraphica https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220967 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 10:31 EST --- Umm, again ping, Paul? -- 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 222571] Review Request: kalgebra - MathML-based graph calculator 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: kalgebra - MathML-based graph calculator for KDE https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222571 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 10:32 EST --- Updated: Spec URL: http://tux.u-strasbg.fr/~chit/RPMS/kalgebra.spec SRPM URL: http://tux.u-strasbg.fr/~chit/RPMS/kalgebra-0.5-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 223598] Review Request: liborigin - Library for reading OriginLab OPJ project files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: liborigin - Library for reading OriginLab OPJ project files https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=223598 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 10:59 EST --- With the $RPM_OPT_FLAGS I'm having these: g++ -c -pipe -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m32 -march=i386 -mtune=generic -fasynchronous-unwind-tables -Wall -W -O2 -fPIC OPJFile.cpp -o OPJFile.o OPJFile.cpp: In member function 'int OPJFile::Parse()': OPJFile.cpp:76: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:109: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:116: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:123: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:126: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:129: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:139: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:147: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:156: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:165: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:201: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:207: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:215: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:216: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:254: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:269: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:272: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:286: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:288: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:289: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:298: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:305: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:321: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:323: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:324: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:326: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:328: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:382: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:392: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:408: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute warn_unused_result OPJFile.cpp:430: warning: ignoring return value of 'size_t fread(void*, size_t, size_t, FILE*)', declared with attribute
[Bug 220926] Review Request: ingo - The Horde email filter manager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: ingo - The Horde email filter manager https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220926 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 11:08 EST --- Module imported, added to owners.list, devel built, and FC[56] branches requested. Thanks again 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 186452] Review Request: kdebluetooth: The KDE Bluetooth 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: kdebluetooth: The KDE Bluetooth Framework https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=186452 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 11:31 EST --- i could get a pin dialog using bluetooth-applet and indeed pin helpers have gone away. from what i can tell whats needed is to add dbus support to register and listen for pin requests. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 219972] Review Request: poker-network - A poker server, client and abstract user interface library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: poker-network - A poker server, client and abstract user interface library Alias: poker-network https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=219972 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 11:43 EST --- The selinux stuff is in a state of flux right now as wart and I implement it. I added both a poker-network and poker-bot selinux subpackage for consistency. I planned to ask wart if this was necessary, but I have not had the chance to ask him yet. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 186452] Review Request: kdebluetooth: The KDE Bluetooth 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: kdebluetooth: The KDE Bluetooth Framework https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=186452 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 12:05 EST --- Just to clarify, did you test the package from comment 29? Here's how Debian has apparently solved the pin dialog issue; doesn't look too clean to me but anyway: http://bugs.debian.org/383877 http://ftp.debian.org/debian/pool/main/k/kdebluetooth/kdebluetooth_0.99+1.0beta2-2.diff.gz -- 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 221372] Review Request: rb_libtorrent - A C++ BitTorrent library aiming to be the best alternative
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: rb_libtorrent - A C++ BitTorrent library aiming to be
the best alternative
Alias: rb_libtorrent
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221372
--- Additional Comments From [EMAIL PROTECTED] 2007-01-27 12:26 EST ---
Well, for 0.11-3:
* License:
- Well, some sources are not licensed under BSD.
A. Some files are under the Boost Software License
(perhaps okay)
[under include/libtorrent/]:
asio.hpp asio/*.hpp asio/*/*.hpp
invariant_check.cpp
B. Need some verification
[under include/libtorrent/]
utf8.hpp
--
1. The origin of this software must not be misrepresented; you must not
claim that you wrote the original software. If you use this software
in a product, an acknowledgment in the product documentation would be
appreciated but is not required.
2. Altered source versions must be plainly marked as such, and must not be
misrepresented as being the original software.
3. This notice may not be removed or altered from any source distribution.
---
* Fedora specific documentation
- Please include your name.
* Timestamps
- This package try to install many text files/image files
and in that case keeping on those files are preferable.
For this package please:
%install
rm -rf %{buildroot}
export CPPROG=%{__cp} -p
make install DESTDIR=%{buildroot} INSTALL=%{__install} -c -p
* Requires, etc
/usr/lib/pkgconfig/libtorrent.pc includes:
Libs: -L${libdir} -lz -lboost_date_time -lboost_filesystem -lboost_thread -lz
-ltorrent
* I doubt that addtional linkage is needed as
--
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 221372] Review Request: rb_libtorrent - A C++ BitTorrent library aiming to be the best alternative
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: rb_libtorrent - A C++ BitTorrent library aiming to be the best alternative Alias: rb_libtorrent https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221372 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 12:48 EST --- (In reply to comment #10) B. Need some verification [under include/libtorrent/] utf8.hpp Okay, this is zlib, free and GPL-compatible. So, this is licensed under 3 kinds of license, BSD, Boost and zlib. -- 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 221669] Review Request: Deluge - A Python BitTorrent client with support for UPnP and DHT
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: Deluge - A Python BitTorrent client with support for UPnP and DHT Alias: deluge https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221669 [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-27 12:58 EST --- ( As python-libtorrent (bug 221376) seems to be accepted soon after rb_libtorrent (bug 221372) is completed, I assign this bug to myself. ) -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 219972] Review Request: poker-network - A poker server, client and abstract user interface library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: poker-network - A poker server, client and abstract user interface library Alias: poker-network https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=219972 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 13:22 EST --- We've discussed this a bit on IRC, and I think I've found a reasonable solution. One selinux subpackage will be needed (poker-selinux), and it will contain only the selinux policy files, not the wrapper scripts. poker-selinux will not need 'Requires: poker-server'. Since the wrapper scripts are generic enough to be used when selinux is enabled or disabled, there doesn't need to be a conditional in the init script to call them if they are present. The wrapper scripts can safely move to the corresponding poker-bot and poker-server subpackages. poker-web requires the selinux policies in order to function, since it requires giving permission to httpd to connect to the pokerd_port_t ports. So poker-web will need Requires: poker-selinux. If poker-web is installed with poker-server and/or poker-bot, this will enforce selinux protection on those two packages as well. The protection for poker-bot/poker-server can be disabled, however, by using 'setsebool pokerd_disable_trans on' without affecting the selinux rules needed for poker-web. But if poker-server/poker-bot are installed without poker-web, then the selinux protection is only available if the admin installs poker-selinux manually. I'm testing out these changes now and will attach updates for the spec file and init scripts when ready. -- 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 224684] Review Request: cronolog - Web log rotation program for Apache
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: cronolog - Web log rotation program for Apache https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=224684 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 13:43 EST --- All these items addressed, thanks. ftp://ftp.tummy.com/pub/tummy/RPMS/SRPMS/cronolog-1.6.2-4.src.rpm ftp://ftp.tummy.com/pub/tummy/RPMS/SRPMS/cronolog.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 186452] Review Request: kdebluetooth: The KDE Bluetooth 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: kdebluetooth: The KDE Bluetooth Framework https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=186452 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 13:59 EST --- yes i used the package from comment 29 ill try with debian's patch -- 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 225038] Review Request: medit - Another very nice Gtk+ text 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: medit - Another very nice Gtk+ text editor https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225038 [EMAIL PROTECTED] changed: What|Removed |Added Summary|Review Request: medit - |Review Request: medit - |Another very nice Gtk+ text |Another very nice Gtk+ text |editor |editor --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 14:01 EST --- In fields %install, in your desktop-file-install line commande, --delete-original was pasted twice...? --vendor=Fedora is not required anymore, juste set --vendor=. you shoudn't past all BuildRequires on the same line. -- 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 221188] Review Request: pdns-recursor - Modern, advanced and high performance recursing/non authoritative nameserver
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: pdns-recursor - Modern, advanced and high performance recursing/non authoritative nameserver https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221188 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|163778 |163779 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 14:34 EST --- Sorry it took me so long to get back to looking at this. 1-4 all look solved. One new issue... the init file's Description doesn't look right: # description: This is a daemon which periodically checks for updates \ # and can send notifications via mail, dbus or syslog. Thats a pretty minor issue, and you should be able to change it before you import. I don't see any further issues, so this package is APPROVED. Don't forget to close this review request NEXTRELEASE once it's been imported and built. Also, consider reviewing some other waiting packages to help spread out the reviewing load. :) -- 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 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 14:52 EST --- (In reply to comment #1) - usage of service and chkconfig in %pre/%post impose Require chkconfig,service It doesn't use /sbin/service, but it does use /sbin/chkconfig so I added the proper requires. 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-2.fc6.src.rpm * Sat Jan 27 2007 Bernard Johnson [EMAIL PROTECTED] - 1.12-2 - add Requires(preun), Requires(post) for chkconfig -- 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 224684] Review Request: cronolog - Web log rotation program for Apache
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: cronolog - Web log rotation program for Apache https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=224684 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|163778 |163779 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 15:21 EST --- Looks good, 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 201502] Review Request: php-pear-PhpDocumentor
Please 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-PhpDocumentor Alias: phpDocumentor https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=201502 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 15:56 EST --- 1. Because php-pear-PhpDocumentor is a library that phpdoc uses. Think of them as curl and libcurl. 2. I don't think enabling web access by default is a good idea -- it's not a common usage, and that would make it depend on httpd, which is not necessary for most uses of phpdoc. -- 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 225038] Review Request: medit - Another very nice Gtk+ text 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: medit - Another very nice Gtk+ text editor https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225038 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 15:59 EST --- * must not include /usr/share/icons/hicolor/icon-theme.cache * missing scriptlets for gtk-update-icon-cache (hicolor icons) http://fedoraproject.org/wiki/Packaging/ScriptletSnippets * must not include /usr/share/mime/mime.cache * contents of /usr/share/mime/ and subdirs are highly questionable and need a very close 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 188542] Review Request: hylafax
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: hylafax https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=188542 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 16:10 EST --- I've updated the SPEC and the SRPM: http://osdn.dl.sourceforge.net/sourceforge/hylafax/hylafax.spec http://osdn.dl.sourceforge.net/sourceforge/hylafax/hylafax-5.0.4-1.src.rpm rpmlint gives me these warnings/errors: E: hylafax invalid-soname /usr/lib/libfaxutil.so.5.0.4 libfaxutil.so E: hylafax invalid-soname /usr/lib/libfaxserver.so.5.0.4 libfaxserver.so W: hylafax devel-file-in-non-devel-package /usr/lib/libfaxserver.so E: hylafax non-readable /var/spool/hylafax/etc/hosts.hfaxd 0600 E: hylafax executable-marked-as-config-file /etc/cron.hourly/hylafax E: hylafax subdir-in-bin /usr/sbin/faxmail/image E: hylafax script-without-shebang /usr/sbin/faxsetup.linux E: hylafax executable-marked-as-config-file /etc/cron.daily/hylafax E: hylafax subdir-in-bin /usr/sbin/faxmail/application/octet-stream E: hylafax subdir-in-bin /usr/sbin/faxmail/image/tiff E: hylafax non-standard-dir-perm /var/spool/hylafax/archive 0700 E: hylafax subdir-in-bin /usr/sbin/faxmail/application/pdf W: hylafax devel-file-in-non-devel-package /usr/lib/libfaxutil.so E: hylafax non-standard-dir-perm /var/spool/hylafax/doneq 0700 E: hylafax non-standard-dir-perm /var/spool/hylafax/sendq 0700 E: hylafax non-standard-dir-perm /var/spool/hylafax/tmp 0700 E: hylafax non-standard-dir-perm /var/spool/hylafax/docq 0700 E: hylafax subdir-in-bin /usr/sbin/faxmail/application W: hylafax non-conffile-in-etc /etc/hylafax/faxcover_example_sgi.ps E: hylafax non-standard-dir-perm /var/spool/hylafax/pollq 0700 E: hylafax executable-marked-as-config-file /etc/rc.d/init.d/hylafax Please let me know what you would like me to do from here. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 188542] Review Request: hylafax
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: hylafax https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=188542 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 16:21 EST --- Thanks Howard, I will look at the files later this weekend, I promise. There was something seriously wrong with you latest package (the ones I built and the binary you gave me), the permissions were messed up and files were not executable, the %attr statements don't work as expected. I'll need to investigate this further. Please stay tuned. -- 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 CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 16:27 EST --- I believe this fixes the earlier issues. http://fetter.org/~shackle/postgresql-dbi-link.spec http://fetter.org/~shackle/postgresql-dbi-link-2.0.0-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 221669] Review Request: Deluge - A Python BitTorrent client with support for UPnP and DHT
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: Deluge - A Python BitTorrent client with support for UPnP and DHT Alias: deluge https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221669 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 16:42 EST --- What do you mean by hanging up? Does it draw the window; but have it unresponsive to input? Does it simply not function right? I'm using FC-6 (with a lot of X-related stuff from Rawhide) and have seen no such 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 225045] New: Review Request: incron - Inotify cron system
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=225045 Summary: Review Request: incron - Inotify cron system 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://rubenkerkhof.com/packages/incron.spec SRPM URL: http://rubenkerkhof.com/packages/incron-0.4.0-1.src.rpm Description: This program is an inotify cron system. It consists of a daemon and a table manipulator. You can use it a similar way as the regular cron. The difference is that the inotify cron handles filesystem events rather than time periods. -- 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 225045] Review Request: incron - Inotify cron 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: incron - Inotify cron system https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225045 [EMAIL PROTECTED] changed: What|Removed |Added Summary|Review Request: incron -|Review Request: incron - |Inotify cron system |Inotify cron system Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] OtherBugsDependingO|163776 |163778 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 17:04 EST --- Per my comment on IRC, I would be happy to review this package... Look for a formal review later tonight. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 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 [EMAIL PROTECTED] changed: What|Removed |Added Resolution|RAWHIDE |NEXTRELEASE --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 17:09 EST --- With minor problems, but a package is already available also in FC-5 and FC-6. Thanks for your help. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 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 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] OtherBugsDependingO|163776, 177841 |163778 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 17:30 EST --- I'd be happy to review this package. Look for a full review here in a bit... -- 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 198816] Review Request: gfs-utils - global file system userland utilities
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: gfs-utils - global file system userland utilities https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=198816 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED -- 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 209511] Review Request: gnbd - global network block device
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: gnbd - global network block device https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209511 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED -- 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 221372] Review Request: rb_libtorrent - A C++ BitTorrent library aiming to be the best alternative
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: rb_libtorrent - A C++ BitTorrent library aiming to be
the best alternative
Alias: rb_libtorrent
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221372
--- Additional Comments From [EMAIL PROTECTED] 2007-01-27 17:58 EST ---
I've noted the separate licenses; but AFAICT they all are very similar to the
BSD license (mostly do what you want with the code, so long as the original
authors' copyrights are left intact and the license text is not altered)
* Fedora specific documentation
- Please include your name.
Fixed in release 4.
* Timestamps
- This package try to install many text files/image files
and in that case keeping on those files are preferable.
For this package please:
Fixed in release 4.
* Requires, etc
/usr/lib/pkgconfig/libtorrent.pc includes:
Libs: -L${libdir} -lz -lboost_date_time -lboost_filesystem -lboost_thread -lz
-ltorrent
* I doubt that addtional linkage is needed as
Ah I see. Since we're linking against the system zlib, both the the @ZLIB@ and
@LIBS@ macros in the input file are expanded to contain '-lz' in the final
libtorrent.pc file. I've fixed this in release 4.
Spec: http:/thecodergeek.com/downloads/fedora/rb_libtorrent.spec
SRPM: http:/thecodergeek.com/downloads/fedora/rb_libtorrent-0.11-4.src.rpm
Thanks for your time and comments!
--
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 221188] Review Request: pdns-recursor - Modern, advanced and high performance recursing/non authoritative nameserver
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: pdns-recursor - Modern, advanced and high performance recursing/non authoritative nameserver https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221188 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 18:03 EST --- Hi Kevin, Fixed, and build successfully on -devel. Also, consider reviewing some other waiting packages to help spread out the reviewing load. :) Working on that :-) Thanks for your 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 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 AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED] CC||[EMAIL PROTECTED] OtherBugsDependingO|163778 |163776, 177841 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 18:09 EST --- Must: - package conforms to package naming guidlines - spec file name matches basename - the package is licensed with an open source compatible license - package includes license in %doc - spec file is written in american english - spec file is legible - sources match upstream (sha1sum): 08bbe7f6e66f1b14b40ce7da8caa716b6ee122e3 mailgraph-1.12.tar.gz - package successfully built on my test box (i386) - no need for exclude arch (noarch package) - files in %doc does not affect runtime of application - does not contain a pkgconfig file - does not contain any libraries - does not have a devel package - does not contain any .la files - package does not have a gui - package does not own files/directories owned by other packages. - no libraries, no need to run ldconfig - not relocatable - package owns everything it creates - package does not duplicate files it owns - consistently uses macros - code, not content Needs work: - rpmlint errors: E: mailgraph non-standard-gid /var/cache/mailgraph apache. This can be ignored, but you should Require httpd, otherwise you can't be sure this user exists. E: mailgraph non-standard-dir-perm /var/cache/mailgraph 0775. Can't you make apache the owner of the directory? - Do you need the repotag in Release? - Please preserve timestamps when installing files (Packaging/Guidelines/Timestamps) I see you have not been sponsored yet, and I'm not a sponsor, so I'll have to put the FE-NEEDSPONSOR blocker back. -- 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 221669] Review Request: Deluge - A Python BitTorrent client with support for UPnP and DHT
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: Deluge - A Python BitTorrent client with support for UPnP and DHT Alias: deluge https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221669 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 18:41 EST --- Well, hang up means... When I launch deluge * the window is drawn, with characters on the window * the message (in comment 7) is printed out * but the window does not response to any mouse click anymore... -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 219972] Review Request: poker-network - A poker server, client and abstract user interface library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: poker-network - A poker server, client and abstract user interface library Alias: poker-network https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=219972 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 18:45 EST --- Created an attachment (id=146755) -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=146755action=view) selinux file updates The attached tarball contains the selinux policy files and the corresponding changes to the init scripts and spec file. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 216723] Review Request: libsieve - A library for parsing, sorting and filtering your mail
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: libsieve - A library for parsing, sorting and filtering your mail https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216723 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 19:12 EST --- Well, * Why do you call autoconf while there is a original configure script? * Remove static archive (*.a) * Usually the dependency for the main package should be given with full version-release dependency. * Please specify the URL of SOURCE0 * Is %repotag needed? As least this is not needed for Fedora. * And.. I don't know, however are many files under www/ or rfc/ can be ignored? -- 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 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 19:41 EST --- I could. but if I change it apache.apache 0755, then I'm pretty sure rpmlint will complain about non-standard ownership. Is there a flaw to the way that I've done it, or are you expressing a preference, or pointing me to a guideline? I think rpmlint is complaining about the 0775, which probably should be a 0755 :-) -- 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 173388] Review Request: mod_evasive - Denial of Service evasion module for Apache
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: mod_evasive - Denial of Service evasion module for
Apache
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=173388
[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-27 19:59 EST ---
Hi Konstant,
Review for release 1.10.1-1
* RPM name is OK
* Builds fine in mock
* rpmlint looks OK
* File list looks OK
* Config files of mod_evasive look OK
Needs work:
* Source 0 is not available
(http://www.nuclearelephant.com/projects/mod_evasive/
mod_evasive_1.10.1.tar.gz). The project is now at
http://www.zdziarski.com/projects/mod_evasive/
* Spec file: some paths are not replaced with RPM macros
Please replace /usr/sbin/apxs with %{_sbindir}/apxs
--
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 216723] Review Request: libsieve - A library for parsing, sorting and filtering your mail
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: libsieve - A library for parsing, sorting and filtering your mail https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216723 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 20:11 EST --- (In reply to comment #9) Well, * Why do you call autoconf while there is a original configure script? At one point I was using a newer release that did not have a configure script. I forgot to change it back. Fixed. * Remove static archive (*.a) Fixed. * Usually the dependency for the main package should be given with full version-release dependency. Fixed. * Please specify the URL of SOURCE0 Fixed. * Is %repotag needed? As least this is not needed for Fedora. No, at the moment it's for my convenience so I don't need two spec files for non-FE builds. * And.. I don't know, however are many files under www/ or rfc/ can be ignored? www files are the smarty templated files that are for the web pages hosted at sf.net. In my opinion the RFCs should not be included. rfc/README: These RFC's are copied here for convenience only! In no way does the libSieve project make any claims as to ownership, authorship or stewardship over these documents. Please do not make reference to this mini-repository in any way; rather, point to the home repository hosted by the IETF: http://www.ietf.org/internet-drafts/ Spec URL: http://www.symetrix.com/~bjohnson/projects/Fedora-Extras/libsieve.spec SRPM URL: http://www.symetrix.com/~bjohnson/projects/Fedora-Extras/libsieve-2.1.13-4.fc6.src.rpm * Sat Jan 27 2007 Bernard Johnson [EMAIL PROTECTED] - 2.1.13-4 - add fully versioned dependency on main package for -devel - remove .a library from -devel - do not call autoconf, use configure file - direct download url for source0 - remove buildrequires for autoconf and m4 -- 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 216723] Review Request: libsieve - A library for parsing, sorting and filtering your mail
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: libsieve - A library for parsing, sorting and filtering your mail https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216723 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] OtherBugsDependingO|163776 |163778 nThis|| -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 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 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 20:32 EST --- (In reply to comment #7) I could. but if I change it apache.apache 0755, then I'm pretty sure rpmlint will complain about non-standard ownership. Is there a flaw to the way that I've done it, or are you expressing a preference, or pointing me to a guideline? I think rpmlint is complaining about the 0775, which probably should be a 0755 :-) You are correct, but I think you missed my point. If you change the ownership from root.apache to apache.apache and the mode from 0775 to 0755, you trade this error message: E: mailgraph non-standard-dir-perm /var/cache/mailgraph 0775 for this error message: E: mailgraph non-standard-uid /var/cache/mailgraph apache I just checked it on my system and indeed that is what rpmlint says. -- 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 223592] Review Request: wuja - Gnome desktop applet for integration with Google calendar
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: wuja - Gnome desktop applet for integration with Google calendar https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=223592 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 21:00 EST --- According to the naming guidelines the name of the spec should be gnome-applet-wuja.spec now. I think you should replace wuja with gnome-applet-wuja now in %description, too. You can remove the 'killall -HUP gconfd-2 ...'-lines from the scriptlets, since bug #173869 is fixed. The 'Obsoletes:'-tag is only needed - if a package named wuja had already been published or - if there are 3rd party packages named wuja and we want to replace them. So for this special package you can remove it I think. -- 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 221669] Review Request: Deluge - A Python BitTorrent client with support for UPnP and DHT
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: Deluge - A Python BitTorrent client with support for UPnP and DHT Alias: deluge https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221669 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 21:30 EST --- Same thing happens on Development (x86_64) -- 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
--- Additional Comments From [EMAIL PROTECTED] 2007-01-27 21:34 EST ---
Just to fill in some additional information here:
- David is the upstream author of this package. (from comment #20).
- The Oracle (and excel) dependencies were due to examples of how to use this
package, there is no real dependency on them in the package itself.
- David is fine with Devrim maintaining the Fedora version of this package now.
- The 2.0.0 version was released upstream, including moving those test scripts
to a 'examples' directory, and having them be shipped as doc files.
That said, looking at the package in comment #20:
1. You should use the %{?dist} tag:
http://www.fedoraproject.org/wiki/Packaging/DistTag
2. You might change the
%doc %{_datadir}/%{name}/examples
to just
%doc examples
Although I guess it's just a matter of taste where the example files are...
I don't see anything else thats a blocker with the package in comment #20.
--
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 225045] Review Request: incron - Inotify cron 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: incron - Inotify cron system https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225045 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 22:25 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) 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: 61bfebf6814ee893d4ff42628a823404 incron-0.4.0.tar.bz2 61bfebf6814ee893d4ff42628a823404 incron-0.4.0.tar.bz2.1 6fbef5fdda790abbf187d3c43d79d9e85ff9a59e incron-0.4.0.tar.bz2 6fbef5fdda790abbf187d3c43d79d9e85ff9a59e incron-0.4.0.tar.bz2.1 OK - 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 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: OK - Should build in mock. OK - Should build on all supported archs See below - Should function as described. OK - Should have dist tag OK - Should package latest version Issues: 1. rpmlint says: E: incron setuid-binary /usr/bin/incrontab root 04755 E: incron non-standard-executable-perm /usr/bin/incrontab 04755 I think this is required for handling user incron's. 2. The post/preun/postun scriptlets aren't right. Installing gives you: error reading information on service incron: No such file or directory error: %post(incron-0.4.0-1.fc6.i386) scriptlet failed, exit status 1 The name of the service is 'incrond' 3. Testing on my fc6 laptop with selinux enabled shows that it works. It didn't seem to pick up my user incrontab at first, and I had to restart the daemon, but then it started working ok. Using the IN_NO_LOOP is very important ( set a monitor for create in /tmp to mail me, and it looped and sent me about 1000 emails before I stopped it. ;) The only blocker I see is fixing up the scriptlets. Fix that and I will approve the package. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 223724] Review Request: fvwm - 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: fvwm - window manager https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=223724 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 22:52 EST --- Thanks for the excellent comments! Working... -- 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 218556] Review Request: ecryptfs-utils - Linux eCryptfs utilities
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: ecryptfs-utils - Linux eCryptfs utilities https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=218556 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 23:15 EST --- In reply to comment #24: Are you using a x86_64 machine? I see the same thing on my x86_64 machine here. I'll attach the dmesg from the machine. It does sound like a locking issue. Michael: Could you post an updated package addressing the issues in comments #17 and #18? -- 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 218556] Review Request: ecryptfs-utils - Linux eCryptfs utilities
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: ecryptfs-utils - Linux eCryptfs utilities https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=218556 --- Additional Comments From [EMAIL PROTECTED] 2007-01-27 23:17 EST --- Created an attachment (id=146760) -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=146760action=view) backtrace from dmesg on x86_64 machine -- 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 216723] Review Request: libsieve - A library for parsing, sorting and filtering your mail
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: libsieve - A library for parsing, sorting and
filtering your mail
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216723
--- Additional Comments From [EMAIL PROTECTED] 2007-01-28 00:13 EST ---
Well,
* License
- Some sources are not licensed under LGPL.
+ GPL
sv_util/*print*.[hc]
sv_util/xsize.h
As GPL is more strict than LGPL, please modify
the whole license of this package to GPL
(not a blocker, however, would you tell upstream
about this? License text is LGPL, however this
package is surely GPL)
* Timestamps
- This package installs several header files, and
in that case keeping timestamps on these files
are preferred. For this package,
--
make install DESTDIR=$RPM_BUILD_ROOT INSTALL=%{__install} -c -p
--
will keep timestamps.
* Source:
- It seems that the newest is 2.2.4.
--
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 221004] Review Request: jeta - Horde SSH Java Applet
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: jeta - Horde SSH Java Applet
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221004
[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-28 02:14 EST ---
I promise not to take so long with this one.
First off, the release should be 0.1.rc2%{?dist}; you can append .1 once the
package is in the repo and you need to tweak a non-development branch without
rebuilding any of the newer branches.
About registry.php, what stops us from just shipping the horde package with jeta
enabled in registry.php? I don't think it's worth it to try and change it; that
can get rather complicated and it's best to avoid that kind of thing unless it's
really trivial to do, and even then it's still usually better to just leave it
up to the admin.
I guess the fundamental question is why upstream doesn't enable jeta by default.
I suppose it's because it hasn't been released yet. Or do things break if
status is set to active when the application isn't installed?
--
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
