[Bug 434954] Review Request: writRecogn - A CJK handwriting recognizer

2008-02-26 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: writRecogn - A CJK handwriting recognizer


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-27 02:40 EST ---
Jens,
Thanks for your comments.

Regarding comment #2:
The building requires are mainly for koji build, otherwise the koji cannot find
essential libraries.

Regarding comment #3:
The lightbulb.png shows a small icon in "Recognize" buttom. This will be fixed
in next release.

Regarding comment #4,#5:
Addressed accordingly.

Regards,
Ding-Yi Chen





-- 
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 226507] Merge Review: tux

2008-02-26 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: tux


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium
Product|Fedora Extras   |Fedora
Version|devel   |rawhide

[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||CANTFIX




--- Additional Comments From [EMAIL PROTECTED]  2008-02-27 02:22 EST ---
Tux has been removed from Fedora.

-- 
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 434593] Review Request: stalonetray - A stand alone notification area implementation

2008-02-26 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: stalonetray - A stand alone notification area 
implementation


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-27 02:16 EST ---
(In reply to comment #1)
> Good to use make install command as
> make install DESTDIR=$RPM_BUILD_ROOT INSTALL="install -p"

done

> License looks to me GPLv2+

Ok. Changed.

> Can we have summary like this?
> Summary:A stand alone notification area

Sure. Done.

Spec URL: http://svahl.fedorapeople.org/stalonetray/stalonetray.spec
SRPM URL: 
http://svahl.fedorapeople.org/stalonetray/stalonetray-0.7.6-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 433135] Review Request: anyRemote - Bluetooth remote control

2008-02-26 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: anyRemote - Bluetooth remote control


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-27 01:36 EST ---
(In reply to comment #10)
> (In reply to comment #9)
> >To make it sure that all installed files are correctly built from
>   open source files, we do not allow to install pre-compiled files
>   (like .jar files) directly.
> It is written in packaging/Guidlines:
> : Some software (usually related to compilers or cross-compiler
> : environments) cannot be build without the use of a previous toolchain
> : or development environment (open source). If you have a package which meets
> : this criteria, contact the Fedora Packaging Committee for approval.
> To build *jar i use Sun WTK, which can be considered as "toolchain", but 
> it is not open source at all. I need an advice, how it is possible to 
> distribute *jar anyway (Livna ?) because without *jar this project will be 
> almost unuseful.


Isn't is possible to build jar files in this package by using
java related packages provided by Fedora? As far as I know
most Fedora java packages use ant to rebuild jar files.


-- 
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 434547] Review Request: aprsd - Internet gateway and client access to amateur radio APRS packet data

2008-02-26 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: aprsd - Internet gateway and client access to amateur 
radio APRS packet data


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-02-27 00:36 EST ---
I know upstream seems dead, I know there are certain issues in operating
a APRSd gateway but again there are people using this so I think this
package will help as there are a lot of fedora users who compile this from 
source.

-- 
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 434593] Review Request: stalonetray - A stand alone notification area implementation

2008-02-26 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: stalonetray - A stand alone notification area 
implementation


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 23:49 EST ---
Good to use make install command as
make install DESTDIR=$RPM_BUILD_ROOT INSTALL="install -p"

License looks to me GPLv2+

Can we have summary like this?
Summary:A stand alone notification area


-- 
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 435064] New: Review Request: midisport-firmware - firmware files for M-Audio/Midiman USB MIDI and Audio devices

2008-02-26 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=435064

   Summary: Review Request: midisport-firmware - firmware files for
M-Audio/Midiman USB MIDI and Audio devices
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://www.codemonkey.org.uk/junk/midisport-firmware.spec
SRPM URL: http://www.codemonkey.org.uk/junk/midisport-firmware-1.2-1.src.rpm
Description: Firmware for M-Audio/Midiman USB MIDI and Audio devices

I based this specfile on the ivtv-firmware spec.

TODO: add udev rules to autoload firmware.

-- 
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 381241] Review Request: ncl - NCAR Command Language and NCAR Graphics

2008-02-26 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: ncl -  NCAR Command Language and NCAR Graphics


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   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 381241] Review Request: ncl - NCAR Command Language and NCAR Graphics

2008-02-26 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: ncl -  NCAR Command Language and NCAR Graphics


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review-




--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 23:04 EST ---
After reviewing all the comments here it seems that Patrice has done a
very thorough review.  The only thing missing seems to be the "source 
matches upstream" check which was omitted due to Pat's (understandable) 
stance on the annoying must-register-to-download primary web site.  After
registering I can confirm: 

 + source matches upstream with sha1sum:
eca618a1179356e2950608da0e9ca210f1caa1a8  ncl_ncarg_src-5.0.0.tar.gz
eca618a1179356e2950608da0e9ca210f1caa1a8  ../ncl_ncarg_src-5.0.0.tar.gz
 + specfile is clean and legible
 + license appears to be correct
 + builds in mock on F8 x86_64

Since there appear to be no blockers and since Patrice has already done a 
very thorough review, I'd like to APPROVE this 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 226447] Merge Review: sysfsutils

2008-02-26 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: sysfsutils


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 23:00 EST ---
Whoops, sorry, my mistake... Got close-happy plowing through bugs yesterday... 
:)

-- 
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 433735] Review Request: libgcroots - Root acquisition library for Garbage Collector

2008-02-26 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: libgcroots - Root acquisition library for Garbage 
Collector


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 22:43 EST ---
Review:
+ package builds in mock (rawhide i386).
koji build => http://koji.fedoraproject.org/koji/taskinfo?taskID=472349
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
f5fa9b60e506c70a041ef03ab26dbd18  libgcroots-0.2.1.tar.bz2
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ BuildRequires are proper.
+ Compiler flags used correctly.
+ defattr usage is correct.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no static libraries.
+ gcroots.pc file present.
+ -devel subpackage exists.
+ no .la files.
+ no translations are available.
+ Does owns the directories it creates.
+ ldconfig scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ Package libgcroots-0.2.1-2.fc9->
  Provides: libgcroots.so.0
  Requires: libc.so.6 libc.so.6(GLIBC_2.0) libc.so.6(GLIBC_2.1)
libc.so.6(GLIBC_2.1.3) libdl.so.2 libgcroots.so.0 rtld(GNU_HASH)
+ Package libgcroots-devel-0.2.1-2.fc9 ->
  Requires: libgcroots.so.0 pkgconfig libgcroots = 0.2.1-2.fc9
+ Not a GUI App.
APPROVED.
 

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

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


[Bug 433735] Review Request: libgcroots - Root acquisition library for Garbage Collector

2008-02-26 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: libgcroots - Root acquisition library for Garbage 
Collector


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 22:20 EST ---
Thanks for catching that up. updated:

http://tagoh.fedorapeople.org/libgcroots/libgcroots.spec
http://tagoh.fedorapeople.org/libgcroots/libgcroots-0.2.1-2.fc9.src.rpm


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

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


[Bug 435058] New: Review Request: splat - Analyze point-to-point terrestrial RF communication links

2008-02-26 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=435058

   Summary: Review Request: splat -  Analyze point-to-point
terrestrial RF communication links
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://bjensen.fedorapeople.org/pkgs/hams/SPECS/splat.spec
SRPM URL: 
http://bjensen.fedorapeople.org/pkgs/hams/SRPMS/splat-1.2.1-4.fc9.src.rpm
Description:


SPLAT! is a Surface Path Length And Terrain analysis application written for
Linux and Unix workstations. SPLAT! analyzes point-to-point terrestrial RF 
communication links, and provides information useful to communication system
designers and site engineers.

-- 
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 434954] Review Request: writRecogn - A CJK handwriting recognizer

2008-02-26 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: writRecogn - A CJK handwriting recognizer


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 22:01 EST ---
Again about naming I would suggest naming the programs something like
%{_bindir}/WritRecogn and %{_bindir}/WritRecogn-manager.

It would be nice to have a .desktop file for the main program.

-- 
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 434954] Review Request: writRecogn - A CJK handwriting recognizer

2008-02-26 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: writRecogn - A CJK handwriting recognizer


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 22:02 EST ---
Created an attachment (id=296018)
 --> (https://bugzilla.redhat.com/attachment.cgi?id=296018&action=view)
writRecogn.spec-1.patch

minor spec cleanup suggestions

-- 
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 434954] Review Request: writRecogn - A CJK handwriting recognizer

2008-02-26 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: writRecogn - A CJK handwriting recognizer


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 21:51 EST ---
Program seems to run and work ok - I realise it is still alpha at this stage.
I just note a warning at runtime though:

** (writRecogn:9633): WARNING **: Couldn't find pixmap file: lightbulb.png

and also some debug output (but that is ok at this stage).

Also just wondering if it is necessary to ship all the files in
"/usr/share/writRecogn/data/"?

-- 
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 434954] Review Request: writRecogn - A CJK handwriting recognizer

2008-02-26 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: writRecogn - A CJK handwriting recognizer


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 21:39 EST ---
Personally I would suggest calling it WritRecogn or writrecogn.
The mixed lower- and uppercase looks a little awkward to me,
for the record anyway.

| BuildRequires:  gtk2-devel >= 2.10 libxml2-devel >= 2.6 sqlite-devel >= 3.0
libtool >= 1.5
| Requires:   gtk2 >= 2.10  libxml2 >= 2.6  sqlite >= 3.0

I think it should sufficient to specific the minimum versions for the 
buildrequires and then the explicit Requires can be droppped.
They are all satisfied by Fedora 7 anyway so not really necessary
for Fedora 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 434954] Review Request: writRecogn - A CJK handwriting recognizer

2008-02-26 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: writRecogn - A CJK handwriting recognizer


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 21:31 EST ---
rpmlint says:

writRecogn.src: W: summary-ended-with-dot A handwriting recognizer that allows
users to input Chinese characters used in Chinese, Japanese and Korean.
writRecogn.src: E: summary-too-long A handwriting recognizer that allows users
to input Chinese characters used in Chinese, Japanese and Korean.
writRecogn.src: E: description-line-too-long writRecogn recognizes east Asian
handwritten characters for Chinese, Japanese, and Korean (CJK) and will
interface to input methods such as SCIM. Unlike some implementations which
require to build a huge set of character recognition rules, we recognize
radicals of Chinese characters, i.e. the word root of the character, then use a
character-structure-based input method to search for the word. This saves us
from writing recognition rules for tens of thousands of CJK characters. This
should provide better recognition accuracy than current open source handwriting
recognition libraries, like tomoe.
writRecogn.src: E: description-line-too-long The main program which provides the
GUI is writRecogn and there is a commandline character data maintenance program
cdMgr.


-- 
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 434727] Review Request: libpst - utilities to convert Outlook .pst files to other formats

2008-02-26 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: libpst - utilities to convert Outlook .pst files to 
other formats


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 19:56 EST ---
Thanks! 

What is the policy on the change log in the .spec file versus the gnu
ChangeLog/NEWS files? Are we supposed to duplicate every entry from ChangeLog
into the .spec file, or is the changelog in the spec file only for packaging
related changes?

Yes, the name is strange, but that is the original name of the project, and I
did not want to arbitrarily change it. At some point we may be able to
standardize the interface enough to create a libpst.so used by the actual
utilities, and also usable by other projects. That will further add to the
naming confusion.

-- 
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 226447] Merge Review: sysfsutils

2008-02-26 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: sysfsutils


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 19:42 EST ---
Yep, I wanted to look back over it before I added the fedora-review +, since it
was a month ago that I did the review and my memory needs frequent refreshing. 
It looks like everything is good.  Thanks Jarod (and Kevin for poking :).

-- 
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 227121] Review Request: wstx-2.9.3-1jpp - Woodstox Stax Implementation

2008-02-26 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: wstx-2.9.3-1jpp - Woodstox Stax Implementation


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||DUPLICATE




--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 19:29 EST ---
I have reopened the bug. Note that if you are interested in this package I
highly encourage you to take over ownership. I cant commit to maintaining
packages for the next little while,. I am sure there is a way to transfer
ownership of the review/package to you and I would be happy to do the review (if
I am allowed to somehow be the reporter and the reviewer..)

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

-- 
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 252110] Review Request: wstx - Woodstox Stax Implementation

2008-02-26 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: wstx - Woodstox Stax Implementation


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 19:29 EST ---
*** Bug 227121 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 432808] Review Request: figtoipe - FIG to IPE conversion tool

2008-02-26 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: figtoipe - FIG to IPE conversion tool


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 19:24 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 252110] Review Request: wstx - Woodstox Stax Implementation

2008-02-26 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: wstx - Woodstox Stax Implementation


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Version|devel   |rawhide

[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|CLOSED  |ASSIGNED
   Keywords||Reopened
 Resolution|DUPLICATE   |




-- 
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 226447] Merge Review: sysfsutils

2008-02-26 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: sysfsutils


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 18:53 EST ---
Todd: Did you want to mark this with fedora-review + ? 

Jarod: You shouldn't usually close until it's been approved... ;) 

-- 
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 250804] Review Request: perl-Encode-Detect - Detects the encoding of data

2008-02-26 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: perl-Encode-Detect - Detects the encoding of data
Alias: perl-Encode-Detect

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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 18:48 EST ---
Thanks Ville.  Just FYI, I believe I've now completed all of the prerequisite
steps described at:

http://fedoraproject.org/wiki/PackageMaintainers/Join


