[Bug 455555] New: Review Request: libhocr - A Hebrew optical character recognition library

2008-07-16 Thread bugzilla
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/show_bug.cgi?id=45

   Summary: Review Request: libhocr - A Hebrew optical character
recognition library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://oron.fedorapeople.org/libhocr/libhocr.spec
SRPM URL: http://oron.fedorapeople.org/libhocr/libhocr-0.10.11-1.fc8.src.rpm

Description:
  LibHocr is a GNU Hebrew optical character recognition library. It scans
  document images, improve the image, analyses the page layout, recognises
  the characters and outputs the text. The output texts are now editable
  text, ready for your blog, word processor or any other use.

Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=719405

$ rpmlint RPMS/i386/libhocr-*fc8*rpm
libhocr-gtk.i386: W: no-documentation
libhocr-python.i386: W: no-documentation
5 packages and 0 specfiles checked; 0 errors, 2 warnings.

Notes:
 * Base package name same as upstream. Otherwise I would have chosen
   hocr (maybe with hocr-libs subpackage).
 * Split to:
   - libhocr (libraries and the command line utility)
   - libhocr-devel
   - libhocr-python (python bindings)
   - libhocr-gtk (a GUI app and sane plugin [both python])
 * Minor patches (in SRPM) sent upstream. When they are merged, we can
   remove the autoreconf in the %pre section.
 * Lot's of compiler warnings. Will have to work with upstream developer
   to help him improve quality.
 * x86_64 build fails during install:
   - This is due to the wrong install path of python modules
   - The offending lines are in bindings/python/Makefile.am
   - In the next build I'll patch it to separate the modules
 into python_sitelib and python_sitearch.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455555] Review Request: libhocr - A Hebrew optical character recognition library

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libhocr - A Hebrew optical character recognition 
library


https://bugzilla.redhat.com/show_bug.cgi?id=45


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||177841
  nThis||




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 453016] Review Request: un-core-fonts - Korean TrueType fonts

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: un-core-fonts - Korean TrueType fonts


https://bugzilla.redhat.com/show_bug.cgi?id=453016





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 03:35 EST ---
Created an attachment (id=311926)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=311926action=view)
un-core-fonts.spec-5.patch

Here is a patch which implements the subpackaging with a macro.

I put in a couple of FIXME for the ko text in the macro - please have a look
and fix them up as appropriate. :)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 453847] Review Request: gpt - The Grid Packaging Toolkit

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: gpt - The Grid Packaging Toolkit


https://bugzilla.redhat.com/show_bug.cgi?id=453847





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 03:37 EST ---
Sorry for the delay of this iteration. Globus made a new release of the globus
toolkit, so I had to update my packages accordingly.

After communicating with the upstream GPT, I have been informed that the
maintenance of GPT has been transferred from NCSA to the Globus Alliance, so I
have based the new GPT SRPM on the latest tarfile from globus.org. (This
transfer is not indicated anywhere on the www.gridpackagingtools.org website
though.)

