[Bug 278481] Review Request: linuxdcpp - Client for the direct connect protocol

2007-09-07 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: linuxdcpp - Client for the direct connect protocol


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 02:55 EST ---
I correct those errors.

Spec URL: http://zprikryl.fedorapeople.org/linuxdcpp.spec
SRPM URL: http://zprikryl.fedorapeople.org/linuxdcpp-0-0.1.20070905cvs.src.rpm
Description: Linux DC++ is a program the uses the Direct Connect protocol. It is
compatible with other DC clients, such as the original DC from Neomodus, DC++
and derivatives. Linux DC++ also interoperates with all common DC hub software.

-- 
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 280211] Review Request: aspell-sk - Slovak dictionaries for Aspell

2007-09-07 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: aspell-sk - Slovak dictionaries for Aspell


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 02:30 EST ---
Package built for FC6, F7, F8 (devel) and EL5. Unable to build for EL4 according
to dependency problems.


-- 
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 247227] Review Request: nabi - hangul and hanja X input method

2007-09-07 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: nabi - hangul and hanja X input method
Alias: nabi

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 02:21 EST ---
New Package CVS Request
===
Package Name: nabi
Short Description: Hangul X Input Method 
Owners: sbehera
Branches: F-7
InitialCC: petersen
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 281891] Review Request: cheese - A cheesy program to take pictures and videos from your web cam

2007-09-07 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: cheese - A cheesy program to take pictures and videos 
from your web cam


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 02:16 EST ---
After a quick glance at your spec, there are 2 things that my packaging does 
better:

- don't use a vendor prefix for the desktop file

- produce a meaningful debuginfo 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 281901] Review Request: cheese - A webcam application for snapshots and movies

2007-09-07 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: cheese - A webcam application for snapshots and movies


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 02:10 EST ---
Oh, what timing!

Please discuss with Rahul who will be the maintainer of cheese
(bug 281891)


-- 
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 260441] Review Request: pguiman - The PostgreSQL database server managing tool

2007-09-07 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: pguiman - The PostgreSQL database server managing tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED]
   Flag|fedora-review?  |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 273761] Review Request: tomcat-native - Tomcat native library

2007-09-07 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: tomcat-native - Tomcat native library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 02:17 EST ---
Built for devel an 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 248314] Review Request: cheese - A cheesy program to take pictures and videos from your web cam

2007-09-07 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: cheese  - A cheesy program to take pictures and videos 
from your web cam


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Resolution|WONTFIX |DUPLICATE




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 02:07 EST ---


*** This bug has been marked as a duplicate of 281891 ***

-- 
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 243955] Review Request: python-xlrd - Library to extract data from Microsoft Excel (tm) spreadsheet files

2007-09-07 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-xlrd - Library to extract data from Microsoft 
Excel (tm) spreadsheet files


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 07:28 EST ---
Sean, can you review my changes and package functionality?

-- 
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 243955] Review Request: python-xlrd - Library to extract data from Microsoft Excel (tm) spreadsheet files

2007-09-07 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-xlrd - Library to extract data from Microsoft 
Excel (tm) spreadsheet files


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review+  |




-- 
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 281891] Review Request: cheese - A cheesy program to take pictures and videos from your web cam

2007-09-07 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: cheese - A cheesy program to take pictures and videos 
from your web cam


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Resolution|WONTFIX |DUPLICATE




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 08:26 EST ---


*** This bug has been marked as a duplicate of 281901 ***

-- 
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 281901] Review Request: cheese - A webcam application for snapshots and movies

2007-09-07 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: cheese - A webcam application for snapshots and movies


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 08:26 EST ---
*** Bug 281891 has been marked as a duplicate of this bug. ***

-- 
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 281891] Review Request: cheese - A cheesy program to take pictures and videos from your web cam

2007-09-07 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: cheese - A cheesy program to take pictures and videos 
from your web cam


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||WONTFIX




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 08:19 EST ---
Closing off bug report. 

-- 
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 281901] Review Request: cheese - A webcam application for snapshots and movies

2007-09-07 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: cheese - A webcam application for snapshots and movies


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 10:23 EST ---
From the spec file:
BuildRequires: gstreamer-plugins-good-devel 

That doesn't seem to be checked for or used anywhere. Did you mean to have the
package itself as a requires instead? (make sure you require a version of
gstreamer-plugins-good that has v4l2 plugins as well)

BuildRequires: xorg-x11-proto-devel
BuildRequires: libXxf86vm-devel

what are those used for? There doesn't seem to be any X checks or headers used
in the sources.

Should the Source be that instead?
http://live.gnome.org/Cheese/Releases?action=AttachFiledo=gettarget=%{name}-%{version}.tar.gz

-- 
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 282261] Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes

2007-09-07 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: isync - Tool to synchronize IMAP4 and Maildir mailboxes
Alias: isync-review

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


[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 282261] New: Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes

2007-09-07 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=282261

   Summary: Review Request: isync - Tool to synchronize IMAP4 and
Maildir mailboxes
   Product: Fedora
   Version: devel
  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/lkundrak/SPECS/isync.spec
SRPM URL: 
http://people.redhat.com/lkundrak/mock-results/isync-1.0.3-1.fc7.i386/isync-1.0.3-1.fc7.src.rpm
Description: Tool to synchronize IMAP4 and Maildir mailboxes

isync is a command line application which synchronizes mailboxes; currently 
Maildir and IMAP4 mailboxes are supported. New messages, message deletions and 
flag changes can be propagated both ways. isync is suitable for use in 
IMAP-disconnected mode.

-- 
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 225679] Merge Review: dejagnu

2007-09-07 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: dejagnu


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 07:28 EST ---
Ok:
- rpmlint output
  W: dejagnu devel-file-in-non-devel-package /usr/share/dejagnu/testglue.c
  W: dejagnu devel-file-in-non-devel-package /usr/include/dejagnu.h
  W: dejagnu devel-file-in-non-devel-package /usr/share/dejagnu/stub-loader.c
  - the *.c files are part of the testsuite
  - dejagnu.h could be placed in devel subpackage, but in this case I think it's
ok to keep it as it is
- the package is named according to the Package Naming Guidelines
- the spec file name matches the base package %{name}
- the package is licensed with a Fedora approved license
- the License field in the package spec file matches the actual license (GPLv2+)
- file containing the text of the license(s) for the package is included in %doc
- the spec file is written in American English
- the spec file for the package is legible
- the sources used to build the package match the upstream source
- the package owns all directories that it creates
- the package does not contain any duplicate files in the %files listing
- permissions on files are set properly
- the package has a %clean section, which contains rm -rf $RPM_BUILD_ROOT
- the package consistently uses macros
- the package contains code, or permissible content
- files included as %doc don't affect the runtime of the application
- the package does not own files or directories already owned by other packages
- at the beginning of %install, the package runs rm -rf $RPM_BUILD_ROOT
- all filenames in rpm packages are valid UTF-8