-- 
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 381241] Review Request: ncl - NCAR Command Language and NCAR Graphics

2008-02-26 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: ncl -  NCAR Command Language and NCAR Graphics


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||434349
  nThis||




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

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


[Bug 435042] New: Review Request: gpsman - GPS data manager

2008-02-26 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=435042

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


Spec URL: http://bjensen.fedorapeople.org/pkgs/hams/SPECS/gpsman.spec
SRPM URL: 
http://bjensen.fedorapeople.org/pkgs/hams/SRPMS/gpsman-6.3.2-3.fc9.src.rpm
Description:

GPS Manager (GPSMan) is a graphical manager of GPS data that makes possible
the preparation, inspection and edition of GPS data in a friendly environment.

GPSMan supports communication and real-time logging with both Garmin and
Lowrance receivers and accepts real-time logging information in NMEA 0183
from any GPS receiver.

-- 
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 226452] Merge Review: system-config-bind

2008-02-26 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: system-config-bind


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 18:20 EST ---
for #3: 

I am not a employee of Red Hat, so I can't show you any specific place, but I
was told to refer you to "CopyrightGuidelines on the intranet". Is that
something you can find? Or does that mean anything to you? 

for #4: 

Take a look at: https://fedorahosted.org/web/faq and see the "How can I publish
archive releases (tgz, zip, etc) for my project?" You should be able to use that
to upload releases and then you can point your Source0: to that url.


-- 
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 241487] Review Request: cduce - An XML-oriented functional language (ocaml)

2008-02-26 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: cduce - An XML-oriented functional language (ocaml)


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 17:47 EST ---
Here's a new release based on 0.5.2-1.  Please don't use '-' character in
upstream release numbers!  It makes it hard to package!

Anyway:

Spec URL: http://www.annexia.org/tmp/ocaml/cduce.spec
SRPM URL: http://www.annexia.org/tmp/ocaml/cduce-0.5.2.1-1.fc9.src.rpm

  * Tue Feb 26 2008 Richard W.M. Jones <[EMAIL PROTECTED]> - 0.5.2.1-1
  - New upstream release 0.5.2-1.

Here are the rpmlint errors:

  $ rpmlint /home/rjones/rpmbuild/SRPMS/cduce-0.5.2.1-1.fc9.src.rpm
  cduce.src:63: E: configure-without-libdir-spec

This is a problem in rpmlint.  See bug 433783.

  $ rpmlint /home/rjones/rpmbuild/RPMS/i386/cduce-0.5.2.1-1.fc9.i386.rpm
  cduce.i386: E: only-non-binary-in-usr-lib
  cduce.i386: W: ocaml-naming-policy-not-applied 
/usr/lib/ocaml/cduce/cduce_lib.cmi

See also bug 433783.

  $ rpmlint /home/rjones/rpmbuild/RPMS/i386/cduce-devel-0.5.2.1-1.fc9.i386.rpm
  cduce-devel.i386: W: ocaml-naming-policy-not-applied 
/usr/lib/ocaml/cduce/cduce_lib.a

As above.

-- 
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 434727] Review Request: libpst - utilities to convert Outlook .pst files to other formats

2008-02-26 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: libpst - utilities to convert Outlook .pst files to 
other formats


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 17:04 EST ---
OK, there are a few things that you need to change in this package:

* Please don't override localstatedir. This is set properly in the Fedora 
config.

* In the Summary, please describe the package without using its name. Tell me
what it does, not what it is called. :) Also, please be sure to capitalize the
first word (and don't end it with a period).

* Release should not be wholly macro driven, this is something that you want to
increment by hand. I strongly suggest that you consider using the Dist Tag
system for this, see: http://fedoraproject.org/wiki/Packaging/DistTag

* We have a specific, explicit system for identifying the License tag. Your
package should have "License: GPLv2+", see:
http://fedoraproject.org/wiki/Packaging/LicensingGuidelines

* Please use the %{name} and %{version} macros everywhere that you have "libpst"
and "0.6.7" hardcoded (except in Name: and Version:, of course). This ensures a
much easier update path, with less work on your part. For example, use them in
the Source: field, in your mkdir and mv commands, and in the %files entries.

* Please use one of the approved BuildRoot settings, which are listed here:
http://fedoraproject.org/wiki/Packaging/Guidelines#head-b4fdd45fa76cbf54c885ef0836361319ab962473

* Do not set Vendor or Packager in the spec file. The Fedora buildsystem will
set these values automatically.

* Do not use AutoReqProv. This is one of the big benefits of rpm! We want those
provides!

* In your %description, please don't have any line longer than 80 characters.
This ensures that it displays cleanly on a terminal screen. You can have
multiple lines. :)

* Please use %setup -q instead of just %setup. This quiets the output from the
tarball unpacking, which helps us save on logfile size in the buildsystem.

* In %build, please replace your long configure invocation with %configure,
which is a macro that evaluates to the same thing (and more).

* In %build, please use make %{_?smp_mfiles} instead of just make. See:
http://fedoraproject.org/wiki/Packaging/Guidelines#head-525c7d76890cb22df33b759c65c35c82bf434d2e

* In %install, there is no reason to check the buildroot before deleting it.
Since you've set it in the spec file, it will always be safe to simply delete it
as the first step in the %install process.

* In %install, replace your long make install invocation with simply:
make DESTDIR=$RPM_BUILD_ROOT install
It will achieve the same result, but will avoid embedding the buildroot into the
installed files (and is also much smaller).