(In reply to comment #7)

 In fact the renaming of the package cannot be done only in fedora,
 the naming guidelines mandate that the package is called gpt in
 that case.

So the name is back to gpt, new version is here:

SRPM:
http://www.grid.tsl.uu.se/repos/globus/fedora/9/src/SRPMS/gpt-3.2-10.fc9.src.rpm
SPEC: http://www.grid.tsl.uu.se/repos/globus/fedora/9/info/gpt.spec

(In reply to comment #5)

 (It is FHS, not HFS). This patch too should be made acceptable by 
 upstream. It uses configure, so the standard paths could be passed to
 the application. They should be used unless the environment variable
 is set, in which case the environment variable should be used.

I have split the large FHS patch to several smaller ones and submitted them
upstream.

   Why do you change _ in - in script file names?
  
  The upstream code is inconsistent in its naming. It has 20 script names that
  have - and 8 that uses _, for no apparent good reason. Making the naming of 
  the
  scripts consistent is more userfriendly.
 
 Maybe, but this is something that should be changed upstream and not in 
 fedora.

OK, I have removed this.

   Reading the /usr/share/gpt/lib/perl/Grid/GPT/LocalEnv.pm
   file it seems that gtar/gzip is used, so a Requires should be needed,
   in stead of the perl(Archive::Tar) Requires. Also rpm and rpmbuild
   seems to be used so maybe some Requires are missing.
  
  I have added Requires for tar and gzip. (I did not do that originally since
  those were listed as exceptions that were not needed to be listed as build
  requirements. But thinking about it that is not quite the same thing.) The
  perl(Archive::Tar) requirement is automatically picked up by rpm, and not
  mentioned in the spec file. And I think it should be there.
 
 I don't think that perl(Archive::Tar) is used when gtar/gzip is used, so
 it shouldn't be required.

The system tar/gzip is only used for packaging/unpackaging. The Archive::Tar
perl module is still used for extracting metadata about tarfiles, like the
number of files in the archive. (Yes, I did try to filter out the requirement
and the globus build then failed).

  I did not add any requires for rpm and rpmbuild since their use inside gpt 
  is a
  rather exotic use of gpt. Using gpt to produce rpms that way will create bad
  rpms, with no proper sources.
 
 I don't think it is an issue. If it is what gpt user are used to.

I have added requires on rpm and rpm-build.

   Also gpt-bootstrap.sh seems to require all the autotools and it is not
   very clear where it is documented.
  
  Yes bootstrapping requires the autotools, but that is normally the case, so 
  I
  can't see that it requires any special documentation. Please clarify what 
  you
  meant by this comment, since I don't get what you are suggesting.
 
 If gpt-bootstrap.sh is meant to be used by gpt users, Requires in gpt
 are needed for the autotools.

I have added requires on autotools (automake, autoconf, libtool). (I hope you
won't reject the package now due to rpmlint complaining about the libtool
requirement.)

   Do you really need to rerun the autotools? It doesn't seems clear
   to me based on the patches.
  
  Patches change configure.in and Makefile.am, so yes.
 
 Ah, I missed them. Looking at them they seem simple enough that patching
 configure and Makefile.in additionally would allow not to rerun the
 autotools.

OK, The patches grew quite a bit in order to propagate the changes, but it was
not impossible, so I have done it this way in the new package.

(In reply to comment #6)

 In the next iterations, it could be nice to have in the spec file the 
 the links to the bugzilla you put in Comment #4 in comment near the 
 patch.

Due to the change of maintenance to the Globus Alliance I was asked to resubmit
the patches to the globus bugzilla by the current maintainer. I have put links
to these new bugzilla reports in the spec file.

 I have 2 additional comments regarding the changelog section
 * you should remove %{?dist} from the changelog
 * I personally prefer when there is a blank line between 2 changelog
   entries, but this is a matter of preference.

OK, done in the new package.


-- 
Configure bugmail: 

[Bug 442507] Review Request: libspe2 - SPE Runtime Management Library

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libspe2 - SPE Runtime Management Library


https://bugzilla.redhat.com/show_bug.cgi?id=442507





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 04:08 EST ---
Please change to:

 %define spuinclude %{_includedir}/spu

 %{_sysconfdir}/rc.d/init.d/ == %{_initrddir}

I have done a 32bit and 64bit test build and it looks pretty good. There are
still a few things which should be fixed:

elfspe2.ppc: W: service-default-enabled /etc/rc.d/init.d/elfspe2
elfspe2.ppc: E: incoherent-subsys /etc/rc.d/init.d/elfspe2 elfspe

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 454215] Review Request: stk - Synthesis ToolKit in C++

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: stk - Synthesis ToolKit in C++


https://bugzilla.redhat.com/show_bug.cgi?id=454215





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 04:11 EST ---
Spec URL: http://thm.fedorapeople.org/stk.spec
SRPM URL: http://thm.fedorapeople.org/stk-4.3.1-2.fc9.src.rpm

%changelog
* Tue Jul 15 2008 Thomas Moschny .. - 4.3.1-2
- Update sharedlib patch, fixes alsa problem.
- Fix path for include files in -devel.
- Change path for docs in -devel.


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 453503] Review Request:zenon - Automated theorem prover for first-order classical logic

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request:zenon - Automated theorem prover for first-order 
classical logic


https://bugzilla.redhat.com/show_bug.cgi?id=453503





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 04:13 EST ---
(In reply to comment #3)
 Since the Fedora guidelines don't say anything about where tests
 or examples go, I went and looked the Debian guidelines, figuring
 that if Debian preferred something and Fedora didn't care, it would be
 convenient to users if Debian/Ubuntu and Fedora did the same thing.

There is no specific recommendation, except that, since it is some
documentation is should be in %doc.

 These files should not be referenced
 by any program: they're there for the benefit of the system administrator and
 users as documentation only.

That's in the guideline.

 in /usr/share/doc/PACKAGENAME-VERSION/examples/
 
 Sound reasonable?

In general, the examples are indeed put in %doc examples/
or, if there are not so much files in %doc directly in %doc.
I don't think a formal guideline is needed, there are already a 
lot of guidelines. You could add it to 
http://fedoraproject.org/wiki/PackageMaintainers/PackagingTricks
however.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 454215] Review Request: stk - Synthesis ToolKit in C++

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: stk - Synthesis ToolKit in C++


https://bugzilla.redhat.com/show_bug.cgi?id=454215





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 04:15 EST ---
Here's the list of files with (at least to me) unclear legal status

 - src/include/dsound.h
 - src/include/soundcard.h
 - projects/demo/scores/*.ski
 - projects/examples/scores/*.ski
 - projects/examples/midifiles/*.mid
 - doc/html/tutorial/*.ski

It would be nice if someone could comment or help me sorting this out.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 435227] Review Request: xpsk31 - GTK+ graphical version of lpsk31

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xpsk31 - GTK+ graphical version of lpsk31


https://bugzilla.redhat.com/show_bug.cgi?id=435227


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
 Status|NEEDINFO|ASSIGNED
   Flag|needinfo?([EMAIL PROTECTED]|
   |g)  |




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 04:31 EST ---
Hello,

I'm taking over this review request because bob has some problems
I've corrected spec file and bumped version

new files:
http://lucilanga.fedorapeople.org/xpsk31.spec
http://lucilanga.fedorapeople.org/xpsk31-0.8-2.fc9.src.rpm


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455558] Review Request: udns - DNS resolver library for both synchronous and asynchronous DNS queries

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: udns - DNS resolver library for both synchronous and 
asynchronous DNS queries


https://bugzilla.redhat.com/show_bug.cgi?id=48


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-review?




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455426] Review Request: gstreamer-java - Java interface to the gstreamer framework

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: gstreamer-java - Java interface to the gstreamer 
framework


https://bugzilla.redhat.com/show_bug.cgi?id=455426





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 05:46 EST ---
final version of 0.8 released here is the new
SRPM URL:
http://www.lfarkas.org/linux/packages/fedora/9/SRPMS/gstreamer-java-0.8-1.fc9.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 442507] Review Request: libspe2 - SPE Runtime Management Library

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libspe2 - SPE Runtime Management Library


https://bugzilla.redhat.com/show_bug.cgi?id=442507





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 05:44 EST ---
Thanks for your help Adrian.

I changed what you asked for and fixed the errors and warnings. 
Furthermore I found some dependency problems which I resolved as well. 

You'll find the latest version here: 

Spec URL: ftp://testcase.software.ibm.com/fromibm/linux/libspe2.spec
SRPM URL: 
ftp://testcase.software.ibm.com/fromibm/linux/libspe2-2.2.80-95-3.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455558] Review Request: udns - DNS resolver library for both synchronous and asynchronous DNS queries

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: udns - DNS resolver library for both synchronous and 
asynchronous DNS queries


https://bugzilla.redhat.com/show_bug.cgi?id=48


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 05:50 EST ---
looks good.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 442507] Review Request: libspe2 - SPE Runtime Management Library

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libspe2 - SPE Runtime Management Library


https://bugzilla.redhat.com/show_bug.cgi?id=442507





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 06:46 EST ---
Adrian, sorry for blaiming you, but from latest /usr/lib/rpm/macros (rpm 
4.5.90):

%_initddir  %{_sysconfdir}/rc.d/init.d
# Deprecated misspelling, present for backwards compatibility.
%_initrddir %{_initddir}

So %{_initrddir} shouldn't be used. Either the other macro of what Jochen 
already 
did.

Jochen, were you able to solve incoherent-subsys or do you need help there?

Adrian, AFAIK the warning service-default-enabled can be ignored, because 
libspe2 doesn't make sense when it's not enabled, thus I would suggest to leave
it simply on per default.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 442507] Review Request: libspe2 - SPE Runtime Management Library

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libspe2 - SPE Runtime Management Library


https://bugzilla.redhat.com/show_bug.cgi?id=442507





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 07:12 EST ---
(In reply to comment #34)
 Adrian, sorry for blaiming you, but from latest /usr/lib/rpm/macros (rpm 
 4.5.90):
 
 %_initddir  %{_sysconfdir}/rc.d/init.d
 # Deprecated misspelling, present for backwards compatibility.
 %_initrddir %{_initddir}
 
 So %{_initrddir} shouldn't be used. Either the other macro of what Jochen 
 already 
 did.

I remember once we discussed this, however current Fedora packaging guidelines
suggests
to use %_initrddir :
https://fedoraproject.org/wiki/Packaging/SysVInitScript#Initscripts_on_the_filesystem

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 442507] Review Request: libspe2 - SPE Runtime Management Library

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libspe2 - SPE Runtime Management Library


https://bugzilla.redhat.com/show_bug.cgi?id=442507





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 07:19 EST ---
Very good. Another reviewer. The problem with %_initddir is that it only works
with the rpm version (4.5.90). It does not exist for F8/F9. Makes sense to use
the right one for rawhide. Jochen, you just need to remember to change it for
the other branches.

I am okay with service-default-enabled. Seems to make sense.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455575] New: Review Request: dbus-c++ - Native C++ bindings for D-Bus

2008-07-16 Thread bugzilla
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/show_bug.cgi?id=455575

   Summary: Review Request: dbus-c++ - Native C++ bindings for D-Bus
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://tgmweb.at/gadllah/dbus-c++.spec
SRPM URL: http://tgmweb.at/gadllah/dbus-c++-0.5.0-0.1.20080716git.fc9.src.rpm
Description: 
Native C++ bindings for D-Bus for use in C++ programs.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 442507] Review Request: libspe2 - SPE Runtime Management Library

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libspe2 - SPE Runtime Management Library


https://bugzilla.redhat.com/show_bug.cgi?id=442507


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Flag||fedora-review?




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455575] Review Request: dbus-c++ - Native C++ bindings for D-Bus

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: dbus-c++ - Native C++ bindings for D-Bus


https://bugzilla.redhat.com/show_bug.cgi?id=455575


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 07:46 EST ---
Can you change the snapshot generation to include last commit id? You can see
http://cvs.fedoraproject.org/viewcvs/rpms/xorg-x11-drv-nouveau/devel/xorg-x11-drv-nouveau.spec?rev=1.4view=auto
for an example.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 442507] Review Request: libspe2 - SPE Runtime Management Library

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libspe2 - SPE Runtime Management Library


https://bugzilla.redhat.com/show_bug.cgi?id=442507





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 08:13 EST ---
(In reply to comment #34)
 Jochen, were you able to solve incoherent-subsys or do you need help there?

Yes, I was able to solve this issue. Thanks. 

(In reply to comment #36)
 I am okay with service-default-enabled. Seems to make sense.

libspe2 can be used without the elfspe2 init script. But so called spulets
which are standalone spu programs can not run without this service. 
I changed the init script to be disabled per default. But as you don't mind I'll
change it back to be enabled per default. Thanks.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455581] New: Review Request: php-pecl-ssh2 - php bindings to the functions of libssh2

2008-07-16 Thread bugzilla
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/show_bug.cgi?id=455581

   Summary: Review Request: php-pecl-ssh2 - php bindings to the
functions of libssh2 
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://ispbrasil.com.br/php-pecl-ssh2/php-pecl-ssh2.spec
SRPM URL: http://ispbrasil.com.br/php-pecl-ssh2/php-pecl-ssh2-0.10-1.fc9.src.rpm
Description:
since libssh2 [1] was included in fedora, should be nice to have php-pecl-ssh2 
[2], because this will allow write php scripts with ssh2 support.

[1] - https://bugzilla.redhat.com/show_bug.cgi?id=398881
[2] - http://pecl.php.net/package/ssh2

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455500] Need OpenGTL package

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Need OpenGTL package


https://bugzilla.redhat.com/show_bug.cgi?id=455500





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 08:45 EST ---
Err, when saying that, I kinda forgot that akademy is coming up real fast, so my
free time in the short term may be insufficient to get this out until after I
get back.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455581] Review Request: php-pecl-ssh2 - php bindings to the functions of libssh2

2008-07-16 Thread bugzilla
Please 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-pecl-ssh2 - php bindings to the functions of 
libssh2


https://bugzilla.redhat.com/show_bug.cgi?id=455581


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Review Request: php-pecl-  |Review Request: php-pecl-
   |ssh2 - php bindings to the|ssh2 - php bindings to the
   |functions of libssh2   |functions of libssh2




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455575] Review Request: dbus-c++ - Native C++ bindings for D-Bus

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: dbus-c++ - Native C++ bindings for D-Bus


https://bugzilla.redhat.com/show_bug.cgi?id=455575





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 08:59 EST ---
OK, done.
New spec: http://tgmweb.at/gadllah/dbus-c++.spec
New SRPM: 
http://tgmweb.at/gadllah/dbus-c++-0.5.0-0.2.20080716git1337c65.fc9.src.rpm


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455585] New: Review Request: packagekit-qt - QT bindings for packagekit

2008-07-16 Thread bugzilla
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/show_bug.cgi?id=455585

   Summary: Review Request: packagekit-qt - QT bindings for
packagekit
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


SPEC  SRPM are here http://tuxbrewr.fedorapeople.org/kpackagekit/
Description: QT bindings needed for kpackagekit

Build logs  http://koji.fedoraproject.org/koji/taskinfo?taskID=718807

Requires Qt4.4 so currently only builds for rawhide.  Will be included in F9 
when KDE4.1 is released

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455585] Review Request: packagekit-qt - QT bindings for packagekit

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: packagekit-qt - QT bindings for packagekit


https://bugzilla.redhat.com/show_bug.cgi?id=455585





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 09:28 EST ---
Quick comments:

1. Is BR: kdelibs-devel really needed here?  If so, use kdelibs4-devel instead.
2. Same for qt, use BR: qt4-devel - 4.4
3. Preferable to use %{cmake} macro like:
%{cmake} .
instead of manually doing:
cmake . -DCMAKE_INSTALL_PREFIX=/usr/


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455585] Review Request: packagekit-qt - QT bindings for packagekit

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: packagekit-qt - QT bindings for packagekit


https://bugzilla.redhat.com/show_bug.cgi?id=455585





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 09:34 EST ---
and
4.  add %changelog

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455575] Review Request: dbus-c++ - Native C++ bindings for D-Bus

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: dbus-c++ - Native C++ bindings for D-Bus


https://bugzilla.redhat.com/show_bug.cgi?id=455575





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 09:38 EST ---
formal review:

OK  source files match upstream
OK  package meets naming and versioning guidelines.
OK  specfile is properly named, is cleanly written and uses macros 
consistently.
OK  dist tag is present.
OK  build root is correct.
OK  license field matches the actual license.
OK  license is open source-compatible (LGPLv2+). License text included in 
package.
OK  latest version is being packaged.
OK  BuildRequires are proper.
BAD compiler flags are appropriate.
OK  %clean is present.
OK  package builds in mock (Rawhide/x86_64).
BAD debuginfo package looks complete.
OK  rpmlint is silent.
OK  final provides and requires look sane.
N/A %check is present and all tests pass.
OK  shared libraries are added to the regular linker search paths, correct
scriptlets exists
OK  owns the directories it creates.
OK  doesn't own any directories it shouldn't.
OK  no duplicates in %files.
OK  file permissions are appropriate.
OK  correct scriptlets present.
OK  code, not content.
OK  documentation is small, so no -docs subpackage is necessary.
OK  %docs are not necessary for the proper functioning of the package.
OK  headers in devel subpackage.
OK  pkgconfig files in devel subpackage.
OK  no libtool .la droppings.
OK  not a GUI app.


- wrong flags are passed to the compiler, looks like package's own
- can't see any source files in the debuginfo package (depends on the flags 
issue)
- and I would use the git-archive to generate the source archive, but that's not
any blocker, it would decrease the size from 750 KB to only 50 KB ;-)


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 435542] Review Request: xdemorse - GTK based application for decoding and displaying Morse code signals

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xdemorse - GTK based application for decoding and 
displaying Morse code signals


https://bugzilla.redhat.com/show_bug.cgi?id=435542


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 09:49 EST ---
OK, now is this package APPROVED


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 435542] Review Request: xdemorse - GTK based application for decoding and displaying Morse code signals

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xdemorse - GTK based application for decoding and 
displaying Morse code signals


https://bugzilla.redhat.com/show_bug.cgi?id=435542


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 10:01 EST ---
New Package CVS Request
===
Package Name: xdemorse
Short Description: GTK based application for decoding and displaying Morse code
signals
Owners: lucilanga,bjensen
Branches: F-8 F-9
InitialCC: 
Cvsextras Commits: yes



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455544] Review Request: cx18-firmware - Firmware for Conexant cx23418-based video capture devices

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: cx18-firmware - Firmware for Conexant cx23418-based 
video capture devices


https://bugzilla.redhat.com/show_bug.cgi?id=455544


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 10:00 EST ---
New Package CVS Request
===
Package Name: cx18-firmware
Short Description: Firmware for Conexant cx23418-based video capture devices
Owners: jwilson
Branches: F-8 F-9
InitialCC:
Cvsextras Commits: yes


Nb: the intention here is to build in F8 and let that be inherited up through F9
and devel, since this is a noarch firmware package.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455585] Review Request: packagekit-qt - QT bindings for packagekit

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: packagekit-qt - QT bindings for packagekit


https://bugzilla.redhat.com/show_bug.cgi?id=455585





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 10:05 EST ---
Done 

New build logs  http://koji.fedoraproject.org/koji/taskinfo?taskID=719895

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455558] Review Request: udns - DNS resolver library for both synchronous and asynchronous DNS queries

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: udns - DNS resolver library for both synchronous and 
asynchronous DNS queries


https://bugzilla.redhat.com/show_bug.cgi?id=48


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 10:06 EST ---
New Package CVS Request
===
Package Name: udns
Short Description: DNS resolver library for both synchronous and asynchronous
DNS queries
Owners: adrian
Branches: F-8 F-9
InitialCC: ben
Cvsextras Commits: yes


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 428686] Review Request: tex-simplecv - latex class for writing curricula vitae

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: tex-simplecv - latex class for writing curricula vitae


https://bugzilla.redhat.com/show_bug.cgi?id=428686





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 10:23 EST ---
tex-simplecv-1.6-6.fc9.1 has been submitted as an update for Fedora 9

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 428686] Review Request: tex-simplecv - latex class for writing curricula vitae

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: tex-simplecv - latex class for writing curricula vitae


https://bugzilla.redhat.com/show_bug.cgi?id=428686





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 10:25 EST ---
tex-simplecv-1.6-6.fc8 has been submitted as an update for Fedora 8

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455575] Review Request: dbus-c++ - Native C++ bindings for D-Bus

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: dbus-c++ - Native C++ bindings for D-Bus


https://bugzilla.redhat.com/show_bug.cgi?id=455575





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 10:25 EST ---
(In reply to comment #3)
 formal review:

 BAD   compiler flags are appropriate.
Fixed.

 BAD   debuginfo package looks complete.
Fixed.

 - wrong flags are passed to the compiler, looks like package's own
Fixed now. (see above)
 - can't see any source files in the debuginfo package (depends on the flags 
 issue)
Fixed too.
 - and I would use the git-archive to generate the source archive, but that's 
 not
 any blocker, it would decrease the size from 750 KB to only 50 KB ;-)
Done.

New spec: http://tgmweb.at/gadllah/dbus-c++.spec
New SRPM: 
http://tgmweb.at/gadllah/dbus-c++-0.5.0-0.3.20080716git1337c65.fc9.src.rpm


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 428686] Review Request: tex-simplecv - latex class for writing curricula vitae

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: tex-simplecv - latex class for writing curricula vitae


https://bugzilla.redhat.com/show_bug.cgi?id=428686


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 10:33 EST ---
Packages built to all supported Fedora versions.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455575] Review Request: dbus-c++ - Native C++ bindings for D-Bus

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: dbus-c++ - Native C++ bindings for D-Bus


https://bugzilla.redhat.com/show_bug.cgi?id=455575


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 10:47 EST ---
looks OK now, so this package is APPROVED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455575] Review Request: dbus-c++ - Native C++ bindings for D-Bus

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: dbus-c++ - Native C++ bindings for D-Bus


https://bugzilla.redhat.com/show_bug.cgi?id=455575


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 10:59 EST ---
Thanks for the review.
-
New Package CVS Request
===
Package Name: dbus-c++
Short Description: Native C++ bindings for D-Bus
Owners: drago01
Branches: F-8 F-9
InitialCC:
Cvsextras Commits: yes


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 454895] Review Request: sitecopy - Tool for easily maintaining remote web sites

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: sitecopy - Tool for easily maintaining remote web sites


https://bugzilla.redhat.com/show_bug.cgi?id=454895





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 11:00 EST ---
1. Fixed

2. neon-devel - fixed
   internationalization - fixed
   openssl-devel - fixed 
   There are two messages:
configure: zlib not supported
configure: Thread-safe SSL not supported
   zlib not supported - From Changelog  configure script it looks like zlib 
code was breaking and they have disabled its use and bypassing functions. I 
will try to confirm by digging further today. 

SRPM: http://rakesh.fedorapeople.org/srpm/sitecopy-0.16.3-3.fc9.src.rpm
SPEC: http://rakesh.fedorapeople.org/spec/sitecopy.spec


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455149] Review Request: latency-policy - Policy for system wide latency

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: latency-policy - Policy for system wide latency


https://bugzilla.redhat.com/show_bug.cgi?id=455149


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 11:53 EST ---
Can't we hide the knobs for default setups?

I doubt that servers processing financial data need:
- ALSA powersave
- WiFi poll powersave

I'd also like to know how this would work with the cpuspeed daemon, which is in
the default installation, and supports more governors/setups.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455149] Review Request: latency-policy - Policy for system wide latency

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: latency-policy - Policy for system wide latency


https://bugzilla.redhat.com/show_bug.cgi?id=455149





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 11:57 EST ---
Right, the knobs are not meant to be changed, only if an admin knows what they
are doing, or to work round a kernel bug. As for the cpuspeed interaction I'm
not sure. I'll have to do some tests.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 435542] Review Request: xdemorse - GTK based application for decoding and displaying Morse code signals

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xdemorse - GTK based application for decoding and 
displaying Morse code signals


https://bugzilla.redhat.com/show_bug.cgi?id=435542


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 12:04 EST ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455544] Review Request: cx18-firmware - Firmware for Conexant cx23418-based video capture devices

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: cx18-firmware - Firmware for Conexant cx23418-based 
video capture devices


https://bugzilla.redhat.com/show_bug.cgi?id=455544


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 12:06 EST ---
cvs done. 


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455575] Review Request: dbus-c++ - Native C++ bindings for D-Bus

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: dbus-c++ - Native C++ bindings for D-Bus


https://bugzilla.redhat.com/show_bug.cgi?id=455575


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 12:13 EST ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455558] Review Request: udns - DNS resolver library for both synchronous and asynchronous DNS queries

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: udns - DNS resolver library for both synchronous and 
asynchronous DNS queries


https://bugzilla.redhat.com/show_bug.cgi?id=48


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 12:29 EST ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 454895] Review Request: sitecopy - Tool for easily maintaining remote web sites

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: sitecopy - Tool for easily maintaining remote web sites


https://bugzilla.redhat.com/show_bug.cgi?id=454895





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 12:36 EST ---
Letest release today solves these messages:
I have updated to latest release.
SPEC: http://rakesh.fedorapeople.org/spec/sitecopy.spec
SRPM: http://rakesh.fedorapeople.org/srpm/sitecopy-0.16.5-1.fc9.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455149] Review Request: latency-policy - Policy for system wide latency

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: latency-policy - Policy for system wide latency


https://bugzilla.redhat.com/show_bug.cgi?id=455149


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 12:45 EST ---
rpmlint output:
--
latency-policy.noarch: E: executable-marked-as-config-file
/etc/rc.d/init.d/latency-policy
latency-policy.noarch: E: executable-marked-as-config-file 
/etc/sysconfig/latency
latency-policy.noarch: E: executable-marked-as-config-file
/etc/latency-policy/config.d/aspm
latency-policy.noarch: E: executable-marked-as-config-file
/etc/latency-policy/config.d/ondemand
latency-policy.noarch: E: executable-marked-as-config-file
/etc/latency-policy/config.d/alpm
latency-policy.noarch: E: only-non-binary-in-usr-lib
latency-policy.noarch: W: conffile-without-noreplace-flag
/etc/latency-policy/config.d/alpm
latency-policy.noarch: W: conffile-without-noreplace-flag
/etc/latency-policy/config.d/aspm
latency-policy.noarch: W: conffile-without-noreplace-flag
/etc/latency-policy/config.d/ondemand
latency-policy.noarch: W: conffile-without-noreplace-flag
/etc/rc.d/init.d/latency-policy
latency-policy.noarch: W: conffile-without-noreplace-flag /etc/sysconfig/latency
latency-policy.noarch: W: service-default-enabled 
/etc/rc.d/init.d/latency-policy
latency-policy.noarch: W: no-reload-entry /etc/rc.d/init.d/latency-policy
latency-policy.noarch: E: subsys-not-used /etc/rc.d/init.d/latency-policy
latency-policy.src:53: W: libdir-macro-in-noarch-package %dir
%{_libdir}/latency-policy
latency-policy.src:54: W: libdir-macro-in-noarch-package 
%{_libdir}/latency-policy/*
2 packages and 0 specfiles checked; 7 errors, 9 warnings.
--

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 438543] Review Request: Synopsis - Source-code introspection tool.

2008-07-16 Thread bugzilla
Please 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/show_bug.cgi?id=438543





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 13:09 EST ---
ping again?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 244593] Review Request: postgresql-pgbouncer - Lightweight connection pooler for PostgreSQL

2008-07-16 Thread bugzilla
Please 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-pgbouncer - Lightweight connection pooler 
for PostgreSQL


https://bugzilla.redhat.com/show_bug.cgi?id=244593


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|NEEDINFO
   Flag||needinfo?([EMAIL PROTECTED]
   ||pt.com)




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 13:07 EST ---
ping again?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455575] Review Request: dbus-c++ - Native C++ bindings for D-Bus

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: dbus-c++ - Native C++ bindings for D-Bus


https://bugzilla.redhat.com/show_bug.cgi?id=455575


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455622] New: Review Request: scriptaculous-js - JavaScript libraries for web user interfaces

2008-07-16 Thread bugzilla
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/show_bug.cgi?id=455622

   Summary: Review Request: scriptaculous-js - JavaScript libraries
for web user interfaces
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://dmalcolm.fedorapeople.org/scriptaculous-js.spec
SRPM URL: http://dmalcolm.fedorapeople.org/scriptaculous-js-1.8.1-1.src.rpm
Description:
JavaScript libraries for web user interfaces

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 449869] Review Request: tasque - A simple task management app

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: tasque - A simple task management app


https://bugzilla.redhat.com/show_bug.cgi?id=449869


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 13:32 EST ---
For 0.1.6-2:

* Dependency
  - Okay, it seems to work now (perhaps rawhide mono was just broken...)
However currently I get the message like below:
-
[EMAIL PROTECTED] tasque]$ tasque 

** (Tasque:16280): WARNING **: The following assembly referenced from
/usr/lib/tasque/Tasque.exe could not be loaded:
 Assembly:   NDesk.DBus.GLib(assemblyref_index=8)
 Version:1.0.0.0
 Public Key: f6716e4f9b2ed099
The assembly was not found in the Global Assembly Cache, a path listed in the
MONO_PATH environment variable, or in the location of the executing assembly
(/usr/lib/tasque/).


** (Tasque:16280): WARNING **: Could not load file or assembly 'NDesk.DBus.GLib,
Version=1.0.0.0, Culture=neutral, PublicKeyToken=f6716e4f9b2ed099' or one of its
dependencies.

** (Tasque:16280): WARNING **: Missing method Init in assembly
/usr/lib/tasque/Tasque.exe, type NDesk.DBus.BusG
[Debug]: Tasque remote control disabled (DBus exception): Could not load file or
assembly 'NDesk.DBus.GLib, Version=1.0.0.0, Culture=neutral,
PublicKeyToken=f6716e4f9b2ed099' or one of its dependencies.
[Debug]: Found Available Backend: Tasque.Backends.RtmBackend.RtmBackend
-
Perhaps adding Requires: ndesk-dbus-glib is preferable.

  - Also, for directory ownership issue (%{_datadir}/dbus-1/services/), 
it may be preferable that you write the explicit dependency: Requires: 
dbus.

* Desktop files
  - Please remove the category X-SuSE-Core-Office from the desktop file
(as we are not on SuSE).

Then:
-
NOTE: Before being sponsored:

This package will be accepted with another few work. 
But before I accept this package, someone (I am a candidate) 
must sponsor you.

Once you are sponsored, you have the right to review other 
submitters' review requests and approve the packages formally. 
For this reason, the person who want to be sponsored (like you) 
are required to show that you have an understanding 
of the process and of the packaging guidelines as is described
on :
http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored

Usually there are two ways to show this.
A. submit other review requests with enough quality.
B. Do a pre-review of other person's review request
   (at the time you are not sponsored, you cannot do
   a formal review)

When you have submitted a new review request or have pre-reviewed other 
person's review request, please write the bug number on this bug report 
so that I can check your comments or review request.

Fedora package collection review requests which are waiting for someone to
review can be checked on:
http://fedoraproject.org/PackageReviewStatus/NEW.html
(NOTE: please don't choose Merge Review)


Review guidelines are described mainly on:
http://fedoraproject.org/wiki/Packaging/ReviewGuidelines
http://fedoraproject.org/wiki/Packaging/Guidelines
http://fedoraproject.org/wiki/Packaging/ScriptletSnippets



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 454220] Review Request: germanium - a download manager for eMusic.com

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: germanium - a download manager for eMusic.com


https://bugzilla.redhat.com/show_bug.cgi?id=454220





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 13:46 EST ---
Would you post the full URLs of your newest spec/srpm files so that reviewers
can easily find out them?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 452427] Review Request: awesome - Extremely fast, small, dynamic and awesome floating and tiling window manager

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: awesome - Extremely fast, small, dynamic and awesome 
floating and tiling window manager
Alias: awesome

https://bugzilla.redhat.com/show_bug.cgi?id=452427





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 13:52 EST ---
FYI your latest srpm does not build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=720416

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 435542] Review Request: xdemorse - GTK based application for decoding and displaying Morse code signals

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xdemorse - GTK based application for decoding and 
displaying Morse code signals


https://bugzilla.redhat.com/show_bug.cgi?id=435542





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 14:06 EST ---
xdemorse-0.9-4.fc9 has been submitted as an update for Fedora 9

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 435542] Review Request: xdemorse - GTK based application for decoding and displaying Morse code signals

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xdemorse - GTK based application for decoding and 
displaying Morse code signals


https://bugzilla.redhat.com/show_bug.cgi?id=435542


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 447847] Review Request: unbound - Validating, recursive, and caching DNS(SEC) resolver

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: unbound - Validating, recursive, and caching DNS(SEC) 
resolver


https://bugzilla.redhat.com/show_bug.cgi?id=447847





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 14:36 EST ---
updated to 1.0.1

Spec URL: ftp://ftp.xelerance.com/unbound/unbound.spec
SRPM URL: ftp://ftp.xelerance.com/unbound/unbound-1.0.1-1.fc9.src.rpm 



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455149] Review Request: latency-policy - Policy for system wide latency

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: latency-policy - Policy for system wide latency


https://bugzilla.redhat.com/show_bug.cgi?id=455149


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 14:40 EST ---
Feature page is here: https://fedoraproject.org/wiki/Features/LatencyPolicy

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 452427] Review Request: awesome - Extremely fast, small, dynamic and awesome floating and tiling window manager

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: awesome - Extremely fast, small, dynamic and awesome 
floating and tiling window manager
Alias: awesome

https://bugzilla.redhat.com/show_bug.cgi?id=452427





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 14:43 EST ---
(In reply to comment #12)
 FYI your latest srpm does not build:
 http://koji.fedoraproject.org/koji/taskinfo?taskID=720416

Indeed. Thanks for noting that Mamoru-san.

See %%description

* http://mnowak.fedorapeople.org/awesome/awesome.spec

* http://mnowak.fedorapeople.org/awesome/awesome-2.3.2-5.fc9.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 452427] Review Request: awesome - Extremely fast, small, dynamic and awesome floating and tiling window manager

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: awesome - Extremely fast, small, dynamic and awesome 
floating and tiling window manager
Alias: awesome

https://bugzilla.redhat.com/show_bug.cgi?id=452427





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 15:13 EST ---
I'd guess the Requires: libconfuse = 2.6 is autodetected and should thus be
removed, and having the libconfuse = 2.6 in BuildReq is sufficient. At least
it was with my package. :-)


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 195015] Review Request: xpa

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xpa


https://bugzilla.redhat.com/show_bug.cgi?id=195015


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 15:34 EST ---
Package Change Request
==
Package Name: foobar
New Branches: EL-5


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 352741] Review Request: tkimg - More Image Formats for Tk

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: tkimg - More Image Formats for Tk


https://bugzilla.redhat.com/show_bug.cgi?id=352741


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 15:36 EST ---
Package Change Request
==
Package Name: tkimg
New Branches: EL-5


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 233352] Review Request: funtools - FITS library and utilities

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: funtools - FITS library and utilities


https://bugzilla.redhat.com/show_bug.cgi?id=233352


[EMAIL PROTECTED] changed:

   What|Removed |Added

Product|Fedora Extras   |Fedora
Version|devel   |rawhide

[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 15:38 EST ---
Package Change Request
==
Package Name: funtools
New Branches: EL-5


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 195921] Review Request: sextractor

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: sextractor


https://bugzilla.redhat.com/show_bug.cgi?id=195921


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 15:48 EST ---
Package Change Request
==
Package Name: sextractor
New Branches: EL-5


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 184450] Review Request: wcstools

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: wcstools


https://bugzilla.redhat.com/show_bug.cgi?id=184450


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 15:45 EST ---
Package Change Request
==
Package Name: wcstools
New Branches: EL-5


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 452427] Review Request: awesome - Extremely fast, small, dynamic and awesome floating and tiling window manager

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: awesome - Extremely fast, small, dynamic and awesome 
floating and tiling window manager
Alias: awesome

https://bugzilla.redhat.com/show_bug.cgi?id=452427





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 15:56 EST ---
(In reply to comment #14)
 I'd guess the Requires: libconfuse = 2.6 is autodetected and should thus be
 removed, 

No, is not. 

Requires: ... libconfuse.so.0 ...

And because on F8 there's stil v2.5 I found it useful ATM.

 and having the libconfuse = 2.6 in BuildReq is sufficient. At least
 it was with my package. :-)

you mean -devel, right?


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455654] New: Review Request: pads - Passive Asset Detection System

2008-07-16 Thread bugzilla
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/show_bug.cgi?id=455654

   Summary: Review Request: pads - Passive Asset Detection System
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://people.redhat.com/sgrubb/files/pads.spec
SRPM URL: http://people.redhat.com/sgrubb/files/pads-1.2-1.fc9.src.rpm

Description: 
PADS is a libpcap based detection engine used to passively
detect network assets.  It is designed to complement IDS
technology by providing context to IDS alerts. When new assets
are found, it can send IDMEF alerts via prelude.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 452427] Review Request: awesome - Extremely fast, small, dynamic and awesome floating and tiling window manager

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: awesome - Extremely fast, small, dynamic and awesome 
floating and tiling window manager
Alias: awesome

https://bugzilla.redhat.com/show_bug.cgi?id=452427





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 16:12 EST ---
* http://mnowak.fedorapeople.org/awesome/awesome.spec

* http://mnowak.fedorapeople.org/awesome/awesome-2.3.2-6.fc9.src.rpm

One more update -- libXinerama-devel was missing. Now it builds inside mock.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 452388] Review Request: python-migrate - Schema migration tools for SQLAlchemy

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: python-migrate - Schema migration tools for SQLAlchemy


https://bugzilla.redhat.com/show_bug.cgi?id=452388


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]   |[EMAIL PROTECTED]
   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 16:23 EST ---
With regard to the naming issue, I'm also in favor of 
s/migrate/sqlalchemy-migrate/.

? rpmlint complaint: python-migrate.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/migrate/versioning/templates/manage.py_tmpl 
0644
* Package name is OK, and I'd be fine with the s/migrate/sqlalchemy-migrate/
rename as well.
* Package meets the packaging guidelines
* License is valid and meets guidelines
* Specfile is clean
* Source file matches upstream
* Package successfully builds in mock
* Requires and BuildRequires are correct
* Package owns everything it creates
* Does not contain duplicate files
* Permissions are set properly
* %clean section exists
* consistent use of macros
* code, not content
* small amounts of documentation, no subpackage
* %docs do not effect runtime
* buildroot is wiped before installation
* filenames are all valid utf-8

I don't see the rpmlint error as an issue, since it is a template that turns
into a script once installed.

So, the rename is up to you guys -- whether we want python-migrate or
python-sqlalchemy-migrate (I tend to lean towards the latter).  Other than 
that...

Approved!

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450466] Review Request: clive - Video extraction tool for user-uploaded video hosts

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: clive - Video extraction tool for user-uploaded video 
hosts


https://bugzilla.redhat.com/show_bug.cgi?id=450466





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 17:00 EST ---
clive-0.4.18-1.fc8 has been submitted as an update for Fedora 8

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450466] Review Request: clive - Video extraction tool for user-uploaded video hosts

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: clive - Video extraction tool for user-uploaded video 
hosts


https://bugzilla.redhat.com/show_bug.cgi?id=450466





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 17:02 EST ---
clive-0.4.18-1.fc9 has been submitted as an update for Fedora 9

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 454128] Review Request: Thibault-fonts - Collection of fonts from thibault.org

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: Thibault-fonts - Collection of fonts from thibault.org


https://bugzilla.redhat.com/show_bug.cgi?id=454128





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 17:22 EST ---
Reviewing...
rpmlint cleanups requested.

thibault-fonts.src: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 
12)
thibault-fonts.src: E: description-line-too-long A collection of fonts from
thibault.org, including Isabella, Essays1743, StayPuft, and Rockets.
thibault-fonts.src: W: no-version-in-last-changelog
thibault-fonts.src: W: invalid-license LGPLv2.1
thibault-fonts-essays1743.noarch: E: summary-too-long Thibault.org font based on
the typeface used in a 1743 English translation of Montaigne's Essays
thibault-fonts-essays1743.noarch: W: no-version-in-last-changelog
thibault-fonts-essays1743.noarch: W: invalid-license LGPLv2.1
thibault-fonts-rockets.noarch: W: no-version-in-last-changelog
thibault-fonts-staypuft.noarch: W: no-version-in-last-changelog
thibault-fonts-isabella.noarch: W: no-version-in-last-changelog
5 packages and 0 specfiles checked; 2 errors, 8 warnings.

License tag should be LGPLv2+, and only at the top, no extra one needed in
each %package.

BuildRoot could be better.  Preferred is
%(mktemp -ud %{_tmppath}/%{name}-%{version}-%{release}-XX)

other cleanups noted on IRC.  Not sure what to do about each of the subpackages
owning /usr/share/fonts/thibault/.  But seems nuts to make 4 new subdirs under
there just to solve it.


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 454128] Review Request: Thibault-fonts - Collection of fonts from thibault.org

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: Thibault-fonts - Collection of fonts from thibault.org


https://bugzilla.redhat.com/show_bug.cgi?id=454128





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 17:39 EST ---
Updated per above mentioned suggestions.

http://www.oslb.net/fonts/thibault-fonts-0.1-1.fc9.src.rpm
http://www.oslb.net/fonts/thibault-fonts.spec

Lyos Gemini Norezel

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 227115] Review Request: saxon8-B.8.7-1jpp - Java Basic XPath 2.0, XSLT 2.0, and XQuery 1.0 implementation

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: saxon8-B.8.7-1jpp - Java  Basic XPath 2.0, XSLT 2.0, 
and XQuery 1.0 implementation


https://bugzilla.redhat.com/show_bug.cgi?id=227115


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 17:50 EST ---
Retracting review request.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 452427] Review Request: awesome - Extremely fast, small, dynamic and awesome floating and tiling window manager

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: awesome - Extremely fast, small, dynamic and awesome 
floating and tiling window manager
Alias: awesome

https://bugzilla.redhat.com/show_bug.cgi?id=452427





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 18:22 EST ---
Meh. Of course I meant BuildRequires: libconfuse-devel = 2.6.

I'd add a comment on the reason for the Requires: libconfuse = 2.6, as that
is not at all obvious.


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 226319] Merge Review: procps

2008-07-16 Thread bugzilla
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: procps


https://bugzilla.redhat.com/show_bug.cgi?id=226319


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag|fedora-review+  |fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 18:22 EST ---
Then somebody else re-review this and see whether LGPLv2 and
GPLv2 must be added to the licence tag (comment 5).
The GFDL is not a code licence.


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455500] Need OpenGTL package

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Need OpenGTL package


https://bugzilla.redhat.com/show_bug.cgi?id=455500





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 18:41 EST ---
Ok, I'll try to take a look when I get time. Meanwhile, with Eric Noulard's help
I managed to get cpack working; I have a usable RPM but said RPM is in gross
violation of about half of Fedora's packaging policies. My initial impression is
that it won't be possible to use cpack to make Fedora RPM's (though, I'm not
sure it would be desired anyway).

IOW I have a work-around for now so the urgency (for me, anyway) isn't quite so
high, but we still need a proper RPM for the eventual Fedora inclusion of
KOffice2. (And it would be very awesome if Fedora devs had an /official/ RPM, as
it seems there currently is not any official one for /any/ distro.)

Bug 455502 is still high priority, since it's blocking *any* build of OpenGTL on
x86_64.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 452388] Review Request: python-migrate - Schema migration tools for SQLAlchemy

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: python-migrate - Schema migration tools for SQLAlchemy


https://bugzilla.redhat.com/show_bug.cgi?id=452388





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 18:48 EST ---
Cool.  The package naming is a bit of a pain:

If we think of this as primarily a python module with a program attached to it
then it should be: python-migrate to follow the naming guidelines for python
modules (import migrate = python-migrate)

If we think of it as a program with most functionality implemented in a module,
then it could be named migrate or sqlalchemy-migrate.

debian has python-migrate.  Gentoo doesn't seem to have it packaged yet.  So I
think I'll leave it at python-migrate for now.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 452388] Review Request: python-migrate - Schema migration tools for SQLAlchemy

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: python-migrate - Schema migration tools for SQLAlchemy


https://bugzilla.redhat.com/show_bug.cgi?id=452388


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 18:51 EST ---
New Package CVS Request
===
Package Name: python-migrate
Short Description: Schema migration tools for SQLAlchemy
Owners: toshio lmacken ricky
Branches: F-8 F-9 devel EL-5 
InitialCC:
Cvsextras Commits: yes


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 247402] Review Request: GspiceUI - A GUI to freely available Spice Electronic circuit similators

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: GspiceUI - A GUI to freely available Spice Electronic 
circuit similators


https://bugzilla.redhat.com/show_bug.cgi?id=247402


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEEDINFO|ASSIGNED
   Flag|needinfo?([EMAIL PROTECTED]) |




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 19:45 EST ---
Sorry, my gmail account is currently down since few days due to google server
troubles.
So i couldn't be highlighted earlier.

Well, 
Correct, the lastest version build fine and seem works pretty nice.
i'll upload it by tomorrow.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 352741] Review Request: tkimg - More Image Formats for Tk

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: tkimg - More Image Formats for Tk


https://bugzilla.redhat.com/show_bug.cgi?id=352741


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 20:09 EST ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 195015] Review Request: xpa

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xpa


https://bugzilla.redhat.com/show_bug.cgi?id=195015


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 20:08 EST ---
I take it you meant the Package Name to be xpa here?

cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 233352] Review Request: funtools - FITS library and utilities

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: funtools - FITS library and utilities


https://bugzilla.redhat.com/show_bug.cgi?id=233352


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 20:09 EST ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 184450] Review Request: wcstools

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: wcstools


https://bugzilla.redhat.com/show_bug.cgi?id=184450


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 20:14 EST ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 352761] Review Request: ds9 - Astronomical Data Visualization Application

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ds9 - Astronomical Data Visualization Application


https://bugzilla.redhat.com/show_bug.cgi?id=352761


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 20:14 EST ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 195921] Review Request: sextractor

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: sextractor


https://bugzilla.redhat.com/show_bug.cgi?id=195921


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 20:15 EST ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 452388] Review Request: python-migrate - Schema migration tools for SQLAlchemy

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: python-migrate - Schema migration tools for SQLAlchemy


https://bugzilla.redhat.com/show_bug.cgi?id=452388


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 20:17 EST ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455684] New: Review Request: perl-DateTime-Format-DB2 - Parse and format DB2 dates and times

2008-07-16 Thread bugzilla
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/show_bug.cgi?id=455684

   Summary: Review Request: perl-DateTime-Format-DB2 - Parse and
format DB2 dates and times
   Product: Fedora
   Version: rawhide
  Platform: All
   URL: http://search.cpan.org/dist/DateTime-Format-DB2/
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


SRPM URL: 
http://fedora.biggerontheinside.net/review/perl-DateTime-Format-DB2-0.05-1.fc8.src.rpm
SPEC URL: 
http://fedora.biggerontheinside.net/review/perl-DateTime-Format-DB2.spec

Description:
This module understands the formats used by DB2 for its DATE, TIME, and
TIMESTAMP data types. It can be used to parse these formats in order to
create DateTime objects, and it can take a DateTime object and produce a
string representing it in the DB2 format.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 454895] Review Request: sitecopy - Tool for easily maintaining remote web sites

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: sitecopy - Tool for easily maintaining remote web sites


https://bugzilla.redhat.com/show_bug.cgi?id=454895


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|177841  |
  nThis||
   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 20:25 EST ---
That does indeed solve all the issues I see, so this package is APPROVED. 

I have sponsored you, so you should continue the process at: 
http://fedoraproject.org/wiki/PackageMaintainers/Join#Add_Package_to_CVS_and_Set_Owner
for this and your other approved package.

Please let me know via email or irc if I can assist you any with the process...
welcome!

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 445027] Review Request: dnrd - A caching, forwarding DNS proxy server

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: dnrd - A caching, forwarding DNS proxy server


https://bugzilla.redhat.com/show_bug.cgi?id=445027


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|177841  |
  nThis||
   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 20:25 EST ---
That solves all the issues I see here, so this package is APPROVED. 

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455056] Review Request: weex - Fast WEb EXchanger non-interactive FTP client

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: weex - Fast WEb EXchanger non-interactive FTP client


https://bugzilla.redhat.com/show_bug.cgi?id=455056


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|177841  |
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 20:27 EST ---
Removing needsponsor, I have sponsored Rakesh. 

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 454395] Review Request: php-xmpphp - PHP XMPP Library

2008-07-16 Thread bugzilla
Please 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-xmpphp - PHP XMPP Library


https://bugzilla.redhat.com/show_bug.cgi?id=454395


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|177841  |
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 20:28 EST ---
Removing needsponsor, I have sponsored Rakesh.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 453520] Review Request: libUnihan - C library for Unihan character database in 5NF

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libUnihan - C library for Unihan character database in 
5NF


https://bugzilla.redhat.com/show_bug.cgi?id=453520


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review?




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 455039] Review Request: php-oauth - PHP Authentication library for desktop to web applications

2008-07-16 Thread bugzilla
Please 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-oauth - PHP Authentication library for desktop to 
web applications


https://bugzilla.redhat.com/show_bug.cgi?id=455039


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|177841  |
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 20:28 EST ---
Removing needsponsor, I have sponsored Rakesh.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 448397] Review Request: ntop - A network traffic probe similar to the UNIX top command

2008-07-16 Thread bugzilla
Please 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_package

https://bugzilla.redhat.com/show_bug.cgi?id=448397


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|177841  |
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 20:28 EST ---
Removing needsponsor, I have sponsored Rakesh.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 453519] Review Request: libUnihan-data - The Unihan character database in 5NF

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libUnihan-data - The Unihan character database in 5NF


https://bugzilla.redhat.com/show_bug.cgi?id=453519


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-review?




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 450323] Review Request: coq - Coq proof management system

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: coq - Coq proof management system


https://bugzilla.redhat.com/show_bug.cgi?id=450323





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 21:01 EST ---
coq-8.1pl3-1.fc9 has been submitted as an update for Fedora 9

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 449869] Review Request: tasque - A simple task management app

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: tasque - A simple task management app


https://bugzilla.redhat.com/show_bug.cgi?id=449869





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 21:14 EST ---
(In reply to comment #6)
 For 0.1.6-2:
 
 * Dependency
   - Okay, it seems to work now (perhaps rawhide mono was just broken...)
 However currently I get the message like below:
 -
 [EMAIL PROTECTED] tasque]$ tasque 
 
 ** (Tasque:16280): WARNING **: The following assembly referenced from
 /usr/lib/tasque/Tasque.exe could not be loaded:
  Assembly:   NDesk.DBus.GLib(assemblyref_index=8)
  Version:1.0.0.0
  Public Key: f6716e4f9b2ed099
 The assembly was not found in the Global Assembly Cache, a path listed in the
 MONO_PATH environment variable, or in the location of the executing assembly
 (/usr/lib/tasque/).
 
 
 ** (Tasque:16280): WARNING **: Could not load file or assembly 
 'NDesk.DBus.GLib,
 Version=1.0.0.0, Culture=neutral, PublicKeyToken=f6716e4f9b2ed099' or one of 
 its
 dependencies.
 
 ** (Tasque:16280): WARNING **: Missing method Init in assembly
 /usr/lib/tasque/Tasque.exe, type NDesk.DBus.BusG
 [Debug]: Tasque remote control disabled (DBus exception): Could not load file 
 or
 assembly 'NDesk.DBus.GLib, Version=1.0.0.0, Culture=neutral,
 PublicKeyToken=f6716e4f9b2ed099' or one of its dependencies.
 [Debug]: Found Available Backend: Tasque.Backends.RtmBackend.RtmBackend
 -
 Perhaps adding Requires: ndesk-dbus-glib is preferable.
 
   - Also, for directory ownership issue (%{_datadir}/dbus-1/services/), 
 it may be preferable that you write the explicit dependency: Requires: 
 dbus.
 
 * Desktop files
   - Please remove the category X-SuSE-Core-Office from the desktop file
 (as we are not on SuSE).
 
 Then:
 -
 NOTE: Before being sponsored:
 
 This package will be accepted with another few work. 
 But before I accept this package, someone (I am a candidate) 
 must sponsor you.
 
 Once you are sponsored, you have the right to review other 
 submitters' review requests and approve the packages formally. 
 For this reason, the person who want to be sponsored (like you) 
 are required to show that you have an understanding 
 of the process and of the packaging guidelines as is described
 on :
 http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored
 
 Usually there are two ways to show this.
 A. submit other review requests with enough quality.
 B. Do a pre-review of other person's review request
(at the time you are not sponsored, you cannot do
a formal review)
 
 When you have submitted a new review request or have pre-reviewed other 
 person's review request, please write the bug number on this bug report 
 so that I can check your comments or review request.
 
 Fedora package collection review requests which are waiting for someone to
 review can be checked on:
 http://fedoraproject.org/PackageReviewStatus/NEW.html
 (NOTE: please don't choose Merge Review)
 
 
 Review guidelines are described mainly on:
 http://fedoraproject.org/wiki/Packaging/ReviewGuidelines
 http://fedoraproject.org/wiki/Packaging/Guidelines
 http://fedoraproject.org/wiki/Packaging/ScriptletSnippets
 
 

Latest revision:
http://kaylor.info/fedora/SPEC/tasque.spec
http://kaylor.info/fedora/SRPM/tasque-0.1.6-3.fc9.src.rpm

ndesk-dbus-glib is definitely preferable.  I tried installing revision 2 on a
clean Fedora 9 live usb and the program started but would later crash.

The SuSE reference is now removed.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 449869] Review Request: tasque - A simple task management app

2008-07-16 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: tasque - A simple task management app


https://bugzilla.redhat.com/show_bug.cgi?id=449869





--- Additional Comments From [EMAIL PROTECTED]  2008-07-16 21:30 EST ---
I noticed my last post was not clear.  Without ndesk-dbus-glib, tasque would
start and later crash.  With ndesk-dbus-glib, it ran fine.


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


  1   2   >