Need some work:
- -n dejagnu-%{version} in %setup isn't necessary
- make in %build can be removed, there is nothing to build
- expect dependency doesn't need to use a version, 5.21 is very old
- tcl dependency is not required, expect depends on tcl
- jadetex docbook-utils-pdf tetex-dvips are not required during build, the
tarball contains generated overview.pdf 
- mv doc/html doc/overview doesn't seem to be necessary
- please use make DESTDIR=$RPM_BUILD_ROOT install-man for man installation
- BuildArchitecture should be just BuildArch
- %{version} could be used in Source
- expect should be in BuildRequires for %check. But the test suite seems to have
tests that are supposed to fail, so I'd suggest removing %check completely or
modifying the tests so make check doesn't need -k and || :, and regressions will
be caught in the build


-- 
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 253781] Review Request: outerspace - client for 4x on-line strategy game

2007-09-07 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: outerspace - client for 4x on-line strategy game


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 06:36 EST ---
I'm sorry, I've been busy last few days. I'll fix issues and provide updated
package by the end of this weekend.

-- 
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 282261] Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes

2007-09-07 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: isync - Tool to synchronize IMAP4 and Maildir mailboxes
Alias: isync-review

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  Alias||isync-review




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 08:58 EST ---
rpmlint is silent on all generated packages.
The build logs from mock can help you review the package:
http://people.redhat.com/lkundrak/mock-results/isync-1.0.3-1.fc7.i386/
Thanks for the 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 244192] Review Request: eclipse-anyedit - AnyEdit plugin for Eclipse

2007-09-07 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: eclipse-anyedit - AnyEdit plugin for Eclipse


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Product|Fedora Extras   |Fedora




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 10:45 EST ---
The package don't build with IcedTea (derived OpenJDK normally shipped with F8)

...
[javac] 119. ERROR in
/home/fons/rpmbuild/BUILD/eclipse-anyedit-1.8.0/build/plugins/de.loskutov.anyedit.AnyEditTools/src/de/loskutov/anyedit/util/TextUtil.java
(at line 856)
[javac] case '?':
[javac] 
[javac] Duplicate case
[javac] --
[javac] 119 problems (46 errors, 73 warnings)

Like Peter say the correct buildpath used in Fedora is 
%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)

If you fix the above build problems and the buildpath, I would be happy to
pre-review this package.


-- 
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 249236] Review Request: kde-style-domino - KDE style with a soft look

2007-09-07 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: kde-style-domino - KDE style with a soft look


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 09:48 EST ---
ping kelly ?

-- 
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 278481] Review Request: linuxdcpp - Client for the direct connect protocol

2007-09-07 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: linuxdcpp - Client for the direct connect protocol


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 11:12 EST ---
I will review it now completely. Here is a little hint: Do not write the release
in the 0.1 from the release in the tarball oder directory. Otherwise you will
have to rebuild the tarball everytime to only want to increase the release
without changing the tarball, e.g. due to Requires-adjustment oder because of a
mandatory rebuild.

-- 
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 244192] Review Request: eclipse-anyedit - AnyEdit plugin for Eclipse

2007-09-07 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: eclipse-anyedit - AnyEdit plugin for Eclipse


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 11:13 EST ---
(In reply to comment #4)
 again, that is not the preferred buildpath.  read the packaging guidelines 
 link
 i posted in comment #2.

Strange I have never see such a buildpath in a package but yes if it's in the
guideline that may do the trick.

-- 
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 253781] Review Request: outerspace - client for 4x on-line strategy game

2007-09-07 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: outerspace - client for 4x on-line strategy game


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 06:28 EST ---
ping?

-- 
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 282261] Review Request: isync - Tool to synchronize IMAP4 and Maildir mailboxes

2007-09-07 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: isync - Tool to synchronize IMAP4 and Maildir mailboxes
Alias: isync-review

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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 09:16 EST ---
I added an openssl-devel dependency to enable SSL support and bumped the
Revision by one. And thus, here are the actual files:
Spec URL: http://people.redhat.com/lkundrak/SPECS/isync.spec
SRPM URL:
http://people.redhat.com/lkundrak/mock-results/isync-1.0.3-2.fc7.i386/isync-1.0.3-2.fc7.src.rpm

Mock output is
http://people.redhat.com/lkundrak/mock-results/isync-1.0.3-2.fc7.i386/

-- 
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 281901] Review Request: cheese - A webcam application for snapshots and movies

2007-09-07 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: cheese - A webcam application for snapshots and movies


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 12:22 EST ---
Operator error...

-- 
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 282521] New: Review Request: dblatex - DocBook to LaTeX/ConTeXt Publishing

2007-09-07 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=282521

   Summary: Review Request: dblatex - DocBook to LaTeX/ConTeXt
Publishing
   Product: Fedora
   Version: devel
  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: https://nbecker.dyndns.org/RPM/dblatex.spec
SRPM URL: https://nbecker.dyndns.org/RPM/dblatex-0.2.7-1.fc7.src.rpm
Description: dblatex is a program that transforms your SGML/XMLDocBook
documents to DVI, PostScript or PDF by translating them
into pure LaTeX as a first process.  MathML 2.0 markups
are supported, too. It started as a clone of DB2LaTeX.

Authors:

   BenoƮt Guillon [EMAIL PROTECTED]
   Andreas Hoenen [EMAIL PROTECTED]

-- 
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 281901] Review Request: cheese - A webcam application for snapshots and movies

2007-09-07 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: cheese - A webcam application for snapshots and movies


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 11:52 EST ---
With the BRs/Requires for gstreamer-plugins-good(-devel) fixed, this looks good
to push.

-- 
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 281891] Review Request: cheese - A cheesy program to take pictures and videos from your web cam

2007-09-07 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: cheese - A cheesy program to take pictures and videos 
from your web cam


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 07:48 EST ---
Vendor prefixes just don't make sense.

Your bug number is lower, so you get to pick which review request you want to
close...

-- 
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 281901] Review Request: cheese - A webcam application for snapshots and movies

2007-09-07 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: cheese - A webcam application for snapshots and movies


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 10:56 EST ---
 Missing BRs in gtk+?

No, the cheese configure checks for 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 281901] Review Request: cheese - A webcam application for snapshots and movies

