[Bug 639529] Review Request: apache-commons-pool - rename of jakarta-commons-pool

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Chris Spike  changed:

   What|Removed |Added

 Blocks||588764

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639529] New: Review Request: apache-commons-pool - rename of jakarta-commons-pool

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: apache-commons-pool - rename of jakarta-commons-pool

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

   Summary: Review Request: apache-commons-pool - rename of
jakarta-commons-pool
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: chris.sp...@arcor.de
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


This is a re-review of existing package!

Spec URL:
http://spike.fedorapeople.org/apache-commons-pool/apache-commons-pool.spec
SRPM URL:
http://spike.fedorapeople.org/apache-commons-pool/apache-commons-pool-1.5.5-1.fc13.src.rpm

Description: The goal of Pool package is it to create and maintain an object
(instance) pooling package to be distributed under the ASF license. The package
should support a variety of pool implementations, but encourage support of an
interface that makes these implementations interchangeable.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 636250] Review Request: HotEqn - Java Library to display LateX Code - Dependency for Freemind

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #9 from Orion Poplawski  2010-10-02 00:24:55 
EDT ---
GPL+ is fine.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639518] Review Request: gst-entrans - Plug-ins and tools for transcoding and recording with GStreamer

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Theodore Lee  2010-10-01 22:06:44 EDT ---
[antiaircr...@aa-netbook SRPMS]$ rpmlint gst-entrans-0.10.2-1.fc13.src.rpm
gst-entrans.src: W: spelling-error Summary(en_US) transcoding -> trans coding,
trans-coding, transcending
gst-entrans.src: W: spelling-error %description -l en_US GEntrans -> Gentians,
Gen trans, Gen-trans
gst-entrans.src: W: spelling-error %description -l en_US transcoding -> trans
coding, trans-coding, transcending
gst-entrans.src: W: spelling-error %description -l en_US GStreamer -> G
Streamer, Streamer, Streamliner
gst-entrans.src: W: invalid-url URL: http://gentrans.sourceforge.net/ 
gst-entrans.src: W: no-buildroot-tag
gst-entrans.src: W: invalid-url Source0:
http://downloads.sourceforge.net/gentrans/gst-entrans-0.10.2.tar.gz 
1 packages and 0 specfiles checked; 0 errors, 7 warnings.

The invalid URL warnings are the fault of my university's proxy.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639518] Review Request: gst-entrans - Plug-ins and tools for transcoding and recording with GStreamer

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Theodore Lee  changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)

--- Comment #1 from Theodore Lee  2010-10-01 21:50:55 EDT ---
Koji scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=2503553

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639518] New: Review Request: gst-entrans - Plug-ins and tools for transcoding and recording with GStreamer

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: gst-entrans - Plug-ins and tools for transcoding and 
recording with GStreamer

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

   Summary: Review Request: gst-entrans - Plug-ins and tools for
transcoding and recording with GStreamer
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: theo...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://www.masteryofthearts.co.cc/random/gst-entrans.spec
SRPM URL:
http://www.masteryofthearts.co.cc/random/gst-entrans-0.10.2-1.fc13.src.rpm
Description:
GEntrans is a software package providing a collection of plug-ins and tools 
for the GStreamer multimedia framework specifically geared towards transcoding 
and recording purposes.

GStreamer allows for easy multimedia processing and creation of multimedia 
applications, as e.g. demonstrated by a number of players and some other 
applications already built on it. The purpose here is to concentrate on using 
the framework for transcoding purposes.

This is my first package for Fedora, so I'm in need of a sponsor.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 226349] Merge Review: pyxf86config

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Parag AN(पराग)  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||panem...@gmail.com
 AssignedTo|nob...@fedoraproject.org|panem...@gmail.com
   Flag||fedora-review?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 226547] Merge Review: x86info

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Parag AN(पराग)  changed:

   What|Removed |Added

 AssignedTo|da...@redhat.com|panem...@gmail.com
   Flag||fedora-review?

--- Comment #16 from Parag AN(पराग)  2010-10-01 21:21:23 
EDT ---
This is merge-review so should be assigned to the reviewer only till it gets
closed.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 226547] Merge Review: x86info

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Parag AN(पराग)  changed:

   What|Removed |Added

   Flag|fedora-review?  |

--- Comment #15 from Parag AN(पराग)  2010-10-01 21:20:28 
EDT ---
sure I will check this and provide cleanup patch.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 588692] Review Request: rubygem-rerun - Restarts your app when file changes

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #13 from Fedora Update System  
2010-10-01 21:13:35 EDT ---
rubygem-rerun-0.5.2-3.fc14 has been pushed to the Fedora 14 testing repository.
 If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update rubygem-rerun'.  You can
provide feedback for this update here:
https://admin.fedoraproject.org/updates/rubygem-rerun-0.5.2-3.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 634258] Review Request: spew - I/O performance measurement and load generation tool

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #12 from Fedora Update System  
2010-10-01 21:13:30 EDT ---
spew-1.0.8-3.fc14 has been pushed to the Fedora 14 testing repository.  If
problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update spew'.  You can provide
feedback for this update here:
https://admin.fedoraproject.org/updates/spew-1.0.8-3.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 529441] Review Request: pdfbox - Java PDF library

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Bug 529441 depends on bug 529433, which changed state.

Bug 529433 Summary: Update to 2.4.1, 2.9.0
https://bugzilla.redhat.com/show_bug.cgi?id=529433

   What|Old Value   |New Value

 Resolution||RAWHIDE
 Status|NEW |CLOSED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 633215] Review Request: SimplyHTML - Application and a java component for rich text processing

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from hannes  2010-10-01 16:07:18 
EDT ---
Spec URL: http://hannes.fedorapeople.org/SimplyHTML/SimplyHTML.spec
SRPM URL:
http://hannes.fedorapeople.org/SimplyHTML/SimplyHTML-0.13.1-3.fc13.src.rpm

I hope that I fixed everything! Made an extra subdirectory for the symlinks.
All the best and a nice weekend!

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 633215] Review Request: SimplyHTML - Application and a java component for rich text processing

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from hannes  2010-10-01 16:08:25 
EDT ---
Sorry a typo in the SRPM URL:
http://hannes.fedorapeople.org/SimplyHTML/SimplyHTML-0.13.1-3.fc14.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 226547] Merge Review: x86info

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #14 from Dave Jones  2010-10-01 16:11:01 EDT ---
anyone see anything else I need to fix ? 

$ rpmlint x86info.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612581] Review Request: spacewalk-backend - Common programs needed to be installed on the Spacewalk servers/proxies

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Jesse Keating  changed:

   What|Removed |Added

 Blocks||639391

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 626666] Review Request: groonga - An Embeddable Fulltext Search Engine

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Mamoru Tasaka  2010-10-01 
14:54:28 EDT ---
Some notes (checked 1.0.2-2)

* BR
  - Looking into bindings/ directory, it seems that some languages bindings
are available for this package (python, php, and also perhaps ruby?).
Would you try to enable these bindings?

* License
---
doc/ja/README says LGPLv2

BSD
doc/ja/html/_static/

MIT or GPLv2
doc/ja/html/_static/jquery.js
resource/admin_html/css/ui-lightness/jquery-ui-1.8.1.custom.css
resource/admin_html/js/jquery-1.4.2.min.js
--
  - So -libs license should be "LGPLv2 and (MIT or GPLv2)" and -doc license
should be "LGPLv2 and BSD".

* Timestamps
  - Please consider to use
---
make install DESTDIR=$RPM_BUILD_ROOT INSTALL~="install -p"
---
to keep timestamps on installed files as much as possible.
This method usually works for Makefiles generated by recent autotool
scripts.

* Macros
  - Please use macros for standard directories. /var should be %_var or
%_localstatedir, /usr/sbin should be %_sbin, and etc
https://fedoraproject.org/wiki/Packaging/RPMMacros

* SysV script related issues
  - Please use %{_initddir} (/etc/rc.d/init.d) instead of %_sysconfdir/init.d
   
