[Bug 209906] Review Request: elektra - A key/value pair database to store software configurations
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: elektra - A key/value pair database to store software configurations https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209906 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 03:57 EST --- (In reply to comment #8) About the source script error: the /etc/profiles.d/ directory i have do not contain any file (which are sh or csh - so executable by definition) which do not have executable bit set... (if you have some example i will check) They are wrong. These are core packages that were not reviewed until recently and many had packaging issues, this will hopefully be solved soon. (you could even report those issues on the merge review tickets, though somebody already did it unless I'm wrong). I've checked for examples in glib2 or krb5 but non of theses file remove it... they do not use %config for one or %config(noreplace) for the other. I think this file isn't aimed to be checked by used to define their environnement but use the environnement that conform to the package version... (i will check by use) Anyway The packaging guideline should state about this question... It is a question that isn't completely agreed. My personal opinion is that scripts in /etc/init.d and /etc/profile.d shouldn't be %config at all. If I recall well, previously I did differently - and I accept other contributors view. I plan to release cinepaint which requires oyranos which requires elektra (do you know kafka?!) So hope it will help to do so. I will use it as a base for I think that you could submit them right now, and have oyranos depend on that ticket oyranos packaging i also hope it will help to solve differents question open here and to know how to fix them by the examples... Requires(postun): /sbin/ldconfig isn't needed, with %postun -p the dependency is added automatically. Otherwise - I'd like to keep the statically compiled executable, but it could be done later when elektra is used in such a way that it needs to be there for recovery. What I'd like is to have the library static package be called static-devel (or devel-static) such that the static name is kept for later use for executables. - your way to package doc is certainly better than mine because it will certainly work better with --short-circuit - about putting things not in /bin and /sbin, it's fine for me, also could be reverted if elektra is needed for recovery - Putting the include file in elektra/ subdirectory implies a change in the .pc file(s) I still haven't ssen the resulting names, to see if it isn't too much transformation and if it works. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 209906] Review Request: elektra - A key/value pair database to store software configurations
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: elektra - A key/value pair database to store software
configurations
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209906
--- Additional Comments From [EMAIL PROTECTED] 2007-02-21 04:11 EST ---
I sense some progress ;)
Some remarks at random items:
* Append --includedir=%{_includedir}/elektra to %configure, and all the include
file handling magic in your current spec is superfluous.
* remove --libdir=%{_libdir} from %configure.
It's redundant
* *-devel must own
%{_includedir}/elektra
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225694] Merge Review: dictd
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: dictd https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225694 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 04:56 EST --- dictd-1.9.15-9 has the most common review issues 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 226535] Merge Review: w3m
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: w3m https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226535 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 04:59 EST --- Bug #222914 is now Fixed in w3m-0.5.1-16.fc7. Thanks for your feedback. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226407] Merge Review: sendmail
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: sendmail https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226407 [EMAIL PROTECTED] changed: What|Removed |Added Flag|needinfo?([EMAIL PROTECTED]|needinfo?([EMAIL PROTECTED] |ct.org) |m) --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 05:53 EST --- Grr, reporter is nobody; whose bright idea was that? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226671] Merge Review: zlib
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: zlib https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226671 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 07:12 EST --- Fixed in zlib-1.2.3-7.fc7. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226448] Merge Review: sysklogd
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: sysklogd https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226448 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 07:15 EST --- 2) We can do nothing. I don't want to use upstream sources, because upstream is dead and some other distros are using our sources. I'd like to rather merge patches with RH sources, but need to be sure the sysklogd is stable as much as possible. I can remove URL(it doesn't point to our sources), if you don't like it. 5) There was a dist tag in spec file, wasn't 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 228707] Review Request: KoboDeluxe - 3'rd person scrolling 2D shooter
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: KoboDeluxe - 3'rd person scrolling 2D shooter
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=228707
--- Additional Comments From [EMAIL PROTECTED] 2007-02-21 07:26 EST ---
(In reply to comment #4)
By the way..
Does this game try to create some files under
%{_var}/games/kobo-deluxe (e.g. some score files)?
If so, there is a possibility that this directory is
not deleted when trying to uninstall this package.
Yes it does create files there and yes the dir will be left around after
uninstall which is why the group isn't deleted on uninstall either.
I could write a %postun which detect if this is a true uninstall or just an
upgrade and on a true uninstall does rm -fr on the dir and then removes the
group.
(In reply to comment #5)
Well, for -0.2.pre10:
* Documentation
- Perhaps README.xkobo is missing
Your right
- One more non-UTF8 document found (note: this is not detected
by rpmlint)
ChangeLogISO-8859-1
Ok, I'll fix this and the README.xkobo as soon as its clear what todo with the
left-over highscore 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 228295] Review Request: kbilliards - A Fun Billiards Simulator 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: kbilliards - A Fun Billiards Simulator Game https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=228295 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] OtherBugsDependingO|163776 |163778 nThis|| -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 228707] Review Request: KoboDeluxe - 3'rd person scrolling 2D shooter
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: KoboDeluxe - 3'rd person scrolling 2D shooter
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=228707
--- Additional Comments From [EMAIL PROTECTED] 2007-02-21 07:38 EST ---
Well, according to your message it seems usual that
the game specific directories under %{_var}/games are
not deleted, and I don't disagree to leave the directories
undeleted.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 220736] Review Request: doclifter - lifts man pages and other troff-based markups to XML-DocBook
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: doclifter - lifts man pages and other troff-based markups to XML-DocBook https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220736 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |CLOSED Resolution||CANTFIX CC|[EMAIL PROTECTED]| OtherBugsDependingO|163776, 177841 |201449 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 07:54 EST --- https://www.redhat.com/archives/fedora-devel-list/2007-February/msg01006.html -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 228782] Review Request: tcl-html - HTML documentation for Tcl/Tk
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: tcl-html - HTML documentation for Tcl/Tk https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=228782 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |CLOSED Resolution||WONTFIX --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 08:03 EST --- These are nice arguments, but many packages have documentation included in one package. I don't want split them now. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 229477] New: Review Request: xblast-data - Data files for the game xblast
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=229477 Summary: Review Request: xblast-data - Data files for the game xblast 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://people.atrpms.net/~hdegoede/xblast-data.spec SRPM URL: http://people.atrpms.net/~hdegoede/xblast-data-2.10.0-1.fc7.src.rpm Description: This package contains the data files for XBlast, a multiplayer game where the purpose is to Blast the other players of the gamefield by laying bombs close to them. While at the same time you must avoid being blown up yourself. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 229476] Review Request: xblast - Lay bombs and Blast the other players of the field (SDL version)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: xblast - Lay bombs and Blast the other players of the field (SDL version) https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=229476 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 08:17 EST --- Notice that the review request for the required xblast-data is in bug 229477 -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 229477] Review Request: xblast-data - Data files for the game xblast
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: xblast-data - Data files for the game xblast https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=229477 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 08:18 EST --- Notice that the review request for the xblast game-engine is in bug 229476 -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 229478] New: Review Request: qdbm - Quick Database Manager
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=229478 Summary: Review Request: qdbm - Quick Database Manager 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.ioa.s.u-tokyo.ac.jp/~mtasaka/dist/extras/development/SPECS/qdbm.spec SRPM URL: http://www.ioa.s.u-tokyo.ac.jp/~mtasaka/dist/extras/development/SRPMS/qdbm-1.8.74-1.fc7.src.rpm Mockbuild log on FC7 i386: http://www.ioa.s.u-tokyo.ac.jp/~mtasaka/dist/extras/development/LOGS/MOCK-qdbm.log Description: QDBM is an embedded database library compatible with GDBM and NDBM. It features hash database and B+ tree database and is developed referring to GDBM for the purpose of the following three points: higher processing speed, smaller size of a database file, and simpler API. rpmlint on binary rpm: W: qdbm++-devel no-documentation W: qdbm-java devel-file-in-non-devel-package /usr/lib/libjqdbm.so Well, I don't know well about java package, however I think that there is no need to create another -japa-devel package for this .so file. Co-maintainers are welcome!! -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 229478] Review Request: qdbm - Quick Database 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: qdbm - Quick Database Manager https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=229478 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 08:25 EST --- *** Bug 206238 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 229319] Review Request: dekorator - KDE window decoration 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: dekorator - KDE window decoration engine https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=229319 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 09:20 EST --- New build: Spec URL: http://www.snoekie.com/rpm/dekorator.spec SRPM URL: http://www.snoekie.com/rpm/dekorator-0.3-3.src.rpm Changes: - Added Requires: kdebase as this package is useless without kwin - Install the ugly theme (moved it from %doc) - Created remove_theme_paths_button patch to remove the config dialog's set theme paths button - Created default_theme patch to set a first-time default theme - Fixed the Bushido-Yellow theme's masks to make it usable - Install the Bushido-Yellow theme by default - Added dekorator.fedora to %doc to explain why some extra themes are modified/omitted - Created config_help_tab patch to make the config dialog use HTML help (and fix some typos) Notes: (In reply to comment #6) #002: I modified the masks for the Bushido-Yellow-theme to make it work properly, and decided to include it. I did, however create the dekorator.fedora file detailing the reason for this modification and why K-Style: Infinity was left out. #005: I have decided _not_ to drop the XHTML documentation and rather modified the application to make use of it; it looks *much* better this way, especially because of the use of tables instead of dodgy ascii art. Besides, we now have the added advantage of being able to read the documentation without using kcontrol. In addition, I added the 1/2 lines that were missing from it (which were present in the text-only kcontrol module's original docs. This patch is fairly portable; if the documentation file is not found, the application will fall back to the original text... and yes, I fixed that typo :-) (#003). I also fixed a typo on the about dialog. #006: Done. The engine now loads the Default-theme by default (even when the user clicks on the Defaults button), and I've eliminated the Set theme button completely - configuring the engine now feels much more like configuring a normal kwin decoration. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225797] Merge Review: gimp-data-extras
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Merge Review: gimp-data-extras
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225797
[EMAIL PROTECTED] changed:
What|Removed |Added
AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED]
External Bug||http://bugzilla.gnome.org/sh
Reference||ow_bug.cgi?id=410387
Flag|fedora-review- |fedora-review?
--- Additional Comments From [EMAIL PROTECTED] 2007-02-21 09:20 EST ---
(In reply to comment #1)
Random first notes:
* Release (1.1.1) should be an integer. You may also add %{?dist} if you wish.
* separate the BuildRequires into two lines. It's not very obvious now.
* change BuildRoot to
%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
* remove the dot at the end of Summary.
* As /usr/bin is in the execution path, saying /usr/bin/gimptool in the
install
section is not necessary. Just use gimptool, which makes the spec more
legible.
all fixed in CVS
* Use %defattr (-, root, root, -) instead of %defattr (-, root, root)
I've used %defattr (-, root, root, 0755) . Using - as dirmode (instead of
none) doesn't make a difference, both mean keep mode of the directory as
found.
* The license filed says GPL, while there is nothing in the source tarball
that
confirms that. The COPYING file is also empty. (BLOCKER)
filed upstream at http://bugzilla.gnome.org/show_bug.cgi?id=410387
* You should ship some of the files as %doc (at least AUTHORS, ChangeLog,
NEWS,
and README).
* The spec file is not UTF-8. (BLOCKER)
* The install root is not cleaned at the beginning of %install
all fixed in CVS
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 220630] Review Request: qpidc - C++ implementation of AMQP messaging spec from Apache Qpid. Upstream for Red Hat Messaging.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: qpidc - C++ implementation of AMQP messaging spec from Apache Qpid. Upstream for Red Hat Messaging. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220630 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 09:20 EST --- Main owner is myself, fedora account aconway. cvsextras requested, mailed [EMAIL PROTECTED] directly with contact info to talk today. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225797] Merge Review: gimp-data-extras
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: gimp-data-extras https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225797 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 09:21 EST --- Added Adrian on Cc, as he is (was) upstream, at least partly. Adrian, can you shed some light on the licensing question? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225992] Merge Review: libcap
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: libcap https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225992 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 09:43 EST --- libcap-1.10-28 has some common review issues 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 226050] Merge Review: libtool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: libtool https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226050 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 10:08 EST --- libtool-1.5.22-10 prepared for 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 229490] New: Review Request: pypar2 - graphical frontend to par2cmdline
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=229490 Summary: Review Request: pypar2 - graphical frontend to par2cmdline 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://carron.maxime.free.fr/fedora/rpms/pypar2.spec SRPM URL: http://carron.maxime.free.fr/fedora/rpms/pypar2-1.2-1.fc6.src.rpm Description: A graphical frontend to par2cmdline written in Python. This is the first RPM I propose on Extras. I've been helped by the french community (a small review have already been done : http://forums.fedora-fr.org/viewtopic.php?id=18581) As it's the first time I contribute, I also need a sponsor. Thanks, Maxime -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226100] Merge Review: lksctp-tools
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: lksctp-tools https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226100 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 10:16 EST --- lksctp-tools-1.0.6-3.fc7 prepared for 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 225683] Merge Review: dev86
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: dev86 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225683 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 10:23 EST --- I patched the makefile to pass the RPM_OPT_FLAGS to the dev86 part compiled by gcc, and here we go: (seems like ncc needs additional fixing...) ncc -c -Mn -O -D__LIBC__ -D__LIBC_VER__='0.16.17' -o crt0.o crt0.c *** buffer overflow detected ***: ncc terminated === Backtrace: = /lib/libc.so.6(__chk_fail+0x41)[0x245361] /lib/libc.so.6[0x245aa8] ncc[0x8049047] ncc[0x804ae62] /lib/libc.so.6(__libc_start_main+0xdc)[0x179f2c] ncc[0x8048901] === Memory map: 00147000-0016 r-xp 03:02 3820099/lib/ld-2.5.so 0016-00161000 r-xp 00018000 03:02 3820099/lib/ld-2.5.so 00161000-00162000 rwxp 00019000 03:02 3820099/lib/ld-2.5.so 00164000-0029b000 r-xp 03:02 3820100/lib/libc-2.5.so 0029b000-0029d000 r-xp 00137000 03:02 3820100/lib/libc-2.5.so 0029d000-0029e000 rwxp 00139000 03:02 3820100/lib/libc-2.5.so 0029e000-002a1000 rwxp 0029e000 00:00 0 04225000-0423 r-xp 03:02 3819945 /lib/libgcc_s-4.1.1-20070105.so.1 0423-04231000 rwxp a000 03:02 3819945 /lib/libgcc_s-4.1.1-20070105.so.1 08048000-0804d000 r-xp 03:02 4285554 /home/jnovy/CVS/dev86/devel/dev86-0.16.17/bin/ncc 0804d000-0804e000 rwxp 4000 03:02 4285554 /home/jnovy/CVS/dev86/devel/dev86-0.16.17/bin/ncc 09b79000-09b9a000 rwxp 09b79000 00:00 0 4000-40001000 r-xp 4000 00:00 0 [vdso] 40001000-40002000 rw-p 40001000 00:00 0 4001a000-4001b000 rw-p 4001a000 00:00 0 bfe8d000-bfea2000 rw-p bfe8d000 00:00 0 [stack] make[4]: *** [crt0.o] Aborted make[4]: Leaving directory `/home/jnovy/CVS/dev86/devel/dev86-0.16.17/libc' make[3]: *** [library] Error 2 make[3]: Leaving directory `/home/jnovy/CVS/dev86/devel/dev86-0.16.17' make[2]: *** [all] Error 2 make[2]: Leaving directory `/home/jnovy/CVS/dev86/devel/dev86-0.16.17' make[1]: *** [all] Error 2 make[1]: Leaving directory `/home/jnovy/CVS/dev86/devel/dev86-0.16.17' error: Bad exit status from /var/tmp/rpm-tmp.64192 (%build) -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226314] Merge Review: prctl
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: prctl https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226314 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 10:38 EST --- prctl-1.5-2 prepared for 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 222039] Review Request: ogdi - Open Geographic Datastore Interface
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: ogdi - Open Geographic Datastore Interface https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222039 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 10:38 EST --- no 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 228707] Review Request: KoboDeluxe - 3'rd person scrolling 2D shooter
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: KoboDeluxe - 3'rd person scrolling 2D shooter https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=228707 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|163778 |163779 nThis|| Flag||fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 10:47 EST --- (In reply to comment #8) This should be the last iteration. Actually!! --- This package (KoboDeluxe) is APPROVED by me --- Well, the process of importing new packages changed. So pleace recheck from step 8 of http://fedoraproject.org/wiki/Extras/NewPackageProcess . -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 220393] Review Request: synopsis - Source-code Introspection Tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: synopsis - Source-code Introspection Tool https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220393 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|NEEDINFO Flag||needinfo?([EMAIL PROTECTED] ||ca) --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 10:49 EST --- 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 225957] Merge Review: k3b
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: k3b https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225957 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||229420 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 225957] Merge Review: k3b
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Merge Review: k3b
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225957
--- Additional Comments From [EMAIL PROTECTED] 2007-02-21 10:51 EST ---
Once merge is complete, *this* package can/should:
Obsoletes: k3b-extras %{version}-%{release}
Provides: k3b-extras = %{version}-%{release}
BuildRequires: libmpcdec-devel
BuildRequires: libsndfile-devel
and add %configure options: --with-musepack --with-sndfile
See also bug #229420
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226526] Merge Review: vim
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: vim https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226526 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 10:52 EST --- fixed in vim-7.0.195-2.fc7 -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 228425] Review Request: gtkpod - Graphical song management program for Apple's iPod
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: gtkpod - Graphical song management program for Apple's iPod Alias: gtkpod-review https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=228425 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 11:18 EST --- So the section in the guidelines on using the vendor option to desktop-file-install is incorrect? Are you sure? The section you linked to previously wasn't related to desktop-file-install but instead to Vendor: tags in the specfile header. Do you think you could help me out with a link to any discussion on where this was changed? I could easily have missed it if the packaging committee recently changed this. 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 227108] Review Request: plexus-xmlrpc-1.0-0.b4.3jpp - Plexus XML RPC Component
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: plexus-xmlrpc-1.0-0.b4.3jpp - Plexus XML RPC Component https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227108 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED] Flag||fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 11:51 EST --- OK, thanks for addressing those, I agree they're fine... I'm marking as fedora-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 227095] Review Request: plexus-appserver-1.0-0.a5.3jpp - Plexus Application 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: plexus-appserver-1.0-0.a5.3jpp - Plexus Application
Server
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227095
[EMAIL PROTECTED] changed:
What|Removed |Added
AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED]
--- Additional Comments From [EMAIL PROTECTED] 2007-02-21 11:53 EST ---
NO * rpmlint on this package.srpm gives no output
- justify warnings if you think they shouldn't be there
--
$ rpmlint plexus-appserver-1.0-0.1.a5.3jpp.1.src.rpm
W: plexus-appserver mixed-use-of-spaces-and-tabs (spaces: line 9, tab: line
91)
(minor warnings, should be fine)
Again, rpmlint does not generate this warning for me.
Btw, this section:
%if %{gcj_support}
if [ -x %{_bindir}/rebuild-gcj-db ]
then
%{_bindir}/rebuild-gcj-db
fi
%endif
seems to be repeated in the spec file (lines 177-182)
Fixed.
If there's anything else let me know. When I hear back from you, I'll build it
in mock.
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 228295] Review Request: kbilliards - A Fun Billiards Simulator 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: kbilliards - A Fun Billiards Simulator Game https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=228295 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 12:00 EST --- Well, for 0.8.7b-1: * gettext po file - Can't the gettext po files under .po/ directory be used for this program? (One possibility is that with adding gettext as BuildRequires, gettext may try to compile po files automatically, or you have to use msgfmt manually anyway). * Documentation - What is the following files? --- ./src/NOATUN_AUTHORS --- * index.docbook Usually KDE applications try to install index.docbook (here ./doc/en/index.docbook) under /usr/share/doc/HTML/. For this packages, can this be ignored? * Other miscs - What are the files under ./wafadmin 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 223588] Review Request: rudeconfig - C++ library for manipulating config 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: rudeconfig - C++ library for manipulating config files https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=223588 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 12:09 EST --- New Package CVS Request === Package Name: rudeconfig Short Description: C++ library for manipulating configuration and .ini files Owners: [EMAIL PROTECTED] Branches: FC-6 FC-5 InitialCC: [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 226053] Merge Review: libusb
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: libusb https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226053 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 12:19 EST --- 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 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|ASSIGNED|NEEDINFO Flag||needinfo?([EMAIL PROTECTED] ||.com) --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 12:32 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 225119] Review Request: pythoncad - PythonCAD scriptable CAD 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: pythoncad - PythonCAD scriptable CAD package https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225119 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|NEEDINFO Flag||needinfo?([EMAIL PROTECTED]) --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 12:35 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 209906] Review Request: elektra - A key/value pair database to store software configurations
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: elektra - A key/value pair database to store software configurations https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209906 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 12:39 EST --- SRPMS: http://kwizart.free.fr/fedora/6/testing/elektra/elektra-0.6.4-4.kwizart.fc6.src.rpm SPECS: http://kwizart.free.fr/fedora/6/testing/elektra/elektra.spec Summary: A key/value pair database to store software configurations I've added %config(noreplace) as rpmlint remain silent this way. I asked confirmation on the extras ml for this... Also used static-devel so rpmlint consider it as a devel package. I will check how it works with mock fc7 and also start a sight on oyranos... -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 222042] Review Request: GDAL - Geospatial Data Abstraction 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: GDAL - Geospatial Data Abstraction Library https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222042 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 12:53 EST --- Updated. Spec URL: http://openrisc.rdsor.ro/gdal.spec SRPM URL: http://openrisc.rdsor.ro/gdal-1.4.0-6.src.rpm More docs included, and busted out rpath forever by simply using external libtool.It builds fine in mock, rpmlint not complain. Please help with a 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 226496] Merge Review: tn5250
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: tn5250 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226496 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 12:55 EST --- I usually prefer setting all permissions in the %files section as you can't end up with some unwantend world-writable files this way. But I agree that this can only happen when you rely on 'make install' and not when you run the install binary on your own. Anyway, fixed in -11 -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 227099] Review Request: plexus-cdc-1.0-0.a4.2jpp - Plexus Component Descriptor Creator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: plexus-cdc-1.0-0.a4.2jpp - Plexus Component Descriptor Creator https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227099 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED] Flag||fedora-review- --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 12:59 EST --- Here are the links to an updated spec file and source rpm: SPEC FILE: https://mwringe.108.redhat.com/files/documents/175/234/plexus-cdc.spec SOURCE RPM: https://mwringe.108.redhat.com/files/documents/175/235/plexus-cdc-1.0-0.1.a4.2jpp.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 228894] Review Request: rpcbind - converts RPC program numbers into universal addresses
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: rpcbind - converts RPC program numbers into universal addresses https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=228894 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|NEEDINFO Flag||needinfo?([EMAIL PROTECTED]) --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 13:26 EST --- You're supposed to add in the spec comments about the changes you've applied. There are still mixed spaces/tabs, (spaces: line 3, tab: line 48) Requires: libtirpc is redundant. RPM automatically picks up and adds a requirement on libtirpc.so.1()(64bit) (for 64bit builds). You don't need to list the libtirpc Requirement. rpcbind is enabled by default, is there good reason for this? Must be asked of any service that is enabled by default. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 222042] Review Request: GDAL - Geospatial Data Abstraction 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: GDAL - Geospatial Data Abstraction Library https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222042 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 13:39 EST --- Updated. Spec URL: http://openrisc.rdsor.ro/gdal.spec SRPM URL: http://openrisc.rdsor.ro/gdal-1.4.0-7.src.rpm sorry. i forgot to require 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 226479] Merge Review: tcl
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Merge Review: tcl
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226479
--- Additional Comments From [EMAIL PROTECTED] 2007-02-21 13:40 EST ---
Finally I built tcl with
ln -s %{_datadir}/%{name}%{majorver}
%{buildroot}%{_prefix}/lib/%{name}%{majorver}
When I change it, I couldn't built it on x86_64 so I ended with this solution
for backward compatibility.
Tcl-html wasn't be separated, because:
- many packages have documentation in one package.
- the documentation isn't 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 227105] Review Request: plexus-runtime-builder-1.0-0.a9.2jpp - Plexus Component Descriptor Creator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: plexus-runtime-builder-1.0-0.a9.2jpp - Plexus Component Descriptor Creator https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227105 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 228894] Review Request: rpcbind - converts RPC program numbers into universal addresses
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: rpcbind - converts RPC program numbers into universal addresses https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=228894 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 15:04 EST --- Spec URL: http://people.redhat.com/steved/rpcbind/rpcbind.spec SRPM URL: http://people.redhat.com/steved/rpcbind/rpcbind-0.1.4-1.fc7.src.rpm have been updated! -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are 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 Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 227095] Review Request: plexus-appserver-1.0-0.a5.3jpp - Plexus Application 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: plexus-appserver-1.0-0.a5.3jpp - Plexus Application Server https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227095 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED] Flag||fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 15:58 EST --- Marking fedora-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 227047] Review Request: classworlds-1.1-0.a2.2jpp - Classworlds Classloader 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: classworlds-1.1-0.a2.2jpp - Classworlds Classloader Framework https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227047 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 16:12 EST --- New Package CVS Request === Package Name: classworlds-1.1-0.a2.2jpp Short Description: Classworlds Classloader Framework Owners: [EMAIL PROTECTED] Branches: FC-7 InitialCC: [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 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 Alias: ntop https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=219025 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 16:13 EST --- Created an attachment (id=148534) -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=148534action=view) Proposed patch to remove Apache 2 licensed material - if needed -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 227050] Review Request: dtdparser-1.21-3jpp - A Java DTD Parser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: dtdparser-1.21-3jpp - A Java DTD Parser https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227050 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 16:14 EST --- New Package CVS Request === Package Name: dtdparser-1.21-3jpp Short Description: A Java DTD Parser Owners: [EMAIL PROTECTED] Branches: FC-7 InitialCC: [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 227075] Review Request: jtidy-1.0-0.20000804r7dev.6jpp - HTML syntax checker and pretty printer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: jtidy-1.0-0.2804r7dev.6jpp - HTML syntax checker and pretty printer https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227075 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 16:17 EST --- New Package CVS Request === Package Name: jtidy-1.0-0.2804r7dev.6jpp Short Description: HTML syntax checker and pretty printer Owners: [EMAIL PROTECTED] Branches: FC-7 InitialCC: [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 227112] Review Request: relaxngDatatype-1.0-3jpp - RELAX NG Datatype API
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: relaxngDatatype-1.0-3jpp - RELAX NG Datatype API https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227112 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review- |fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 16:20 EST --- Marking fedora-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 226192] Merge Review: net-snmp
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: net-snmp https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226192 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|NEEDINFO Flag||needinfo?([EMAIL PROTECTED]) --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 16:21 EST --- By the soon to be ratified process, further work is required by the package owner, (and even though it isn't entirely logical) NEEDINFO is to point at the package owner until that work is done. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 227112] Review Request: relaxngDatatype-1.0-3jpp - RELAX NG Datatype API
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: relaxngDatatype-1.0-3jpp - RELAX NG Datatype API https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227112 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 16:24 EST --- New Package CVS Request === Package Name: relaxngDatatype-1.0-3jpp Short Description: RELAX NG Datatype API Owners: [EMAIL PROTECTED] Branches: FC-7 InitialCC: [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 227119] Review Request: ws-jaxme-0.5.1-1jpp - Open source implementation of JAXB
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: ws-jaxme-0.5.1-1jpp - Open source implementation of JAXB https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227119 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 16:25 EST --- New Package CVS Request === Package Name: ws-jaxme-0.5.1-1jpp Short Description: Open source implementation of JAXB Owners: [EMAIL PROTECTED] Branches: FC-7 InitialCC: [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 220890] Review Request: libcdaudio - Control operation of a CD-ROM when playing audio CDs
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: libcdaudio - Control operation of a CD-ROM when playing audio CDs https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220890 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 16:27 EST --- -bi --short-circuit builds work just fine for me with a mktemp -ud BuildRoot when it's specified in a specfile. Define work just fine. You do get a new buildroot with every invocation of rpmbuild, don't you? Effectively, these newly created buildroots pile up to a gigantic pile of crap. Subsequent invocations of rpmbuild do not remove the old buildroots automatically as it is done with a predictable buildroot filename. In situations when --short-circuit is really helpful, the pile of buildroots gets inconvenient to navigate in and requires work-arounds. Hence a mandatory buildroot that uses mktemp would turn against packagers and would be a bad decision. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 228894] Review Request: rpcbind - converts RPC program numbers into universal addresses
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: rpcbind - converts RPC program numbers into universal addresses https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=228894 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|163778 |163779 nThis|| Flag||fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 16:29 EST --- Looks good now. The last thing would be coordinating the Obsoletes/Provides with portmap for the removal of portmap. I'm approving this package (although I'm still not happy about the init script being marked as a config 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 227123] Review Request: xmldb-api-0.1-0.20041010.3jpp - XML:DB API for Java
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: xmldb-api-0.1-0.20041010.3jpp - XML:DB API for Java https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227123 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 16:31 EST --- New Package CVS Request === Package Name: xmldb-api-0.1-0.20041010.3jpp Short Description: XML:DB API for Java Owners: [EMAIL PROTECTED] Branches: FC-7 InitialCC: [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 227127] Review Request: xpp3-1.1.3.4-1.o.2jpp - XML Pull Parser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: xpp3-1.1.3.4-1.o.2jpp - XML Pull Parser https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227127 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 16:35 EST --- New Package CVS Request === Package Name: xpp3-1.1.3.4-1.o.2jpp Short Description: XML Pull Parser Owners: [EMAIL PROTECTED] Branches: FC-7 InitialCC: [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 220890] Review Request: libcdaudio - Control operation of a CD-ROM when playing audio CDs
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: libcdaudio - Control operation of a CD-ROM when playing audio CDs https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220890 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 16:37 EST --- (In reply to comment #16) You do get a new buildroot with every invocation of rpmbuild, don't you? No, only on rpmbuilds that go into %install. That's due to using the -u switch of mktemp. Quite clever and credits go to Ville for that. In situations when --short-circuit is really helpful, the pile of buildroots gets inconvenient Forget I mentioned --short-circuit, Ville's trick deals fine with that, and I simply trusted some mails I had read w/o checking, so --short-circuit is not an issue. The only remaining bit is running %install w/o %clean. This will indeed leave buildroots behind. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 228159] Review Request: new-stuff-manager - program that runs in the background and downloads/installs plugins
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: new-stuff-manager - program that runs in the background and downloads/installs plugins https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=228159 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 16:38 EST --- Source0 should be what you have for URL: - a full url path to the source. URL should be a url to the webpage of the project or other information on 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 197565] Review Request: buildbot
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: buildbot https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=197565 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 229250] Review Request: koji - Build system tools
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: koji - Build system tools https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=229250 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 16:46 EST --- New Package CVS Request === Package Name: koji Short Description: Build system tools Owners: [EMAIL PROTECTED],[EMAIL PROTECTED] Branches: FC-6 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 227126] Review Request: xpp2-2.1.10-6jpp - XML Pull Parser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: xpp2-2.1.10-6jpp - XML Pull Parser https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227126 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED] Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 16:49 EST --- New Package CVS Request === Package Name: xpp2 Short Description: XML Pull Parser Owners: [EMAIL PROTECTED] Branches: InitialCC: [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 229591] New: Review Request: lshw - Hardware Lister (lshw)
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=229591 Summary: Review Request: lshw - Hardware Lister (lshw) 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://ezix.org/software/files/lshw-B.02.10.tar.gz SRPM URL: http://ezix.org/software/files/lshw-B.02.10.tar.gz Description: lshw (Hardware Lister) is a small tool to provide detailed information on the hardware configuration of the machine. It can report exact memory configuration, firmware version, mainboard configuration, CPU version and speed, cache configuration, bus speed, etc. on DMI-capable x86 or EFI (IA-64) systems and on some PowerPC machines PS/ I have found that tools very useful and already available on various distribution -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 227125] Review Request: xom-1.0-3jpp - XML Pull Parser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: xom-1.0-3jpp - XML Pull Parser https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227125 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 227127] Review Request: xpp3-1.1.3.4-1.o.2jpp - XML Pull Parser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: xpp3-1.1.3.4-1.o.2jpp - XML Pull Parser https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227127 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 227123] Review Request: xmldb-api-0.1-0.20041010.3jpp - XML:DB API for Java
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: xmldb-api-0.1-0.20041010.3jpp - XML:DB API for Java https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227123 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 227119] Review Request: ws-jaxme-0.5.1-1jpp - Open source implementation of JAXB
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: ws-jaxme-0.5.1-1jpp - Open source implementation of JAXB https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227119 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 196837] Review Request: php-pear-PHPUnit - Regression testing framework for unit tests
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-PHPUnit - Regression testing framework for unit tests Alias: pear-PHPUnit3 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=196837 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 17:21 EST --- SPEC: http://tkmame.retrogames.com/fedora-extras/php-pear-PHPUnit.spec SRPM: http://tkmame.retrogames.com/fedora-extras/php-pear-PHPUnit-3.0.4-1.src.rpm %changelog * Wed Feb 21 2007 Christohper Stone [EMAIL PROTECTED] 3.0.4-1 - Upstream sync -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 227075] Review Request: jtidy-1.0-0.20000804r7dev.6jpp - HTML syntax checker and pretty printer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: jtidy-1.0-0.2804r7dev.6jpp - HTML syntax checker and pretty printer https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227075 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 227057] Review Request: gnu-regexp-1.1.4-10jpp - Java NFA regular expression engine implementation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: gnu-regexp-1.1.4-10jpp - Java NFA regular expression engine implementation https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227057 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 227047] Review Request: classworlds-1.1-0.a2.2jpp - Classworlds Classloader 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: classworlds-1.1-0.a2.2jpp - Classworlds Classloader Framework https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227047 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 227050] Review Request: dtdparser-1.21-3jpp - A Java DTD Parser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: dtdparser-1.21-3jpp - A Java DTD Parser https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=227050 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 220890] Review Request: libcdaudio - Control operation of a CD-ROM when playing audio CDs
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: libcdaudio - Control operation of a CD-ROM when playing audio CDs https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220890 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 17:27 EST --- (In reply to comment #16) You do get a new buildroot with every invocation of rpmbuild, don't you? Effectively, these newly created buildroots pile up to a gigantic pile of crap. (In reply to comment #18) Please do try the proposed buildroot with --short-circuit and prove that it doesn't do what I've written in comment 16. Option -u does not prevent mktemp from returning a new path with every invocation. Prove? It's true that it *returns* a new buildroot on every invocation, but it does not *create* it, no gigantic pile of crap is generated by merely invoking rpmbuild as you wrote. E.g. rpmbuild -bp/-bc are not creating anything. And that's why --short-circuit has nothing to do with this here, unless you would had been suggesting packages that use %buildroot in %prep and %build, which (almost) all agree to be A Bad Thing. Anyway whoever is smart enough to make diligent use of --short-circuit will be able to use his favourite external %buildroot setting as well. But for the normal non-short-circuit case there is still a small bitter pill left, that seperated %install and %clean of rpmbuilds will leave buildroots behind. Ceterum censeo clausulum definendam buildrootium coactum esse esse delendam. Ceterum censeo -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 220890] Review Request: libcdaudio - Control operation of a CD-ROM when playing audio CDs
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: libcdaudio - Control operation of a CD-ROM when playing audio CDs https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=220890 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 17:41 EST --- I still only refer to --short-circuit commands. Get back to the quote that started this: -bi --short-circuit builds work just fine for me with a mktemp -ud BuildRoot when it's specified in a specfile. Do you see? -bi --short-circuit Anyway whoever is smart enough to make diligent use of --short-circuit will be able to use his favourite external %buildroot setting as well. So, I must specify a BuildRoot in the spec only to override it in ~/.rpmmacros because it is over-engineered crap? Sad. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 221906] Review Request: gmediaserver - UPnP compatible media server for the GNU 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: gmediaserver - UPnP compatible media server for the GNU system Alias: gmediaserver https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221906 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Alias||gmediaserver Flag||fedora-review? --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 17:48 EST --- I will provide a review, but you will need to provide an init file first. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 209906] Review Request: elektra - A key/value pair database to store software configurations
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: elektra - A key/value pair database to store software
configurations
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209906
--- Additional Comments From [EMAIL PROTECTED] 2007-02-21 17:49 EST ---
(In reply to comment #11)
I've added %config(noreplace) as rpmlint remain silent this way.
It is better to be correct than shut rpmlint. %config(noreplace)
seems clearly wrong to me. %config could be acceptable, however.
* the following is unusefull:
Requires(postun): /sbin/ldconfig
* /usr/bin/elektra-kdb_static
should be removed from the main package
* /usr/lib/elektra/*.la shouldn't be shipped
* the scripts would better be in %doc than in /usr/share/elektra/scripts
in my opinion. Or some could be in %doc and others in
/usr/share/elektra/scripts
* I won't object the prefixing with elektra although I don't like
it that much. However 2 man pages became wrong:
/usr/share/man/man5/elektra-elektra.5.gz
/usr/share/man/man7/elektra-elektra.7.gz
* The *.la files should not be in elektra-static-devel
* The following is a bad idea (although it is likely that I added it):
Requires: %{_sysconfdir}/profile.d
* to have rpmbuild -bi --short-circuit working, you should add, before
mkdir __doc and mkdir __doc-devel:
rm -rf __doc __doc-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 225774] Merge Review: ftp
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: ftp https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225774 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review- |fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 17:56 EST --- Thanks. I don't see any further blockers so this package is 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 228159] Review Request: new-stuff-manager - program that runs in the background and downloads/installs plugins
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: new-stuff-manager - program that runs in the background and downloads/installs plugins https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=228159 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 18:10 EST --- Thanks for the advice. I changed it and updated files are available in the same location as above. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 229154] Review Request: konwert - Converter of character encodings
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: konwert - Converter of character encodings https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=229154 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 18:12 EST --- Hi, attempt #3 http://users.jyu.fi/~divanov/devel/konwert.spec http://users.jyu.fi/~divanov/devel/konwert-1.8-9.fc7.src.rpm Xavier, how did you get warning about the spaces/tabs? rpmlint didn't give it for me. should I use some policy? Thanks, Daniil. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 209906] Review Request: elektra - A key/value pair database to store software configurations
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: elektra - A key/value pair database to store software
configurations
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209906
--- Additional Comments From [EMAIL PROTECTED] 2007-02-21 18:16 EST ---
Ok i will fix theses...
It builds fine in mock for fc7...
http://kwizart.free.fr/fedora/6/testing/elektra/build.log
quotes:
checking dynamic linker characteristics... cat: ld.so.conf.d/*.conf: No such
file or directory (several times...)
...
Requires(postun): /sbin/ldconfig /sbin/ldconfig
yes i see, i will fix that.
About scripts, is it fine to ln -s to %{_datadir}/elektra from doc since i'm not
sure they were files in doc will have a executable bit set if needed?
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are 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 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 18:19 EST --- Thanks Chris, E: gfs-utils missing-mandatory-lsb-tag Description E: gfs-utils missing-mandatory-lsb-tag Short-Description Those errors are about the init file rpmlint checks that file for tags between ### BEGIN INIT INFO and ### END INIT INFO The checks are defined in /usr/share/rpmlint/InitScriptCheck.py A few new rpmlint warnings have popped up: [EMAIL PROTECTED] i386]$ rpmlint gfs-utils-0.1.11-3.i386.rpm W: gfs-utils dangling-relative-symlink /sbin/mount.gfs mount.gfs2 W: gfs-utils dangling-relative-symlink /sbin/umount.gfs umount.gfs2 [EMAIL PROTECTED] i386]$ rpmlint gfs-utils-debuginfo-0.1.11-3.i386.rpm W: gfs-utils-debuginfo spurious-executable-perm /usr/src/debug/gfs-0.1.11/libgfs/log.c W: gfs-utils-debuginfo spurious-executable-perm /usr/src/debug/gfs-0.1.11/libgfs/fs_dir.c W: gfs-utils-debuginfo spurious-executable-perm /usr/src/debug/gfs-0.1.11/libgfs/util.c W: gfs-utils-debuginfo spurious-executable-perm /usr/src/debug/gfs-0.1.11/libgfs/inode.c W: gfs-utils-debuginfo spurious-executable-perm /usr/src/debug/gfs-0.1.11/libgfs/fs_bits.c W: gfs-utils-debuginfo spurious-executable-perm /usr/src/debug/gfs-0.1.11/libgfs/rgrp.c W: gfs-utils-debuginfo spurious-executable-perm /usr/src/debug/gfs-0.1.11/libgfs/file.c W: gfs-utils-debuginfo spurious-executable-perm /usr/src/debug/gfs-0.1.11/libgfs/fs_bmap.c W: gfs-utils-debuginfo spurious-executable-perm /usr/src/debug/gfs-0.1.11/libgfs/bitmap.c W: gfs-utils-debuginfo spurious-executable-perm /usr/src/debug/gfs-0.1.11/libgfs/fs_inode.c W: gfs-utils-debuginfo spurious-executable-perm /usr/src/debug/gfs-0.1.11/libgfs/block_list.c W: gfs-utils-debuginfo spurious-executable-perm /usr/src/debug/gfs-0.1.11/libgfs/bio.c -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225655] Merge Review: coreutils
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Merge Review: coreutils
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225655
--- Additional Comments From [EMAIL PROTECTED] 2007-02-21 18:20 EST ---
(In reply to comment #16)
Depending on itself: Er.. can you be more specific?
The build is depending on coreutils commands, like mkdir
Are we talking about BuildRequires tags or Requires tags? Please be specific
with the changes you'd like me to make -- also it would help if you number
them. :-)
I am talking about BuildRequires, and I don't really want that you
do changes, since it seems impossible to me to remove those bootstrapping
issues. But I'd like something like your opinion on that subject.
There is still one /usr/bin:
for i in env cut; do ln -sf ../../bin/$i $RPM_BUILD_ROOT/usr/bin; done
Take a look at the wider picture there: we are making compatibility symlinks
for
binaries that used to be in /usr/bin but are now in /bin. We explicitly know
from the history of the packages where these binaries used to be. Changing a
variable is not going to change that. Changing /usr/bin here to be %{_bindir}
would be incorrect.
I wouldn't say incorrect, but not using a macro seems correct.
A suggestions: use the install-info scriptlets from the guidelines, they
are simpler. So replace, in %preun
[ -f %{_infodir}/%{name}.info.gz ] \
/sbin/install-info --delete %{_infodir}/%{name}.info* \
%{_infodir}/dir || :
with
/sbin/install-info --delete %{_infodir}/%{name}.info %{_infodir}/dir || :
and, in %post
[ -f %{_infodir}/%{name}.info* ] \
/sbin/install-info %{_infodir}/%{name}.info* %{_infodir}/dir || :
with
/sbin/install-info %{_infodir}/%{name}.info %{_infodir}/dir || :
I find it very strange to use perl to do simple substitutions when a
sed call does exactly the same.
I don't have any other remark.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225296] Merge Review: autoconf
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: autoconf https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225296 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 18:22 EST --- * there should be a comment explaining why there are the %define for the find_provides (something along # filter out bogus perl(Autom4te*) dependencies And for ./configure instead of %configure a comment is needed too. * sed in BuildRequires is unneeded * gawk doesn't seem to be needed as BuildRequires nor as Requires Suggestions: * call the dependency generator files along autoconf-filter-provides.sh * remove .gz in install-info scriptlets -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226671] Merge Review: zlib
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: zlib https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226671 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 18:30 EST --- * I don't like that much adding autotools support with a patch. Did upstream accept the patch? * zutil.h shouldn't be shipped * timestamp of .h and man files should be kept. * there is no description of the API. Please consider shipping the html pages as I suggest above (or any other description of the API). Suggestion: * in the libtool comment, replace bogus with unuseful, libtool is right in installing .la 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 209906] Review Request: elektra - A key/value pair database to store software configurations
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: elektra - A key/value pair database to store software
configurations
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209906
--- Additional Comments From [EMAIL PROTECTED] 2007-02-21 18:34 EST ---
(In reply to comment #13)
About scripts, is it fine to ln -s to %{_datadir}/elektra from doc since i'm
not
sure they were files in doc will have a executable bit set if needed?
Not really (%doc must not be runtome), but those files should be shipped
with executable bit unset if they are documentation. The user should know
how to set the exec bit or call them with an interpreter.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 229600] New: Review Request: specto - An desktop application that will watch configurable events
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=229600 Summary: Review Request: specto - An desktop application that will watch configurable events 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://blog.fedora-fr.org/public/smootherfrogz/SPECS/specto.spec SRPM URL: http://blog.fedora-fr.org/public/smootherfrogz/RPMs/specto-0.2.0-1.fc7.noarch.rpm Description: Specto is a desktop application that will watch configurable events (such as website updates, emails, file and folder changes, system processes, etc) and then trigger notifications. Specto can watch a website for updates and notify you when there is activity (otherwise, Specto will just stay out of the way). This changes the way you work, because you can be informed of events instead of having to look out for them. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226398] Merge Review: scim-sinhala
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: scim-sinhala https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226398 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-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 229391] Review Request: system-config-kdump - graphical tool for configuring kernel crash dumps
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: system-config-kdump - graphical tool for configuring kernel crash dumps https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=229391 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 18:40 EST --- This package only intends to configure crash dumping via kdump/kexec, hence the 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 229291] Review Request: thinkfinger - A driver for the UPEK/SGS Thomson Microelectronics fingerprint reader
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: thinkfinger - A driver for the UPEK/SGS Thomson Microelectronics fingerprint reader https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=229291 [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 226448] Merge Review: sysklogd
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Merge Review: sysklogd
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226448
[EMAIL PROTECTED] changed:
What|Removed |Added
Flag|fedora-review- |fedora-review+
--- Additional Comments From [EMAIL PROTECTED] 2007-02-21 18:50 EST ---
2. ok.
Checking out a copy from the above cvs, making a .tar.gz and diffing between
that version and whats shipped in your src rpm shows one diffrence...
you have a sysklogd.spec file in your tar.gz, which doesn't exist in cvs.
I assume thats generated from the sysklogd.spec.in, so it can be ignored.
Can you add a comment to the spec before the Source line indicating how someone
could check out the source from rhlinux cvs? something like:
# The source for this package was pulled from cvs.
# Use the following commands to generate the tarball:
# export CVSROOT=:pserver:[EMAIL PROTECTED]:/usr/local/CVS
# cvs login (hit return)
# cvs co sysklogd
# mv sysklogd sysklogd-%{version}rh
# tar -czvf sysklogd-%{version}rh.tar.gz sysklogd-%{version}rh
5. There is indeed a dist tag. That was an old/outdated comment.
I see no further blockers here. If you could make that comment change, we can
call this package APPROVED.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 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-02-21 18:56 EST --- ok, that version from comment #28 looks good to me, so I would APPROVE it (again). Mamoru: Could you look over the package in comment #28 and confirm that I haven't missed anything? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226448] Merge Review: sysklogd
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: sysklogd https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226448 --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 18:58 EST --- For generating the tarball, see the tag-archive/create-archive sections in the Makefile. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 228555] Review Request: Fedora Directory 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: Fedora Directory Server https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=228555 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs+ --- Additional Comments From [EMAIL PROTECTED] 2007-02-21 19:19 EST --- as this caused a problem using the old fedora-ds package provided by Directory Server Team and it only provides the base ldap server functionality. none of the admin tools It has been renamed in fedora cvs to fedora-ds-base with the intention when there is a full replacement in fedora we can have a fedora-ds metapackage that provides the full user experience -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