2007-09-07 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: cheese - A webcam application for snapshots and movies


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 10:46 EST ---
(In reply to comment #5)
  That doesn't seem to be checked for or used anywhere. Did you mean to have 
  the
  package itself as a requires instead? (make sure you require a version of
  gstreamer-plugins-good that has v4l2 plugins as well)
 
 Probably right.

FYI, it's gstreamer-plugins-good = 0.10.6-2

  BuildRequires: xorg-x11-proto-devel
  BuildRequires: libXxf86vm-devel
  
  what are those used for? There doesn't seem to be any X checks or headers 
  used
  in the sources.
 
 I added them because the build fails without them...

Missing BRs in gtk+?

  Should the Source be that instead?
 

http://live.gnome.org/Cheese/Releases?action=AttachFiledo=gettarget=%{name}-%{version}.tar.gz
 
 In the best of all possible worlds, yes
 in the real world, make source can't handle the ? in the url

Never mind then.

-- 
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 276961] Review Request: straw - Desktop news aggregator

2007-09-07 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: straw - Desktop news aggregator


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 11:39 EST ---
python-bsddb3 is no longer needed independently as it comes along with python.
did a little bit change in spec file .
koji build from scratch ran succesfully.

see here for koji build status.

[http://koji.fedoraproject.org/koji/getfile?taskID=151311name=build.log]

-- 
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 281901] Review Request: cheese - A webcam application for snapshots and movies

2007-09-07 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: cheese - A webcam application for snapshots and movies


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 10:40 EST ---
rpmlint output:
W: cheese mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 47)

(unimportant)

- MUST: The package must be named according to the Package Naming Guidelines.

OK

- MUST: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption on Package Naming Guidelines.

OK

- MUST: The package must meet the Packaging Guidelines.



- MUST: The package must be licensed with a Fedora approved license and meet the
Licensing Guidelines.

OK

- MUST: The License field in the package spec file must match the actual 
license.

OK

- MUST: If (and only if) the source package includes the text of the license(s)
in its own file, then that file, containing the text of the license(s) for the
package must be included in %doc.

OK

- MUST: The spec file must be written in American English.

No problems there, although this sounds a bit weird (unneeded?)
It also includes fancy graphical effects based on the 
gstreamer-backend.

- MUST: The spec file for the package MUST be legible. If the reviewer is unable
to read the spec file, it will be impossible to perform a review. Fedora is not
the place for entries into the Obfuscated Code Contest (http://www.ioccc.org/).

OK

- MUST: The sources used to build the package must match the upstream source, as
provided in the spec URL. Reviewers should use md5sum for this task. If no
upstream URL can be specified for this package, please see the Source URL
Guidelines for how to deal with this.

OK.

- MUST: The package must successfully compile and build into binary rpms on at
least one supported architecture.

OK.

- MUST: All build dependencies must be listed in BuildRequires, except for any
that are listed in the exceptions section of Packaging Guidelines; inclusion of
those as BuildRequires is optional. Apply common sense.

See above.

- MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.

OK.

- MUST: Every binary RPM package which stores shared library files (not just
symlinks) in any of the dynamic linker's default paths, must call ldconfig in
%post and %postun.

N/A

- MUST: If the package is designed to be relocatable, the packager must state
this fact in the request for review, along with the rationalization for
relocation of that specific package. Without this, use of Prefix: /usr is
considered a blocker.

N/A

- MUST: A package must own all directories that it creates. If it does not
create a directory that it uses, then it should require a package which does
create that directory. Refer to the Guidelines for examples.

OK

- MUST: A package must not contain any duplicate files in the %files listing.

OK.

- MUST: Permissions on files must be set properly. Executables should be set
with executable permissions, for example. Every %files section must include a
%defattr(...) line.

OK

- MUST: Each package must have a %clean section, which contains rm -rf
%{buildroot} (or $RPM_BUILD_ROOT).

OK

- MUST: Each package must consistently use macros, as described in the macros
section of Packaging Guidelines.

OK.

- MUST: The package must contain code, or permissable content. This is described
in detail in the code vs. content section of Packaging Guidelines.

OK.

- MUST: Large documentation files should go in a -doc subpackage. (The
definition of large is left up to the packager's best judgement, but is not
restricted to size. Large can refer to either size or quantity)

N/A

- MUST: If a package includes something as %doc, it must not affect the runtime
of the application. To summarize: If it is in %doc, the program must run
properly if it is not present.

OK

- MUST: Header files must be in a -devel package.

N/A

- MUST: Static libraries must be in a -static package.

N/A

- MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig' (for
directory ownership and usability).

N/A

- MUST: If a package contains library files with a suffix (e.g. libfoo.so.1.1),
then library files that end in .so (without suffix) must go in a -devel package.

N/A

- MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency: Requires: %{name} =
%{version}-%{release} 

N/A

- MUST: Packages must NOT contain any .la libtool archives, these should be
removed in the spec.

N/A

- MUST: Packages containing GUI applications must include a %{name}.desktop
file, and that file must be properly installed with desktop-file-install in the
%install section. This is described in detail in the desktop files section of
Packaging Guidelines. 

[Bug 281901] Review Request: cheese - A webcam application for snapshots and movies

2007-09-07 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: cheese - A webcam application for snapshots and movies


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 10:43 EST ---
 That doesn't seem to be checked for or used anywhere. Did you mean to have the
 package itself as a requires instead? (make sure you require a version of
 gstreamer-plugins-good that has v4l2 plugins as well)

Probably right.

 BuildRequires: xorg-x11-proto-devel
 BuildRequires: libXxf86vm-devel
 
 what are those used for? There doesn't seem to be any X checks or headers used
 in the sources.

I added them because the build fails without them...

 Should the Source be that instead?

http://live.gnome.org/Cheese/Releases?action=AttachFiledo=gettarget=%{name}-%{version}.tar.gz

In the best of all possible worlds, yes
in the real world, make source can't handle the ? in the url


-- 
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 239336] Review Request: thunderbird-enigmail - Enigmail extension for Mozilla Thunderbird

2007-09-07 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: thunderbird-enigmail - Enigmail extension for Mozilla 
Thunderbird


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||WONTFIX




-- 
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 205798] Review Request: xine-lib - The Xine library

2007-09-07 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: xine-lib - The Xine library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 10:32 EST ---
Package Change Request
==
Package Name: xine-lib
New Branches: EL-5
Updated EPEL Owners: scop,abompard


-- 
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 281901] Review Request: cheese - A webcam application for snapshots and movies

2007-09-07 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: cheese - A webcam application for snapshots and movies


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 12:15 EST ---
New Package CVS Request
===
Package Name: cheese
Short Description: A webcam application for snapshots and movies
Owners: mclasen
Branches: 
InitialCC: 
Cvsextras Commits:

-- 
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 271861] Review Request: vbetool - Run real-mode video BIOS code to alter hardware state

2007-09-07 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: vbetool -  Run real-mode video BIOS code to alter 
hardware state
Alias: vbetool

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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 12:13 EST ---
Spec URL: http://till.fedorapeople.org/review/vbetool.spec
SRPM URL: http://till.fedorapeople.org/review/vbetool-0.7-2.fc7.src.rpm

- ExcludeArch: ppc and ppc64 (does not build there)
- add Conflicts with old pm-utils 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 271661] Review Request: radeontool - Backlight and video output configuration tool for radeon cards