* In %install, you do not need to make the docdir and manually copy the files
into it. rpm will do this for you, for any files marked in %files as doc. 
To do this, just get rid of the mkdir and mv commands from %install, and delete
these lines from %files:
%doc %{_mandir}/*
%docdir %{_datadir}/doc/libpst-0.6.7
%{_datadir}/doc/libpst-0.6.7
Then, add this line to %files:
%doc AUTHORS COPYING ChangeLog NEWS README
That's it! Same end result, so much less pain. :)

* You do not need to have empty %pre, %post, %preun, and %postun entries. Just
take them out entirely.

* In your %clean entry, please add: rm -rf $RPM_BUILD_ROOT

* In %files, please use %defattr(-,root,root,-) instead of %defattr(-,root,root)

* In %files, please don't use %doc %{_mandir}/*. There are two problems with
this line:
1. %{_mandir} is automagically marked as %doc when it is used in %files, you do
not need to specify this.
2. %{_mandir}/* will cause this package to own all of the created directories
under %{_mandir}, which in the case of your package are "man1/" and "man5/". You
only want to own the manpages installed in those directories, so replace the
line with:
%{_mandir}/man1/*
%{_mandir}/man5/*

* In the %changelog, please ensure the following:
Everytime that you make any change, no matter how small or trivial, add a
changelog entry, in a supported format. The supported formats are described
here:
http://fedoraproject.org/wiki/Packaging/Guidelines#head-b7d622f4bb245300199c6a33128acce5fb453213

* The naming of this package "libpst" is a little strange, given that it doesn't
actually have any libraries in it. A better name might be "pst-utils". Normally,
I wouldn't say anything, but since I know that you are the upstream, I figured I
would throw it out there. If you like the "libpst" name, that's fine, you can
keep using it here.

I highly recommend that you read through:
http://fedora

[Bug 434996] Review Request: certmaster - a certificate distribution system

2008-02-26 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: certmaster - a certificate distribution system


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 16:56 EST ---
rpmlint certmaster-0.18-1.fc8.src.rpm 
certmaster.src:76: E: hardcoded-library-path in /usr/lib/lsb/install_initd
certmaster.src:77: E: hardcoded-library-path in /usr/lib/lsb/install_initd
certmaster.src:78: E: hardcoded-library-path in /usr/lib/lsb/install_initd
certmaster.src:98: E: hardcoded-library-path in /usr/lib/lsb/remove_initd
certmaster.src:99: E: hardcoded-library-path in /usr/lib/lsb/remove_initd
certmaster.src:100: E: hardcoded-library-path in /usr/lib/lsb/remove_initd

and that's from the suse section. otherwise all looks good.


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

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


[Bug 426026] Review Request: gnome-menu-extended - Gnome Menu with KDE directory

2008-02-26 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: gnome-menu-extended - Gnome Menu with KDE directory


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 16:38 EST ---
Sorry, I've uploaded the noarch.rpm, it's fix 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 227121] Review Request: wstx-2.9.3-1jpp - Woodstox Stax Implementation

2008-02-26 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: wstx-2.9.3-1jpp - Woodstox Stax Implementation


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 16:00 EST ---
ping?

Vivek, it looks like Rafael does not have the time for this package. I would
like to suggest that we reopen bug 252110 and close this one as a dup of 252110
instead of the otherway around, then I can review your newer package in bug 
252110.


-- 
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 307821] Review Request: sooperlooper - Realtime software looping sampler

2008-02-26 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: sooperlooper - Realtime software looping sampler


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 16:02 EST ---
Fernando: ping?

All you need todo is ask for cvs creation, import and fire a build.


-- 
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 231809] Review Request: xmlrpc-epi - An implementation of the xmlrpc protocol in C

2008-02-26 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: xmlrpc-epi - An implementation of the xmlrpc protocol 
in C
Alias: xmlrpc-epi

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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 16:01 EST ---
Callum, ping?


-- 
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 433220] Review Request: kAnyREmote - KDE frontend for anyremote

2008-02-26 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: kAnyREmote - KDE frontend for anyremote


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 15:46 EST ---
Updated ...

Spec URL: 
http://downloads.sourceforge.net/anyremote/kanyremote-fedora.spec?use_mirror=osdn
SRPM URL: 
http://downloads.sourceforge.net/anyremote/kanyremote-4.6-2.fc8.src.rpm?use_mirror=osdn


-- 
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 433219] Review Request: ganyremote - GTK frontend for anyremote

2008-02-26 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: ganyremote - GTK frontend for anyremote


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 15:47 EST ---
Updated ...

Spec URL:  
http://downloads.sourceforge.net/anyremote/ganyremote-fedora.spec?use_mirror=osdn

SRPM URL: 
http://downloads.sourceforge.net/anyremote/ganyremote-2.6-3.fc8.src.rpm?use_mirror=osdn

-- 
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 433135] Review Request: anyRemote - Bluetooth remote control

2008-02-26 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: anyRemote - Bluetooth remote control


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 15:31 EST ---
Updated:

Spec URL:  
http://downloads.sourceforge.net/anyremote/anyremote-fedora.spec?use_mirror=osdn

SRPM URL:
http://downloads.sourceforge.net/anyremote/anyremote-4.3-3.fc8.src.rpm?use_mirror=osdn

-- 
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 433135] Review Request: anyRemote - Bluetooth remote control

2008-02-26 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: anyRemote - Bluetooth remote control


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 15:30 EST ---
(In reply to comment #9)

>* Why do you want to split -data, -doc subpackages ? 
To have possibility release them separately. (for example:
if only -data package was changed)

> Please remove Redundant Requires/BuildRequires
OK. Done except bc. bc is not covered by "exception" list.

> is bc really needed?
Yes

>Also please remove redundant BuildRequires.
OK. Done.

>On the other hands, at least some BuildRequries are missing.
OK. libXtst-devel was added.

>Please consider to use %configure (check what %configure does
OK. Done.

>Stripping the rebuilt binaries by yourself is forbidden to create
OK. Done.

>For man files, please use %{_mandir}/
OK. Done.

>Please make it sure that all directories created when installing
  a rpm are owned by the rpm.
OK. Done.

  
>Document files should usually be installed under
  %{_defaultdocdir}/%{name}-%{version}, not %{_datadir}/%{name}.
OK. %{_datadir}/%{name} changed to %{_defaultdocdir}/%{name}

>To make it sure that all installed files are correctly built from
  open source files, we do not allow to install pre-compiled files
  (like .jar files) directly.
It is written in packaging/Guidlines:
: Some software (usually related to compilers or cross-compiler
: environments) cannot be build without the use of a previous toolchain
: or development environment (open source). If you have a package which meets
: this criteria, contact the Fedora Packaging Committee for approval.
To build *jar i use Sun WTK, which can be considered as "toolchain", but 
it is not open source at all. I need an advice, how it is possible to 
distribute *jar anyway (Livna ?) because without *jar this project will be 
almost unuseful.


-- 
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 435018] New: Review Request: clipper - crystallographic object oriented library

2008-02-26 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=435018

   Summary: Review Request: clipper - crystallographic object
oriented library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://www.stanford.edu/~fenn/packs/clipper.spec
SRPM URL: http://www.stanford.edu/~fenn/packs/clipper-2.0-10.f8.src.rpm
Description: The aim of the project is to produce a set of object-oriented 
libraries for the organisation of crystallographic data and the performance of 
crystallographic computation. The libraries are designed as a framework for new 
crystallographic software, which will allow the full power of modern 
programming techniques to be exploited by the developer. This will lead to 
greater functionality from simpler code which will be easier to develop and 
debug.

Also see:
http://www.ysbl.york.ac.uk/~cowtan/clipper/clipper.html

-- 
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 435017] New: Review Request: SSM - coordinate superposition library

2008-02-26 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=435017

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


Spec URL: http://www.stanford.edu/~fenn/packs/ssm.spec
SRPM URL: http://www.stanford.edu/~fenn/packs/ssm-0.1-2.f8.src.rpm
Description:  SSM is a macromolecular coordinate superposition library, written 
by Eugene Krissinel of the EBI.

The library implements the SSM algorithm of protein structure comparison in 
three dimensions, which includes an original procedure of matching graphs built 
on the protein's secondary-structure elements, followed by an iterative 
three-dimensional alignment of protein backbone Calpha atoms.

Also see:
http://www.ebi.ac.uk/msd-srv/ssm/
http://www.bioxray.dk/~mok/ssm.php

-- 
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 435015] New: Review Request: libGPP4 - LGPL CCP4 library

2008-02-26 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=435015

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


Spec URL: http://www.stanford.edu/~fenn/packs/gpp4.spec
SRPM URL: http://www.stanford.edu/~fenn/packs/gpp4-1.0.4-2.f8.src.rpm
Description: The ggp4 library is a v2.1 LGPL version of the 5.0.2 CCP4 library 
(http://www.ccp4.ac.uk/main.html), used in crystallography data storage and I/O 
routines. This version was patched by Ralf Grosse-Kunstleve to address some of 
the more serious deficiencies of the older libraries and a GNU autotools build 
environment developed by Paul Emsley and Morten Kjeldgaard.

Also see:
http://www.bioxray.dk/~mok/gpp4.php
http://www.ccp4.ac.uk/main.html

-- 
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 435016] New: Review Request: libmmdb - MMDB coordinate library

2008-02-26 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=435016

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


Spec URL: http://www.stanford.edu/~fenn/packs/mmdb.spec
SRPM URL: http://www.stanford.edu/~fenn/packs/mmdb-1.09.1-2.f8.src.rpm
Description: MMDB is CCP4's macromolecular coordinate library, written by 
Eugene Krissinel of the EBI. The Coordinate Library is designed to assist CCP4 
developers in working with coordinate files. The major source of coordinate 
information remains the PDB files, although more information becomes available 
in mmCIF format.

The Library features working with both file formats plus an internal binary 
format portable between different platforms. This is achieved at uniformity of 
the Library's interface functions, so that there is no difference in handling 
different formats.

The Library provides various high-level tools for working with coordinate 
files, which include not only reading and writing, but also 
orthogonal-fractional coordinate transforms, generation of symmetry mates, 
editing the molecular structure and some others. The Library is supposed as a 
general low-level tool for unifying the coordinate-related operations.

This distribution is based on the last free version of mmdb, namely 1.09. 
Eugene's latest version of mmdb (1.10) is licensed under the non-free CCP4 
license.

Also see:
http://www.ebi.ac.uk/~keb/cldoc/
http://www.bioxray.dk/~mok/mmdb.php

-- 
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 431857] Review Request: pakchois - PKCS#11 wrapper library

2008-02-26 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: pakchois - PKCS#11 wrapper library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 15:24 EST ---
Many thanks for taking care of this, Michael and Kevin!

-- 
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 426884] Review Request: eclipse-epic - Perl Eclipse plugin

2008-02-26 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-epic - Perl Eclipse plugin


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 15:02 EST ---
I assume this is because the Eclipse guidelines aren't yet finished.  I don't
believe they should be blocking this, but that's not really my call.

-- 
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 426884] Review Request: eclipse-epic - Perl Eclipse plugin

2008-02-26 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-epic - Perl Eclipse plugin


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 14:58 EST ---
Can anybody tell me *what* I'm doing wrong, even. Bugzilla needs a preview
button, obviously. :-)

-- 
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 426884] Review Request: eclipse-epic - Perl Eclipse plugin

2008-02-26 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-epic - Perl Eclipse plugin


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 14:56 EST ---
I notice this has been tagged as blocking F-GUIDELINES. Can anybody tell me I'm
doing wrong?

-- 
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 434996] New: Review Request: certmaster - a certificate distribution system

2008-02-26 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=434996

   Summary: Review Request: certmaster - a certificate distribution
system
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: 
http://mdehaan.fedorapeople.org/files/certmaster/certmaster.spec

SRPM URL: 
http://mdehaan.fedorapeople.org/files/certmaster/certmaster-0.18-1.fc8.src.rpm

Description: certmaster is a certificate distribution system used in Func that 
we are splitting off into a seperate package so other applications can take 
advantage of it for exchange/provisioning of SSL certificates

-- 
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 434973] Review Request: scidavis

2008-02-26 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: scidavis


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 13:31 EST ---
Could you retry to download it ? It seems the package on the website was bad.
It's now solved.

-- 
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 428718] Review Request: pysvn - Python bindings for Subversion

2008-02-26 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: pysvn - Python bindings for Subversion


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Review Request: python-svn -|Review Request: pysvn -
   |Python bindings for |Python bindings for
   |Subversion  |Subversion




--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 13:30 EST ---
So...I've been able to make all of the requested changes but one: pycxx 
dependency.

The others were easy, even over-sights on my part, but pysvn and pycxx are very
interwoven, in fact pycxx only exists for pysvn (same developer/maintainer, and
only went as far as making pycxx so he could get pysvn done).

If pycxx needs its own package (i.e. people would like to use it), then I'll
look at it.  As it is, pysvn pulls in the pycxx source and headers itself and
compiles pycxx itself, statically, for itself.  pycxx by itself does not have a
fully (Red Hat/Fedora) usable makefile and pysvn's needs tweaking for support,
and I'll have to learn a lot more about C++ and compiling C++ for use in a
shared library environment.

I know that in the past, syslog-ng 1.x had a dependency that was included so it
could build.  What is the policy now?

For now (http://koji.fedoraproject.org/koji/taskinfo?taskID=471353):
http://yum.virtualxistenz.com/pysvn/pysvn.spec
http://yum.virtualxistenz.com/pysvn/pysvn-1.5.2-6.f8.src.rpm (do I reset the
release version after the name changes?)

-- 
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 434557] Review Request: perl-Log-Trivial - Very simple tool for writing very simple log files

2008-02-26 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: perl-Log-Trivial - Very simple tool for writing very 
simple log files


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 13:21 EST ---
Checked in and built.

-- 
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 431857] Review Request: pakchois - PKCS#11 wrapper library

2008-02-26 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: pakchois - PKCS#11 wrapper library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 13:10 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 434973] Review Request: scidavis

2008-02-26 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: scidavis


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 13:10 EST ---
 rpm -i scidavis-0.1.2-1.fc8.src.rpm
warning: user tanguy-e does not exist - using root
warning: group tanguy-e does not exist - using root
warning: user tanguy-e does not exist - using root
warning: group tanguy-e does not exist - using root
warning: user tanguy-e does not exist - using root
warning: group tanguy-e does not exist - using root
error: unpacking of archive failed on 
file 
/home/nbecker/RPM/SOURCES/scidavis-0.1.2_translations_2008-02-03.tar.bz2;47c455cb:
 
cpio: read

-- 
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 432542] Review Request: autogen - Automated text file generator

2008-02-26 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: autogen - Automated text file generator


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 12:48 EST ---
Spec: http://rishi.fedorapeople.org/autogen.spec
SRPM: http://rishi.fedorapeople.org/autogen-5.9.4-4.fc8.src.rpm

To omit unused direct shared library dependencies, I copied /usr/bin/libtool and
modified it as before.

-- 
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 434973] Review Request: scidavis

2008-02-26 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: scidavis


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 12:07 EST ---
$ rpmlint scidavis-0.1.2-1.fc8.src.rpm
nothing

$ rpmlint scidavis-manual-0.1.2-1.fc8.i386.rpm
nothing

$ rpmlint scidavis-debuginfo-0.1.2-1.fc8.i386.rpm
nothing

$ rpmlint scidavis-0.1.2-1.fc8.i386.rpm 
scidavis.i386: W: non-conffile-in-etc /etc/scidavisrc.pyc
scidavis.i386: W: non-conffile-in-etc /etc/scidavisrc.pyo
scidavis.i386: W: non-conffile-in-etc /etc/scidavisrc.py

>From upstream "The reason why scidavisrc.py is in /etc is that we consider it 
>to
be a configuration file (and, to the best of my knowledge, /etc is the standard
location for system-wide config files). Like many other config files, it can be
overridden on a per-user basis by creating a file ~/.scidavisrc.py. Maybe
scidavisrc.pyo and scidavisrc.pyc are not exactly config files; but then again,
files like /etc/modprobe.conf and /etc/mtab are also auto-generated."

-- 
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 434973] New: Review Request: scidavis

2008-02-26 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=434973

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


Spec Name or Url: 
http://laurence.bernard.pagesperso-orange.fr/eric/scidavis.spec
SRPM Name or Url:
http://laurence.bernard.pagesperso-orange.fr/eric/scidavis-0.1.2-1.fc8.src.rpm
Description: SciDAVis is a user-friendly data analysis and visualization program
primarily aimed at high-quality plotting of scientific data. It strives to
combine an intuitive, easy-to-use graphical user interface with powerful
features such
as Python scriptability.

-- 
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 433187] Review Request: boinc-client - A platform for distributed computing

2008-02-26 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: boinc-client - A platform for distributed computing
Alias: boinc-client-review

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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 11:54 EST ---
This does not build on dist-f9, at least on i386.

http://koji.fedoraproject.org/koji/taskinfo?taskID=471083
http://koji.fedoraproject.org/koji/getfile?taskID=471086&name=build.log

It seems that most (all?) error was caused by missing headers
like "#include " or so.

-- 
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 433228] Review Request: distcc - Distributed C/C++ compilation

2008-02-26 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: distcc - Distributed C/C++ compilation


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 11:49 EST ---
Uh, yeah, there some issues with IPv6. I guess I need to fix those. Should be
trivial to fix, though. IPv6 addresses need to be formatted a [address]:port 
instead of just address:port as for IPv4.

-- 
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 226039] Merge Review: libraw1394

2008-02-26 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: libraw1394


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
   Flag|fedora-cvs+ |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 11:38 EST ---
Looks good! Closing this one out.

-- 
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 416461] Review Request: xmms-pulse - XMMS output plugin for the PulseAudio sound server.

2008-02-26 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:  xmms-pulse -  XMMS output plugin for the PulseAudio 
sound server.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 11:34 EST ---
Well, I cannot see your srpm are updated as your latest srpm
has "xmms-pulse-0.9.4-1.fc8".

Please change the version of your srpm each time you modify your
srpm to avoid confusion. Also, please write the URL of spec/srpm
from which we can download them directly by "wget -N" (for example)
on 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 429221] Review Request: dzcomm - Dzcomm a RS-232 API/lib

2008-02-26 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: dzcomm - Dzcomm a RS-232 API/lib


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 11:26 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 233975] Review Request: freefem++ - PDE solving tool

2008-02-26 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: freefem++ - PDE solving tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Resolution|CURRENTRELEASE  |NEXTRELEASE




-- 
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 226202] Merge Review: nspr

2008-02-26 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: nspr


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 10:42 EST ---
I think the above should be sufficent, once committed to cvs. :)

-- 
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 426026] Review Request: gnome-menu-extended - Gnome Menu with KDE directory

2008-02-26 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: gnome-menu-extended - Gnome Menu with KDE directory


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 10:42 EST ---
http://home.scarlet.be/~tsi30161/gnome-menu-extended-0.7-3.fc8.src.rpm

seems 404...

-- 
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 434906] Review Request: xosview - OS resource viewer

2008-02-26 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: xosview - OS resource viewer


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|CLOSED  |ASSIGNED
   Keywords||Reopened
 Resolution|NOTABUG |




-- 
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 434906] Review Request: xosview - OS resource viewer

2008-02-26 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: xosview - OS resource viewer


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 10:37 EST ---
Bah, no.  I don't know how that happened -- must have been an errant mouse
click.  I've re-opened it.

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

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


[Bug 226202] Merge Review: nspr

2008-02-26 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: nspr


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 10:25 EST ---
(In reply to comment #1)
> Unversion obsoletes for mozilla-nspr and mozilla-nspr-devel.  Probably want to
> correct this.

It obsoletes *any* mozilla-nspr package, because we've stopped shipping NSPR as
part of the mozilla packages, but started to ship it as a standalone package.

Do you still think a version number must get included?
I would have to research which version number was last, at some point before 
FC6.
Would it have to be the exact version number?


> nspr.src: E: no-cleaning-of-buildroot %install
> You should clean $RPM_BUILD_ROOT in the %clean section and just after the
> beginning of %install section. Use "rm -Rf $RPM_BUILD_ROOT".

The %clean section already has a rm statement.
I've changed it to use uppercase -R
I've added it to the beginning of the %install section.


> nspr.src: W: mixed-use-of-spaces-and-tabs (spaces: line 13, tab: line 1)
> The specfile mixes use of spaces and tabs for indentation, which is a
> cosmetic annoyance.  Use either spaces or tabs for indentation, not both.

fixed


> There are no documentation files in either the main package or -devel.
> If there are none, this is OK, which seems to be the case.

find . -type f | grep -v \.c$ |grep -v \.a$ | grep -v \.so$ | grep -v \.o$ |
grep -vw CVS | grep -v \.h$ | grep -v \.orig$ | grep -v \.rej$ | grep -v
Makefile$  |grep -v \.mk$ | grep -v \.~ | grep -v \.bck$ | grep -v \.mn | grep
-v '/\.#' | grep -v \.def$ | grep -v \.chk$ | grep -v \.s$  |grep -vw tests |
grep -vw OBJ | grep -v \.cfg$ | grep -v Makefile.in$ |grep -v \.cpp$ |grep -vw
cvsignore  |grep -v \.mk.in$ | grep -v \.sh\.in$ | grep -v \.pl$ | grep -v \.m4$
|grep -v \.asm$


I looked at all the files this command produced.

./nsprpub/pr/src/threads/combined/README
./nsprpub/lib/libc/README
./nsprpub/lib/libc/src/README
These look like a developer oriented quick dump of thoughts from 1996/7, but
they don't seem to be worthy to be called "documentation".


> The Source0 tag lacks a URL, and I can't locate the appropriate tarball on the
> upstream site, so I can't compare the upstream tarball and SRPM tarball.  If
> this a modifed upstream tarball or cvs snapshot, please include a script that
> creates this tarball from upstream.

I've added a comment that points to the download directory where official
releases are available.
Sometimes we do package non-release snapshots. If we do, there is no matching
tarball available, but it always will be based on a CVS tag (the changelog shall
make it clear what CVS tag is being used).


-- 
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 434880] Review Request: tanukiwrapper - Java Service Wrapper

2008-02-26 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: tanukiwrapper - Java Service Wrapper


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||177841
  nThis||




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

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


[Bug 434890] Review Request: LinLog - A ham radio logbook for linux

2008-02-26 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: LinLog - A ham radio logbook for linux


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 09:59 EST ---
Correct, it's in multiple source files. Sorry, my bad.

-- 
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 433488] Review Request: dayplanner - An easy and clean Day Planner

2008-02-26 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: dayplanner - An easy and clean Day Planner


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 09:52 EST ---
Could someone review this please?

-- 
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 434911] Review Request: gmfsk - A Gnome Multimode HF Terminal for Ham Radio

2008-02-26 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: gmfsk - A Gnome Multimode HF Terminal for Ham Radio


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 09:53 EST ---
Release number is fine as it is now. Guidelines say you should append "pre",
"alpha", "beta" and vcs snapshot tags to the release tag.

-- 
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 434954] New: Review Request: writRecogn - A CJK handwriting recognizer

2008-02-26 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=434954

   Summary: Review Request: writRecogn - A CJK handwriting
recognizer
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: 
http://optusnet.dl.sourceforge.net/sourceforge/writrecogn/writRecogn.spec
SRPM URL: 
http://optusnet.dl.sourceforge.net/sourceforge/writrecogn/writRecogn-0.1.1-0.fc8.src.rpm
Description: writRecogn recognizes east Asian handwritten characters for 
Chinese, Japanese, and Korean (CJK) and will interface to input methods such as 
SCIM. Unlike some implementations which require to build a huge set of 
character recognition rules, we recognize radicals of Chinese characters, i.e. 
the word root of the character, then use a character-structure-based input 
method to search for the word.

-- 
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 226039] Merge Review: libraw1394

2008-02-26 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: libraw1394


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 09:39 EST ---
Thanks much, committed to rawhide cvs a few minutes ago.

-- 
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 434911] Review Request: gmfsk - A Gnome Multimode HF Terminal for Ham Radio

2008-02-26 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: gmfsk - A Gnome Multimode HF Terminal for Ham Radio


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 09:32 EST ---
The release number in the spec file should be set to 1 too just to be "standard"
rather than the multi-part "pre" version number like it is 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 434911] Review Request: gmfsk - A Gnome Multimode HF Terminal for Ham Radio

2008-02-26 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: gmfsk - A Gnome Multimode HF Terminal for Ham Radio


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 09:25 EST ---
Created an attachment (id=295915)
 --> (https://bugzilla.redhat.com/attachment.cgi?id=295915&action=view)
rawhide-i386-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 434911] Review Request: gmfsk - A Gnome Multimode HF Terminal for Ham Radio

2008-02-26 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: gmfsk - A Gnome Multimode HF Terminal for Ham Radio


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 09:24 EST ---
RPM Lint: OK
Package name: OK
Spec file: OK
License: OK
Actual License: OK
%doc License: OK
Spec file language: OK
Spec file readable: OK
Upstream source vs. used tarball: OK (md5sum match)
Compile and Build:
 - F-7: OK
 - F-8: OK
 - rawhide: FAILED (logs attached)
 - EL-5: FAILED (but not targeted anyway)

Applicable Package Guidelines:
 - -debuginfo package: OK (~592K)

Locales: OK
Shared libs: N/A
Relocatable: N/A
Directory and file ownership: OK
No duplicate files in %files: OK
File Permissions: OK
Macro usage: OK
Code vs. Content: OK
(Large) Documentation: OK
%doc affecting runtime: OK
Header files in -devel package: N/A
Static Libraries in -static package: N/A
pkgconfig Requires: N/A
Library files: N/A
Devel requires base package: N/A
.la libtool archives: N/A
Duplicate ownership of files/directories: OK
Remove BuildRoot: OK
UTF-8 filenames: OK

Please examine the build log files attached

-- 
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 434890] Review Request: LinLog - A ham radio logbook for linux

2008-02-26 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: LinLog - A ham radio logbook for linux


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 09:26 EST ---
Upon further investigation this should in fact be GPLv2+

-- 
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 226039] Merge Review: libraw1394

2008-02-26 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: libraw1394


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 09:21 EST ---
However, I can do a review quickly for you. :)

The only issues I see are: 
1. The License tag is incorrect, it should be LGPLv2+.
2. You're using a tab separator on line 11, and spaces everywhere else.

Since these items are pedantic, and easily corrected in CVS, this package is
approved. Please commit those changes. :)

Here's the full review notes:

Good:

- rpmlint checks return:
libraw1394.src: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 11)
libraw1394.src: W: invalid-license LGPL
libraw1394.x86_64: W: invalid-license LGPL
libraw1394-devel.x86_64: W: invalid-license LGPL
libraw1394-debuginfo.x86_64: W: invalid-license LGPL

Needs fixing in CVS, not a blocker for review.

- package meets naming guidelines
- package meets packaging guidelines (except for license tag)
- license (LGPLv2+) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream
- package compiles on devel (x86_64)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- no need for .desktop file 
- devel package ok
- no .la files
- post/postun ldconfig ok
- devel requires base package n-v-r

-- 
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 434642] Review Request: xastir - Amateur Station Tracking and Reporting system for amateur radio

2008-02-26 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: xastir - Amateur Station Tracking and Reporting system 
for amateur radio


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


[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 434547] Review Request: aprsd - Internet gateway and client access to amateur radio APRS packet data

2008-02-26 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: aprsd - Internet gateway and client access to amateur 
radio APRS packet data


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|NEEDINFO
   Flag||needinfo?([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 434547] Review Request: aprsd - Internet gateway and client access to amateur radio APRS packet data

2008-02-26 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: aprsd - Internet gateway and client access to amateur 
radio APRS packet data


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 09:11 EST ---
This project is dead upstream best I can tell. There may also be some issues
with how the software works, I am looking for input from some more knowledgeable
in the situation. I think this review request might need to be closed.

-- 
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 434547] Review Request: aprsd - Internet gateway and client access to amateur radio APRS packet data

2008-02-26 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: aprsd - Internet gateway and client access to amateur 
radio APRS packet data


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


[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 434890] Review Request: LinLog - A ham radio logbook for linux

2008-02-26 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: LinLog - A ham radio logbook for linux


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 09:03 EST ---
APPROVED

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

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


[Bug 434890] Review Request: LinLog - A ham radio logbook for linux

2008-02-26 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: LinLog - A ham radio logbook for linux


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 08:47 EST ---
Corrected License.

Spec URL: http://bjensen.fedorapeople.org/pkgs/hams/SPECS/LinLog.spec
SRPM URL: 
http://bjensen.fedorapeople.org/pkgs/hams/SRPMS/LinLog-0.3-4.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 241487] Review Request: cduce - An XML-oriented functional language (ocaml)

2008-02-26 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: cduce - An XML-oriented functional language (ocaml)


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 07:52 EST ---
Is there a packaging problem with cduce 0.5.2?  It seems like the
file dtd2cduce.ml is missing, so when I build it I get this error:

Pack cduce_lib.cmo
Build cduce_lib.cma
make[1]: Leaving directory `/home/rjones/rpmbuild/BUILD/cduce-0.5.2'
make: *** No rule to make target `tools/dtd2cduce.ml', needed by `dtd2cduce'.  
Stop.
error: Bad exit status from /var/tmp/rpm-tmp.31858 (%build)

Below is a diff of the manifests for 0.5.1 and 0.5.2:

@@ -51,6 +51,8 @@
 ./LICENSE
 ./Makefile
 ./Makefile.conf.template
+./Makefile.macosx
+./make-package-macosx
 ./META.in
 ./misc/bool.ml
 ./misc/bool.mli
@@ -146,10 +148,6 @@
 ./schema/schema_xml.ml
 ./schema/schema_xml.mli
 ./schema/TODO
-./tools/.cvsignore
-./tools/dtd2cduce.ml
-./tools/pull.ml
-./tools/validate.ml
 ./types/atoms.ml
 ./types/atoms.mli
 ./types/boolean.ml


-- 
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 226452] Merge Review: system-config-bind

2008-02-26 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: system-config-bind


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 05:40 EST ---
ad 3.
Yes, I'm pretty sure it's ok, almost all system-config tools are GPLv2+. I don't
know about any Red Hat policy against GPL >2.

ad 4.
I made a note to https://fedorahosted.org/system-config-bind/wiki/ on how to get
sources and tarballs of older releases. The real tarballs are in distCVS and
source RPMs, I hope that suffice, as I would not be happy having to upload each
release twice :-)


-- 
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 434911] Review Request: gmfsk - A Gnome Multimode HF Terminal for Ham Radio

2008-02-26 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: gmfsk - A Gnome Multimode HF Terminal for Ham Radio


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


[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 434904] Review Request: dxcc - Small utility which determines the ARRL DXCC entity of a ham radio callsign

2008-02-26 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: dxcc - Small utility which determines the ARRL DXCC 
entity of a ham radio callsign


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 05:23 EST ---
RPM Lint: OK
Package name: OK
Spec file: OK
License: OK
Actual License: OK
%doc License: OK
Spec file language: OK
Spec file readable: OK
Upstream source vs. used tarball: OK (md5sum match)
Compile and Build:
 - F-7: OK
 - F-8: OK
 - rawhide: OK
 - EL-5: OK (although not targeted)

Applicable Package Guidelines: N/A so OK

Locales: N/A
Shared libs: N/A
Relocatable: N/A
Directory and file ownership: OK
No duplicate files in %files: OK
File Permissions: OK
Macro usage: OK
Code vs. Content: OK
(Large) Documentation: N/A
%doc affecting runtime: OK
Header files in -devel package: N/A
Static Libraries in -static package: N/A
pkgconfig Requires: N/A
Library files: N/A
Devel requires base package: N/A
.la libtool archives: N/A
Duplicate ownership of files/directories: OK
Remove BuildRoot: OK
UTF-8 filenames: OK

APPROVED

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

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


[Bug 434890] Review Request: LinLog - A ham radio logbook for linux

2008-02-26 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: LinLog - A ham radio logbook for linux


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 05:14 EST ---
RPM Lint: OK
Package name: OK
Spec file: OK
License: OK, GPLv2+
Actual License: OK, but source (src/linlog.cpp) says "GPL see COPYING file
shipped", COPYING file just says GPLv2
%doc License: OK
Spec file language: OK
Spec file readable: OK
Upstream source vs. used tarball: OK (md5sum match)
Compile and Build:
 - F-7: OK
 - F-8: OK
 - rawhide: OK
 - EL-5: FAILED: No package qt4-devel, but not a target, so it's OK.

Applicable Package Guidelines:
 - -debuginfo package: OK (~1.9M)

Locales: N/A
Shared libs: N/A
Relocatable: N/A
Directory and file ownership: OK
No duplicate files in %files: OK
File Permissions: OK
Macro usage: OK
Code vs. Content: OK
(Large) Documentation: .pdf file (manual) included, not very large though: OK
%doc affecting runtime: OK
Header files in -devel package: N/A
Static Libraries in -static package: N/A
pkgconfig Requires: N/A
Library files: N/A
Devel requires base package: N/A
.la libtool archives: N/A
Duplicate ownership of files/directories: OK
Remove BuildRoot: OK
UTF-8 filenames: OK

Problems:
The license of the package should be GPLv2 according to src/linlog.cpp, and I
can't find any other mention off 'GPL' or 'or later' in the code using a simple
'grep -rin  *'. Please let me know where it says v2+ if it is really 
v2+.

Another thing: You did well on the .desktop and icon file ;-)

-- 
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 434904] Review Request: dxcc - Small utility which determines the ARRL DXCC entity of a ham radio callsign

2008-02-26 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: dxcc - Small utility which determines the ARRL DXCC 
entity of a ham radio callsign


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


[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 434890] Review Request: LinLog - A ham radio logbook for linux

2008-02-26 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: LinLog - A ham radio logbook for linux


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


[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 433735] Review Request: libgcroots - Root acquisition library for Garbage Collector

2008-02-26 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: libgcroots - Root acquisition library for Garbage 
Collector


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 03:22 EST ---
missing Requires:pkgconfig on -devel subpackage.

-- 
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 431857] Review Request: pakchois - PKCS#11 wrapper library

2008-02-26 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: pakchois - PKCS#11 wrapper library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-02-26 03:09 EST ---
New Package CVS Request
===
Package Name: pakchois
Short Description: PKCS#11 wrapper library
Owners: jorton
Branches: 
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