https://fedoraproject.org/wiki/Packaging/SysVInitScript#Initscripts_on_the_filesystem

  - For registering groonga user or group, please follow
https://fedoraproject.org/wiki/Packaging/UsersAndGroups
! Note
  The current way of yours is anyway confusing because doing 
  'echo "Unexpected error adding group \"groonga\". Aborting
installation."'
  does not actually abort installation (because $ echo foo will perhaps
exit
  with status 0).

---
 /usr/sbin/useradd -r -s /sbin/nologin -c 'groonga' \
  -d %{_localstatedir}/lib/groonga --create-home \
---
- $ env LANG=C man useradd says "--create-home" will copy files in skeleton
  directory into the created home directory. Is it needed?
- And anyway %_localstatedir/lib/groonga must also be listed in %files
  entry, so I don't think --create-home option is needed here.

  - The lines
---
%post
/bin/mkdir -p /var/run/groonga
/bin/chown -R groonga:groonga /var/run/groonga
---
is wrong. In this case, the directory %_localstatedir/run/%name must be
registered in %files entry with proper %attr added.

---
%post munin-plugins
/usr/sbin/munin-node-configure --shell --remove-also | grep -e 'groonga_' | sh
[ -f /var/lock/subsys/munin-node ] && \
 /sbin/service munin-node restart > /dev/null 2>&1
---
  - Some Requires(post) is required here (Requires(post): munin-node)
  - Doesn't munin-node service support condrestart?

  - We don't usually call userdel / groupdel on scriptlets:
https://fedoraproject.org/wiki/Packaging/UsersAndGroups

  - Would you explain why this is needed?

%postun munin-plugins
if [ $1 -eq 0 ]; then
 rm %{_sysconfdir}/munin/plugins/groongar_* > /dev/null 2>&1


  - By the way calling /sbin/service or /sbin/chkconfig or getent or
so requires writing Proper "Requires(foo): bar"s. Please
refer to:
https://fedoraproject.org/wiki/Packaging/UsersAndGroups
   
https://fedoraproject.org/wiki/Packaging/SysVInitScript#Initscripts_in_spec_file_scriptlets

* %files
  - We now prefer to use %defattr(-,root,root,-)

  - Files under %_mandir (note: please use %_mandir instead of %_datadir/man)
are
automatically marked as %doc, so there is no need to write explicit %doc
attribute
for %_mandir/foo.

  - Please take care of directory ownership issue.
   
https://fedoraproject.org/wiki/Packaging/Guidelines#File_and_Directory_Ownership
https://fedoraproject.org/wiki/Packaging/UnownedDirectories

! The point is that when installing a (binary) rpm creates some new
directories,
  the created directories should also be owned by the (just installed) rpm.
  For example, when installing groonga-libs rpm, installing
  %_libdir/groonga/modules/suggest/suggest.so (on i686)
  newly creates the following directories:
--
%_libdir/%name
%_lib

[Bug 631763] Review Request: zif - Simple wrapper for rpm

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #13 from Richard Hughes  2010-10-01 14:44:50 
EDT ---
New Package SCM Request
===
Package Name: zif
Short Description: Simple wrapper for rpm
Owners: rhughes
Branches: f14
InitialCC: rhughes

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631763] Review Request: zif - Simple wrapper for rpm

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Richard Hughes  changed:

   What|Removed |Added

   Flag||needinfo?, fedora-cvs?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631763] Review Request: zif - Simple wrapper for rpm

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Michal Schmidt  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #12 from Michal Schmidt  2010-10-01 14:41:22 
EDT ---
Everything is good now.

Current scratch build in dist-f15:
http://koji.fedoraproject.org/koji/taskinfo?taskID=2506735

APPROVED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 635654] Review Request: perl-Text-Hunspell - Perl interface to the Hunspell library

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #13 from Paul Howarth  2010-10-01 13:51:18 EDT 
---
Everything's built and pushed to stable now except the EL-6 version (see Bug
#636763), which probably won't be resolved until RHEL-6 GA.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631763] Review Request: zif - Simple wrapper for rpm

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #11 from Richard Hughes  2010-10-01 13:43:44 
EDT ---
(In reply to comment #10)
>   [??] Requires
> I wonder what is special about sqlite-devel that it is required
> explicitly by zif-devel, but libarchive-devel is left to be pulled
> automatically via rpm's pkgconfig dependency extraction.

You're correct. sqlite-devel gets pulled in automatically. I've removed that
line.

>   [!!] macros
>  %{_mandir} should be used instead of %{_datadir}/man/

Fixed.

> %{_datadir}/gtk-doc/html is owned neither by zif-devel nor any
> Required package. Should Require gtk-doc?

I wasn't sure. Looking into it, I shouldn't just add a Req for gtk-doc, as
there's been a big push to not pull it in unless it's really required. I've
just taken ownership of all the gtk-doc directory, like has been done to my
other packages by other people.

> [!!] owning of directories, already noted above

Fixed.

New spec and SRPMS for review:

http://people.freedesktop.org/~hughsient/temp/zif.spec
http://people.freedesktop.org/~hughsient/temp/zif-0.1.0-4.fc14.src.rpm

Thanks,

Richard.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631763] Review Request: zif - Simple wrapper for rpm

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #10 from Michal Schmidt  2010-10-01 13:20:15 
EDT ---
> $ rpmlint {SRPMS,RPMS/x86_64}/zif*-3.*rpm
> zif.src: I: enchant-dictionary-not-found en_US
> zif.src: W: non-standard-group Unspecified
> zif.src: W: no-cleaning-of-buildroot %clean
> zif.src: W: no-buildroot-tag
> zif.src: W: no-%clean-section
> zif.x86_64: W: non-standard-group Unspecified

All OK with current rpm.

> zif.x86_64: W: shared-lib-calls-exit /usr/lib64/libzif.so.1.0.1 
> e...@glibc_2.2.5

OK. exit() is called only from egg_error_real() which is never used in the
library.
BTW, a clever use of unsafe linker options might even elimitate the function.

