[Bug 239096] Review Request: perl-libwhisker2 - Perl module geared specificly for HTTP testing
Please 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-libwhisker2 - Perl module geared specificly for HTTP testing https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239096 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 01:53 EST --- OK, builds fine and rpmlint is quiet. You don't need BuildRequires: perl but it doesn't hurt anything to have it. Note for others who may check for BuildRequires: needed for the perl-devel split: this module doesn't build like a regular module and has a custom Makefile.pl which doesn't make use of the regular Perl build infrastructure. Yes, this package does at least try to use MD5 at some point although it will use an internal implementation if the module isn't available. At least according to my reading of src/mdx.pl. The only issue at this point that concerns me is the name of the package. Our naming guidelines indicate that packages should be named according to how they're carried in CPAN, but of course this module isn't in CPAN. According to CPAN rules it wold be called perl-LW2, but that seems somewhat suboptimal as well. Frankly I favor the current naming, but before I approve I'll gather a couple of other opinions. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239525] New: Review Request: gammu - Command Line utility and Libraries to work with mobile phones
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239525 Summary: Review Request: gammu - Command Line utility and Libraries to work with mobile phones Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: fedora-package-review@redhat.com Spec URL: http://download.tuxfamily.org/lxtnow/fedora/SPECS/gammu.spec SRPM URL: http://download.tuxfamily.org/lxtnow/temp/gammu-1.10.0-1.fc7.src.rpm MockLog: http://download.tuxfamily.org/lxtnow/temp/gammu-1.10.0-1-build.log Description: Gammu is command line utility and library to work with mobile phones from many vendors. Support for different models differs, but basic functions should work with majority of them. Program can work with contacts, messages (SMS, EMS and MMS), calendar, todos, filesystem, integrated radio, camera, etc. It also supports daemon mode to send and receive SMSes. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239526] New: Review Request: python-gammu - Python bindings for Gammu
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239526 Summary: Review Request: python-gammu - Python bindings for Gammu Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: fedora-package-review@redhat.com Spec URL: http://download.tuxfamily.org/lxtnow/fedora/SPECS/python-gammu.spec SRPM URL: http://download.tuxfamily.org/lxtnow/temp/python-gammu-0.19-1.src.rpm MockLog: http://download.tuxfamily.org/lxtnow/temp/python-gammu-0.19-build.log Description: Python bindings for Gammu library. It currently does not support all Gammu features,but range of covered functions is increasing,if you need some specific, feel free to use bug tracking system for feature requests. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239526] Review Request: python-gammu - Python bindings for Gammu
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-gammu - Python bindings for Gammu https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239526 [EMAIL PROTECTED] changed: What|Removed |Added BugsThisDependsOn||239525 -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239525] Review Request: gammu - Command Line utility and Libraries to work with mobile phones
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: gammu - Command Line utility and Libraries to work with mobile phones https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239525 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||239526 nThis|| -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239527] New: Review Request: wammu - Mobile Phone Manager Frontend - Gammu GUI
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239527 Summary: Review Request: wammu - Mobile Phone Manager Frontend - Gammu GUI Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: fedora-package-review@redhat.com Spec URL: http://download.tuxfamily.org/lxtnow/fedora/SPECS/wammu.spec SRPM URL: http://download.tuxfamily.org/lxtnow/temp/wammu-0.19-1.fc7.src.rpm MockLog: http://download.tuxfamily.org/lxtnow/temp/wammu-0.19-build.log Description: Wammu is mobile phone manager running on Linux, Windows and possibly other platforms,where Gammu and wxPython works. The communication is made by Gammu library. Currently supported features: * complete support (can read/edit/delete/copy) for contacts, todo, calendar * can read/create/save/send/backup smses * sending files to phone (OBEX and Sony Ericsson phones only) * sms composer for multi part smses (currently only text and predefined bitmap/sound can be edited) * display message including pictures and ringtones playback * support for backup and import in various formats (vCard, iCalendar,...) * export messages to mail (IMAP4, maildir and mailbox storages are supported) * searching for phone * translated into several languages * rated as best on many software servers -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239527] Review Request: wammu - Mobile Phone Manager Frontend - Gammu GUI
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: wammu - Mobile Phone Manager Frontend - Gammu GUI https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239527 [EMAIL PROTECTED] changed: What|Removed |Added BugsThisDependsOn||239526 -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239526] Review Request: python-gammu - Python bindings for Gammu
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-gammu - Python bindings for Gammu https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239526 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||239527 nThis|| -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239087] Review Request: perl-Nmap-Parser - Parse nmap scan data with perl
Please 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-Nmap-Parser - Parse nmap scan data with perl Alias: perl-Nmap-Parser https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239087 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 03:25 EST --- Some remarks: * You are using dos2unix to convert files. Please use /usr/bin/dos2unix instead and add BuildRequires: /usr/bin/dos2unix * Please move file conversion and permission fixing to %prep instead of %install and fix the source files instead of the installed files. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 230560] Review Request: wqy-bitmap-fonts - a fine-tuned Chinese bitmap font
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: wqy-bitmap-fonts - a fine-tuned Chinese bitmap font
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=230560
--- Additional Comments From [EMAIL PROTECTED] 2007-05-09 03:41 EST ---
0.8.1-4 with bdf looks ok, but please do not rename the spec file in the
package.
I guess wqy-bitmapfont-bdf-all-cvs20070506 is a cvs snapshot: I think it would
be preferable to make the initial package with an official release.
So IMHO best to use wqy-bitmapfont-pcf-0.8.1-7.tar.gz currently or wait for
the next official release.
(I see wqy-bitmapfont-ttf-0.8.1-7.tar.gz was also released.:)
I don't think you need to buildrequire make or %{_bindir}/env.
perl is also listed in
http://fedoraproject.org/wiki/Packaging/Guidelines#Exceptions.
It is generally better to avoid using explicit file requires since
for them yum may need to download filelists.xml.gz which is rather big.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 230560] Review Request: wqy-bitmap-fonts - a fine-tuned Chinese bitmap font
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: wqy-bitmap-fonts - a fine-tuned Chinese bitmap font https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=230560 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 03:43 EST --- Created an attachment (id=154379) -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=154379action=view) wqy-bitmap-fonts.spec-4.patch some more suggested fixes and cleanup One more update and I think I would like to proceed to the formal review. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225010] Review Request: glob2 - An innovative RTS game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: glob2 - An innovative RTS game https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225010 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 04:26 EST --- I built .23 locally, and it doesn't show any campaign (while .21 does). I am running on x86_64. Other then that it is a very nice game! -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239525] Review Request: gammu - Command Line utility and Libraries to work with mobile phones
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: gammu - Command Line utility and Libraries to work with mobile phones https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239525 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Flag||fedora-review? --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 04:49 EST --- like to review this package. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239527] Review Request: wammu - Mobile Phone Manager Frontend - Gammu GUI
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: wammu - Mobile Phone Manager Frontend - Gammu GUI https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239527 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239526] Review Request: python-gammu - Python bindings for Gammu
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-gammu - Python bindings for Gammu https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239526 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239154] Review Request: referencer - A document organiser and bibliography manager for Gnome
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: referencer - A document organiser and bibliography
manager for Gnome
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239154
--- Additional Comments From [EMAIL PROTECTED] 2007-05-09 05:10 EST ---
Created an attachment (id=154382)
-- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=154382action=view)
mock-referencer-build.log
I am not yet a package-maintainer.
This is my pre-review to exercise reviewing.
I checked this package as follows.
rpmlint is silent on both of this SRPM and the generated binary package.
package name: reference: OK.
spec file name: reference.spec: OK.
license:GPL:
%license field matches the actual license,
COPYING is listed in %doc, OK.
spec file: written in English, legible:OK.
upstream source:
http://icculus.org/referencer/downloads/referencer-1.0.3.tar.gz:
md5sum:1179e0eafc17ef130b8d669255bef0b3
source: referencer-1.0.3.tar.gz:
md5sum:1179e0eafc17ef130b8d669255bef0b3,
matches to the upstream source, OK.
build: mock successfully built this package on fc-6/i386,
see the attached build.log, OK.
BuildRequires: poppler-devel libgnomeuimm26-devel boost-devel gcc-c++
gettext perl(XML::Parser) desktop-file-utils:
all required packages are listed, so mock builds successfully, OK.
locale handling:
by %find_lang %{name}, not by %{_datadir}/locale/*: OK.
%prefix:not used: OK.
shared library problem: not installed: OK.
ownership of directories and files:
all listed under %doc, ${_bindir} and ${_datadir}: OK.
owning directories created by other packages: none: OK.
duplicate listing in %file: none: OK.
%defattr: (-,root,root,-):OK.
at the beginning of %install:
rm -rf %{buildroot}:OK.
%clean: rm -rf %{buildroot} OK.
usage of macros:consistent: OK.
desktop file:
%{name}.desktop is installed with desktop-file-install in %install:
OK.
%post, %postun:
update desktop database, mime database and icon cache:
OK.
This packaging seems to be complete!
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 238248] Review Request: ddccontrol - TFT monitor parameters control
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: ddccontrol - TFT monitor parameters control
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=238248
[EMAIL PROTECTED] changed:
What|Removed |Added
CC||[EMAIL PROTECTED]
--- Additional Comments From [EMAIL PROTECTED] 2007-05-09 05:26 EST ---
I am not yet a package-maintainer. This is my pre-review to exercise
reviewing.
rpmbuild builds rpms successfully from the source with ddccontrol.spec,
and rpm -i installs the generated binary rpms (libddccontrol, ddccontrol,
ddccontrol-applet) successfully. mock, however, ends with errors. This
is due to lack of gettext in BuildRequires. Adding this to the spec
file, mock builds rpms successfully.
rpmlint says on ddccontrol-0.4.2-1.fc6.src.rpm:
start here-
W: ddccontrol non-standard-group System/Configuration/Hardware
The value of the Group tag in the package is not valid. Valid groups are:
Amusements/Games, Amusements/Graphics, Applications/Archiving,
Applications/Communications, Applications/Databases,
Applications/Editors, Applications/Emulators, Applications/Engineering,
Applications/File, Applications/Internet, Applications/Multimedia,
Applications/Productivity, Applications/Publishing, Applications/System,
Applications/Text, Development/Debug, Development/Debuggers,
Development/Languages, Development/Libraries, Development/System,
Development/Tools, Documentation, System Environment/Base, System
Environment/Daemons, System Environment/Kernel, System
Environment/Libraries, System Environment/Shells, User
Interface/Desktops, User Interface/X, User Interface/X Hardware Support.
W: ddccontrol hardcoded-path-in-buildroot-tag /var/tmp/%{name}-buildroot
A path is hardcoded in your Buildroot tag. It should be replaced
by something like %{_tmppath}/%name-root.
E: ddccontrol 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.
E: ddccontrol no-cleaning-of-buildroot %clean
You should clean $RPM_BUILD_ROOT in the %clean section and just after the
beginning of %install section. Use rm -Rf $RPM_BUILD_ROOT.
W: ddccontrol no-%clean-section
The spec file doesn't contain a %clean section to remove the files installed
by the %install section.
end here
rpmlint says on the generated ddccontrol-0.4.2-1.fc6.i368.rpm:
(Hereafter, I remove the same message about grouping above.)
start here-
E: ddccontrol standard-dir-owned-by-package /usr/share/man/man1
This package owns a directory that is part of the standard hierarchy, which
can lead to default directory permissions or ownerships being changed to
something non-standard.
E: ddccontrol setuid-binary /usr/bin/ddcpci root 04711
The file is setuid, this may be dangerous, especially if this
file is setuid root.
E: ddccontrol non-standard-executable-perm /usr/bin/ddcpci 04711
A standard executable should have permission set to 0755. If you get this
message, it means that you have a wrong executable permissions in some files
included in your package.
W: ddccontrol incoherent-version-in-changelog 0.4.2-alt2 0.4.2-1.fc6
The last entry in %changelog contains a version identifier that is not
coherent with the epoch:version-release tuple of the package.
W: ddccontrol unstripped-binary-or-object /usr/bin/ddccontrol
W: ddccontrol unstripped-binary-or-object /usr/bin/ddcpci
W: ddccontrol unstripped-binary-or-object /usr/bin/gddccontrol
W: ddccontrol one-line-command-in-%post #%update_menus
You should use %post -p command instead of using:
%post
command
It will avoid the fork of a shell interpreter to execute your command as
well as allows rpm to automatically mark the dependency on your command
for the excecution of the scriptlet.
W: ddccontrol one-line-command-in-%postun #%clean_menus
You should use %postun -p command instead of using:
%postun
command
It will avoid the fork of a shell interpreter to execute your command as
well as allows rpm to automatically mark the dependency on your command
for the excecution of the scriptlet.
end here
rpmlint says on the generated ddccontrol-applet-0.4.2-1.fc6.i368.rpm:
start here-
W: ddccontrol-applet no-documentation
The package contains no documentation (README, doc, etc).
You have to include documentation files.
W: ddccontrol-applet unstripped-binary-or-object /usr/lib/ddccontrol/ddcc-applet
end here
rpmlint says on the generated
[Bug 239525] Review Request: gammu - Command Line utility and Libraries to work with mobile phones
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: gammu - Command Line utility and Libraries to work with mobile phones https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239525 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 05:47 EST --- rpmlint on SRPM reports W: gammu mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 12) The specfile mixes use of spaces and tabs for indentation, which is a cosmetic annoyance. Use either spaces or tabs for indentation, not both. rpmlint on RPM reports W: gammu spurious-executable-perm /usr/share/doc/gammu-1.10.0/examples/ppp/startppp The file is installed with executable permissions, but was identified as one that probably should not be executable. Verify if the executable bits are desired, and remove if not. W: gammu spurious-executable-perm /usr/share/doc/gammu-1.10.0/examples/fax/faxsend The file is installed with executable permissions, but was identified as one that probably should not be executable. Verify if the executable bits are desired, and remove if not. W: gammu spurious-executable-perm /usr/share/doc/gammu-1.10.0/examples/fax/faxreceive The file is installed with executable permissions, but was identified as one that probably should not be executable. Verify if the executable bits are desired, and remove if not. W: gammu spurious-executable-perm /usr/share/doc/gammu-1.10.0/examples/ppp/gprs The file is installed with executable permissions, but was identified as one that probably should not be executable. Verify if the executable bits are desired, and remove if not. W: gammu spurious-executable-perm /usr/share/doc/gammu-1.10.0/examples/ppp/statsppp The file is installed with executable permissions, but was identified as one that probably should not be executable. Verify if the executable bits are desired, and remove if not. W: gammu spurious-executable-perm /usr/share/doc/gammu-1.10.0/examples/ppp/data The file is installed with executable permissions, but was identified as one that probably should not be executable. Verify if the executable bits are desired, and remove if not. W: gammu doc-file-dependency /usr/share/doc/gammu-1.10.0/examples/fax/faxreceive /bin/sh An included file marked as %doc creates a possible additional dependency in the package. Usually, this is not wanted and may be caused by eg. example scripts with executable bits set included in the package's documentation. W: gammu doc-file-dependency /usr/share/doc/gammu-1.10.0/examples/fax/faxsend /bin/sh An included file marked as %doc creates a possible additional dependency in the package. Usually, this is not wanted and may be caused by eg. example scripts with executable bits set included in the package's documentation. W: gammu doc-file-dependency /usr/share/doc/gammu-1.10.0/examples/ppp/data /bin/sh An included file marked as %doc creates a possible additional dependency in the package. Usually, this is not wanted and may be caused by eg. example scripts with executable bits set included in the package's documentation. W: gammu doc-file-dependency /usr/share/doc/gammu-1.10.0/examples/ppp/gprs /bin/sh An included file marked as %doc creates a possible additional dependency in the package. Usually, this is not wanted and may be caused by eg. example scripts with executable bits set included in the package's documentation. W: gammu doc-file-dependency /usr/share/doc/gammu-1.10.0/examples/ppp/startppp /bin/sh An included file marked as %doc creates a possible additional dependency in the package. Usually, this is not wanted and may be caused by eg. example scripts with executable bits set included in the package's documentation. W: gammu doc-file-dependency /usr/share/doc/gammu-1.10.0/examples/ppp/statsppp /bin/sh An included file marked as %doc creates a possible additional dependency in the package. Usually, this is not wanted and may be caused by eg. example scripts with executable bits set included in the package's documentation. rpmlint on gammu-devel reports W: gammu-devel no-dependency-on gammu W: gammu-devel summary-not-capitalized gammu development files Summary doesn't begin with a capital letter. W: gammu-devel non-standard-group Development/libraries The value of the Group tag in the package is not valid. Valid groups are: Amusements/Games, Amusements/Graphics, Applications/Archiving, Applications/Communications, Applications/Databases, Applications/Editors, Applications/Emulators, Applications/Engineering, Applications/File, Applications/Internet, Applications/Multimedia, Applications/Productivity, Applications/Publishing, Applications/System, Applications/Text, Development/Debug, Development/Debuggers, Development/Languages, Development/Libraries, Development/System, Development/Tools, Documentation, System Environment/Base, System Environment/Daemons, System Environment/Kernel, System Environment/Libraries,
[Bug 237373] Review Request: mcpp - Alternative C/C++ preprocessor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: mcpp - Alternative C/C++ preprocessor https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=237373 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 06:00 EST --- I have just written pre-review on the following two new packages. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239154 referencer: a Gnome application to organise documents or references https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=238248 ddccontrol: control monitor parameters,like brightness and contrast Writing review is a hard task than I expected! -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239234] Review Request: arp-scan - Scanning and fingerprinting tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: arp-scan - Scanning and fingerprinting tool https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239234 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239525] Review Request: gammu - Command Line utility and Libraries to work with mobile phones
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: gammu - Command Line utility and Libraries to work with mobile phones https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239525 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 07:09 EST --- Created an attachment (id=154387) -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=154387action=view) Modified SPEC I did some workaround for your SPEC to solve most of rpmlint errors and warnings. kindly update SPEC,bump release and provide a new SRPM -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239546] New: Review Request: mdsplib - METAR Decoder Software Package Library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239546 Summary: Review Request: mdsplib - METAR Decoder Software Package Library Product: Fedora Extras Version: devel Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: fedora-package-review@redhat.com Spec URL: http://ftp.es6.freshrpms.net/tmp/extras/mdsplib/mdsplib.spec SRPM URL: http://ftp.es6.freshrpms.net/tmp/extras/mdsplib/mdsplib-0.11-2.src.rpm Description: METAR is a French acronym for Aviation Routine Weather Report. It is the standard format for reporting meteorological conditions. As a free service, the National Weather Service continually updates a server with METARs from all over the world. The MDSP Library provides a programmer with two functions, the major one being DcdMETAR, which decodes a METAR into structures provided by the library. Also prtDMETR, which prints out a decoded METAR structure. -- Packager note : It would be best to have this package patched to generate a shared library instead of only the default static library. It's not trivial since it doesn't use libtool, but would make sense since this version has been the latest for over 5 years now. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 230549] Review Request: oxine - Lightweight, purely OSD based xine frontend
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: oxine - Lightweight, purely OSD based xine frontend https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=230549 [EMAIL PROTECTED] changed: What|Removed |Added Severity|normal |medium Priority|normal |medium [EMAIL PROTECTED] changed: What|Removed |Added BugsThisDependsOn||239546 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 07:14 EST --- Spec URL: http://ftp.es6.freshrpms.net/tmp/extras/oxine/oxine.spec SRPM URL: http://ftp.es6.freshrpms.net/tmp/extras/oxine/oxine-0.6-2.src.rpm * Tue May 8 2007 Matthias Saou http://freshrpms.net/ 0.6-2 - Enable weather now that mdsplib is built. And make this bug depend on bug #239546 (mdsplib review request). -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239546] Review Request: mdsplib - METAR Decoder Software Package Library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: mdsplib - METAR Decoder Software Package Library https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239546 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||230549 nThis|| -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233596] Review Request: python-Coherence - Python framework to participate in digital living networks
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-Coherence - Python framework to participate in digital living networks https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233596 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 07:16 EST --- New Package CVS Request === Package Name: python-Coherence Short Description: Python framework to participate in digital living networks Owners: [EMAIL PROTECTED] Branches: devel FC-6 FC-5 EL-4 EL-5 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233598] Review Request: elisa - Media Center
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: elisa - Media Center https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233598 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 07:18 EST --- Spec URL: http://ftp.es6.freshrpms.net/tmp/extras/elisa/elisa.spec SRPM URL: http://ftp.es6.freshrpms.net/tmp/extras/elisa/elisa-0.1.6-3.src.rpm * Tue May 8 2007 Matthias Saou http://freshrpms.net/ 0.1.6-3 - Change Coherence requirement to python-Coherence to match package name change. - python-Coherence should be imported and built shortly. - python-tag is the only dependency left... I didn't hear back from the author yet :-( -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233782] Review Request: vegastrike - 3D OpenGL spaceflight simulator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: vegastrike - 3D OpenGL spaceflight simulator https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233782 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 07:32 EST --- Builds. rpmlint clean with one exception: W: vegastrike patch-not-applied Patch4: vegastrike-0.4.2-opengl-fix.patch A patch is included in your package but was not applied. Refer to the patches documentation to see what's wrong. Looks like the line including the patch needs to be commented out. Meets naming guidelines. Spec name is good. Meets packaging guidelines. (albeit with some interesting scriptlets) License is good, but COPYING needs to be included in %doc, along with AUTHORS, DOCUMENTATION, etc. Spec is legible American English. Source does not match upstream tarball, but there is no such thing, and spec includes tar.bz2 generation steps. Hence, no URL. Builds on fc6/i386. More coming. . . -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 210823] Review Request: wxsvg - wxSVG is C++ library to create, manipulate and render SVG files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: wxsvg - wxSVG is C++ library to create, manipulate and render SVG files https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=210823 [EMAIL PROTECTED] changed: What|Removed |Added Priority|normal |medium [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 07:35 EST --- Good question. I have a wxsvg package in the pipe too. If the initial reporter doesn't show any interest in getting the package in anymore, I think I'll be submitting mine directly. Current version here : http://ftp.es6.freshrpms.net/tmp/extras/wxsvg/wxsvg.spec http://ftp.es6.freshrpms.net/tmp/extras/wxsvg/wxsvg-1.0-0.2.b7.src.rpm I haven't looked at the packages from this bug, but the first major issue I do see is that the version is wrong, as this b7 is a beta7, not some post 1.0 release, as Michael already pointed out. It builds fine on FC6 x86_64 for me. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239234] Review Request: arp-scan - Scanning and fingerprinting tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: arp-scan - Scanning and fingerprinting tool https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239234 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 07:52 EST --- Review: + package builds in mock (development i386). + rpmlint is silent for SRPM and RPM. + source files match upstream. eb24303c6eb4d77c3abf23511efce642 arp-scan-1.6.tar.gz + 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 COPYING is included in package. + %doc is small so no need of -doc subpackage. + BuildRequires are proper. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code, not content. + no static libraries. + no .pc files are present. + no -devel subpackage exists. + no .la files. + no translations are available. + Does owns the directories it creates. + no duplicates in %files. + file permissions are appropriate. + no scriptlets are used. + Requires: /usr/bin/perl libc.so.6 libc.so.6(GLIBC_2.0) libc.so.6(GLIBC_2.1) libc.so.6(GLIBC_2.3) libc.so.6(GLIBC_2.3.4) libc.so.6(GLIBC_2.4) libpcap.so.0.9 perl(Getopt::Std) perl(LWP::Simple) perl(LWP::Simple) perl(strict) perl(warnings) rtld(GNU_HASH) APPROVED. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239525] Review Request: gammu - Command Line utility and Libraries to work with mobile phones
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: gammu - Command Line utility and Libraries to work with mobile phones https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239525 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 08:02 EST --- I saw neither python-gammu nor wammu uses gammu-libs rpm. So is there any other package needs gammu-libs? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239234] Review Request: arp-scan - Scanning and fingerprinting tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: arp-scan - Scanning and fingerprinting tool https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239234 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 08:23 EST --- New Package CVS Request === Package Name: arp-scan Short Description: Scanning and fingerprinting tool Owners: [EMAIL PROTECTED] Branches: FC-5 FC-6 EL-4 EL-5 InitialCC: [EMAIL PROTECTED] -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239087] Review Request: perl-Nmap-Parser - Parse nmap scan data with perl
Please 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-Nmap-Parser - Parse nmap scan data with perl Alias: perl-Nmap-Parser https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239087 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 08:24 EST --- Updated: - Move end-of-line fix to %prep - Update dos2unix BR Spec URL: http://folk.ntnu.no/sindrb/packages/green_nyc/perl-Nmap-Parser.spec SRPM URL: http://folk.ntnu.no/sindrb/packages/green_nyc/perl-Nmap-Parser-1.05-3.fc7.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 234749] Review Request: arm-linux-binutils - Cross Compiling GNU binutils targeted at arm-linux
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: arm-linux-binutils - Cross Compiling GNU binutils targeted at arm-linux https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=234749 [EMAIL PROTECTED] changed: What|Removed |Added Priority|normal |medium [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 08:43 EST --- Shouldn't this be based on the Fedora binutils package, but with the suitable target definition? I did specfiles based on Fedora's for cross-compilation (to i686) once upon a time. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233782] Review Request: vegastrike - 3D OpenGL spaceflight simulator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: vegastrike - 3D OpenGL spaceflight simulator https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233782 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 08:58 EST --- I think the BRs are OK, but I'm building in mock to test. Taking awhile. .. No locales, shared libraries. Not relocatable. No ownership issues evident. Permissions, defattr OK. %clean present and correct. Macros look OK. No overly large docs. No header files or static libraries. No pkgconfig No versioned .so No devel package. No .la. Desktop file present and handled properly. No ownership conflicts. install begins with buildroot wipe. All UTF-8 filenames. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 234749] Review Request: arm-linux-binutils - Cross Compiling GNU binutils targeted at arm-linux
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: arm-linux-binutils - Cross Compiling GNU binutils targeted at arm-linux https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=234749 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 09:01 EST --- Fedora's binutils spec is IMHO somewhat ugly/messy. If it turns out that this will become just as ugly/messy I might go that way. But for now I prefer to start clean. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233782] Review Request: vegastrike - 3D OpenGL spaceflight simulator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: vegastrike - 3D OpenGL spaceflight simulator https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233782 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 09:03 EST --- Mock builds fine. BRs are OK. All other MUSTS are met. Looks like it's just the %doc issue and the patch question. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 234749] Review Request: arm-linux-binutils - Cross Compiling GNU binutils targeted at arm-linux
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: arm-linux-binutils - Cross Compiling GNU binutils targeted at arm-linux https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=234749 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 09:03 EST --- Also this is not for arm-linux-fedora, but for arm-linux, the first target I have in mind is my gp2x handheld game device. But I'm trying to make this more generic then that. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 234749] Review Request: arm-linux-binutils - Cross Compiling GNU binutils targeted at arm-linux
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: arm-linux-binutils - Cross Compiling GNU binutils targeted at arm-linux https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=234749 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 09:08 EST --- (In reply to comment #4) Also this is not for arm-linux-fedora, This would be arm-redhat-linux. but for arm-linux, arm-linux is not useful, because there exist many flavors of *-linux toolchains and each vendor/distributor ships it's own variant of toolchains. the first target I have in mind is my gp2x handheld game device. But I'm trying to make this more generic then that. This won't work. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239119] Review Request: perl-Class-C3-XS - XS speedups for Class::C3
Please 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-Class-C3-XS - XS speedups for Class::C3 Alias: perl-Class-C3-XS https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239119 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 10:03 EST --- New Package CVS Request === Package Name: perl-Class-C3-XS Short Description: XS speedups for Class::C3 Owners: [EMAIL PROTECTED] Branches: FC-5, FC-6, devel InitialCC: [EMAIL PROTECTED] -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239119] Review Request: perl-Class-C3-XS - XS speedups for Class::C3
Please 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-Class-C3-XS - XS speedups for Class::C3 Alias: perl-Class-C3-XS https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239119 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239369] Review Request: perl-Sys-Syscall - Access system calls that Perl doesn't normally provide access to
Please 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-Sys-Syscall - Access system calls that Perl doesn't normally provide access to Alias: perl-Sys-Syscall https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239369 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 10:06 EST --- APPROVED -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239087] Review Request: perl-Nmap-Parser - Parse nmap scan data with perl
Please 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-Nmap-Parser - Parse nmap scan data with perl Alias: perl-Nmap-Parser https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239087 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 10:16 EST --- Now that tibbs and Ralf have done the hard work, I'll post my review :) Note that you can also use sed itself to fix end-of-line-encoding warnings; sed -i '/\r//' filename will do it. I'm just pointing this out, it's certainly not a blocker to use dos2unix. + source files match upstream: 5d5f113a9d166b07e041a5dc52f9c3ee Nmap-Parser-1.05.tar.gz 5d5f113a9d166b07e041a5dc52f9c3ee ../Nmap-Parser-1.05.tar.gz + package meets naming and versioning guidelines. + specfile is properly named, is cleanly written and uses macros consistently. + dist tag is present. + build root is correct. + license field matches the actual license. + license is open source-compatible. License text included in package. + latest version is being packaged. + BuildRequires are proper. + compiler flags are appropriate. (noarch) + %clean is present. + package installs properly + rpmlint is silent. + final provides and requires are sane: ** perl-Nmap-Parser-1.05-3.fc6.noarch.rpm == rpmlint == provides perl(Nmap::Parser) = 1.05 perl(Nmap::Parser::Host) perl(Nmap::Parser::Host::OS) perl(Nmap::Parser::Host::Service) perl(Nmap::Parser::Session) perl-Nmap-Parser = 1.05-3.fc6 == requires perl(:MODULE_COMPAT_5.8.8) perl(Storable) perl(XML::Twig) perl(strict) perl(vars) + %check is present and all tests pass: All tests successful. Files=4, Tests=171, 1 wallclock secs ( 1.07 cusr + 0.28 csys = 1.35 CPU) + no shared libraries are added to the regular linker search paths. + owns the directories it creates. + doesn't own any directories it shouldn't. + no duplicates in %files. + file permissions are appropriate. + no scriptlets present. + code, not content. + documentation is small, so no -docs subpackage is necessary. + %docs are not necessary for the proper functioning of the package. + no headers. + no pkgconfig files. + no libtool .la droppings. + not a GUI app. APPROVED -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239087] Review Request: perl-Nmap-Parser - Parse nmap scan data with perl
Please 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-Nmap-Parser - Parse nmap scan data with perl Alias: perl-Nmap-Parser https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239087 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review? |fedora-review+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239161] Review Request: perl-Net-Libdnet - Perl Interface to libdnet
Please 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-Net-Libdnet - Perl Interface to libdnet
Alias: perl-Net-Libdnet
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239161
--- Additional Comments From [EMAIL PROTECTED] 2007-05-09 10:30 EST ---
In %files, %{perl_vendorarch}/Net/* should be %{perl_vendorarch}/Net.
Missing br's on:
perl(ExtUtils::MakeMaker)
perl(Test::More)
Fix those two items, and I'll approve.
+ source files match upstream:
cec04f6a07cf9e9ec90077c7f3970ffc Net-Libdnet-0.01.tar.gz
cec04f6a07cf9e9ec90077c7f3970ffc ../Net-Libdnet-0.01.tar.gz
+ package meets naming and versioning guidelines.
+ specfile is properly named, is cleanly written and uses macros consistently.
+ dist tag is present.
+ build root is correct.
+ license field matches the actual license.
+ license is open source-compatible. License text included in package.
+ latest version is being packaged.
X BuildRequires are proper.
+ compiler flags are appropriate.
+ %clean is present.
+ debuginfo package looks complete.
+ rpmlint is silent.
+ final provides and requires are sane:
** perl-Net-Libdnet-0.01-1.fc6.x86_64.rpm
== rpmlint
== provides
Libdnet.so()(64bit)
perl(Net::Libdnet) = 0.01
perl-Net-Libdnet = 0.01-1.fc6
== requires
libc.so.6()(64bit)
libc.so.6(GLIBC_2.2.5)(64bit)
libc.so.6(GLIBC_2.4)(64bit)
libdnet.so.1()(64bit)
perl = 0:5.006
perl(:MODULE_COMPAT_5.8.8)
perl(AutoLoader)
perl(Carp)
perl(DynaLoader)
perl(Exporter)
perl(strict)
perl(warnings)
** perl-Net-Libdnet-debuginfo-0.01-1.fc6.x86_64.rpm
== rpmlint
== provides
Libdnet.so.debug()(64bit)
perl-Net-Libdnet-debuginfo = 0.01-1.fc6
== requires
+ %check is present and all tests pass:
All tests successful.
Files=10, Tests=59, 1 wallclock secs ( 0.26 cusr + 0.16 csys = 0.42 CPU)
+ no shared libraries are added to the regular linker search paths.
X owns the directories it creates.
+ doesn't own any directories it shouldn't.
+ no duplicates in %files.
+ file permissions are appropriate.
+ no scriptlets present.
+ code, not content.
+ documentation is small, so no -docs subpackage is necessary.
+ %docs are not necessary for the proper functioning of the package.
+ no headers.
+ no pkgconfig files.
+ no libtool .la droppings.
+ not a GUI app.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 234749] Review Request: arm-linux-binutils - Cross Compiling GNU binutils targeted at arm-linux
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: arm-linux-binutils - Cross Compiling GNU binutils targeted at arm-linux https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=234749 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 10:39 EST --- (In reply to comment #5) (In reply to comment #4) Also this is not for arm-linux-fedora, This would be arm-redhat-linux. but for arm-linux, arm-linux is not useful, because there exist many flavors of *-linux toolchains and each vendor/distributor ships it's own variant of toolchains. the first target I have in mind is my gp2x handheld game device. But I'm trying to make this more generic then that. This won't work. Hmm, I thought that with gcc-multilib, it should be possible to have 2 generic toolchains: arm-linux and arm-uclinux, so that atleast the toolchain could be shared. But I guess when you add things like g++ ABI, that then even that is not possible. So does this mean that it would be better to just do an arm-gp2x-linux toolchain then? That would actually make things quite a bit easier. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233782] Review Request: vegastrike - 3D OpenGL spaceflight simulator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: vegastrike - 3D OpenGL spaceflight simulator https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233782 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 10:52 EST --- (In reply to comment #14) Looks like the line including the patch needs to be commented out. It has this comment above it: # not applied as no longer needed, kept for reference Commenting it out will exclude it from the srpm and from cvs-import, which is not what I want. License is good, but COPYING needs to be included in %doc, along with AUTHORS, DOCUMENTATION, etc. Good catch will fix. (In reply to comment #16) Mock builds fine. BRs are OK. All other MUSTS are met. Looks like it's just the %doc issue and the patch question. See above, here is a new version with the %doc fixed: http://people.atrpms.net/~hdegoede/vegastrike.spec http://people.atrpms.net/~hdegoede/vegastrike-0.4.3-3.fc7.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233782] Review Request: vegastrike - 3D OpenGL spaceflight simulator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: vegastrike - 3D OpenGL spaceflight simulator https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233782 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 10:56 EST --- Ok. Sounds reasonable. APPROVED. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233782] Review Request: vegastrike - 3D OpenGL spaceflight simulator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: vegastrike - 3D OpenGL spaceflight simulator https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233782 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 11:03 EST --- Thanks for the review!! New Package CVS Request === Package Name: vegastrike Short Description: 3D OpenGL spaceflight simulator Owners:[EMAIL PROTECTED] Branches: FC-6 devel InitialCC: empty -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225575] Review Request: roundcubemail - Round Cube Webmail is a browser-based multilingual IMAP client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: roundcubemail - Round Cube Webmail is a browser-based multilingual IMAP client https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225575 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 11:15 EST --- My permission errors brought about by log/temp/config path changes: [EMAIL PROTECTED] SPECS]$ rpmlint -i ../RPMS/noarch/roundcubemail-0.1-beta2.2.3.noarch.rpm E: roundcubemail non-standard-dir-perm /etc/roundcubemail 0640 A standard directory should have permission set to 0755. If you get this message, it means that you have wrong directory permissions in some dirs included in your package. E: roundcubemail non-readable /etc/roundcubemail/db.inc.php.dist 0640 The file can't be read by everybody. If this is expected (for security reasons), contact your rpmlint distributor to get it added to the list of exceptions for your distro (or add it to your local configuration if you installed rpmlint from the source tarball). E: roundcubemail non-standard-gid /var/log/roundcubemail apache A file in this package is owned by a non standard group. Standard groups are: root, bin, daemon, sys, adm, tty, disk, lp, mem, kmem, wheel, mail, news, uucp, man, games, gopher, dip, ftp, lock, nobody, users E: roundcubemail non-standard-dir-perm /var/log/roundcubemail 0775 A standard directory should have permission set to 0755. If you get this message, it means that you have wrong directory permissions in some dirs included in your package. E: roundcubemail non-readable /etc/httpd/conf.d/roundcubemail.conf 0640 The file can't be read by everybody. If this is expected (for security reasons), contact your rpmlint distributor to get it added to the list of exceptions for your distro (or add it to your local configuration if you installed rpmlint from the source tarball). W: roundcubemail spurious-executable-perm /usr/share/doc/roundcubemail-0.1/SQL/postgres.initial.sql The file is installed with executable permissions, but was identified as one that probably should not be executable. Verify if the executable bits are desired, and remove if not. E: roundcubemail non-readable /etc/roundcubemail/main.inc.php.dist 0640 The file can't be read by everybody. If this is expected (for security reasons), contact your rpmlint distributor to get it added to the list of exceptions for your distro (or add it to your local configuration if you installed rpmlint from the source tarball). W: roundcubemail conffile-without-noreplace-flag /etc/roundcubemail/db.inc.php.dist A configuration file is stored in your package without the noreplace flag. A way to resolve this is to put the following in your SPEC file: %config(noreplace) /etc/your_config_file_here W: roundcubemail conffile-without-noreplace-flag /etc/roundcubemail/main.inc.php.dist A configuration file is stored in your package without the noreplace flag. A way to resolve this is to put the following in your SPEC file: %config(noreplace) /etc/your_config_file_here -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239161] Review Request: perl-Net-Libdnet - Perl Interface to libdnet
Please 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-Net-Libdnet - Perl Interface to libdnet Alias: perl-Net-Libdnet https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239161 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 11:22 EST --- Updated: - Added missing BRs - Fixed %files list Spec URL: http://folk.ntnu.no/sindrb/packages/green_nyc/perl-Net-Libdnet.spec SRPM URL: http://folk.ntnu.no/sindrb/packages/green_nyc/perl-Net-Libdnet-0.01-2.fc7.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239087] Review Request: perl-Nmap-Parser - Parse nmap scan data with perl
Please 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-Nmap-Parser - Parse nmap scan data with perl Alias: perl-Nmap-Parser https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239087 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 11:26 EST --- New Package CVS Request === Package Name: perl-Nmap-Parser Short Description: Parse nmap scan data with perl Owners: [EMAIL PROTECTED] Branches: FC-5 FC-6 EL-4 EL-5 InitialCC: [EMAIL PROTECTED] -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 238562] Review Request: machineball - A futuristic ball game with simple rules
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: machineball - A futuristic ball game with simple rules
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=238562
[EMAIL PROTECTED] changed:
What|Removed |Added
Flag||fedora-review?
--- Additional Comments From [EMAIL PROTECTED] 2007-05-09 11:57 EST ---
REVIEW:
* rpmlint is quiet
!* missing BR: mesa-libGLU-devel
* licensed under GPL license and its text's included
* required %post(un) supplied
* dist tag present
!* no macros in Source0 tag
* buildroot's fine
* package doesn't create any new directories
* proper permissions of all files
* desktop file seems fine
* %clean section is fine
* source's md5sum is good, but
!* curiously enough, its name is different than the offical one. Adding
%{version} tag should fix that problem
THINGS TO DO:
- add missing BR: mesa-libGLU-devel
- add at least %{version} to Source0 tag
- upload proper source code file to srpm (current one has a wrong name)
The last issue forces change of versioning. Probably you can just replace -
with ., but if aby other (better) ideas are very welcome.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 235741] Review Request: hippo-canvas - Crossplatform canvas using cairo
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: hippo-canvas - Crossplatform canvas using cairo https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235741 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|177841 | nThis|| Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 11:58 EST --- New Package CVS Request === Package Name: hippo-canvas Short Description: Crossplatform canvas using cairo Owners: [EMAIL PROTECTED] Branches: InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 238562] Review Request: machineball - A futuristic ball game with simple rules
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: machineball - A futuristic ball game with simple rules https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=238562 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 12:05 EST --- (In reply to comment #6) The last issue forces change of versioning. Probably you can just replace - with ., but if aby other (better) ideas are very welcome. I should have written but any other (better) ideas are very welcome ;) -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 237373] Review Request: mcpp - Alternative C/C++ preprocessor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: mcpp - Alternative C/C++ preprocessor https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=237373 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 12:08 EST --- Well, * mcpp packaging/performance is okay * For pre-reviewing: - for referencer, actually Deji Akingunola is a good packager and the spec file seems no problem at a glance (I didn't check in detail) - for ddccontrol, according to your comment the spec file seems to have many problem. I guess more problem may be found (for this package, I have not seen the spec file yet) (In reply to comment #16) Writing review is a hard task than I expected! Perhaps. While I have reviewed about 140 packages and have submitted about 40 review requests, reviewing srpm was very difficult for me first. Okay!! -- This package (mcpp) is APPROVED by me -- Please follow: http://fedoraproject.org/wiki/PackageMaintainers/Join from Get a Fedora Account. I will receive a mail which notices that you need a sponsor and then I will sponsor you. NOTE: devel repository is currently under the way of the merge of Core and Extras and you may meet some trouble. If you have some trouble, please let me know. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239154] Review Request: referencer - A document organiser and bibliography manager for Gnome
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: referencer - A document organiser and bibliography manager for Gnome https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239154 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239154] Review Request: referencer - A document organiser and bibliography manager for Gnome
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: referencer - A document organiser and bibliography manager for Gnome https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239154 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239161] Review Request: perl-Net-Libdnet - Perl Interface to libdnet
Please 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-Net-Libdnet - Perl Interface to libdnet Alias: perl-Net-Libdnet https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239161 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 12:05 EST --- APPROVED -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 223586] Review Request: strigi - A desktop search program for KDE
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: strigi - A desktop search program for KDE https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=223586 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 12:58 EST --- If there's a popular request for it, i can create an autostart desktop file to start the daemon along with the desktop session. If you do that, you should really split off a strigi-libs subpackage so merely installing my KDE 4 packages won't force in a daemon some users may not want. That's how beagle is packaged, too (as well as other stuff such as mysql), so it wouldn't be a first. Still, I think this (both the .desktop file and the strigi-libs subpackage) would be the best solution. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239369] Review Request: perl-Sys-Syscall - Access system calls that Perl doesn't normally provide access to
Please 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-Sys-Syscall - Access system calls that Perl doesn't normally provide access to Alias: perl-Sys-Syscall https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239369 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 13:00 EST --- New Package CVS Request === Package Name: perl-Sys-Syscall Short Description: Access system calls that Perl doesn't normally provide access to Owners: [EMAIL PROTECTED] Branches: FC-5 FC-6 -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239154] Review Request: referencer - A document organiser and bibliography manager for Gnome
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: referencer - A document organiser and bibliography
manager for Gnome
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239154
--- Additional Comments From [EMAIL PROTECTED] 2007-05-09 13:05 EST ---
Almost okay.
One issue:
* Timestamps
- This package installs some files which were not built or
modified during rebuilding (such as .png, .svg, .glade..) and
keeping timestamps on these files is recommended.
(actually for this package only 3 files are built or modified)
For this package, please use the following:
---
make DESTDIR=%{buildroot} INSTALL=%{__install} -p install
---
Other things are okay.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225575] Review Request: roundcubemail - Round Cube Webmail is a browser-based multilingual IMAP client
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: roundcubemail - Round Cube Webmail is a browser-based
multilingual IMAP client
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225575
--- Additional Comments From [EMAIL PROTECTED] 2007-05-09 13:47 EST ---
(In reply to comment #38)
%attr(0644,root,apache) %config(noreplace) %{_sysconfdir}/*
%attr(0644,root,apache) %config(noreplace) %{_sysconfdir}/%{name}/*
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 188477] Review Request: maildrop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: maildrop https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=188477 [EMAIL PROTECTED] changed: What|Removed |Added Priority|normal |medium --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 13:49 EST --- Axel is there any progress in maildrop package submit? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225575] Review Request: roundcubemail - Round Cube Webmail is a browser-based multilingual IMAP client
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: roundcubemail - Round Cube Webmail is a browser-based
multilingual IMAP client
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225575
--- Additional Comments From [EMAIL PROTECTED] 2007-05-09 13:57 EST ---
(In reply to comment #38)
W: roundcubemail spurious-executable-perm
Change the mode on these files to 0644 with chmod or install -m in the
%install
section.
Sorry for the followups, I missed a few things without looking at the spec. I
don't think you need any executables in the source files, so you should be able
to:
In %prep (after %setup), run
find . -type f -print | xarg chmod a-x
Then all those %attr lines with %roundcubemail can be replaced by just:
%{roundcubedir}
because there won't be permissions to fix (that's causing duplicate file
listings in the rpm anyway).
These odd permissions should be reported upstream as undesirable.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 223422] Review Request: mrxvt - Multi-tabbed terminal emulator.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: mrxvt - Multi-tabbed terminal emulator. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=223422 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 13:58 EST --- ping again? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225114] Review Request: jamin - JACK Audio Mastering interface
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: jamin - JACK Audio Mastering interface https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225114 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 13:59 EST --- ping again? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 221054] Review Request: gerbv - A gerber file viewer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: gerbv - A gerber file viewer https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=221054 [EMAIL PROTECTED] changed: What|Removed |Added Severity|normal |medium Priority|normal |medium [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |NEEDINFO Flag||needinfo?([EMAIL PROTECTED]) --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 14:04 EST --- ping, Robert? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225575] Review Request: roundcubemail - Round Cube Webmail is a browser-based multilingual IMAP client
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: roundcubemail - Round Cube Webmail is a browser-based
multilingual IMAP client
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225575
--- Additional Comments From [EMAIL PROTECTED] 2007-05-09 14:05 EST ---
(In reply to comment #39)
(In reply to comment #38)
%attr(0644,root,apache) %config(noreplace) %{_sysconfdir}/*
%attr(0644,root,apache) %config(noreplace) %{_sysconfdir}/%{name}/*
Thanks, I intuited that. :)
(In reply to comment #40)
(In reply to comment #38)
W: roundcubemail spurious-executable-perm
Change the mode on these files to 0644 with chmod or install -m in the
%install
section.
Sorry for the followups, I missed a few things without looking at the spec. I
don't think you need any executables in the source files, so you should be
able to:
In %prep (after %setup), run
find . -type f -print | xarg chmod a-x
Then all those %attr lines with %roundcubemail can be replaced by just:
%{roundcubedir}
because there won't be permissions to fix (that's causing duplicate file
listings in the rpm anyway).
These odd permissions should be reported upstream as undesirable.
That's great, but what's xarg?
[EMAIL PROTECTED] SPECS]$ rpmbuild -ba roundcubemail.spec
Executing(%prep): /bin/sh -e /var/tmp/rpm-tmp.91340
+ umask 022
+ cd /usr/src/redhat/BUILD
+ LANG=C
+ export LANG
+ unset DISPLAY
+ cd /usr/src/redhat/BUILD
+ rm -rf roundcubemail-0.1beta2
+ /bin/gzip -dc /usr/src/redhat/SOURCES/roundcubemail-0.1beta2.2.tar.gz
+ tar -xf -
+ STATUS=0
+ '[' 0 -ne 0 ']'
+ cd roundcubemail-0.1beta2
++ /usr/bin/id -u
+ '[' 500 = 0 ']'
++ /usr/bin/id -u
+ '[' 500 = 0 ']'
+ /bin/chmod -Rf a+rX,u+w,g-w,o-w .
+ find . -type f -print
+ xarg chmod a-x
/var/tmp/rpm-tmp.91340: line 37: xarg: command not found
error: Bad exit status from /var/tmp/rpm-tmp.91340 (%prep)
RPM build errors:
Bad exit status from /var/tmp/rpm-tmp.91340 (%prep)
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239525] Review Request: gammu - Command Line utility and Libraries to work with mobile phones
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: gammu - Command Line utility and Libraries to work with mobile phones https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239525 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 14:06 EST --- Wow..!! It seem that the link above point to the obsolete built package. here is the right below. SRPM: http://download.tuxfamily.org/lxtnow/temp/gammu-1.10.0-1.fc6.src.rpm --- rpmlint output from main package: -- W: gammu spurious-executable-perm /usr/share/doc/gammu-1.10.0/examples/ppp/startppp W: gammu spurious-executable-perm /usr/share/doc/gammu-1.10.0/examples/fax/faxsend W: gammu spurious-executable-perm /usr/share/doc/gammu-1.10.0/examples/fax/faxreceive W: gammu spurious-executable-perm /usr/share/doc/gammu-1.10.0/examples/ppp/gprs W: gammu spurious-executable-perm /usr/share/doc/gammu-1.10.0/examples/ppp/statsppp W: gammu spurious-executable-perm /usr/share/doc/gammu-1.10.0/examples/ppp/data - These files are examples script that the end-user can use for more accebilities in his action with the Gammu command line. These files shouldn't be removed from its default executable permission. These above warning are harmless and can be ignored. --- I saw neither python-gammu nor wammu uses gammu-libs rpm. So is there any other package needs gammu-libs? any reason to have separate -libs rpms? As the upstream/developer mentioned for the build, the package can be build with or without share libraries where share libraries could be use with other applications that aim to work with gammu-libs for developing (such as framework, plugin, GUI, ...). In this case, no...detached libs is not a bad thing. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225575] Review Request: roundcubemail - Round Cube Webmail is a browser-based multilingual IMAP client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: roundcubemail - Round Cube Webmail is a browser-based multilingual IMAP client https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225575 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 14:08 EST --- (In reply to comment #41) That's great, but what's xarg? xargs... sorry -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225575] Review Request: roundcubemail - Round Cube Webmail is a browser-based multilingual IMAP client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: roundcubemail - Round Cube Webmail is a browser-based multilingual IMAP client https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225575 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 14:22 EST --- Ok, that works for perms, save the non-standard warnings. Still breaks the app, Error 500, though. Spec URL: http://zanoni.jcomserv.net/extras/roundcubemail/roundcubemail.3.spec SRPM URL: http://zanoni.jcomserv.net/extras/roundcubemail/roundcubemail-0.1-beta2.2.3.src.rpm Still haven't addressed the DES issue, will soon. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 178922] Review Request: asterisk - The Open Source PBX
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: asterisk - The Open Source PBX https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=178922 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 14:47 EST --- http://nvd.nist.gov/nvd.cfm?cvename=CVE-2007-2488 The IAX2 channel driver (chan_iax2) in Asterisk before 20070504 does not properly null terminate data, which allows remote attackers to trigger loss of transmitted data, and possibly obtain sensitive information (memory contents) or cause a denial of service (application crash), by sending a frame that lacks a 0 byte. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239480] Review Request: penguin-command - Open source arcade game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: penguin-command - Open source arcade game https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239480 [EMAIL PROTECTED] changed: What|Removed |Added Summary|Review Request: penguin-|Review Request: penguin- |command - Open source arcade|command - Open source arcade |game|game Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Flag||fedora-cvs+ --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 14:52 EST --- Good: + Naming seems ok. + tar ball maches with upstream + Rpmlint is quite on source rpm + Package contains verbatin copy of the license + Rpmlint is quite on binary rpm + Package contains verbatin copy of the license. + License is ok. + Local install and uninstall works fine. + Mockbuild works fine. Bad: - Pleae use the -p flag of install to preserve the timestamp of the installed artworks I have APPROVE you and hope, that you may fix the install request before you checkin your package. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226725] Review Request: netgen - LVS netlist comparison tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: netgen - LVS netlist comparison tool https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226725 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 14:53 EST --- The build on x86_64 failed. I thought you tested it under x86_64? http://buildsys.fedoraproject.org/logs/fedora-6-extras/33099-netgen-1.3.7-7.fc6/ however, I've some segmentation fault when using netgen with my netlists. So I'll be working on finding more why and what is failing to contact upstream. Though netgen is important for my daily work, I prefer not to have a broken netgen on fedora repositories. So I'll take my time before closing this bug. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239480] Review Request: penguin-command - Open source arcade game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: penguin-command - Open source arcade game https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239480 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|NEEDINFO Flag|fedora-cvs+ |fedora-review+, needinfo? --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 14:55 EST --- Sory, unfortunately I have set the wrong flag (fedora-cvs instead of fedora-review) -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225575] Review Request: roundcubemail - Round Cube Webmail is a browser-based multilingual IMAP client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: roundcubemail - Round Cube Webmail is a browser-based multilingual IMAP client https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225575 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 15:03 EST --- Added your DES suggestion, got this: W: roundcubemail percent-in-%post E: roundcubemail shell-syntax-error-in-%post Spec URL: http://zanoni.jcomserv.net/extras/roundcubemail/roundcubemail.4.spec SRPM URL: http://zanoni.jcomserv.net/extras/roundcubemail/roundcubemail-0.1-beta2.2.4.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225575] Review Request: roundcubemail - Round Cube Webmail is a browser-based multilingual IMAP client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: roundcubemail - Round Cube Webmail is a browser-based multilingual IMAP client https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225575 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 15:28 EST --- Created an attachment (id=154419) -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=154419action=view) cleanups and fixes Ok, that works for perms, save the non-standard warnings. Still breaks the app, Error 500, though. Here is a diff against your spec in comment #43. It has misc cleanups and fixes. rpmlint output, all expected: E: roundcubemail non-standard-gid /etc/roundcubemail/main.inc.php apache E: roundcubemail non-readable /etc/roundcubemail/main.inc.php 0640 E: roundcubemail non-standard-gid /var/log/roundcubemail apache E: roundcubemail non-standard-dir-perm /var/log/roundcubemail 0775 E: roundcubemail non-standard-gid /etc/roundcubemail/db.inc.php apache E: roundcubemail non-readable /etc/roundcubemail/db.inc.php 0640 W: roundcubemail incoherent-version-in-changelog 0.1-beta2.2.3 0.1-0.1.beta2.2.fc6 (last line will be fixed when you update the changelog) -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 188477] Review Request: maildrop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: maildrop https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=188477 --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 15:33 EST --- Here it is http://dl.atrpms.net/all/maildrop-2.0.3-2.at.src.rpm http://dl.atrpms.net/all/maildrop.spec But I think I'm supposed to reopen a new bugzilla ticket. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225575] Review Request: roundcubemail - Round Cube Webmail is a browser-based multilingual IMAP client
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: roundcubemail - Round Cube Webmail is a browser-based
multilingual IMAP client
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=225575
--- Additional Comments From [EMAIL PROTECTED] 2007-05-09 15:51 EST ---
A definite improvement. I can use the app as installed. Now it's just the
non-standard rpmlint errors and the following from the DES scriptlet on %post:
[EMAIL PROTECTED] limb]# rpm -Uvh roundcubemail-0.1-beta2.3.noarch.rpm
Preparing...### [100%]
1:roundcubemail warning: /etc/roundcubemail/db.inc.php created as
/etc/roundcubemail/db.inc.php.rpmnew
warning: /etc/roundcubemail/main.inc.php created as
/etc/roundcubemail/main.inc.php.rpmnew
### [100%]
/var/tmp/rpm-tmp.92232: line 10: syntax error near unexpected token `let'
/var/tmp/rpm-tmp.92232: line 10: `let rand=${RANDOM}%${max}'
error: %post(roundcubemail-0.1-beta2.3.noarch) scriptlet failed, exit status 2
Spec URL: http://zanoni.jcomserv.net/extras/roundcubemail/roundcubemail.5.spec
SRPM
URL:http://zanoni.jcomserv.net/extras/roundcubemail/roundcubemail-0.1-beta2.3.src.rpm
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239480] Review Request: penguin-command - Open source arcade game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: penguin-command - Open source arcade game https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239480 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEEDINFO|ASSIGNED Flag|needinfo? |fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 15:57 EST --- New Package CVS Request === Package Name: penguin-command Short Description: Open source arcade game Owners: [EMAIL PROTECTED] Branches: FC-6 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 190129] Review Request: python-krbV
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-krbV https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190129 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 212898] Review Request: php-pear-Structures-DataGrid-Renderer-Pager - Renderer driver using PEAR::Pager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Structures-DataGrid-Renderer-Pager - Renderer driver using PEAR::Pager Alias: DG-Renderer-Pager https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=212898 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 212925] Review Request: php-pear-Structures-DataGrid-DataSource-RSS - DataSource driver using RSS files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Structures-DataGrid-DataSource-RSS - DataSource driver using RSS files Alias: pear-DG-RSS https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=212925 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 218226] Review Request: php-pear-Structures-DataGrid-DataSource-MDB2 - DataSource driver using PEAR::MDB2 and an SQL query
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pear-Structures-DataGrid-DataSource-MDB2 - DataSource driver using PEAR::MDB2 and an SQL query Alias: pear-DG-MDB2 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=218226 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233340] Review Request: goocanvas - A new canvas widget for GTK+ that uses cairo for drawing
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: goocanvas - A new canvas widget for GTK+ that uses cairo for drawing https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233340 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233596] Review Request: python-Coherence - Python framework to participate in digital living networks
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-Coherence - Python framework to participate in digital living networks https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233596 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 238932] Review Request: python-decoratortools - Use class and function decorators -- even in Python 2.3
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-decoratortools - Use class and function decorators -- even in Python 2.3 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=238932 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239480] Review Request: penguin-command - Open source arcade game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: penguin-command - Open source arcade game https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239480 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239369] Review Request: perl-Sys-Syscall - Access system calls that Perl doesn't normally provide access to
Please 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-Sys-Syscall - Access system calls that Perl doesn't normally provide access to Alias: perl-Sys-Syscall https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239369 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239234] Review Request: arp-scan - Scanning and fingerprinting tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: arp-scan - Scanning and fingerprinting tool https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239234 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239199] Review Request: perl-Archive-Any - Single interface to deal with file archives
Please 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-Archive-Any - Single interface to deal with file archives Alias: perl-Archive-Any https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239199 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239156] Review Request: perl-DBIx-SQLite-Simple - Easy access to SQLite databases using objects
Please 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-DBIx-SQLite-Simple - Easy access to SQLite databases using objects Alias: DBIx-SQLite-Simple https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239156 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239155] Review Request: perl-Class-Gomor - Another class and object builder
Please 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-Class-Gomor - Another class and object builder Alias: perl-Class-Gomor https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239155 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 222888] Review Request: Tiquit - A PHP5-compatible help desk incident tracking/knowledgebase system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: Tiquit - A PHP5-compatible help desk incident tracking/knowledgebase system https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222888 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 230710] Review Request: boolstuff - Disjunctive Normal Form boolean expression library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: boolstuff - Disjunctive Normal Form boolean expression library https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=230710 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Additional Comments From [EMAIL PROTECTED] 2007-05-09 16:26 EST --- It is a futile effort. Also note, your address is not obfuscated in the spec files. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 235741] Review Request: hippo-canvas - Crossplatform canvas using cairo
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: hippo-canvas - Crossplatform canvas using cairo https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=235741 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 237170] Review Request: repoman - Tool for configuring yum(8) settings and repositories
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: repoman - Tool for configuring yum(8) settings and repositories https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=237170 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 237883] Review Request: perl-SGML-Parser-OpenSP - Perl interface to the OpenSP SGML and XML parser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: perl-SGML-Parser-OpenSP - Perl interface to the OpenSP SGML and XML parser https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=237883 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239087] Review Request: perl-Nmap-Parser - Parse nmap scan data with perl
Please 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-Nmap-Parser - Parse nmap scan data with perl Alias: perl-Nmap-Parser https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239087 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 239089] Review Request: ike-scan - IKE protocol tool to discover, fingerprint and test IPsec VPN se rvers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: ike-scan - IKE protocol tool to discover, fingerprint and test IPsec VPN se rvers https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=239089 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