2007-09-07 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: radeontool - Backlight and video output configuration 
tool for radeon cards
Alias: radeontool

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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 12:12 EST ---
Spec URL: http://till.fedorapeople.org/review/radeontool.spec
SRPM URL: http://till.fedorapeople.org/review/radeontool-1.5-2.fc7.src.rpm

- add Conflicts with old pm-utils 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 244192] Review Request: eclipse-anyedit - AnyEdit plugin for Eclipse

2007-09-07 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: eclipse-anyedit - AnyEdit plugin for Eclipse


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 11:08 EST ---
(In reply to comment #3)
 The package don't build with IcedTea (derived OpenJDK normally shipped with 
 F8)

ok, this package was submitted before icedtea.  i'll take a look at it.  and
thanks for letting me know!

 Like Peter say the correct buildpath used in Fedora is 
 %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)

again, that is not the preferred buildpath.  read the packaging guidelines link
i posted in comment #2.

-- 
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 249365] Review Request: alpine - UW Alpine mail user agent

2007-09-07 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: alpine - UW Alpine mail user agent


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 13:06 EST ---
Naturally the alpine team released a new version while I was
working on it. The rpdoc.patch is no longer necessary, and
the problematic file is gone from the tarball:

http://staff.washington.edu/joshuadf/alpine/alpine-0.-2.fc7.src.rpm
http://staff.washington.edu/joshuadf/alpine/alpine.spec

There is a good chance 0. will become alpine 1.0.
By the way, I have tested an upgrade from livna pine and
everything went smoothly. I have local mbox and remote 
IMAP folders, and my password does not contain any local 
codepage chars.

-- 
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 250924] Review Request: flashrom - Simple program for reading/writing BIOS chips content

2007-09-07 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: flashrom - Simple program for reading/writing BIOS 
chips content


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 05:56 EST ---
New Package CVS Request
===
Package Name: flashrom
Short Description: Simple program for reading/writing BIOS chips content
Owners: peter
Branches: FC-6 F-7
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 251529] Review Request: armacycles-ad - A lightcycle game in 3D

2007-09-07 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: armacycles-ad - A lightcycle game in 3D


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 13:19 EST ---
armacycles-ad-0.2.8.2.1-5.fc7 has been pushed to the Fedora 7 stable 
repository.  If problems still persist, please make note of it in this bug 
report.

-- 
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 249365] Review Request: alpine - UW Alpine mail user agent

2007-09-07 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: alpine - UW Alpine mail user agent


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 13:15 EST ---
OK, proceed with requesting cvs branches... and then import.  Good work.  (and
then folks interesting in comaintaining, like jima and I, can request additional
access).

-- 
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 250924] Review Request: flashrom - Simple program for reading/writing BIOS chips content

2007-09-07 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: flashrom - Simple program for reading/writing BIOS 
chips content


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 05:02 EST ---
You need to follow this
http://fedoraproject.org/wiki/PackageMaintainers/NewPackageProcess
You are at step 8 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 251020] Review Request: libflaim - Flaim Database Engine

2007-09-07 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: libflaim - Flaim Database Engine


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 04:56 EST ---
(In reply to comment #8)

 - todo before import:
 * add a libflaim prefix to the patch, i.e. rename it to:
 libflaim-permissions.patch

Okay, done.

 * fix license tag

Okay, done.

Cheers
Chris

-- 
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 251529] Review Request: armacycles-ad - A lightcycle game in 3D

2007-09-07 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: armacycles-ad - A lightcycle game in 3D


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||ERRATA
   Fixed In Version||0.2.8.2.1-5.fc7




-- 
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 208034] Review Request: HippoDraw - Interactive and Python scriptable data analysis application

2007-09-07 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: HippoDraw - Interactive and Python scriptable data 
analysis application


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|CLOSED  |CLOSED
 Resolution|NEXTRELEASE |ERRATA
   Fixed In Version||1.21.1-2.fc7




-- 
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 252133] Review Request: quicksynergy - GTK frontend for synergy

2007-09-07 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: quicksynergy - GTK frontend for synergy


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 04:10 EST ---
Hello, Andrew:

Well, I must say there are some issues to be fixed on your
spec file. You can check how to write your spec file on
the following URL. Please read them first.

http://fedoraproject.org/wiki/Packaging/Guidelines
http://fedoraproject.org/wiki/Packaging/ReviewGuidelines

A. Description stage:
* SourceURL
  - For sourceforge source, please refer to:
http://fedoraproject.org/wiki/Packaging/SourceURL