> zif.x86_64: W: manual-page-warning /usr/share/man/man1/zif.1.gz 1: warning: 
> macro `\"' not defined
> zif-devel.x86_64: W: non-standard-group Unspecified
> 4 packages and 0 specfiles checked; 0 errors, 8 warnings.

Can be ignored.

Formal review
-
[OK] ... guideline matched
[--] ... irrelevant guideline
[??] ... needs discussion
[!!] ... needs fixing
-

[OK] rpmlint posted
[OK] Naming Guidelines
[OK] spec file name matches package name
[..] Packaging Guidelines:
  [OK] naming
  [OK] version and release
  [OK] legal
  [OK] no pre-built binaries
  [OK] spec legibility
  [OK] architecture support
  [OK] fs layout
  [OK] rpmlint
  [OK] changelog
  [OK] tags
  [OK] BuildRoot tag (not needed)
  [OK] %clean (not needed)
  [??] Requires
I wonder what is special about sqlite-devel that it is required
explicitly by zif-devel, but libarchive-devel is left to be pulled
automatically via rpm's pkgconfig dependency extraction.
  [OK] BuildRequires
  [OK] Summary and description
  [OK] encoding
  [OK] documentation
  [OK] compiler flags
  [OK] debuginfo
  [OK] devel package
  [OK] requiring base package
  [OK] shared libraries
  [--] static libraries
  [OK] no duplication of system libraries
  [OK] no rpaths
  [--] config files
  [--] initscripts
  [--] desktop files
  [!!] macros
 %{_mandir} should be used instead of %{_datadir}/man/
  [--] %global preferred over %define
  [--] handling of locale files
  [OK] timestamps
  [OK] parallel make
  [OK] scriptlets
  [--] conditional deps
  [OK] not relocatable
  [OK] code vs content
  [!!] file and dir ownership
%{_datadir}/gtk-doc/html is owned neither by zif-devel nor any
Required package. Should Require gtk-doc?
  [OK] no duplicate files
  [OK] file permissions
  [--] users and groups
  [--] web apps
  [OK] no conflicts
  [OK] no kernel modules
  [OK] nothing under /srv
  [OK] no bundling
  [--] patches should have upstream bug link or comment
  [--] use of epochs
  [OK] symlinks
  [OK] man pages
  [--] application-specific guidelines
[OK] approved license
[OK] GPLv2+ is correct
[OK] COPYING included in %doc
[OK] American English
[OK] legible spec
[OK] source matches upstream, sha256sum:
  aefac3cce4310e942bf735ea259efc45ba32fbe1c7cb6461e7f74637fd5df2d5
[OK] builds successfully
[--] ExcludeArch
[OK] BuildRequires
[--] locales
[OK] ldconfig called
[OK] no bundling
[OK] not relocatable
[!!] owning of directories, already noted above
[OK] no listing files more than once
[OK] permissions, %defattr usage
[OK] consistent macro usage
[OK] code or permissible content
[--] large docs
[OK] docs not necessary for runtime
[OK] headers in -devel
[--] static libs in -static
[OK] *.so in -devel
[OK] -devel requires base with a fully versioned dep
[OK] no *.la archives
[--] *.desktop for GUI apps
[OK] no owning of directories already owned by other packages
[OK] valid UTF-8 filenames

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 225995] Merge Review: libdaemon

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Parag AN(पराग)  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE
   Flag|fedora-review?  |fedora-review+
Last Closed||2010-10-01 13:12:41

--- Comment #3 from Parag AN(पराग)  2010-10-01 13:12:41 EDT 
---
Fixed in  libdaemon-0.14-1.fc15

APPROVED.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 633215] Review Request: SimplyHTML - Application and a java component for rich text processing

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Mat Booth  2010-10-01 13:09:44 EDT 
---
Okay, this is a full review. Issues I have with the package follow the
checklist.


Key:
- = N/A
x = Check
! = Problem
? = Not evaluated

=== REQUIRED ITEMS ===
[!]  Rpmlint output:
[x]  Package is named according to the Package Naming Guidelines[1].
[x]  Spec file name must match the base package name, in the format
%{name}.spec.
[x]  Package meets the Packaging Guidelines[2].
[x]  Package successfully compiles and builds into binary rpms.
[x]  Buildroot is correct
[x]  Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging
Guidelines[3,4].
[x]  License field in the package spec file matches the actual license.
License type:
[!]  If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
[-]  All independent sub-packages have license of their own
[x]  Spec file is legible and written in American English.
[x]  Sources used to build the package matches the upstream source, as provided
in the spec URL.
MD5SUM this package:b4b594dea5d357909f3619a0c9eccb59
MD5SUM upstream package:b4b594dea5d357909f3619a0c9eccb59
[x]  All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines[5].
[x]  Package must own all directories that it creates.
[!]  Package requires other packages for directories it uses.
[x]  Package does not contain duplicates in %files.
[!]  Permissions on files are set properly.
[!]  Package has a %clean section, which contains rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
[!]  Package consistently uses macros.
[x]  Package contains code, or permissable content.
[x]  Fully versioned dependency in subpackages, if present.
[-]  Package contains a properly installed %{name}.desktop file if it is a GUI
application.
[x]  Package does not own files or directories owned by other packages.
[x]  Javadoc documentation files are generated and included in -javadoc
subpackage
[!]  Packages have proper BuildRequires/Requires on jpackage-utils
[x]  Javadoc subpackages have Require: jpackage-utils
[-]  Package uses %global not %define
[-]  If package uses tarball from VCS include comment how to re-create that
tarball (svn export URL, git clone URL, ...)
[x]  If source tarball includes bundled jar/class files these need to be
removed prior to building
[x]  All filenames in rpm packages must be valid UTF-8.

=== Maven ===
[-]  Use %{_mavenpomdir} macro for placing pom files instead of
%{_datadir}/maven2/poms
[-]  If package uses "-Dmaven.test.skip=true" explain why it was needed in a
comment
[-]  If package uses custom depmap "-Dmaven2.jpp.depmap.file=*" explain why
it's needed in a comment
[-]  Package uses %update_maven_depmap in %post/%postun
[-]  Packages have Requires(post) and Requires(postun) on jpackage-utils (for
%update_maven_depmap macro)

=== Other suggestions ===
[x]  If possible use upstream build method (maven/ant/javac)
[x]  Javadocs are placed in %{_javadocdir}/%{name} with
%{_javadocdir}/%{name}-%{version} symlink
[x]  If package contains pom.xml files install it (including depmaps) even when
building with ant 
[x]  Avoid having BuildRequires on exact NVR unless necessary
[x]  Package has BuildArch: noarch (if possible)
[x]  Latest version is packaged.
[x]  Reviewer should test that the package builds in mock.



=== Issues ===
1. Please don't mix spaces and tabs for spec file indentation.
2. The licence should be included as part of the main package instead of the
javadoc package.
3. The main package should BuildRequires/Requires "jpackage-utils" which owns
the directories you install stuff to.
4. Please be consistent with macro usage and use the %{buildroot} macro at the
top of the %clean and %install sections.
5. Please do not include a class-path attribute in your jar manifests. That
should be patched out if necessary.
6. The script you install does not work because you install it with incorrect
permissions. To make it work, instead of these three lines:

mkdir -p  %{buildroot}%{_bindir}
cp %{SOURCE1} %{buildroot}%{_bindir}
ln -s %(basename %{SOURCE1}) %{buildroot}%{_bindir}/%(basename %{SOURCE1} .sh)

You only need this one line:

install -pD -m755 -T %{SOURCE1} %{buildroot}%{_bindir}/%(basename %{SOURCE1}
.sh)



=== Other Notes ===
1. In order to get around "Javadocs" as a spelling error in rpmlint, I normally
say "API documentation" instead, but you can treat that as a false positive if
you wish.





[1] https://fedoraproject.org/wiki/Packaging:NamingGuidelines
[2] https://fedoraproject.org/wiki/Packaging:Guidelines
[3] https://fedoraproject.org/wiki/Packaging:LicensingGuidelines
[4] https://f

[Bug 226012] Merge Review: libgnomecanvas

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Parag AN(पराग)  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
   Flag|fedora-review?  |fedora-review+
Last Closed||2010-10-01 13:10:54

--- Comment #2 from Parag AN(पराग)  2010-10-01 13:10:54 EDT 
---
Fixed in libgnomecanvas-2.30.1-2.fc15


APPROVED.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 634258] Review Request: spew - I/O performance measurement and load generation tool

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #10 from Fedora Update System  
2010-10-01 12:34:49 EDT ---
spew-1.0.8-3.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/spew-1.0.8-3.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 226655] Merge Review: xorg-x11-xsm

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Parag AN(पराग)  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE
   Flag|fedora-review?  |fedora-review+
Last Closed||2010-10-01 13:00:08

--- Comment #1 from Parag AN(पराग)  2010-10-01 13:00:08 EDT 
---
Fixed in xorg-x11-xsm-1.0.2-15.fc15

Remember Ajax don't want to remove Provides: so I have kept them.


APPROVED.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 226653] Merge Review: xorg-x11-xinit

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Parag AN(पराग)  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE
   Flag|fedora-review?  |fedora-review+
Last Closed||2010-10-01 12:57:48

--- Comment #1 from Parag AN(पराग)  2010-10-01 12:57:48 EDT 
---
Fixed in xorg-x11-xinit-1.0.9-19.fc15

Remember Ajax will be updating this package to latest upstream soon.


APPROVED.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 226656] Merge Review: xorg-x11-xtrans-devel

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Parag AN(पराग)  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE
   Flag|fedora-review?  |fedora-review+
Last Closed||2010-10-01 13:00:47

--- Comment #3 from Parag AN(पराग)  2010-10-01 13:00:47 EDT 
---
Fixed in xorg-x11-xtrans-devel-1.2.2-5.fc15

APPROVED.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 226652] Merge Review: xorg-x11-xfwp

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Parag AN(पराग)  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE
   Flag|fedora-review?  |fedora-review+
Last Closed||2010-10-01 12:58:32

--- Comment #1 from Parag AN(पराग)  2010-10-01 12:58:32 EDT 
---
Fixed in xorg-x11-xfwp-1.0.1-12.fc15

Remember Ajax don't want to remove Provides: so I have kept them.

APPROVED.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 226650] Merge Review: xorg-x11-xdm

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Parag AN(पराग)  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE
Last Closed||2010-10-01 12:38:00

--- Comment #5 from Parag AN(पराग)  2010-10-01 12:38:00 EDT 
---
Its simple cleanup so directly done in git. Fixed in xorg-x11-xdm-1.1.6-21.fc15

Remember Ajax don't want to remove Provides: so I have kept them and he will be
updating this package to latest upstream soon.


APPROVED.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 634258] Review Request: spew - I/O performance measurement and load generation tool

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 634258] Review Request: spew - I/O performance measurement and load generation tool

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #9 from Fedora Update System  2010-10-01 
12:34:43 EDT ---
spew-1.0.8-3.fc12 has been submitted as an update for Fedora 12.
https://admin.fedoraproject.org/updates/spew-1.0.8-3.fc12

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 634258] Review Request: spew - I/O performance measurement and load generation tool

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #11 from Fedora Update System  
2010-10-01 12:34:56 EDT ---
spew-1.0.8-3.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/spew-1.0.8-3.fc13

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 226645] Merge Review: xorg-x11-twm

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Parag AN(पराग)  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE
   Flag|fedora-review?  |fedora-review+
Last Closed||2010-10-01 12:23:50

--- Comment #1 from Parag AN(पराग)  2010-10-01 12:23:50 EDT 
---
Its simple cleanup so directly done in git. Fixed in xorg-x11-twm-1.0.3-7.fc15

Remember Ajax don't want to remove Provides: so I have kept them and he will be
updating this package to latest upstream soon.


APPROVED.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 632051] Review Request: libmeegotouch - MeeGo Touch Framework

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Chen Lei  2010-10-01 11:57:02 EDT ---
New Links:

http://dl.dropbox.com/u/1338197/1/libmeegotouch-0.20.25-2.fc13.src.rpm
http://dl.dropbox.com/u/1338197/1/libmeegotouch.spec

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639380] Review Request: swftools - Utilities for working with Adobe Flash files (SWF files)

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Orion Poplawski  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG
Last Closed||2010-10-01 11:47:05

--- Comment #1 from Orion Poplawski  2010-10-01 11:47:05 
EDT ---
Looks like rpmfusion is reviewing this because of the linking against lame and
avifile.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639380] Review Request: swftools - Utilities for working with Adobe Flash files (SWF files)

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Orion Poplawski  2010-10-01 11:47:22 
EDT ---
https://bugzilla.rpmfusion.org/show_bug.cgi?id=1252

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639380] New: Review Request: swftools - Utilities for working with Adobe Flash files (SWF files)

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: swftools - Utilities for working with Adobe Flash 
files (SWF files)

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

   Summary: Review Request: swftools - Utilities for working with
Adobe Flash files (SWF files)
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: or...@cora.nwra.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://www.cora.nwra.com/~orion/fedora/swftools.spec
SRPM URL: http://www.cora.nwra.com/~orion/fedora/swftools-0.9.1-1.fc13.src.rpm
Description:
SWFTools is a collection of utilities for working with Adobe Flash files (SWF
files). The tool collection includes programs for reading SWF files,
combining them, and creating them from other content (like images, sound
files, videos or source code). SWFTools is released under the GPL.
The current collection is comprised of the programs detailed below:

 * pdf2swf - A PDF to SWF Converter. Generates one frame per page. Enables
   you to have fully formatted text, including tables, formulas, graphics
   etc.  inside your Flash Movie. It's based on the xpdf PDF parser from
   Derek B.  Noonburg.
 * swfcombine - A multi-function tool for inserting SWFs into Wrapper SWFs,
   concatenating SWFs, stacking SWFs or for basic parameter manipulation
   (e.g. changing size).
 * swfstrings - Scans SWFs for text data.
 * swfdump - Prints out various information about SWFs, like contained images/
   fonts/sounds, disassembly of contained code as well as cross-reference and
   bounding box data.
 * jpeg2swf - Takes one or more JPEG pictures and generates a SWF slide show
   from them. Supports motion estimation compression (h.263) for better
   compression of video sequences.
 * png2swf - Like JPEG2SWF, only for PNGs.
 * gif2swf - Converts GIFs to SWF. Also able to handle animated GIFs.
 * wav2swf - Converts WAV audio files to SWFs, using the L.A.M.E. MP3 encoder
   library.
 * font2swf - Converts font files (TTF, Type1) to SWF.
 * swfbbox - Allows to read out, optimize and readjust SWF bounding boxes.
 * swfc - A tool for creating SWF files from simple script files. Includes
   support for both ActionScript 2.0 as well as ActionScript 3.0.
 * swfextract - Allows to extract Movieclips, Sounds, Images etc. from SWF
   files.
 * as3compile - A standalone ActionScript 3.0 compiler. Mostly compatible with
   Flex.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631763] Review Request: zif - Simple wrapper for rpm

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #9 from Richard Hughes  2010-10-01 11:26:16 EDT 
---
(In reply to comment #8)
> Right, rpm considers the Group tag optional since version 4.6. Though I could
> swear Fedora Packaging Guidelines still asked for the tag to be specified, I
> cannot find the requirement anywhere now. Perhaps it's been removed already.

I think so.

> But please be consistent. The -devel subpackage does have a Group tag.

Agreed, apologies.

> I suggest you extend the %description for the main package a bit. Right now it
> only repeats what the Summary says and it is not very informative.

Gotcha. I've added something better.

New spec and SRPMS for review:

http://people.freedesktop.org/~hughsient/temp/zif.spec
http://people.freedesktop.org/~hughsient/temp/zif-0.1.0-3.fc14.src.rpm

Richard.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639369] Review Request: gfs2-utils - Fileysystem utilities for the GFS2 Filesystem

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Steve Whitehouse  2010-10-01 11:13:08 
EDT ---
Created attachment 451033
  --> https://bugzilla.redhat.com/attachment.cgi?id=451033
Draft spec file

Hopefully this isn't too far from the final spec file.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639369] New: Review Request: gfs2-utils - Fileysystem utilities for the GFS2 Filesystem

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: gfs2-utils - Fileysystem utilities for the GFS2 
Filesystem

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

   Summary: Review Request: gfs2-utils - Fileysystem utilities for
the GFS2 Filesystem
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: swhit...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: Attached
SRPM URL: TBD
Description:

The gfs2-utils package contains a number of utilities for creating,
mounting, checking, modifying, and correcting any inconsistencies in GFS2
filesystems.


Important Note: This is complicated so I'm going to be adding to this bug
over the next few days. Feel free to comment in the mean time, but not
everything is ready for a full review right away. The plan is this:

1. To bring back gfs2-utils as a package in its own right
2. To add a new subpackage to gfs2-utils, gfs2-cluster which will contain the
cluster parts of the gfs2 userland infrastructure
3. To drop those items from the pre-existing cluster package (which means
dropping the gfs2-utils binary package in its entirety, and the parts of the
cman binary package which will move into gfs2-cluster)
4. The new gfs2-utils package will then be built in all versions of Fedora and
work alongside the newly adjusted cluster packages

The main reason for doing this is that the releases of the GFS2 packages will
then be independent from those of the rest of the cluster infrastructure. That
means that the GFS2 team can deal with those releases directly, taking some of
the load from the cluster team.

The only missing item at the moment, is an init script for gfs2-cluster which
I'll add to the upstream source shortly. In the mean time, the gfs2-utils spec
file will, I hope, not change very much so that is available for review now.

The upstream source can be found here:
http://git.fedorahosted.org/git/?p=gfs2-utils.git

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631763] Review Request: zif - Simple wrapper for rpm

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #8 from Michal Schmidt  2010-10-01 11:00:41 
EDT ---
(In reply to comment #4)
> (In reply to comment #1)
> > I guess you should specify package group. System Environment/Libraries can 
> > be
> > correct.
> 
> Nahh, this is only for F14+, so the group isn't required at all.

Right, rpm considers the Group tag optional since version 4.6. Though I could
swear Fedora Packaging Guidelines still asked for the tag to be specified, I
cannot find the requirement anywhere now. Perhaps it's been removed already.

But please be consistent. The -devel subpackage does have a Group tag.


I suggest you extend the %description for the main package a bit. Right now it
only repeats what the Summary says and it is not very informative.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 634909] Review Request: v8 - JavaScript Engine

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Peter Lemenkov  changed:

   What|Removed |Added

 CC||lemen...@gmail.com

--- Comment #5 from Peter Lemenkov  2010-10-01 10:31:00 EDT 
---
Btw they just tagged 2.4.7

http://code.google.com/p/v8/source/detail?r=5564

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639348] New: Review Request: gkeyfile-sharp - C# bindings for glib2's keyfile implementation

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: gkeyfile-sharp - C# bindings for glib2's keyfile 
implementation

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

   Summary: Review Request: gkeyfile-sharp - C# bindings for
glib2's keyfile implementation
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: nathan...@natemccallum.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://npmccallum.fedorapeople.org/banshee/gkeyfile-sharp.spec
SRPM URL:
http://repos.fedorapeople.org/repos/npmccallum/banshee/fedora-14/SRPMS/gkeyfile-sharp-0.1-1.fc14.src.rpm
Description: C# bindings for glib2's keyfile implementation

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639351] New: Review Request: gtk-sharp-beans - C# bindings for GTK+ API not included in GTK#

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: gtk-sharp-beans - C# bindings for GTK+ API not 
included in GTK#

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

   Summary: Review Request: gtk-sharp-beans - C# bindings for GTK+
API not included in GTK#
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: nathan...@natemccallum.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://npmccallum.fedorapeople.org/banshee/gtk-sharp-beans.spec
SRPM URL:
http://repos.fedorapeople.org/repos/npmccallum/banshee/fedora-14/SRPMS/gtk-sharp-beans-2.14.0-1.fc14.src.rpm
Description: C# bindings for GTK+ API not included in GTK#

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639350] New: Review Request: gio-sharp - C# bindings for gio

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: gio-sharp - C# bindings for gio

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

   Summary: Review Request: gio-sharp - C# bindings for gio
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: nathan...@natemccallum.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://npmccallum.fedorapeople.org/banshee/gio-sharp.spec
SRPM URL:
http://repos.fedorapeople.org/repos/npmccallum/banshee/fedora-14/SRPMS/gio-sharp-0.2-1.fc14.src.rpm
Description: C# bindings for gio

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639351] Review Request: gtk-sharp-beans - C# bindings for GTK+ API not included in GTK#

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Nathaniel McCallum  changed:

   What|Removed |Added

 Depends on||639350

--- Comment #1 from Nathaniel McCallum  2010-10-01 
10:13:51 EDT ---
Building this package also requires gio-sharp (#639350).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639350] Review Request: gio-sharp - C# bindings for gio

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Nathaniel McCallum  changed:

   What|Removed |Added

 Blocks||639351

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 612581] Review Request: spacewalk-backend - Common programs needed to be installed on the Spacewalk servers/proxies

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Michal Schmidt  changed:

   What|Removed |Added

 CC||mschm...@redhat.com

--- Comment #11 from Michal Schmidt  2010-10-01 10:09:31 
EDT ---
(In reply to comment #8)
> >spacewalk-backend.noarch: W: manual-page-warning
> >/usr/share/man/man8/spacewalk-cfg-get.8.gz 1: warning: `\"' not defined
> >Not sure how to fix that, but it would be good to actually do it.
> Hmmm, I really do not understood it either. I would like to fix it, but I do
> not how. :(

If your mangpage is generated from docbook, see bug 639347.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631763] Review Request: zif - Simple wrapper for rpm

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #7 from Michal Schmidt  2010-10-01 10:10:23 
EDT ---
(In reply to comment #6)
> But that's a bug in docbook2man then.
... filed as bug 639347.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639346] New: Review Request: gudev-sharp - C# bindings for gudev

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: gudev-sharp - C# bindings for gudev

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

   Summary: Review Request: gudev-sharp - C# bindings for gudev
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: nathan...@natemccallum.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://npmccallum.fedorapeople.org/banshee/gudev-sharp.spec
SRPM URL:
http://repos.fedorapeople.org/repos/npmccallum/banshee/fedora-14/SRPMS/gudev-sharp-0.1-1.fc14.src.rpm
Description: C# bindings for gudev

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631763] Review Request: zif - Simple wrapper for rpm

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #6 from Michal Schmidt  2010-10-01 09:52:33 
EDT ---
(In reply to comment #4)
> > >zif.x86_64: W: manual-page-warning /usr/share/man/man1/zif.1.gz 1: 
> > >warning: `\"' >not defined
> > I don't know...
> 
> Not sure, I think it's a rpmlint buglet. All my packages seem to have this
> warning.

rpmlint merely cites a warning detected by running:
cat man/zif.1 |gtbl |groff -mtty-char -Tutf8 -P-c -mandoc -mandoc -wmac
>/dev/null

It does not like the first line:
.\\" auto-generated by docbook2man-spec $Revision$

I believe the correct comment format would be:
.\" auto-generated ...

But that's a bug in docbook2man then.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 532590] Review Request: yaws - Web server for dynamic content written in Erlang

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #13 from Peter Lemenkov  2010-10-01 09:48:35 
EDT ---
Hello All!
Upstream released ver. 1.89 which is now cleanly compiles in Koji - I strongly
encourage you to update src.rpm. There are some notable (for packaging) changes
since 1.88:

+ No longer required patch for compiling using system-wide erlsom.
+ Ver. 1.89 now requires erlang-ibrowse library

I also did some work on shortening list of BuildRequires - you need only the
following Erlang libraries for building:

BuildRequires:  erlang-erts
BuildRequires:  erlang-erlsom
BuildRequires:  erlang-kernel
BuildRequires:  erlang-mnesia
BuildRequires:  erlang-sasl
BuildRequires:  erlang-ssl
BuildRequires:  erlang-stdlib

Unfortunately it doesn't significantly save compilation time.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631763] Review Request: zif - Simple wrapper for rpm

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #5 from Richard Hughes  2010-10-01 09:39:54 EDT 
---
(In reply to comment #2)
> > %define alphatag.20100908git
> This macro is not used anywhere. Also, %global is preferred to %define these
> days.

Eeek, sorry, that got left in from my initial package that was based on the git
checkout. Removed now.

> > %package devel
> [...]
> > Requires: sqlite-devel
> 
> %{?_isa} would be nice here.
> http://www.rpm.org/wiki/PackagerDocs/ArchDependencies

Fixed.

> > %package devel
> [...]
> > Requires: %{name} = %{version}-%{release}
> [...]
> > %files
> > %defattr(-,root,root,-)
> > %doc README AUTHORS NEWS COPYING
> > [...]
> > %files devel
> > %defattr(-,root,root,-)
> > %doc README AUTHORS NEWS COPYING
> 
> No need to duplicate these files here, because -devel Requires the main
> package.

Fixed, thanks.

(In reply to comment #3)
> The files in %{_datadir}/gtk-doc/html/zif/ look like developer's documentation
> to me. Wouldn't they be better placed in the -devel package?

Yup, oops, thanks. New spec and SRPMS for review:

http://people.freedesktop.org/~hughsient/temp/zif.spec
http://people.freedesktop.org/~hughsient/temp/zif-0.1.0-2.fc14.src.rpm

Richard.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639290] Review Request: perl-threads - Perl interpreter-based threads

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Petr Pisar  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #2 from Petr Pisar  2010-10-01 09:36:50 EDT ---
New Package SCM Request
===
Package Name: perl-threads
Short Description: Perl interpreter-based threads
Owners: ppisar, mmaslano, psabata
Branches: f12, f13, f14
InitialCC: perl-sig

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631763] Review Request: zif - Simple wrapper for rpm

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #4 from Richard Hughes  2010-10-01 09:28:42 EDT 
---
(In reply to comment #1)
> My bits:
> >$ rpmlint ../SRPMS/zif-0.1.0-1.fc13.src.rpm 
> >zif.src: W: non-standard-group Unspecified
> I guess you should specify package group. System Environment/Libraries can be
> correct.

Nahh, this is only for F14+, so the group isn't required at all.

> >$ rpmlint ../RPMS/x86_64/zif-*
> >zif.x86_64: W: non-standard-group Unspecified
> >zif.x86_64: W: shared-lib-calls-exit /usr/lib64/libzif.so.1.0.1 
> >e...@glibc_2.2.5
> This is not good. Should be fixed upstream (but I don't examine it deeply)

Although there is a call to exit(), it's in egg_error() which is a shared egg
file between a number of projects. egg_error() is never called in zif, and
certainly not exported, and so it's impossible to call exit.

> >zif.x86_64: W: manual-page-warning /usr/share/man/man1/zif.1.gz 1: warning: 
> >`\"' >not defined
> I don't know...

Not sure, I think it's a rpmlint buglet. All my packages seem to have this
warning.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 632051] Review Request: libmeegotouch - MeeGo Touch Framework

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Chen Lei  2010-10-01 09:24:01 EDT ---
(In reply to comment #1)
> Patches: do we really need all patches? at least missing-Epoch-function is not
> needed anymore as we have Qt 4.7, please update

I removed patch5 from spec. All other patches are remained, they are all
extracted from upstream SRPM, I think it'll be easier to track upstream RPM and
bugzilla system closely. 

Currently, I track F14 with Meego 1.1 which will be released on 2010-10-27.

> Build: not ok
> 
> + ./configure -prefix /usr -libdir /usr/lib64 -release
> 
> QTDIR variable was set but could not find /usr/lib64/qt-3.3/bin/qmake\n
> If your Qt is in a nonstandard location, try:
> QTDIR= ./configure
> error: Bad exit status from /var/tmp/rpm-tmp.bm6HwK (%build)
> 
It's weird, I can build the package in rawhide koji. Maybe this issue is
trigger once the system has both qt3-devel and qt-devel installed.

> 
> PS; could you add me as co-maintainer to MeeGo packages once reviewed? I'm Qt
> co-maintainer and Fedora Mobility guy from Qt/Plasma Netbook side of evil.
> Thanks.

OK, please also apply co-maintainer for exsited meego packages in pkgdb(e.g.
libaccounts-glib libaccounts-qt contextkit libqttracker).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639290] Review Request: perl-threads - Perl interpreter-based threads

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Marcela Mašláňová  changed:

   What|Removed |Added

   Flag||fedora-review+