* BuildRequires
  - gcc is in the expection of BuildRequires (see
Exceptions of
http://fedoraproject.org/wiki/Packaging/Guidelines )

B. %prep/%build/%install stage
* Macros
  - Please use macros properly
http://fedoraproject.org/wiki/Packaging/RPMMacros
For example, /usr/bin should be %{_bindir}.

* Debuginfo issue
  http://fedoraproject.org/wiki/Packaging/Debuginfo
  - Please don't strip binaries by yourself.
Stripping binaries disables to create debuginfo rpm.

* Timestamp
  - Please keep timestamp. When using install command,
add -p option.

* desktop-file-install
  - Desktop file must be installed by using desktop-file-install
(BuildRequires: desktop-file-utils is needed)
  - Note: Category Application is deprecated and should be
  removed (please check desktop-file-install usage).

* GTK+ icon
  - For 48x48 desktop icon, please move it to
%_datadir/icons/hicolor/48x48/apps and call gtk-update-icon-cache
(please check: GTK+ icon cache of
 http://fedoraproject.org/wiki/Packaging/ScriptletSnippets )

C. scriptlets
* update-desktop-database
  - The included desktop file does not contain any mime type
and calling update-desktop-database is not needed.

D. %files entry
* Documents
  - File INSTALL is for people who want to compile and install
this package by themselves and not needed for people
installing this package by using rpm.

* gettext .mo file
  - Gettext .mo files should be installed by using %find_lang.
Check the section Handling Locale Files of
http://fedoraproject.org/wiki/Packaging/Guidelines

Other things are:
* Rebuild
  - Rebuild itself fails.
http://koji.fedoraproject.org/koji/taskinfo?taskID=150879

-- 
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 266261] Review Request: knm_new-fonts - 12x12 JIS X 0208 Bitmap font

2007-09-07 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: knm_new-fonts - 12x12 JIS X 0208 Bitmap font


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 02:49 EST ---
Ok, updated. hopefully it should works enough.

Spec URL: http://tagoh.fedorapeople.org/knm_new-fonts/knm_new-fonts.spec
SRPM URL:
http://tagoh.fedorapeople.org/knm_new-fonts/knm_new-fonts-1.1-2.fc8.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 266261] Review Request: knm_new-fonts - 12x12 JIS X 0208 Bitmap font

2007-09-07 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: knm_new-fonts - 12x12 JIS X 0208 Bitmap font


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 02:39 EST ---
(In reply to comment #3)
 - Building with mock gives an error:
 + install -m 0644 -p 'fonts/*.pcf.gz'
 /var/tmp/knm_new-fonts-1.1-1.fc7-root-mockbuild/usr/share/fonts/knm_new-fonts/
 install: cannot stat `fonts/*.pcf.gz': No such file or directory

Doh. my bad. however:

 - in %prep
 instead of
 %{__mkdir_p} $RPM_BUILD_DIR/%{name}-%{version}
 cd $RPM_BUILD_DIR/%{name}-%{version}
 tar xfz %{SOURCE0}
 this should be used:
 %setup -q -n fonts

I'm worrying about following that since fonts is too generic and it might
affects to others perhaps.


-- 
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 281891] Review Request: cheese - A cheesy program to take pictures and videos from your web cam

2007-09-07 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: cheese - A cheesy program to take pictures and videos 
from your web cam


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 02:32 EST ---

Matthias,

A vendor prefix was added based on an earlier review and a new draft was
proposed to clarify this though it hasn't been ratified yet. I can close this
review request if you want me to.  

-- 
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 281891] Review Request: cheese - A cheesy program to take pictures and videos from your web cam

2007-09-07 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: cheese - A cheesy program to take pictures and videos 
from your web cam


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 02:14 EST ---
Urgh, I did the very same thing today:
https://bugzilla.redhat.com/show_bug.cgi?id=281901

My packaging was done independently; maybe comparing it can lead to an optimal
end result.

-- 
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 281891] Review Request: cheese - A cheesy program to take pictures and videos from your web cam

2007-09-07 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: cheese - A cheesy program to take pictures and videos 
from your web cam


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 02:07 EST ---
*** Bug 248314 has been marked as a duplicate of this bug. ***

-- 
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 243955] Review Request: python-xlrd - Library to extract data from Microsoft Excel (tm) spreadsheet files

2007-09-07 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-xlrd - Library to extract data from Microsoft 
Excel (tm) spreadsheet files


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 07:07 EST ---
I have missed to check noarch rpm. There was many errors. They have been fixed
by me here:
http://www.salstar.sk/pub/fedora/SPECS/python-xlrd.spec
http://www.salstar.sk/pub/fedora/SRPMS/7/python-xlrd-0.6.1-2.fc7.src.rpm

* Fri Sep 7 2007 Jan ONDREJ (SAL) ondrejj(at)salstar.sk 0.6.1-2
- namewithoutprefix removed, too complicated and not required
- added directory xlrd
- permissions for python scripts set to 644
- files converted from DOS line-feeds to UNIX format
- #!/usr/bin/env python added to beginning runxlrd script  
- removed doc and examples from site-packages/xlrd directory 

I have installed and tested this package on my example xls file. Works as 
expected.

Anybody can approve this package?


-- 
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 225849] Merge Review: gnuplot

2007-09-07 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: gnuplot


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 06:01 EST ---
I just build gnuplot-4.2.0-5.fc8 when emacs  files are moved to
/usr/share/emacs/site-lisp/gnuplot subdirectory.

-- 
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 251020] Review Request: libflaim - Flaim Database Engine

2007-09-07 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: libflaim - Flaim Database Engine


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 05:10 EST ---
Package is APPROVED, you need to follow:
http://fedoraproject.org/wiki/PackageMaintainers/NewPackageProcess
(step 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 260441] Review Request: pguiman - The PostgreSQL database server managing tool

2007-09-07 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: pguiman - The PostgreSQL database server managing tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 04:24 EST ---
( Well, at some time there was a suggestion that the assignee should be back
  to the submitter after the review is accepted, however now it is not.
  Please leave the assignee as it is even after the review is accepted )

-- 
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 250924] Review Request: flashrom - Simple program for reading/writing BIOS chips content

2007-09-07 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: flashrom - Simple program for reading/writing BIOS 
chips content


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 04:20 EST ---
OK, thanks.
But I still can't checkout such module:

[EMAIL PROTECTED] FE]$ cvs co flashrom
cvs server: cannot find module `flashrom' - ignored
cvs [checkout aborted]: cannot expand modules
[EMAIL PROTECTED] FE]$


-- 
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 249365] Review Request: alpine - UW Alpine mail user agent

2007-09-07 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: alpine - UW Alpine mail user agent


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 13:48 EST ---
New Package CVS Request
===
Package Name: alpine
Short Description: UW Alpine mail user agent
Owners: joshuadf,jima,rdieter
Branches: FC-6 F-7 EL-4 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 281901] Review Request: cheese - A webcam application for snapshots and movies

2007-09-07 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: cheese - A webcam application for snapshots and movies


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs+




-- 
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 253812] Package Review: taipei-fonts (split from fonts-chinese).

2007-09-07 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: Package Review: taipei-fonts (split from fonts-chinese).


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 04:31 EST ---
I think Jens can be good Reviewer for this package. 
Removing myself from 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 260441] Review Request: pguiman - The PostgreSQL database server managing tool

2007-09-07 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: pguiman - The PostgreSQL database server managing tool


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 03:57 EST ---
New Package CVS Request
===
Package Name: pguiman
Short Description: The PostgreSQL database server managing tool
Owners: odvorace
Branches: FC-6 F-7
InitialCC: 
Cvsextras Commits: no

-- 
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 249236] Review Request: kde-style-domino - KDE style with a soft look

2007-09-07 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: kde-style-domino - KDE style with a soft look


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEEDINFO|ASSIGNED




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 13:38 EST ---
Sorry, back in school, you know.  I'll make the adjustments this weekend.

-- 
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 249365] Review Request: alpine - UW Alpine mail user agent

2007-09-07 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: alpine - UW Alpine mail user agent


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 14:24 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 250463] Review Request: pyvserver - Python interface to the Linux-VServer kernel interface

2007-09-07 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: pyvserver - Python interface to the Linux-VServer 
kernel interface


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 14:24 EST ---
This pre-release should fix the License field and the ChangeLog warning.

New spec file: http://dev.laptop.org/~mstone/releases/SPECS/pyvserver.spec
New SRPM: 
http://dev.laptop.org/~mstone/releases/SRPMS/pyvserver-1.0-0.3.fc7.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 251020] Review Request: libflaim - Flaim Database Engine

2007-09-07 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: libflaim - Flaim Database Engine


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 05:25 EST ---
New Package CVS Request
===
Package Name: libflaim
Short Description: Flaim Database Engine
Owners: snecker
Branches: F-7
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 225764] Merge Review: fonts-ISO8859-2

2007-09-07 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: fonts-ISO8859-2


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED], fedora-
   ||[EMAIL PROTECTED],
   ||[EMAIL PROTECTED]
 AssignedTo|[EMAIL PROTECTED]  |[EMAIL PROTECTED]
  QAContact|fedora-package- |[EMAIL PROTECTED]
   |[EMAIL PROTECTED]   |
   Flag|fedora-review?  |




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 04:31 EST ---
I think Jens can be good Reviewer for this package. 
Removing myself from review.

-- 
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.
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 251020] Review Request: libflaim - Flaim Database Engine

2007-09-07 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: libflaim - Flaim Database Engine


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 14:33 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 250924] Review Request: flashrom - Simple program for reading/writing BIOS chips content

2007-09-07 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: flashrom - Simple program for reading/writing BIOS 
chips content


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 14: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 252133] Review Request: quicksynergy - GTK frontend for synergy

2007-09-07 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: quicksynergy - GTK frontend for synergy


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 03:46 EST ---
You need to Buildrequire: synergy too.

-- 
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 187317] Review Request: mindi

2007-09-07 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: mindi


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 04:54 EST ---
The version of busybox used by mindi is statically linked. So I have a patch to
handle that as it's not supported in busybox 1.2.x. 

I have the need for precise functions built-in and some other *NOT* built in
(such as modules support).

I keep using an older version of busybox as that binary is cfritical for our
tool, and I don't want to pass my time testing newer versions all the time. I
need a very stable busybox for the recovery env, for which e.g. NFS is no
problem ...

3 good reasons IMO for the project to keep a separated package for that.

For afio and buffer I can also provide packages for fedora if there is a need
indeed to provide them (I thought they were already part of the distro and
didn't check that indeed).

-- 
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 205798] Review Request: xine-lib - The Xine library

2007-09-07 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: xine-lib - The Xine library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 14:39 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 281901] Review Request: cheese - A webcam application for snapshots and movies

2007-09-07 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: cheese - A webcam application for snapshots and movies


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 14:42 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 253249] Review Request: classads - a powerful and flexible, semi-structured representation of data

2007-09-07 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: classads - a powerful  and flexible, semi-structured 
representation of data


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 14:54 EST ---
I've done a review of the SRPM provided, here are some comments:

- tarfile matches upstream with the following md5 sum:

$ md5sum classads-1.0rc2.tar.gz 
d3e08a8706b0ad405048192d5f049d28  classads-1.0rc2.tar.gz

- managed to build rpms for i386:

$ sudo rpmbuild -ba classads.spec
error: Failed build dependencies:
pcre-devel is needed by classads-1.0-0.1.rc2.i386

$ rpm -qa|grep pcre
pcre-6.6-1.1

$ sudo rpmbuild -ba classads.spec --nodeps
[...]
Wrote: /usr/src/redhat/SRPMS/classads-1.0-0.1.rc2.src.rpm
Wrote: /usr/src/redhat/RPMS/i386/classads-1.0-0.1.rc2.i386.rpm
Wrote: /usr/src/redhat/RPMS/i386/classads-devel-1.0-0.1.rc2.i386.rpm
Wrote: /usr/src/redhat/RPMS/i386/classads-static-1.0-0.1.rc2.i386.rpm
Wrote: /usr/src/redhat/RPMS/i386/classads-debuginfo-1.0-0.1.rc2.i386.rpm

Note: pcre was installed but pcre-devel was not; rpmbuild still completed
successfully, so you may want to double check the need for pcre-devel.

- rpmlint gives the following output:

$ rpmlint /usr/src/redhat/SRPMS/classads-1.0-0.1.rc2.src.rpm 
W: classads invalid-license Condor Public License
W: classads strange-permission classads.spec 0600

$ rpmlint  /usr/src/redhat/RPMS/i386/classads-1.0-0.1.rc2.i386.rpm
W: classads invalid-license Condor Public License

$ rpmlint  /usr/src/redhat/RPMS/i386/classads-devel-1.0-0.1.rc2.i386.rpm
W: classads-devel invalid-license Condor Public License

$ rpmlint /usr/src/redhat/RPMS/i386/classads-static-1.0-0.1.rc2.i386.rpm
W: classads-static invalid-license Condor Public License

$ rpmlint  /usr/src/redhat/RPMS/i386/classads-debuginfo-1.0-0.1.rc2.i386.rpm
W: classads-debuginfo invalid-license Condor Public License

I understand the license issue is being worked on, the specfile permissions
should be corrected.

- checked provides of the built binary rpms:

$ rpm -qp /usr/src/redhat/RPMS/i386/classads-1.0-0.1.rc2.i386.rpm --provides
libclassad.so.0  
libclassad_ns.so.0  
classads = 1.0-0.1.rc2

$ rpm -qp /usr/src/redhat/RPMS/i386/classads-devel-1.0-0.1.rc2.i386.rpm 
--provides
classads-devel = 1.0-0.1.rc2

$ rpm -qp /usr/src/redhat/RPMS/i386/classads-static-1.0-0.1.rc2.i386.rpm 
--provides
classads-static = 1.0-0.1.rc2

$ rpm -qp /usr/src/redhat/RPMS/i386/classads-debuginfo-1.0-0.1.rc2.i386.rpm
--provides
libclassad.so.0.0.0.debug  
libclassad_ns.so.0.0.0.debug  
classads-debuginfo = 1.0-0.1.rc2


Seems the three remaining issues are dealing with the licensing, fixing the
permissions on the specfile, and confirming or removing the pcre-devel
dependency; everything else looks good.

Nuno


-- 
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 247227] Review Request: nabi - hangul and hanja X input method

2007-09-07 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: nabi - hangul and hanja X input method
Alias: nabi

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 14:37 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 252010] Review Request: rpmrebuild - A tool to build rpm file from rpm database

2007-09-07 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: rpmrebuild - A tool to build rpm file from rpm database


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 15:35 EST ---
Packages updated:

http://www.the-silvas.com/rpmrebuild.spec
http://www.the-silvas.com/rpmrebuild-2.1.1-6.src.rpm

My comments:

- I've read http://fedoraproject.org/wiki/PackageMaintainers/Join. There it
states: Create an account in the  Fedora Account System *when you have found a
sponsor and got your package approved* - My package haven't gotten a sponsor
and I my package hasn't been approved, so I haven't created an account yet.

- I looked around the web, and I didn't find any examples of anyone using a
src.rpm as Source0: on their specs. I have added some comments to document the
fact that the tar.gz upstream file is coming from a src.rpm.

- I have contacted the upstream developer, and I have asked him to make the
tar.gz available officially via the sf.net site.

- I've read the comments on BUG 177841, and I am doing the best I can with this
package with the time I have in my hands, I unfortunately will not have much
time left to review other packages. I might potentially in the future be able to
submit other packages, but I want to be able to get my first one 'officially' in
   to get used to the process. 


-- 
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 283081] New: Review Request: condor - Batch system for High Throughput Computing