--- Comment #1 from Marcela Mašláňová  2010-10-01 09:04:42 
EDT ---
No problems with rpmlint, source or license.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 631763] Review Request: zif - Simple wrapper for rpm

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #3 from Michal Schmidt  2010-10-01 09:02:41 
EDT ---
The files in %{_datadir}/gtk-doc/html/zif/ look like developer's documentation
to me. Wouldn't they be better placed in the -devel package?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639323] New: Review Request: telepathy-qt4 - Qt4 binding for Telepathy

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: telepathy-qt4 - Qt4 binding for Telepathy

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

   Summary: Review Request: telepathy-qt4 - Qt4 binding for
Telepathy
   Product: Fedora
   Version: 14
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: supercyp...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Description:

Telepathy-qt4 is a Qt4 high-level binding for Telepathy.

Links:
http://dl.dropbox.com/u/1338197/1/telepathy-qt4-0.3.10-1.fc13.src.rpm
http://dl.dropbox.com/u/1338197/1/telepathy-qt4.spec

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631763] Review Request: zif - Simple wrapper for rpm

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #2 from Michal Schmidt  2010-10-01 08:53:02 
EDT ---
> %define alphatag.20100908git
This macro is not used anywhere. Also, %global is preferred to %define these
days.

> %package devel
[...]
> Requires: sqlite-devel

%{?_isa} would be nice here.
http://www.rpm.org/wiki/PackagerDocs/ArchDependencies

> %package devel
[...]
> Requires: %{name} = %{version}-%{release}
[...]
> %files
> %defattr(-,root,root,-)
> %doc README AUTHORS NEWS COPYING
> [...]
> %files devel
> %defattr(-,root,root,-)
> %doc README AUTHORS NEWS COPYING

No need to duplicate these files here, because -devel Requires the main
package.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631763] Review Request: zif - Simple wrapper for rpm

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Roman Rakus  changed:

   What|Removed |Added

 CC||rra...@redhat.com

--- Comment #1 from Roman Rakus  2010-10-01 08:29:20 EDT ---
My bits:
>$ rpmlint ../SRPMS/zif-0.1.0-1.fc13.src.rpm 
>zif.src: W: non-standard-group Unspecified
I guess you should specify package group. System Environment/Libraries can be
correct.

>zif.src:53: W: macro-in-comment %{_libdir}
>zif.src: W: no-cleaning-of-buildroot %clean
>zif.src: W: no-buildroot-tag
>zif.src: W: no-%clean-section
>1 packages and 0 specfiles checked; 0 errors, 5 warnings.
it is ok

>$ rpmlint ../RPMS/x86_64/zif-*
>zif.x86_64: W: non-standard-group Unspecified
>zif.x86_64: W: shared-lib-calls-exit /usr/lib64/libzif.so.1.0.1 
>e...@glibc_2.2.5
This is not good. Should be fixed upstream (but I don't examine it deeply)

>zif.x86_64: W: manual-page-warning /usr/share/man/man1/zif.1.gz 1: warning: 
>`\"' >not defined
I don't know...
>zif-devel.x86_64: W: spelling-error Summary(en_US) GLib -> G Lib, Glib, Lib
>zif-devel.x86_64: W: spelling-error %description -l en_US GLib -> G Lib, Glib, 
>>Lib
>3 packages and 0 specfiles checked; 0 errors, 5 warnings.
ok

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 478613] Review Request: ledger - A powerful command-line double-entry accounting system

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #29 from Fedora Update System  
2010-10-01 08:24:28 EDT ---
ledger-2.6.3-2.fc14 has been pushed to the Fedora 14 stable repository.  If
problems still persist, please make note of it in this bug report.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631763] Review Request: zif - Simple wrapper for rpm

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Michal Schmidt  changed:

   What|Removed |Added

 CC||mschm...@redhat.com
  QAContact|extras...@fedoraproject.org |mschm...@redhat.com
   Flag||fedora-review?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631763] Review Request: zif - Simple wrapper for rpm

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Michal Schmidt  changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|mschm...@redhat.com
  QAContact|mschm...@redhat.com |extras...@fedoraproject.org

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 635654] Review Request: perl-Text-Hunspell - Perl interface to the Hunspell library

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #12 from Fedora Update System  
2010-10-01 08:22:09 EDT ---
perl-Text-Hunspell-2.01-2.fc14 has been pushed to the Fedora 14 stable
repository.  If problems still persist, please make note of it in this bug
report.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 620046] Review Request: ghc-terminfo - Haskell terminfo binding

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #12 from Fedora Update System  
2010-10-01 08:23:41 EDT ---
ghc-terminfo-0.3.1.3-1.fc14 has been pushed to the Fedora 14 stable repository.
 If problems still persist, please make note of it in this bug report.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 591737] Review Request: R-ROC - Utilities for ROC

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||R-ROC-1.24.1-1.fc14
 Resolution||ERRATA
Last Closed||2010-10-01 08:21:08

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 620046] Review Request: ghc-terminfo - Haskell terminfo binding

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||ghc-terminfo-0.3.1.3-1.fc14
 Resolution||ERRATA
Last Closed||2010-10-01 08:23:46

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631190] Review Request: libnxt - An utility for flashing LEGO Mindstorms NXT firmware

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #10 from Fedora Update System  
2010-10-01 08:20:50 EDT ---
libnxt-0.3-2.fc14 has been pushed to the Fedora 14 stable repository.  If
problems still persist, please make note of it in this bug report.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 591737] Review Request: R-ROC - Utilities for ROC

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #14 from Fedora Update System  
2010-10-01 08:21:03 EDT ---
R-ROC-1.24.1-1.fc14 has been pushed to the Fedora 14 stable repository.  If
problems still persist, please make note of it in this bug report.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631763] Review Request: zif - Simple wrapper for rpm

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Michal Schmidt  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 631190] Review Request: libnxt - An utility for flashing LEGO Mindstorms NXT firmware

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||libnxt-0.3-2.fc14
 Resolution||ERRATA
Last Closed||2010-10-01 08:20:55

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 620898] Review Request: nxtrc - A program to interact with LEGO NXT via BlueTooth

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||nxtrc-2.3-3.fc14
 Resolution||ERRATA
Last Closed||2010-10-01 08:21:20

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 620898] Review Request: nxtrc - A program to interact with LEGO NXT via BlueTooth

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #12 from Fedora Update System  
2010-10-01 08:21:15 EDT ---
nxtrc-2.3-3.fc14 has been pushed to the Fedora 14 stable repository.  If
problems still persist, please make note of it in this bug report.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639292] New: Review Request: erlang-etorrent - Erlang BitTorrent client and library

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: erlang-etorrent - Erlang BitTorrent client and library

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

   Summary: Review Request: erlang-etorrent - Erlang BitTorrent
client and library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: lemen...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://peter.fedorapeople.org/erlang-etorrent.spec
SRPM URL: http://peter.fedorapeople.org/erlang-etorrent-1.1.1-1.fc12.src.rpm
Description: This is a BitTorrent client and a library written in Erlang. The
focus is on
robustness and scalability in number of torrents rather than in pure speed.
Etorrent is mostly meant for unattended operation, where one just specifies
what files to download and gets a notification when they are.

I'll provide Koji scratchuild and rpmlint as soon as all necessary
buildrequires will be available in Koji

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639292] Review Request: erlang-etorrent - Erlang BitTorrent client and library

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Peter Lemenkov  changed:

   What|Removed |Added

 Depends on||639263, 638909

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 638909] Review Request: erlang-gproc - Extended process registry for Erlang

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Peter Lemenkov  changed:

   What|Removed |Added

 Blocks||639292

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639263] Review Request: erlang-rebar - Erlang Build Tools

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Peter Lemenkov  changed:

   What|Removed |Added

 Blocks||639292

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639291] New: Review Request: erlang-mustache - Mustache template engine for Erlang

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: erlang-mustache - Mustache template engine for Erlang

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

   Summary: Review Request: erlang-mustache - Mustache template
engine for Erlang
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: lemen...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://peter.fedorapeople.org/erlang-mustache.spec
SRPM URL: http://peter.fedorapeople.org/erlang-mustache-0.1.0-1.fc12.src.rpm
Description: An Erlang port of Mustache for Ruby. Mustache is a
framework-agnostic templating
system that enforces separation of view logic from the template file. Indeed,
it
is not even possible to embed logic in the template. This allows templates to
be
reused across language boundaries and for other language independent uses.

This is one of the requirements for erlang-rebar.

rpmlint messages:

Sulaco ~/rpmbuild/SPECS: rpmlint
../RPMS/ppc/erlang-mustache-0.1.0-1.fc12.ppc.rpm 
kojerlang-mustache.ppc: W: spelling-error %description -l en_US templating ->
contemplating, template's, template
ierlang-mustache.ppc: E: no-binary
erlang-mustache.ppc: W: only-non-binary-in-usr-lib
1 packages and 0 specfiles checked; 1 errors, 2 warnings.
Sulaco ~/rpmbuild/SPECS:

All these messages should be ignored - first one is just a failse positive,
while the rest is due to fact that we're installing arch-independent data
(Erlang's VM images) into arch-dependent directory.

Koji scratch build for F-14:
http://koji.fedoraproject.org/koji/taskinfo?taskID=2505964

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639263] Review Request: erlang-rebar - Erlang Build Tools

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Peter Lemenkov  changed:

   What|Removed |Added

  Status Whiteboard|NotReady|

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639263] Review Request: erlang-rebar - Erlang Build Tools

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Peter Lemenkov  changed:

   What|Removed |Added

 Depends on||639291

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639291] Review Request: erlang-mustache - Mustache template engine for Erlang

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Peter Lemenkov  changed:

   What|Removed |Added

 Blocks||639263

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639290] New: Review Request: perl-threads - Perl interpreter-based threads

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: perl-threads - Perl interpreter-based threads

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

   Summary: Review Request: perl-threads - Perl interpreter-based
threads
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: ppi...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://ppisar.fedorapeople.org/perl-threads/perl-threads.spec
SRPM URL:
http://ppisar.fedorapeople.org/perl-threads/perl-threads-1.79-1.fc13.src.rpm
Description:
Since Perl 5.8, thread programming has been available using a model called
interpreter threads which provides a new Perl interpreter for each thread,
and, by default, results in no data or state information being shared
between threads.

(Prior to Perl 5.8, 5005threads was available through the "Thread.pm" API.
This threading model has been deprecated, and was removed as of Perl 5.10.0.)


Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2505960

$ rpmlint perl-threads.spec ../SRPMS/perl-threads-1.79-1.fc13.src.rpm
../RPMS/x86_64/perl-threads-1.79-1.fc13.x86_64.rpm 
perl-threads.spec: W: no-cleaning-of-buildroot %clean
perl-threads.spec: W: no-buildroot-tag
perl-threads.spec: W: no-%clean-section
perl-threads.src: W: no-cleaning-of-buildroot %clean
perl-threads.src: W: no-buildroot-tag
perl-threads.src: W: no-%clean-section
2 packages and 1 specfiles checked; 0 errors, 6 warnings.

This is post-F11 package without BuildRoot.

$ LANG=en rpm -pq -lv ../RPMS/x86_64/perl-threads-1.79-1.fc13.x86_64.rpm 
drwxr-xr-x2 rootroot0 Oct  1 13:37
/usr/lib64/perl5/auto/threads
-rwxr-xr-x1 rootroot44712 Oct  1 13:37
/usr/lib64/perl5/auto/threads/threads.so
-rw-r--r--1 rootroot34251 Sep 24 20:40
/usr/lib64/perl5/threads.pm
drwxr-xr-x2 rootroot0 Oct  1 13:38
/usr/share/doc/perl-threads-1.79
-rw-r--r--1 rootroot 7986 Sep 24 20:40
/usr/share/doc/perl-threads-1.79/Changes
-rw-r--r--1 rootroot  482 Sep 24 20:40
/usr/share/doc/perl-threads-1.79/README
-rw-r--r--1 rootroot12807 Oct  1 13:37
/usr/share/man/man3/threads.3pm.gz

This is standalone package to dual-live with perl's subpackage preparing right
now.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639290] Review Request: perl-threads - Perl interpreter-based threads

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Marcela Mašláňová  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|mmasl...@redhat.com

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 639286] Review Request: rubygem-amazon-ec2 - A Ruby library for accessing the Amazon Web Services

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Michal Fojtik  2010-10-01 07:39:06 EDT 
---
Koji build:

http://koji.fedoraproject.org/koji/taskinfo?taskID=2505945

rpmlint:

$ rpmlint rubygem-amazon-ec2-0.9.15-1.fc13.noarch.rpm 
rubygem-amazon-ec2.noarch: W: no-manual-page-for-binary ec2sh

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639286] New: Review Request: rubygem-amazon-ec2 - A Ruby library for accessing the Amazon Web Services

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: rubygem-amazon-ec2 - A Ruby library for accessing the 
Amazon Web Services

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

   Summary: Review Request: rubygem-amazon-ec2 - A Ruby library
for accessing the Amazon Web Services
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mfoj...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://mifo.sk/RPMS/rubygem-amazon-ec2.spec
SRPM URL: http://mifo.sk/RPMS/rubygem-amazon-ec2-0.9.15-1.fc13.src.rpm
Description:

A Ruby library for accessing the Amazon Web Services EC2, ELB, RDS,
Cloudwatch, and Autoscaling APIs.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639284] New: Review Request: erlang-neotoma - Erlang library and packrat parser-generator for parsing expression grammars

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: erlang-neotoma - Erlang library and packrat 
parser-generator for parsing expression grammars

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

   Summary: Review Request: erlang-neotoma - Erlang library and
packrat parser-generator for parsing expression
grammars
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: lemen...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://peter.fedorapeople.org/erlang-neotoma.spec
SRPM URL: http://peter.fedorapeople.org/erlang-neotoma-1.4-1.fc12.src.rpm
Description: Erlang library and packrat parser-generator for parsing expression
grammars.

This is one of the requirements for erlang-rebar.

rpmlint output:
Sulaco ~/rpmbuild/SRPMS: rpmlint ../RPMS/ppc/erlang-neotoma-1.4-1.fc12.ppc.rpm 
koji erlang-neotoma.ppc: W: spelling-error Summary(en_US) packrat -> pack rat,
pack-rat, Packard
erlang-neotoma.ppc: W: spelling-error %description -l en_US packrat -> pack
rat, pack-rat, Packard
scratch erlang-neotoma.ppc: E: no-binary
erlang-neotoma.ppc: W: only-non-binary-in-usr-lib
1 packages and 0 specfiles checked; 1 errors, 3 warnings.
Sulaco ~/rpmbuild/SRPMS:

All these messages should be ignored - those, regarding grammar, are false
positive, and the rest (no-binary and only-non-binary-in-usr-lib) are due to
fact that arch-independent library is being installed into arch-dependent
directory.

Koji scratch build for F-14:
http://koji.fedoraproject.org/koji/taskinfo?taskID=2505939

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639284] Review Request: erlang-neotoma - Erlang library and packrat parser-generator for parsing expression grammars

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Peter Lemenkov  changed:

   What|Removed |Added

 Blocks||639263

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639263] Review Request: erlang-rebar - Erlang Build Tools

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Peter Lemenkov  changed:

   What|Removed |Added

 Depends on||639284

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639263] Review Request: erlang-rebar - Erlang Build Tools

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Peter Lemenkov  changed:

   What|Removed |Added

 Depends on||639278

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639278] Review Request: erlang-lfe - Lisp Flavoured Erlang

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Peter Lemenkov  changed:

   What|Removed |Added

 Blocks||639263

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639278] New: Review Request: erlang-lfe - Lisp Flavoured Erlang

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: erlang-lfe - Lisp Flavoured Erlang

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

   Summary: Review Request: erlang-lfe - Lisp Flavoured Erlang
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: lemen...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://peter.fedorapeople.org/erlang-lfe.spec
SRPM URL: http://peter.fedorapeople.org/erlang-lfe-0.6.1-1.fc12.src.rpm
Description: Lisp Flavoured Erlang, is a lisp syntax front-end to the Erlang
compiler. Code produced with it is compatible with "normal" Erlang
code. An LFE evaluator and shell is also included.

One of the requirements ("soft-dependency") for erlang-rebar

koji scratch build for F-14:
http://koji.fedoraproject.org/koji/taskinfo?taskID=2505856

rpmlint:
Sulaco ~/rpmbuild/SPECS: rpmlint ../RPMS/ppc/erlang-lfe-0.6.1-1.fc12.ppc.rpm 
erlang-lfe.ppc: E: explicit-lib-dependency erlang-stdlib
erlang-lfe.ppc: E: no-binary
erlang-lfe.ppc: W: only-non-binary-in-usr-lib
1 packages and 0 specfiles checked; 2 errors, 1 warnings.
Sulaco ~/rpmbuild/SPECS

All these messages may be omitted (first one is a false positive due to "lib"
substring in the name of one of the runtime dependency, the rest are due to
fact that arch-independent data is installed into arch-dependent directory)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 635981] Review Request: mojo-signatures - Mojo API signatures project

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Stanislav Ochotnicky  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2010-10-01 06:39:00

--- Comment #5 from Stanislav Ochotnicky  2010-10-01 
06:39:00 EDT ---
Built on koji:
http://koji.fedoraproject.org/koji/taskinfo?taskID=2505767

Thanks

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639272] Review Request: rubygem-rack-accept - HTTP Accept for Ruby/Rack

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

--- Comment #1 from Michal Fojtik  2010-10-01 06:36:43 EDT 
---
I fixed some BuildRequire dependencies, now it builds fine in Koji:

http://koji.fedoraproject.org/koji/taskinfo?taskID=2505839

Spec URL: http://mifo.sk/RPMS/rubygem-rack-accept.spec
SRPM URL:
http://koji.fedoraproject.org/koji/getfile?taskID=2505839&name=rubygem-rack-accept-0.4.3-3.fc13.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639272] New: Review Request: rubygem-rack-accept - HTTP Accept for Ruby/Rack

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: rubygem-rack-accept - HTTP Accept for Ruby/Rack

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

   Summary: Review Request: rubygem-rack-accept - HTTP Accept for
Ruby/Rack
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mfoj...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://mifo.sk/RPMS/rubygem-rack-accept.spec
SRPM URL: http://mifo.sk/RPMS/rubygem-rack-accept-0.4.3-1.fc13.src.rpm
Description:

Rack::Accept is a suite of tools for Ruby/Rack applications that eases the
complexity of building and interpreting the Accept* family of HTTP request
headers.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 638126] Review Request: motoya-lcedar-fonts - Japanese Gothic-typeface TrueType fonts by MOTOYA Co,LTD

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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

Akira TAGOH  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #2 from Akira TAGOH  2010-10-01 05:46:35 EDT ---
Once the review filed, it's categorized into
https://fedoraproject.org/wiki/Category:In-progress_fonts.

Anyway, thank you for reviewing.

New Package SCM Request
===
Package Name: motoya-lcedar-fonts
Short Description: Japanese Gothic-typeface TrueType fonts by MOTOYA Co,LTD
Owners: tagoh
Branches: f12 f13 f14
InitialCC: fonts-sig i18n-team

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 639263] New: Review Request: erlang-rebar - Erlang Build Tools

2010-10-01 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: erlang-rebar - Erlang Build Tools

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

   Summary: Review Request: erlang-rebar - Erlang Build Tools
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: lemen...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://peter.fedorapeople.org/erlang-rebar.spec
SRPM URL: http://peter.fedorapeople.org/erlang-rebar-2-1.fc12.src.rpm
Description: Erlang Build Tools.

This is one of the requirements for etorrent.

rpmlint output:


Sulaco ~/rpmbuild/SRPMS: rpmlint ../RPMS/ppc/erlang-rebar-2-1.fc12.ppc.rpm 
erlang-rebar.ppc: E: explicit-lib-dependency erlang-stdlib
erlang-rebar.ppc: E: no-binary
erlang-rebar.ppc: W: only-non-binary-in-usr-lib
erlang-rebar.ppc: W: devel-file-in-non-devel-package
/usr/lib/erlang/lib/rebar-2/priv/templates/basicnif.c
erlang-rebar.ppc: E: non-executable-script
/usr/lib/erlang/lib/rebar-2/priv/templates/simplenode.erl.script 0644 /bin/bash
erlang-rebar.ppc: W: no-manual-page-for-binary rebar
1 packages and 0 specfiles checked; 3 errors, 3 warnings.
Sulaco ~/rpmbuild/SRPMS: 

First message is a false positive - due to "lib" substring in the name of
explicit requires. The next two (no-binary, only-non-binary-in-usr-lib) should
be ignored too - this is due to the fact that this package is arch-independent
but it is installed into arch-dependent library. Next two messages also should
be ignored - these are templates, required for package's normal operation.

Koji scratch build:

http://koji.fedoraproject.org/koji/taskinfo?taskID=2505758

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


  1   2   >