2007-09-07 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=283081

   Summary: Review Request: condor - Batch system for High
Throughput Computing
   Product: Fedora
   Version: devel
  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://grid.et.redhat.com/files/condor.spec
SRPM URL: http://grid.et.redhat.com/files/condor-6.9.5-0.1.20070907cvs.src.rpm
Description:
Condor is a specialized workload management system for compute-intensive jobs. 
Like other full-featured batch systems, Condor provides a job queueing 
mechanism, scheduling policy, priority scheme, resource monitoring, and 
resource management. Users submit their serial or parallel jobs to Condor, 
Condor places them into a queue, chooses when and where to run the jobs based 
upon a policy, carefully monitors their progress, and ultimately informs the 
user upon completion.

NOTES:
 1) The license is currently unacceptable, but negotiations are nearly complete 
for a compatible license
 2) I'm not sure what to do about the non-standard-uid error
 3) I don't know how to handle the no-soname warning in -devel
 4) The permissions on /var/lib/condor/execute are intentional

-- 
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 244192] Review Request: eclipse-anyedit - AnyEdit plugin for Eclipse

2007-09-07 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: eclipse-anyedit - AnyEdit plugin for Eclipse


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 15:41 EST ---
SPEC URL: 
http://www.stl.gtri.gatech.edu/rmyers/eclipse-anyedit/eclipse-anyedit.spec
SRPM URL:
http://www.stl.gtri.gatech.edu/rmyers/eclipse-anyedit/eclipse-anyedit-1.8.2-1.fc8.src.rpm

changelog: 
* Fri Sep  7 2007 Rob Myers [EMAIL PROTECTED] 1.8.2-1
- update to 1.8.2
- update unicode patch for IcedTea compatibility

-- 
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 253249] Review Request: classads - a powerful and flexible, semi-structured representation of data

2007-09-07 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: classads - a powerful  and flexible, semi-structured 
representation of data


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


[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 252010] Review Request: rpmrebuild - A tool to build rpm file from rpm database

2007-09-07 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: rpmrebuild - A tool to build rpm file from rpm database


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 16:00 EST ---
It is not wrong that you do not have an account. It was the reason why you need
to follow the page I showed you. People that have an account and are sponsored
need to follow the steps on another page. One steps in PackageMaintainers/Join
was to block FE-NEEDSPONSOR, what I also did for you. But I also might have been
wrong, so I explained to you, why I decided to block FE-NEEDSPONSOR. However, I
hope your confusion about comment:10 is now blown away.

If you do not have time to review other packages, then chances to be sponsored
are low, because a sponsor has nothing to decide, whether or not you know the
guidelines well enough.

-- 
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 252010] Review Request: rpmrebuild - A tool to build rpm file from rpm database

2007-09-07 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: rpmrebuild - A tool to build rpm file from rpm database


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 16:33 EST ---
I believe that Anderson is following the proper steps  from
http://fedoraproject.org/wiki/PackageMaintainers/Join

Create an account in the  Fedora Account System when you have found a sponsor
and got your package approved

Anderson does not have a sponsor and does not have his package approved.

It is true that he would be helping by looking at other packages and making
notes but, as said in http://fedoraproject.org/wiki/PackageMaintainers/Join:

Read some other package submissions to learn about packaging and gain
familiarity with the process and requirements.

Once he gets a sponsor, packaged approved, then an account he can get in the
fedorabugs group as stated:

Now, repeat the process using fedorabugs as the Groupname. This will allow you
to make changes to the state of bugs in Bugzilla, which is what you'll need to
do to get them checked in. It will also allow you to do complete package
reviews, including approving packages yourself!

-- 
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 253249] Review Request: classads - a powerful and flexible, semi-structured representation of data

2007-09-07 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: classads - a powerful  and flexible, semi-structured 
representation of data


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 16:47 EST ---
(In reply to comment #3)

 Note: pcre was installed but pcre-devel was not; rpmbuild still completed
 successfully, so you may want to double check the need for pcre-devel.

The code will use POSIX regex (regcomp/regexec) if pcre is not available, and
fall back to no regex support if neither is available. If this is a problem I
could probably change the way configure.ac works to fail if pcre is not
available, and modify the spec to require pcre during configure's run. Though it
might be ok to just not specify --nodeps.

I've updated the SRPM to fix the classads.spec permissions, please re-download.

-- 
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 283081] Review Request: condor - Batch system for High Throughput Computing

2007-09-07 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: condor - Batch system for High Throughput Computing


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 16:54 EST ---
ADDITIONAL NOTE: You will need F7 or RHEL 3 to compile this code. I recommend
compiling with mock and a f7-i386 config. Support for RHEL 4+5 is forthcoming.

-- 
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 249365] Review Request: alpine - UW Alpine mail user agent

2007-09-07 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: alpine - UW Alpine mail user agent


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 16:55 EST ---
OK, devel built successfully:
http://koji.fedoraproject.org/koji/buildinfo?buildID=18087

Since alpine 1.0 release will be very soon, I wouldn't mind 
waiting to build for other tags. Maybe everyone who was watching
this bug could bang on the binary with their own local (al)pine
configurations.

Closing with 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 252010] Review Request: rpmrebuild - A tool to build rpm file from rpm database

2007-09-07 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: rpmrebuild - A tool to build rpm file from rpm database


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC|[EMAIL PROTECTED]|




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 16:58 EST ---
(In reply to comment #14)
 I believe that Anderson is following the proper steps  from
 http://fedoraproject.org/wiki/PackageMaintainers/Join

Did this review request block FE-NEEDSPONSOR before I changed it? No, it did
not. I quote from PackageMaintainers/Join:

| Some potential sponsors will look at the FE-NEEDSPONSOR bug in bugzilla to 
find 
| packages to review. You can add your package to this list by editing your 
review
| request bug and adding FE-NEEDSPONSOR in the 'Bug xyz blocks' field (where 
xyz 
| is the bug number for your review request).

Sorry, I do not know how to express myself in English good enough, if it is
still not clear what I wrote, then please ignore it completely.

-- 
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 253249] Review Request: classads - a powerful and flexible, semi-structured representation of data

2007-09-07 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: classads - a powerful  and flexible, semi-structured 
representation of data


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 18:08 EST ---
Just to clarify, I used --nodeps just to verify the dependency, I'm not
suggesting it should be built that way. I would suggest leaving the
pcre/pcre-devel dependency in, if that results in a better environment for the
program/build.

Just built in mock, all looks good:

$ mock -r fedora-devel-i386.cfg classads-1.0-0.1.rc2.src.rpm
init
clean
prep
This may take a while
create cache
setup
build
ending
done
Results and/or logs in: /var/lib/mock/fedora-development-i386/result

$ ls -alF /var/lib/mock/fedora-development-i386/result
total 4492
drwxr-sr-x 2 nsantos mock4096 Sep  7 18:06 ./
drwxr-sr-x 5 nsantos mock4096 Sep  7 18:00 ../
-rw-r--r-- 1 nsantos mock  200735 Sep  7 18:06 build.log
-rw-rw-r-- 1 nsantos mock  458011 Sep  7 18:06 classads-1.0-0.1.rc2.i386.rpm
-rw-rw-r-- 1 nsantos mock  894714 Sep  7 18:02 classads-1.0-0.1.rc2.src.rpm
-rw-rw-r-- 1 nsantos mock 2317851 Sep  7 18:06
classads-debuginfo-1.0-0.1.rc2.i386.rpm
-rw-rw-r-- 1 nsantos mock  100282 Sep  7 18:06 
classads-devel-1.0-0.1.rc2.i386.rpm
-rw-rw-r-- 1 nsantos mock  557919 Sep  7 18:06 
classads-static-1.0-0.1.rc2.i386.rpm
-rw-r--r-- 1 nsantos mock 165 Sep  7 18:00 mockconfig.log
-rw-r--r-- 1 nsantos mock   19295 Sep  7 18:06 root.log

$ rpmlint /var/lib/mock/fedora-development-i386/result/*.rpm
W: classads invalid-license Condor Public License
W: classads invalid-license Condor Public License
W: classads-debuginfo invalid-license Condor Public License
W: classads-devel invalid-license Condor Public License
W: classads-static invalid-license Condor Public License


-- 
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 283081] Review Request: condor - Batch system for High Throughput Computing

2007-09-07 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: condor - Batch system for High Throughput Computing


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


[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 283081] Review Request: condor - Batch system for High Throughput Computing

2007-09-07 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: condor - Batch system for High Throughput Computing


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 18:17 EST ---
In i386, you should be building with fedora-devel-i386.cfg, as this is what will
go into the next Fedora release (F8).

-- 
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 187828] Review Request: log4net - A tool to output log statements to a variety of output targets

2007-09-07 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: log4net - A tool to output log statements to a variety 
of output targets


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Product|Fedora Extras   |Fedora

[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|CLOSED  |ASSIGNED
   Keywords||Reopened
 Resolution|WONTFIX |




-- 
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 187828] Review Request: log4net - A tool to output log statements to a variety of output targets

2007-09-07 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: log4net - A tool to output log statements to a variety 
of output targets


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-review?




-- 
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 187828] Review Request: log4net - A tool to output log statements to a variety of output targets

2007-09-07 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: log4net - A tool to output log statements to a variety 
of output targets


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




-- 
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 187828] Review Request: log4net - A tool to output log statements to a variety of output targets

2007-09-07 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: log4net - A tool to output log statements to a variety 
of output targets


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 18:30 EST ---
Sorry for the noise folks. I would like to re-open this review request.

Spec URL: http://snecker.fedorapeople.org/log4net/log4net.spec
SRPM URL: http://snecker.fedorapeople.org/log4net/log4net-1.2.10-1.fc7.src.rpm

Comments:

- This is a mono package and find-debuginfo.sh is failing to generate the
requisite package, though I understand this is somewhat commonplace.
- Nant is used to build the package with Strong Name signing.
- Rpmlint is quiet on source, no doc error on devel and complains about no
binary in binary rpm. I understand the latter is also not a blocker.

I am already the maintainer of a couple of other packages and log4net forms part
of around 4-5 which require iFolder to work on Fedora.

Comments appreciated.

Cheers
Chris

-- 
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 241553] Review Request: safekeep - simple, centralized configuration for rdiff-backup

2007-09-07 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: safekeep -  simple, centralized configuration for 
rdiff-backup


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 19:34 EST ---
I have just released version 1.0.2 that addresses (most of) the issues mentioned
above (except for the -client package, as explained above):
   
http://downloads.sourceforge.net/safekeep/safekeep-1.0.2-1.fc7.src.rpm?use_mirror=osdn

-- 
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 283081] Review Request: condor - Batch system for High Throughput Computing

2007-09-07 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: condor - Batch system for High Throughput Computing


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 20:31 EST ---
Please do the review with f7-i386 while I work with upstream to get an F8 port.
The main issues should be in the spec and once they are ironed out rev'ing the
source should be straightforward (and necessary anyway).

-- 
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 241553] Review Request: safekeep - simple, centralized configuration for rdiff-backup

2007-09-07 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: safekeep -  simple, centralized configuration for 
rdiff-backup


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 20:35 EST ---
(In reply to comment #15)

Okay I'll take a look at it tonite. 
For those of you playing along. I'm working as closely as possible with Dimi as
the upstream developer to keep the fedora packaging in sync with the upstream.
I'm still expecting to act as the maintainer for the Fedora packages, and the
gate-keeper to the bits in Fedora cvs until Dimi gets Fedora contributor status
and we can share maintainership duties in Fedora space for this package.

-jef

-- 
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 208034] Review Request: HippoDraw - Interactive and Python scriptable data analysis application

2007-09-07 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: HippoDraw - Interactive and Python scriptable data 
analysis application


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 21:59 EST ---
 * IMPORTANT: license is incorrect: you are licensing this as LGPL, but some
 parts of this application are linking to Qt 3 (free edition), which is
 licensed under the GPL; therefore this application must also be licensed as
 GPL.

This is incorrect. Just as kdelibs can be under the LGPL, this package can too. 
The guidelines explicitly say that the License field should indicate the 
license of the package itself, not the entire linked combination. Now, it is 
true that unless you have a commercial license for Qt, you cannot distribute 
the linked combination under anything other than the GPL. But this still 
doesn't mean the license of the package has to be declared as GPL in the RPM 
License tag, just as in the case of kdelibs.

-- 
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 208034] Review Request: HippoDraw - Interactive and Python scriptable data analysis application

2007-09-07 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: HippoDraw - Interactive and Python scriptable data 
analysis application


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 22:08 EST ---
(In reply to comment #85)
  * IMPORTANT: license is incorrect: you are licensing this as LGPL, but some
  parts of this application are linking to Qt 3 (free edition), which is
  licensed under the GPL; therefore this application must also be licensed as
  GPL.
 
 This is incorrect. Just as kdelibs can be under the LGPL, this package can 
 too. 
..

Please file a new bug report.

-- 
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 208034] Review Request: HippoDraw - Interactive and Python scriptable data analysis application

2007-09-07 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: HippoDraw - Interactive and Python scriptable data 
analysis application


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 22:13 EST ---
And note that 1.21.1. is surely licensed under GPL.

-- 
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 281901] Review Request: cheese - A webcam application for snapshots and movies

2007-09-07 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: cheese - A webcam application for snapshots and movies


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||RAWHIDE




--- Additional Comments From [EMAIL PROTECTED]  2007-09-07 22:15 EST ---
built in rawhide 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


  1   2   >