[Bug 820488] Review Request: mod_auth_xradius - Apache module that provides authentication against RADIUS Servers

2012-05-15 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=820488

--- Comment #9 from Simone Caronni  2012-05-16 02:50:42 
EDT ---
Many thanks, I've added you in CC for the ticket on FPC:

https://fedorahosted.org/fpc/ticket/175

I'll try to do something with the library and see if it can be built as a
shared object.

-- 
Configure 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 676943] Review Request: eiskaltdcpp-qt - Qt frontend to DC++ client library eiskaltdcpp

2012-05-15 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=676943

vasc...@gmail.com changed:

   What|Removed |Added

 CC||vasc...@gmail.com

--- Comment #2 from vasc...@gmail.com 2012-05-16 01:14:32 EDT ---
Jason Tibbitts, please, look at my Review Request 
https://bugzilla.redhat.com/show_bug.cgi?id=821406

-- 
Configure 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 812659] Review Request: par - paragraph reformatter

2012-05-15 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=812659

--- Comment #7 from David Levine  2012-05-15 22:53:10 
EDT ---
Spec URL: http://www.cs.wustl.edu/~levine/par/par-1.52-5.spec
SRPM URL: http://www.cs.wustl.edu/~levine/par/par-1.52-5.fc16.src.rpm

After seeing the discussion about id-utils on the devel mailing list, I think
that the par executable should be installed as /usr/lib/par/par.  So,
par-1.52-5.spec installs these files:

/usr/lib/par/par
/usr/share/doc/par-1.52/par.doc
/usr/share/doc/par-1.52/releasenotes
/usr/share/man/man1/par.1.gz

The defattr directive can be removed if install explicitly creates the doc dir.


Output of rpmlint -o 'ValidLicenses ("Par")':

par.i386: W: spelling-error Summary(en_US) reformatter -> reformatted,
reformatting, reformatory
par.i386: W: spelling-error Summary(en_US) fmt -> ft, mt, fit
par.i386: W: spelling-error %description -l en_US bodiless -> bodices
1 packages and 0 specfiles checked; 0 errors, 3 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 674008] Review Request: openrave - Open Robotics Automation Virtual Environment

2012-05-15 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=674008

--- Comment #93 from Rosen Diankov  2012-05-15 
22:47:48 EDT ---
hi tim,

still working on sympy 0.7.x compat and should be done soon.

If openrave-config is including "-L/usr/lib", it is because some library
openrave links to is including /usr/lib in its linked directories. The stringis
the following:

-L@CMAKE_INSTALL_PREFIX@/lib -lopenrave@OPENRAVE_LIBRARY_SUFFIX@
@OPENRAVE_BOOST_LIB_DIRS@ @Boost_THREAD_LIBRARY_RELEASE@

If you can find out which library (boost most likely?) then you can find out
why it is giving wrong info.


The openrave latest_stable branch now includes fparser 4.5 since it allows us
to compile with LLVM/Clang

As for the Vector casts, we removed the operator (float*) cast because several
compiles failed because it was clashing with operator[]. For example, which
function should v[0] call? I believe it was encouraging unsafe C-type array
practices.

-- 
Configure 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 817984] Review Request: ghc-zlib-conduit - Conduits for (de)compression

2012-05-15 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=817984

Jens Petersen  changed:

   What|Removed |Added

 Depends on||805887

-- 
Configure 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 817984] Review Request: ghc-zlib-conduit - Conduits for (de)compression

2012-05-15 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=817984

Jens Petersen  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

--- Comment #5 from Jens Petersen  2012-05-15 21:30:20 EDT 
---
Built for F18 rawhide.

F17 and earlier need newer zlib-bindings.

-- 
Configure 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 817391] Review Request: jupiter - Jupiter Hardware Control System for Computers

2012-05-15 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=817391

--- Comment #18 from Adrian Alves  2012-05-15 19:39:55 EDT ---
(In reply to comment #16)
> Your Source0 is not a valid URL, see
> 
> https://fedoraproject.org/wiki/Packaging:SourceURL
> 
> You should drop your commented echos.
Check it please, I have to add a comments there:
http://alvesadrian.fedorapeople.org/jupiter.spec
http://alvesadrian.fedorapeople.org/jupiter-0.1.2-6.fc16.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 817391] Review Request: jupiter - Jupiter Hardware Control System for Computers

2012-05-15 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=817391

--- Comment #19 from Adrian Alves  2012-05-15 19:41:37 EDT ---
(In reply to comment #17)
> This sudo hacking seems messy, consider to use pkexec from polkit. 
> A quick intro is found here:
> 
>  https://fedoraproject.org/wiki/Features/UsermodeMigration

How u recommend to use it?
make an script how?
How do I use this:
exec /usr/bin/pkexec  command?
that way? can u help me? how i need to remove the sudo hack for this pkexec?
not sure how.

-- 
Configure 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 821296] Review Request: drpython - python IDE

2012-05-15 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=821296

--- Comment #6 from Adrian Alves  2012-05-15 19:21:23 EDT ---
(In reply to comment #5)
> Some details 
> 
> [!]: SHOULD Spec use %global instead of %define.
>  Note: %define nameup DrPython
> 
> drpython.src: W: spelling-error %description -l en_US customizable ->
> customization
> 
> drpython.src: W: non-standard-group Development/Python
> 
> drpython.src: E: tag-not-utf8 %changelog
> 
> drpython.src: E: non-utf8-spec-file
> /tmp/rpmlint.drpython-3.11.1-2.fc16.src.rpm.Ek4ENv/drpython.spec
> 
> drpython.src:3: W: mixed-use-of-spaces-and-tabs (spaces: line 3, tab: line 2)

Fixed
Spec URL: http://alvesadrian.fedorapeople.org/drpython.spec
SRPM URL: http://alvesadrian.fedorapeople.org/drpython-3.11.1-3.fc16.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 771137] Review Request: decibel-audio-player - straightforward music player

2012-05-15 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=771137

--- Comment #15 from Volker Fröhlich  2012-05-15 17:37:48 EDT 
---

Package Review
==

Key:
- = N/A 
x = Pass
! = Fail
? = Not evaluated



 Generic 
[x]: MUST Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at
 least one supported primary architecture.
[-]: MUST %build honors applicable compiler flags or justifies otherwise.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[x]: MUST Buildroot is not present
 Note: Unless packager wants to package for EPEL5 this is fine
[x]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format. 
[x]: MUST Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT) 
 Note: Clean would be needed if support for EPEL is required
[x]: MUST Sources contain only permissible code or content. 
[x]: MUST Each %files section contains %defattr if rpm < 4.4
 Note: Note: defattr macros not found. They would be needed for EPEL5
[-]: MUST Macros in Summary, %description expandable at SRPM build time.
[!]: MUST Package contains a properly installed %{name}.desktop using desktop-
 file-install file if it is a GUI application.
[x]: MUST Package requires other packages for directories it uses.
[x]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[x]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
 Note: rm -rf would be needed if support for EPEL5 is required
[-]: MUST Large documentation files are in a -doc subpackage, if required.
[x]: MUST If (and only if) the source package includes the text of the
 license(s) in its own file, then that file, containing the text of the
 license(s) for the package is included in %doc.
[x]: MUST License field in the package spec file matches the actual license.
[x]: MUST The spec file handles locales properly.
[!]: MUST Package consistently uses macros (instead of hard-coded directory
 names).
Please use the _prefix macro on your two sed invocations.

[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generate any conflict.
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[x]: MUST Package must own all directories that it creates.
[x]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.
[x]: MUST Requires correct, justified where necessary.
[x]: MUST Rpmlint output is silent.

rpmlint decibel-audio-player-1.08-4.fc18.src.rpm

1 packages and 0 specfiles checked; 0 errors, 0 warnings.


rpmlint decibel-audio-player-1.08-4.fc18.noarch.rpm

1 packages and 0 specfiles checked; 0 errors, 0 warnings.


[x]: MUST Sources used to build the package match the upstream source, as
 provided in the spec URL.
/media/speicher1/makerpm/decibel-audio-player-1.08.tar.gz :
  MD5SUM this package : e8ebaf819c198ff9951903e7c4056aef
  MD5SUM upstream package : e8ebaf819c198ff9951903e7c4056aef

[x]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[-]: MUST Package contains a SysV-style init script if in need of one.
[x]: MUST File names are valid UTF-8.
[-]: MUST Useful -debuginfo package or justification otherwise.
[x]: SHOULD Reviewer should test that the package builds in mock.
[-]: SHOULD If the source package does not include license text(s) as a
 separate file from upstream, the packager SHOULD query upstream to
 include it.
[x]: SHOULD Dist tag is present.
[-]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin,
 /usr/sbin.
[x]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
 --requires).
[x]: SHOULD Package functions as described.
[x]: SHOULD Latest version is packaged.
[x]: SHOULD Package does not include license text files separate from
 upstream.
[x]: SHOULD Scriptlets must be sane, if used.
[x]: SHOULD SourceX is a working URL.
[-]: SHOULD Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: SHOULD Package should compile and build into binary rpms on all supported
 architectures.
[-]: SHOULD %check is present and all tests pass.
[x]: SHOULD Packages should try to preserve tim

[Bug 818589] Review Request: fest-swing-junit - FEST Swing JUnit support

2012-05-15 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=818589

--- Comment #5 from Roman Kennke  2012-05-15 17:01:39 EDT 
---
I added BuildRequires on Maven and fixed the JUnit dependency.

Spec URL:
http://rkennke.fedorapeople.org/fest-swing-junit/3/fest-swing-junit.spec
SRPM URL:
http://rkennke.fedorapeople.org/fest-swing-junit/3/fest-swing-junit-1.2.1-3.fc16.src.rpm

Ok 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 718430] Review Request: unknown-horizons - a 2D RTS game written in python which uses the fife engine

2012-05-15 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=718430

Nelson Marques  changed:

   What|Removed |Added

 Status|CLOSED  |ASSIGNED
 Resolution|WONTFIX |

--- Comment #25 from Nelson Marques  2012-05-15 15:34:35 
EDT ---
Re-opened

Updated previous stuff:

 * http://nmarques.fedorapeople.org/packages/unknown-horizons/

-- 
Configure 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 737286] Review Request: salt - A parallel remote execution system

2012-05-15 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=737286

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|salt-0.9.9.1-1.fc15 |salt-0.9.9.1-1.el6

--- Comment #59 from Fedora Update System  
2012-05-15 15:28:46 EDT ---
salt-0.9.9.1-1.el6 has been pushed to the Fedora EPEL 6 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 744432] Review Request: pyenet - python bindings for ENet

2012-05-15 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=744432

Nelson Marques  changed:

   What|Removed |Added

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

--- Comment #5 from Nelson Marques  2012-05-15 15:12:59 
EDT ---
Re-opening the bug report since current events.

Stuff includes 'SRPM' and 'spec'.

Stuff: http://nmarques.fedorapeople.org/packages/python-enet/

-- 
Configure 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 771137] Review Request: decibel-audio-player - straightforward music player

2012-05-15 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=771137

--- Comment #14 from Volker Fröhlich  2012-05-15 15:16:59 EDT 
---
On future comments, please always post the direct link to spec file and SRPM. 

The fedora-review utility requires that too.

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

[Bug 821890] Review Request: trac-sphinx-plugin - Basic tasks to manage Sphinx documentation in Trac

2012-05-15 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=821890

Kevin Fenzi  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|ke...@scrye.com
   Flag||fedora-review+

--- Comment #1 from Kevin Fenzi  2012-05-15 15:09:04 EDT ---

OK - Package meets naming and packaging guidelines
OK - Spec file matches base package name. 
OK - Spec has consistant macro usage. 
OK - Meets Packaging Guidelines. 
OK - License (BSD)
OK - License field in spec matches
OK - License file included in package
OK - Spec in American English
OK - Spec is legible.
OK - Sources match upstream md5sum:
(Sources do match up, md5 difference is due to timestamps)
87ae6e18ac9b477df50ea8df762c0ea1 
trac-sphinx-plugin-0.2.1.20120515.hge021e691af84.tar.gz
93fb5087f811a3f4d9d02528e31069e4 
trac-sphinx-plugin-0.2.1.20120515.hge021e691af84.tar.gz.orig

OK - BuildRequires correct
OK - Package has %defattr and permissions on files is good. 
OK - Package has a correct %clean section. 
OK - Package has correct buildroot
OK - Package is code or permissible content. 
OK - Packages %doc files don't affect runtime. 
OK - Package has rm -rf RPM_BUILD_ROOT at top of %install

OK - Package compiles and builds on at least one arch. 
OK - Package has no duplicate files in %files. 
OK - Package doesn't own any directories other packages own. 
OK - Package owns all the directories it creates. 
OK - Package obey's FHS standard (except for 2 exceptions)
See below - No rpmlint output. 
OK - final provides and requires are sane.

SHOULD Items:

OK - Should build in mock. 
OK - Should build on all supported archs
OK - Should function as described. 
OK - Should have sane scriptlets. 
OK - Should have dist tag
OK - Should package latest version
OK - Should not use file requires outside of /etc, /bin, /sbin, /usr/bin, or
/usr/sbin

Issues: 

1. rpmlint says: 

trac-sphinx-plugin.src: W: invalid-url Source0:
trac-sphinx-plugin-0.2.1.20120515.hge021e691af84.tar.gz
2 packages and 0 specfiles checked; 0 errors, 1 warnings.

Can be ignored. 

I see no blockers here, so this package is APPROVED.

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

[Bug 820488] Review Request: mod_auth_xradius - Apache module that provides authentication against RADIUS Servers

2012-05-15 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=820488

--- Comment #8 from Jon Ciesla  2012-05-15 14:58:49 EDT 
---
Ugh.  Sigh.

https://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries#Exceptions

File a Trac for FPC and we'll see.  I'm optimistic.  I'd rather see this
packaged separately, so if you can coax a solib out of it that would be better.
 If you can get the Trac in soon we can probably look at it at tomorrow's
meeting.

-- 
Configure 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 821889] Review Request: sphinx-webtools - Sphinx web tools for python web frameworks

2012-05-15 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=821889

Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #2 from Kevin Fenzi  2012-05-15 14:49:38 EDT ---
OK - Package meets naming and packaging guidelines
OK - Spec file matches base package name. 
OK - Spec has consistant macro usage. 
OK - Meets Packaging Guidelines. 
OK - License (BSD)
OK - License field in spec matches
OK - License file included in package
OK - Spec in American English
OK - Spec is legible.
OK - Sources match upstream md5sum:
(md5's don't match because of timestamp differences, contents are the same)
8241b4a44d3af52ab44c2c8f48fed441 
sphinx-webtools-0.2.1.20120515.hg86a2eac02afe.tar.gz
150001f6bf73a0dca6c94092ab2be8be 
sphinx-webtools-0.2.1.20120515.hg86a2eac02afe.tar.gz.orig
OK - BuildRequires correct
OK - Package has %defattr and permissions on files is good. 
OK - Package has a correct %clean section. 
OK - Package has correct buildroot
OK - Package is code or permissible content. 
OK - Packages %doc files don't affect runtime. 
OK - Package has rm -rf RPM_BUILD_ROOT at top of %install

OK - Package compiles and builds on at least one arch. 
OK - Package has no duplicate files in %files. 
OK - Package doesn't own any directories other packages own. 
OK - Package owns all the directories it creates. 
OK - Package obey's FHS standard (except for 2 exceptions)
See below - No rpmlint output. 
OK - final provides and requires are sane.

SHOULD Items:

OK - Should build in mock. 
OK - Should build on all supported archs
OK - Should function as described. 
OK - Should have sane scriptlets. 
OK - Should have dist tag
OK - Should package latest version
OK - Should not use file requires outside of /etc, /bin, /sbin, /usr/bin, or
/usr/sbin

Issues: 

1. There's a test suite, might run it in a %check section? 
python setup.py test

2. rpmlint says: 

sphinx-webtools.noarch: W: spelling-error %description -l en_US functionalities
-> functionalists, functionality, functionalist
sphinx-webtools.src: W: spelling-error %description -l en_US functionalities ->
functionalists, functionality, functionalist
sphinx-webtools.src: W: invalid-url Source0:
sphinx-webtools-0.2.1.20120515.hg86a2eac02afe.tar.gz
2 packages and 0 specfiles checked; 0 errors, 3 warnings.

All can be ignored. 

I don't see any blockers here, so this package is APPROVED. 
Please do consider enabling the tests if they work ok in mock.

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

[Bug 819237] Review Request: vdr-screenshot - Extended screenshot plugin for VDR

2012-05-15 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=819237

--- Comment #12 from MartinKG  2012-05-15 14:48:31 EDT ---
SRPM URL:
https://www.disk.dsl.o2online.de/FclyPlh/RPMS/VDR/vdr-screenshot/vdr-screenshot-0.0.14-3.fc17.src.rpm?a=mpJvUWzKWgc

Spec URL:
https://www.disk.dsl.o2online.de/FclyPlh/RPMS/VDR/vdr-screenshot/vdr-screenshot.spec?a=yszn7xxxQkE

rpmlint output:

rpmlint vdr-screenshot-0.0.14-3.fc17.src.rpm
vdr-screenshot.src: W: spelling-error Summary(en_US) screenshots -> screen
shots, screen-shots, screens hots
vdr-screenshot.src: W: spelling-error %description -l en_US mainmenu -> main
menu, main-menu, trainmen
vdr-screenshot.src: W: spelling-error %description -l en_US mmdd 
vdr-screenshot.src: W: spelling-error %description -l en_US hhmmss 
vdr-screenshot.src: W: spelling-error %description -l en_US jpg -> jog, jg, pg
1 packages and 0 specfiles checked; 0 errors, 5 warnings.

rpmlint vdr-screenshot-0.0.14-3.fc17.x86_64.rpm
vdr-screenshot.x86_64: W: spelling-error Summary(en_US) screenshots -> screen
shots, screen-shots, screens hots
vdr-screenshot.x86_64: W: spelling-error %description -l en_US mainmenu -> main
menu, main-menu, trainmen
vdr-screenshot.x86_64: W: spelling-error %description -l en_US mmdd 
vdr-screenshot.x86_64: W: spelling-error %description -l en_US hhmmss 
vdr-screenshot.x86_64: W: spelling-error %description -l en_US jpg -> jog, jg,
pg
vdr-screenshot.x86_64: W: non-standard-uid /var/cache/vdr/screenshot vdr
1 packages and 0 specfiles checked; 0 errors, 6 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 820488] Review Request: mod_auth_xradius - Apache module that provides authentication against RADIUS Servers

2012-05-15 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=820488

--- Comment #7 from Simone Caronni  2012-05-15 14:48:49 
EDT ---
The libradius bundled has been updated in recent FreeBSD CVS commits at the
previous links.

There are no tarball releases for it, as it is part of the FreeBSD release:

ftp://ftp.freebsd.org/pub/FreeBSD/releases/amd64/9.0-RELEASE/src.txz

bundled(libradius-bsd)?

Thanks,
--Simone

-- 
Configure 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 821838] Review Request: gitolite3 - Highly flexible server for git directory version tracker

2012-05-15 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=821838

--- Comment #8 from Jon Ciesla  2012-05-15 14:46:51 EDT 
---
I believe so.

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

[Bug 820488] Review Request: mod_auth_xradius - Apache module that provides authentication against RADIUS Servers

2012-05-15 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=820488

--- Comment #6 from Jon Ciesla  2012-05-15 14:40:51 EDT 
---
Ok, I've added that md5 implementation, so use bundled(md5-polstra).

Can that libradius build a solib?  Is it updated in the CVS beyond what's in
this module?  Are there tarball releases of it?

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

[Bug 821296] Review Request: drpython - python IDE

2012-05-15 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=821296

--- Comment #5 from Sergio Monteiro Basto  2012-05-15 
14:38:51 EDT ---
Some details 

[!]: SHOULD Spec use %global instead of %define.
 Note: %define nameup DrPython

drpython.src: W: spelling-error %description -l en_US customizable ->
customization

drpython.src: W: non-standard-group Development/Python

drpython.src: E: tag-not-utf8 %changelog

drpython.src: E: non-utf8-spec-file
/tmp/rpmlint.drpython-3.11.1-2.fc16.src.rpm.Ek4ENv/drpython.spec

drpython.src:3: W: mixed-use-of-spaces-and-tabs (spaces: line 3, tab: line 2)

-- 
Configure 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 821917] New: Review Request: mu - maildir utility with Emacs support

2012-05-15 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: mu - maildir utility with Emacs support

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

   Summary: Review Request: mu - maildir utility with Emacs
support
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: maciek.borze...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://fiona.dmcs.pl/~mborzecki/fedora/mu/mu.spec
SRPM URL: http://fiona.dmcs.pl/~mborzecki/fedora/mu/mu-0.9.8.4-1.fc16.src.rpm
Description: mu is a tool for dealing with e-mail messages stored in the
Maildir-format, on Unix-like systems. mu's main purpose is to help you
to find the messages you need, quickly; in addition, it allows you to
view messages, extract attachments, create new maildirs, etc. mu has a
nice Emacs interface provided in emacs-mu4e 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 821838] Review Request: gitolite3 - Highly flexible server for git directory version tracker

2012-05-15 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=821838

--- Comment #7 from Simone Caronni  2012-05-15 14:27:54 
EDT ---
Are these the intended behaviour in the latest revision you posted?

E: script-without-shebang
/usr/share/gitolite3/syntactic-sugar/continuation-lines
E: script-without-shebang
/usr/share/gitolite3/syntactic-sugar/keysubdirs-as-groups

E: non-executable-script /usr/share/gitolite3/VREF/MERGE-CHECK 0644L
/usr/bin/perl

--Simone

-- 
Configure 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 821917] Review Request: mu - maildir utility with Emacs support

2012-05-15 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=821917

maciek.borze...@gmail.com changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)

-- 
Configure 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 736861] Review Request: hgview - A fast Mercurial log navigator for qt4 or curses

2012-05-15 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=736861

Mads Kiilerich  changed:

   What|Removed |Added

Summary|Review Request: hgview - A  |Review Request: hgview - A
   |fast Mercurial log  |fast Mercurial log
   |navigator written in pyqt4  |navigator for qt4 or curses

-- 
Configure 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 821889] Review Request: sphinx-webtools - Sphinx web tools for python web frameworks

2012-05-15 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=821889

Kevin Fenzi  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|ke...@scrye.com
   Flag||fedora-review?

--- Comment #1 from Kevin Fenzi  2012-05-15 14:26:55 EDT ---
I'll review this. Look for a full review in a bit.

-- 
Configure 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 820488] Review Request: mod_auth_xradius - Apache module that provides authentication against RADIUS Servers

2012-05-15 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=820488

--- Comment #5 from Simone Caronni  2012-05-15 14:19:00 
EDT ---
Many thanks for the patch and the the time for the review.

Spec URL: http://slaanesh.fedorapeople.org/mod_auth_xradius.spec
SRPM URL:
http://slaanesh.fedorapeople.org/mod_auth_xradius-0.4.6-4.fc17.src.rpm

- Changed default config to point to localhost only.
- Added patch for rawhide.

Regarding md5 and libradius; it seems is none of the md5 implementations listed
at:

http://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries#cite_note-1 

this is the text file that comes inside the source:

"This software was originally written by John Polstra, and donated to the
FreeBSD project by Juniper Networks, Inc. Oleg Semyonov subsequently
added the ability to perform RADIUS accounting.

The following source files were extracted from a FreeBSD 5-STABLE
system as of 20-Sep-2004:

/usr/src/sys/sys/md5.h
/usr/src/lib/libmd/md5c.c
/usr/src/lib/libradius/libradius.3
/usr/src/lib/libradius/radius.conf.5
/usr/src/lib/libradius/radlib.c
/usr/src/lib/libradius/radlib.h
/usr/src/lib/libradius/radlib_private.h
/usr/src/lib/libradius/radlib_vs.h"

They're still in FreeBSD CVS and licensed as part of the base BSD archive
that's BSD licensed:

http://www.freebsd.org/cgi/cvsweb.cgi/src/lib/libradius/

What should I do?

Thanks,
--Simone

-- 
Configure 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 819237] Review Request: vdr-screenshot - Extended screenshot plugin for VDR

2012-05-15 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=819237

--- Comment #11 from Ville Skyttä  2012-05-15 14:10:46 EDT 
---
(In reply to comment #10)
> vdr-screenshot.x86_64: W: non-standard-uid
> /usr/share/locale/tr_TR/LC_MESSAGES/vdr-screenshot.mo vdr

These errors need fixing, translations should be owned by root, not vdr. 
Dropping all %defattrs and using plain "%attr(-,%{vdr_user},root) %dir
%{vdr_cachedir}/screenshot/" just for the dir would be one way to fix it, or
adding one more %defattr(-,root,root,-) at the end - that's the one that ends
up affecting stuff passed to %files with -f.

Non-blocker, but I'd recommend combining the picture-path and readme patches
into one.

BTW upstream reported that the *.po patch will be included in the next release.

-- 
Configure 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 739015] Review Request: erlang-riak_sysmon - Rate-limiting system_monitor event handler for Riak

2012-05-15 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=739015

Peter Lemenkov  changed:

   What|Removed |Added

  Status Whiteboard|NotReady|

--- Comment #1 from Peter Lemenkov  2012-05-15 13:55:06 EDT 
---
Lifted NotReady and updated to 1.1.2:

http://peter.fedorapeople.org/erlang-riak_sysmon.spec
http://peter.fedorapeople.org/erlang-riak_sysmon-1.1.2-1.fc18.src.rpm

Koji scratchbuild for F-18:
http://koji.fedoraproject.org/koji/taskinfo?taskID=4079575

rpmlint:

sulaco ~/rpmbuild/SPECS: rpmlint
../RPMS/ppc/erlang-riak_sysmon-1.1.2-1.fc18.ppc.rpm
../SRPMS/erlang-riak_sysmon-1.1.2-1.fc18.src.rpm 
erlang-riak_sysmon.ppc: E: explicit-lib-dependency erlang-stdlib

^^^ this one is a false positive (stdlib as a trigger)

erlang-riak_sysmon.ppc: E: no-binary
erlang-riak_sysmon.ppc: W: only-non-binary-in-usr-lib

^^^ this one is tricky. All erlang packages must be installed into
%{_libdir}/erlang/lib so despite of the fact that some of them contains only
arch-independent data they all must be build as arch-dependent. I plan to fix
than but I wouldn't hold my breath.

erlang-riak_sysmon.src: W: invalid-url Source0:
basho-riak_sysmon-1.1.2-0-gaab7706.tar.gz

^^^ blame github for that, not me.

2 packages and 0 specfiles checked; 2 errors, 2 warnings.
sulaco ~/rpmbuild/SPECS:

-- 
Configure 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 820452] Review Request: mediawiki119 - updated mediawiki for EPEL-6

2012-05-15 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=820452

Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #5 from Kevin Fenzi  2012-05-15 13:31:01 EDT ---
Excellent. Thanks!

So, the only outstanding things I see are the rpmlint fixes, which are pretty
minor and updating the License field.

Please do those before checking in, but otherwise this package is APPROVED.

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

[Bug 820452] Review Request: mediawiki119 - updated mediawiki for EPEL-6

2012-05-15 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=820452

Tom "spot" Callaway  changed:

   What|Removed |Added

 Blocks|182235(FE-Legal)|

--- Comment #4 from Tom "spot" Callaway  2012-05-15 
13:24:00 EDT ---
Diane Peters, General Counsel for Creative Commons has agreed to except the
distribution made by Fedora of the unmodified Creative Commons trademarked logo
files as distributed in Mediawiki.

This resolves the issue with the trademark terms to my satisfaction, so I'm
lifting FE-Legal.

-- 
Configure 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 821838] Review Request: gitolite3 - Highly flexible server for git directory version tracker

2012-05-15 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=821838

--- Comment #6 from Jon Ciesla  2012-05-15 13:25:29 EDT 
---
- Added license file, fixed duplicate files, dropped defattr.
- Dropped clean and buildroot.
- Added script to generate tarball in comments.

Right, I'll be building F-17, F-16, and EL-6 only.

SPEC: http://fedorapeople.org/~limb/review/gitolite3/gitolite3.spec
SRPM:
http://fedorapeople.org/~limb/review/gitolite3/gitolite3-3.01-2.fc16.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 820488] Review Request: mod_auth_xradius - Apache module that provides authentication against RADIUS Servers

2012-05-15 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=820488

--- Comment #3 from Jon Ciesla  2012-05-15 13:04:30 EDT 
---
Builds fine on f16, but needs the following patch to build in mock for rawhide:

-- 
Configure 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 820488] Review Request: mod_auth_xradius - Apache module that provides authentication against RADIUS Servers

2012-05-15 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=820488

--- Comment #4 from Jon Ciesla  2012-05-15 13:05:20 EDT 
---
Created attachment 584723
  --> https://bugzilla.redhat.com/attachment.cgi?id=584723
unixd 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 821838] Review Request: gitolite3 - Highly flexible server for git directory version tracker

2012-05-15 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=821838

--- Comment #4 from Simone Caronni  2012-05-15 13:00:15 
EDT ---
Key:
- = N/A
x = Pass
! = Fail
? = Not evaluated

 Generic 
[x]: MUST Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at
 least one supported primary architecture.
[-]: MUST %build honors applicable compiler flags or justifies otherwise.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[!]: MUST Buildroot is not present
 Note: Buildroot is not needed unless packager plans to package for EPEL5
[x]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[!]: MUST Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
 Note: Clean is needed only if supporting EPEL
[x]: MUST Sources contain only permissible code or content.
[x]: MUST %config files are marked noreplace or the reason is justified.
[!]: MUST Each %files section contains %defattr if rpm < 4.4
 Note: defattr() present in %files section. This is OK if packaging
 for EPEL5. Otherwise not needed
[-]: MUST Macros in Summary, %description expandable at SRPM build time.
[x]: MUST Package requires other packages for directories it uses.
[x]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[!]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
 Note: rm -rf is only needed if supporting EPEL5
[-]: MUST Large documentation files are in a -doc subpackage, if required.
[!]: MUST If (and only if) the source package includes the text of the
 license(s) in its own file, then that file, containing the text of the
 license(s) for the package is included in %doc.
[x]: MUST License field in the package spec file matches the actual license.
[x]: MUST Package consistently uses macros (instead of hard-coded directory
 names).
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST No %config files under /usr.
[x]: MUST Package does not generate any conflict.
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[x]: MUST Package must own all directories that it creates.
[x]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.
[x]: MUST Requires correct, justified where necessary.
[!]: MUST Rpmlint output is silent.
[!]: MUST Sources used to build the package match the upstream source, as
 provided in the spec URL.
Package has no sources or they are generated by developer
[x]: MUST Spec file is legible and written in American English.
[x]: MUST Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[-]: MUST Package contains a SysV-style init script if in need of one.
[x]: MUST File names are valid UTF-8.
[-]: MUST Useful -debuginfo package or justification otherwise.
[x]: SHOULD Reviewer should test that the package builds in mock.
[x]: SHOULD If the source package does not include license text(s) as a
 separate file from upstream, the packager SHOULD query upstream to
 include it.
[x]: SHOULD Dist tag is present.
[x]: SHOULD No file requires outside of /etc, /bin, /sbin, /usr/bin,
 /usr/sbin.
[x]: SHOULD Final provides and requires are sane (rpm -q --provides and rpm -q
 --requires).
[x]: SHOULD Package functions as described.
[x]: SHOULD Latest version is packaged.
[x]: SHOULD Package does not include license text files separate from
 upstream.
[-]: SHOULD Scriptlets must be sane, if used.
[x]: SHOULD SourceX is a working URL.
[-]: SHOULD Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: SHOULD Package should compile and build into binary rpms on all supported
 architectures.
[-]: SHOULD %check is present and all tests pass.
[x]: SHOULD Packages should try to preserve timestamps of original installed
 files.
[x]: SHOULD Spec use %global instead of %define.

-- 
Configure 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 821838] Review Request: gitolite3 - Highly flexible server for git directory version tracker

2012-05-15 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=821838

--- Comment #5 from Simone Caronni  2012-05-15 13:01:08 
EDT ---
EPEL 5: Looking at the spec file I assume that the package will not be built
for EPEL 5.

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

The archive includes the "COPYING" file but that's not included in the %doc
section.

[!]: MUST Rpmlint output is silent.

gitolite3.noarch: W: spelling-error %description -l en_US userids -> use rids,
use-rids, user ids
gitolite3.noarch: W: spelling-error %description -l en_US ssh's -> ash's,
sash's
gitolite3.noarch: W: spelling-error %description -l en_US pubkey -> pub key,
pub-key, puberty
gitolite3.noarch: W: spelling-error %description -l en_US gitosis -> mitosis,
halitosis
gitolite3.src: W: spelling-error %description -l en_US Gitolite -> Impolite
gitolite3.src: W: spelling-error %description -l en_US userids -> use rids,
use-rids, user ids
gitolite3.src: W: spelling-error %description -l en_US ssh's -> ash's, sash's
gitolite3.src: W: spelling-error %description -l en_US pubkey -> pub key,
pub-key, puberty
gitolite3.src: W: spelling-error %description -l en_US gitosis -> mitosis,
halitosis
gitolite3.src: W: spelling-error %description -l en_US perl -> Perl, peel, perk

Dictionary errors, can be ignored (halitosis?).


gitolite3.noarch: E: script-without-shebang
/usr/share/perl5/vendor_perl/Gitolite/Triggers/CpuTime.pm
gitolite3.noarch: E: script-without-shebang
/usr/share/gitolite3/lib/Gitolite/Triggers/CpuTime.pm

No interpreter for perl modules, can be ignored.


gitolite3.noarch: E: script-without-shebang
/usr/share/gitolite3/syntactic-sugar/continuation-lines
gitolite3.noarch: E: script-without-shebang
/usr/share/gitolite3/syntactic-sugar/keysubdirs-as-groups

Do these need an interpreter at the top?


gitolite3.noarch: W: non-standard-uid /var/lib/gitolite3/.ssh/authorized_keys
gitolite3
gitolite3.noarch: W: non-standard-gid /var/lib/gitolite3/.ssh/authorized_keys
gitolite3
gitolite3.noarch: W: non-standard-uid /var/lib/gitolite3 gitolite3
gitolite3.noarch: W: non-standard-gid /var/lib/gitolite3 gitolite3
gitolite3.noarch: E: non-standard-dir-perm /var/lib/gitolite3 0750L
gitolite3.noarch: W: non-standard-uid /var/lib/gitolite3/.ssh gitolite3
gitolite3.noarch: W: non-standard-gid /var/lib/gitolite3/.ssh gitolite3
gitolite3.noarch: W: hidden-file-or-dir /var/lib/gitolite3/.ssh

I suppose these can be ignored as per the package structure and the way the
user is confined.


gitolite3.noarch: E: non-standard-dir-perm /var/lib/gitolite3/.ssh 0750L
gitolite3.noarch: W: hidden-file-or-dir /var/lib/gitolite3/.ssh
gitolite3.noarch: E: non-readable /var/lib/gitolite3/.ssh/authorized_keys 0640L
gitolite3.noarch: E: zero-length /var/lib/gitolite3/.ssh/authorized_keys

Maybe (I'm not sure) this line can be removed from the spec file:
%attr(750,%{name},%{name}) %{gitolite_homedir}/.ssh
and a %ghost could be added to the authorized_keys file. Or another option can
be like pcpa stated in Comment #3


gitolite3.noarch: W: no-manual-page-for-binary gitolite

Can be ignored.


gitolite3.noarch: E: non-executable-script
/usr/share/gitolite3/VREF/MERGE-CHECK 0644L /usr/bin/perl

I think this is wrong, all the other scripts in /usr/share/gitolite3/VREF/ are
executable.


gitolite3.src: W: invalid-url Source0:
sitaramc-gitolite-v3.01-0-g88b4c86-nodocs.tar.gz

Missing script to generate tarball in the SRPM.


2 packages and 0 specfiles checked; 9 errors, 20 warnings.


[!]: MUST Sources used to build the package match the upstream source, as
 provided in the spec URL.
Package has no sources or they are generated by developer

Script to generate the sources tarball from the git repository with the
declared components excluded is missing:

http://fedoraproject.org/wiki/Packaging:SourceURL#When_Upstream_uses_Prohibited_Code

-- 
Configure 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 821838] Review Request: gitolite3 - Highly flexible server for git directory version tracker

2012-05-15 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=821838

pcpa  changed:

   What|Removed |Added

 CC||paulo.cesar.pereira.de.andr
   ||a...@gmail.com

--- Comment #3 from pcpa  2012-05-15 
12:56:08 EDT ---
I suggest using

%attr(750,%{name},%{name}) %dir %{gitolite_homedir}
%attr(750,%{name},%{name}) %dir %{gitolite_homedir}/.ssh

instead of

%attr(750,%{name},%{name}) %{gitolite_homedir}
%attr(750,%{name},%{name}) %{gitolite_homedir}/.ssh

to avoid files listed multiple times.

-- 
Configure 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 821889] New: Review Request: sphinx-webtools - Sphinx web tools for python web frameworks

2012-05-15 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: sphinx-webtools - Sphinx web tools for python web 
frameworks

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

   Summary: Review Request: sphinx-webtools - Sphinx web tools for
python web frameworks
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: andrea.v...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL:
http://averi.fedorapeople.org/RPMs/sphinx-webtools/sphinx-webtools.spec
SRPM URL:
http://averi.fedorapeople.org/RPMs/sphinx-webtools/sphinx-webtools-0.2.1-20120515.hg86a2eac02afe.fc16.src.rpm
Description: This package includes some tools to manage sphinx docs on the
Python web framework: generate docs, get contents and links, search
functionalities.

-- 
Configure 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 821890] New: Review Request: trac-sphinx-plugin - Basic tasks to manage Sphinx documentation in Trac

2012-05-15 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: trac-sphinx-plugin - Basic tasks to manage Sphinx 
documentation in Trac

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

   Summary: Review Request: trac-sphinx-plugin - Basic tasks to
manage Sphinx documentation in Trac
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: andrea.v...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL:
http://averi.fedorapeople.org/RPMs/trac-sphinx-plugin/trac-sphinx-plugin.spec
SRPM URL:
http://averi.fedorapeople.org/RPMs/trac-sphinx-plugin/trac-sphinx-plugin-0.2.1-20120515.hge021e691af84.fc16.src.rpm
Description: A trac plugin that implements basic tasks to manage Sphinx
documentation in Trac.

-- 
Configure 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 814916] Review Request: sratom - a C library for serializing LV2 plugins

2012-05-15 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=814916

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #12 from Fedora Update System  
2012-05-15 12:41:56 EDT ---
sratom-0.2.0-3.fc17 has been pushed to the Fedora 17 testing repository.

-- 
Configure 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 812699] Review Request: ghc-happstack-server - Happstack web framework

2012-05-15 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=812699

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #8 from Fedora Update System  2012-05-15 
12:40:55 EDT ---
ghc-sendfile-0.7.6-1.fc17, ghc-happstack-server-7.0.0-2.fc17 has been pushed to
the Fedora 17 testing repository.

-- 
Configure 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 808768] Review Request: annox - Java annotations in XML resources

2012-05-15 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=808768

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #10 from Fedora Update System  
2012-05-15 12:41:49 EDT ---
annox-0.5.0-2.fc17 has been pushed to the Fedora 17 testing repository.

-- 
Configure 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 814924] Review Request: jalv - a simple LV2 host

2012-05-15 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=814924

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #7 from Fedora Update System  2012-05-15 
12:41:20 EDT ---
jalv-1.0.0-2.fc17 has been pushed to the Fedora 17 testing repository.

-- 
Configure 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 820488] Review Request: mod_auth_xradius - Apache module that provides authentication against RADIUS Servers

2012-05-15 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=820488

--- Comment #2 from Jon Ciesla  2012-05-15 12:30:51 EDT 
---
Good:

- rpmlint checks return:

mod_auth_xradius-debuginfo.x86_64: W: spelling-error Summary(en_US) auth ->
auto, Ruth, author
The value of this tag appears to be misspelled. Please double-check.

mod_auth_xradius-debuginfo.x86_64: W: spelling-error Summary(en_US) xradius ->
radius, x radius, radium
The value of this tag appears to be misspelled. Please double-check.

mod_auth_xradius-debuginfo.x86_64: W: spelling-error %description -l en_US auth
-> auto, Ruth, author
The value of this tag appears to be misspelled. Please double-check.

mod_auth_xradius-debuginfo.x86_64: W: spelling-error %description -l en_US
xradius -> radius, x radius, radium
The value of this tag appears to be misspelled. Please double-check.

2 packages and 0 specfiles checked; 0 errors, 4 warnings.

Ignore, otherwise clean.

- package meets naming guidelines
- package meets packaging guidelines

X license ( ASL 2.0 ) OK, text in %doc, matches source except for md5.

- spec file legible, in am. english
- source matches upstream
- package compiles on devel (x86)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- no need for .desktop file 

In auth_xradius.conf, should line 16 possibly be limited to just localhost by
default?  Maybe with what's there present, but commented out?

Bundles md5, which is OK, but needs to Provide bundled(md5-$IMPLEMENTATION). 
Not sure what implementation we have here, might be a new one for the list.

Bundles libradius.  Where's it from?  Is there an upstream we can 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 819237] Review Request: vdr-screenshot - Extended screenshot plugin for VDR

2012-05-15 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=819237

--- Comment #10 from MartinKG  2012-05-15 12:24:58 EDT ---
SRPM URL:
https://www.disk.dsl.o2online.de/FclyPlh/RPMS/VDR/vdr-screenshot/vdr-screenshot-0.0.14-2.fc17.src.rpm?a=ovt8WkmH_3I

Spec URL:
https://www.disk.dsl.o2online.de/FclyPlh/RPMS/VDR/vdr-screenshot/vdr-screenshot.spec?a=A6zmPocvTXo

hunspell-en is now installed

rpmlint output:

rpmlint vdr-screenshot-0.0.14-2.fc17.src.rpm 
vdr-screenshot.src: W: spelling-error Summary(en_US) screenshots -> screen
shots, screen-shots, screens hots
vdr-screenshot.src: W: spelling-error %description -l en_US mainmenu -> main
menu, main-menu, trainmen
vdr-screenshot.src: W: spelling-error %description -l en_US mmdd 
vdr-screenshot.src: W: spelling-error %description -l en_US hhmmss 
vdr-screenshot.src: W: spelling-error %description -l en_US jpg -> jog, jg, pg
1 packages and 0 specfiles checked; 0 errors, 5 warnings.

rpmlint vdr-screenshot-0.0.14-2.fc17.x86_64.rpm
vdr-screenshot.x86_64: W: spelling-error Summary(en_US) screenshots -> screen
shots, screen-shots, screens hots
vdr-screenshot.x86_64: W: spelling-error %description -l en_US mainmenu -> main
menu, main-menu, trainmen
vdr-screenshot.x86_64: W: spelling-error %description -l en_US mmdd 
vdr-screenshot.x86_64: W: spelling-error %description -l en_US hhmmss 
vdr-screenshot.x86_64: W: spelling-error %description -l en_US jpg -> jog, jg,
pg
vdr-screenshot.x86_64: W: non-standard-uid
/usr/share/locale/tr_TR/LC_MESSAGES/vdr-screenshot.mo vdr
vdr-screenshot.x86_64: W: non-standard-uid
/usr/share/locale/es_ES/LC_MESSAGES/vdr-screenshot.mo vdr
vdr-screenshot.x86_64: W: non-standard-uid
/usr/share/locale/fi_FI/LC_MESSAGES/vdr-screenshot.mo vdr
vdr-screenshot.x86_64: W: non-standard-uid /var/cache/vdr/screenshot vdr
vdr-screenshot.x86_64: W: non-standard-uid
/usr/share/locale/de_DE/LC_MESSAGES/vdr-screenshot.mo vdr
vdr-screenshot.x86_64: W: non-standard-uid
/usr/share/locale/ca_ES/LC_MESSAGES/vdr-screenshot.mo vdr
vdr-screenshot.x86_64: W: non-standard-uid
/usr/share/locale/it_IT/LC_MESSAGES/vdr-screenshot.mo vdr
1 packages and 0 specfiles checked; 0 errors, 12 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 820452] Review Request: mediawiki119 - updated mediawiki for EPEL-6

2012-05-15 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=820452

Tom "spot" Callaway  changed:

   What|Removed |Added

 CC||tcall...@redhat.com

--- Comment #3 from Tom "spot" Callaway  2012-05-15 
12:22:28 EDT ---
I've confirmed with Jonathan Snook and Robert Nyman via email that we have
permission to distribute their code (the part of wikibits.js which is
problematic) under CC-BY-SA 2.5.

I've reached out to Creative Commons to try to come up with some sort of
resolution to that issue, please stay tuned.

-- 
Configure 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 821838] Review Request: gitolite3 - Highly flexible server for git directory version tracker

2012-05-15 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=821838

Simone Caronni  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 821838] Review Request: gitolite3 - Highly flexible server for git directory version tracker

2012-05-15 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=821838

--- Comment #2 from Simone Caronni  2012-05-15 12:19:47 
EDT ---
I will review this package

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

[Bug 821838] Review Request: gitolite3 - Highly flexible server for git directory version tracker

2012-05-15 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=821838

Simone Caronni  changed:

   What|Removed |Added

   Flag||fedora-review?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are 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 821838] Review Request: gitolite3 - Highly flexible server for git directory version tracker

2012-05-15 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=821838

Simone Caronni  changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|negativ...@gmail.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 820488] Review Request: mod_auth_xradius - Apache module that provides authentication against RADIUS Servers

2012-05-15 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=820488

Jon Ciesla  changed:

   What|Removed |Added

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

--- Comment #1 from Jon Ciesla  2012-05-15 12:17:30 EDT 
---
In progress.

-- 
Configure 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 821296] Review Request: drpython - python IDE

2012-05-15 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=821296

--- Comment #4 from Sergio Monteiro Basto  2012-05-15 
12:11:44 EDT ---
Spec URL: http://alvesadrian.fedorapeople.org/drpython.spec
SRPM URL: http://alvesadrian.fedorapeople.org/drpython-3.11.1-2.fc16.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 821862] New: Review Request: perl-srpm-macros - RPM macros for building Perl source package from source repository

2012-05-15 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-srpm-macros - RPM macros for building Perl source 
package from source repository

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

   Summary: Review Request: perl-srpm-macros - RPM macros for
building Perl source package from source repository
   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,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://ppisar.fedorapeople.org/perl-srpm-macros/perl-srpm-macros.spec
SRPM URL:
http://ppisar.fedorapeople.org/perl-srpm-macros/perl-srpm-macros-1-1.fc18.src.rpm
Description:
These RPM macros are used for building Perl source packages from source
repositories. They influence build-requires set into the source package.



The only source files will live in Fedora source package git repository.

-- 
Configure 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 821845] Review Request: erlang-egeoip - Erlang IP Geolocation module

2012-05-15 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=821845

Peter Lemenkov  changed:

   What|Removed |Added

 Blocks||182235(FE-Legal)

--- Comment #1 from Peter Lemenkov  2012-05-15 11:00:00 EDT 
---
Cast wizards - raise magic FE-Legal flag.

-- 
Configure 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 821845] New: Review Request: erlang-egeoip - Erlang IP Geolocation module

2012-05-15 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-egeoip - Erlang IP Geolocation module

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

   Summary: Review Request: erlang-egeoip - Erlang IP Geolocation
module
   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,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://peter.fedorapeople.org/erlang-egeoip.spec
SRPM URL:
http://peter.fedorapeople.org/erlang-egeoip-0-0.2.20111025git45c32ad.fc18.src.rpm
Description: Erlang IP Geolocation module, currently supporting the MaxMind
GeoLite City Database.

Koji scratchbuild for F-18:
http://koji.fedoraproject.org/koji/taskinfo?taskID=4078818

rpmlit report:
sulaco ~/rpmbuild/SPECS: rpmlint
../RPMS/ppc/erlang-egeoip-0-0.2.20111025git45c32ad.fc18.ppc.rpm
../SRPMS/erlang-egeoip-0-0.2.20111025git45c32ad.fc18.src.rpm 
erlang-egeoip.ppc: E: explicit-lib-dependency erlang-stdlib

^^^ this one is a false positive (stdlib as a trigger)

erlang-egeoip.ppc: E: no-binary
erlang-egeoip.ppc: W: only-non-binary-in-usr-lib

^^^ this one is tricky. All erlang packages must be installed into
%{_libdir}/erlang/lib so despite of the fact that some of them contains only
arch-independent data they all must be build as arch-dependent. I plan to fix
than but I wouldn't hold my breath.

erlang-egeoip.src: W: invalid-url Source0: mochi-egeoip-45c32ad.tar.gz

^^^ blame github for that, not me.

2 packages and 0 specfiles checked; 2 errors, 2 warnings.
sulaco ~/rpmbuild/SPECS:

This is a popular (among fellow erlangers) Geolocation Erlang library. Note -
content license is originally named ad "Open Data License". AFAIK it looks like
BSD with advertising, but IANAL.

-- 
Configure 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 821838] Review Request: gitolite3 - Highly flexible server for git directory version tracker

2012-05-15 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=821838

Jon Ciesla  changed:

   What|Removed |Added

 CC||lkund...@v3.sk

--- Comment #1 from Jon Ciesla  2012-05-15 10:50:44 EDT 
---
CCing gitolite maintainer.

-- 
Configure 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 821838] New: Review Request: gitolite3 - Highly flexible server for git directory version tracker

2012-05-15 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: gitolite3 - Highly flexible server for git directory 
version tracker

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

   Summary: Review Request: gitolite3 - Highly flexible server for
git directory version tracker
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: limburg...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: Bug
Regression: ---
Mount Type: ---
 Documentation: ---


Description:

Gitolite allows a server to host many git repositories and provide access
to many developers, without having to give them real userids on the server.
The essential magic in doing this is ssh's pubkey access and the authorized
keys file, and the inspiration was an older program called gitosis.

Gitolite can restrict who can read from (clone/fetch) or write to (push) a  
repository. It can also restrict who can push to what branch or tag, which
is very important in a corporate environment. Gitolite can be installed
without requiring root permissions, and with no additional software than git
itself and perl. It also has several other neat features described below and
elsewhere in the doc/ directory.

SPEC: http://fedorapeople.org/~limb/review/gitolite3/gitolite3.spec
SRPM:
http://fedorapeople.org/~limb/review/gitolite3/gitolite3-3.01-1.fc16.src.rpm

Does not Obsolete gitolite because gitolite 2.x will be maintained for some
time, and this facilitates careful migration to 3.x.

-- 
Configure 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 808769] Review Request: jaxb2-common-basics - JAXB2 Basics

2012-05-15 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=808769

--- Comment #3 from gil cattaneo  2012-05-15 10:42:01 EDT 
---
tested on: http://koji.fedoraproject.org/koji/taskinfo?taskID=4078625

-- 
Configure 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 739016] Review Request: erlang-poolboy - A hunky Erlang worker pool factory

2012-05-15 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=739016

Peter Lemenkov  changed:

   What|Removed |Added

  Status Whiteboard|NotReady|

--- Comment #1 from Peter Lemenkov  2012-05-15 10:32:16 EDT 
---
Cleaned up NotReady and updated up to 0.7.0

http://peter.fedorapeople.org/erlang-poolboy.spec
http://peter.fedorapeople.org/erlang-poolboy-0.7.0-1.fc18.src.rpm

Koji scratchbuild for F-18:
http://koji.fedoraproject.org/koji/taskinfo?taskID=4078604

rpmlint:
sulaco ~/rpmbuild/SPECS: rpmlint
../RPMS/ppc/erlang-poolboy-0.7.0-1.fc18.ppc.rpm
../SRPMS/erlang-poolboy-0.7.0-1.fc18.src.rpm 
erlang-poolboy.ppc: E: no-binary
erlang-poolboy.ppc: W: only-non-binary-in-usr-lib

^^^ this one is tricky. All erlang packages must be installed into
%{_libdir}/erlang/lib so despite of the fact that some of them contains only
arch-independent data they all must be build as arch-dependent. I plan to fix
than but I wouldn't hold my breath.

erlang-poolboy.src: W: invalid-url Source0:
devinus-poolboy-0.7.0-0-g6aab8b8.tar.gz

^^^ blame github for that, not me.

2 packages and 0 specfiles checked; 1 errors, 2 warnings.
sulaco ~/rpmbuild/SPECS:

-- 
Configure 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 821802] New: Review Request: erlang-erlando - A set of syntax extensions for Erlang

2012-05-15 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-erlando - A set of syntax extensions for Erlang

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

   Summary: Review Request: erlang-erlando - A set of syntax
extensions 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,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://peter.fedorapeople.org/erlang-erlando.spec
SRPM URL: http://peter.fedorapeople.org/erlang-erlando-2.8.2-1.fc18.src.rpm
Description: A set of syntax extensions for Erlang.

Koji scratchbuild for F-18:
http://koji.fedoraproject.org/koji/taskinfo?taskID=4078450

sulaco ~/rpmbuild/SPECS: rpmlint
../RPMS/ppc/erlang-erlando-2.8.2-1.fc18.ppc.rpm
../SRPMS/erlang-erlando-2.8.2-1.fc18.src.rpm 
erlang-erlando.ppc: E: explicit-lib-dependency erlang-stdlib

^^^ this one is a false positive (stdlib as a trigger)

erlang-erlando.ppc: E: no-binary
erlang-erlando.ppc: W: only-non-binary-in-usr-lib

^^^ this one is tricky. All erlang packages must be installed into
%{_libdir}/erlang/lib so despite of the fact that some of them contains only
arch-independent data they all must be build as arch-dependent. I plan to fix
than but I wouldn't hold my breath.

erlang-erlando.src: W: strange-permission rebar 0764L

^^^ minor issue. I'll report upstream.

erlang-erlando.src: W: invalid-url Source0:
rabbitmq-erlando-rabbitmq_v2_8_2-0-g002ce5e.tar.gz

^^^ blame github for that, not me.

2 packages and 0 specfiles checked; 2 errors, 3 warnings.
sulaco ~/rpmbuild/SPECS: 


This app is growing popularity among Erlang developers quite fast (I'm using it
as well).

-- 
Configure 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 821771] Review Request: erlang-edown - EDoc extension for generating Github-flavored Markdown

2012-05-15 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=821771

--- Comment #1 from Peter Lemenkov  2012-05-15 09:59:03 EDT 
---
rpmlint report:

sulaco ~/rpmbuild/SPECS: rpmlint ../RPMS/ppc/erlang-edown-0.2.4-1.fc18.ppc.rpm
../SRPMS/erlang-edown-0.2.4-1.fc18.src.rpm 
erlang-edown.ppc: E: explicit-lib-dependency erlang-stdlib

^^^ this one is a false positive (stdlib as a trigger)

erlang-edown.ppc: W: spelling-error Summary(en_US) EDoc -> E Doc, Doc, Educ
erlang-edown.ppc: W: spelling-error %description -l en_US EDoc -> E Doc, Doc,
Educ

^^^ likewise

erlang-edown.ppc: E: no-binary
erlang-edown.ppc: W: only-non-binary-in-usr-lib

^^^ this one is tricky. All erlang packages must be installed into
%{_libdir}/erlang/lib so despite of the fact that some of them contains only
arch-independent data they all must be build as arch-dependent. I plan to fix
than but I wouldn't hold my breath.

erlang-edown.src: W: spelling-error Summary(en_US) EDoc -> E Doc, Doc, Educ
erlang-edown.src: W: spelling-error %description -l en_US EDoc -> E Doc, Doc,
Educ

^^^ false positives 

erlang-edown.src: W: invalid-url Source0: esl-edown-v0.2.4-0-gdbdd41e.tar.gz

^^^ blame github for that, not me.

2 packages and 0 specfiles checked; 2 errors, 6 warnings.
sulaco ~/rpmbuild/SPECS:

-- 
Configure 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 821771] New: Review Request: erlang-edown - EDoc extension for generating Github-flavored Markdown

2012-05-15 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-edown - EDoc extension for generating 
Github-flavored Markdown

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

   Summary: Review Request: erlang-edown - EDoc extension for
generating Github-flavored Markdown
   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,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://peter.fedorapeople.org/erlang-edown.spec
SRPM URL: http://peter.fedorapeople.org/erlang-edown-0.2.4-1.fc18.src.rpm
Description: EDoc extension for generating Github-flavored Markdown.

Koji scratchbuild for F-18:
http://koji.fedoraproject.org/koji/taskinfo?taskID=4078397

THis is one of the requirements for updated erlang-gproc, which is also a
requirement for numerous other recently updated Erlang packages.

-- 
Configure 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 808768] Review Request: annox - Java annotations in XML resources

2012-05-15 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=808768

--- Comment #9 from Fedora Update System  2012-05-15 
09:44:58 EDT ---
annox-0.5.0-2.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/annox-0.5.0-2.fc17

-- 
Configure 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 820543] Review Request: libguac-client-vnc - VNC support for guacd

2012-05-15 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=820543

Bug 820543 depends on bug 820542, which changed state.

Bug 820542 Summary: Review Request: libguac - The common library used by all C 
components of Guacamole
https://bugzilla.redhat.com/show_bug.cgi?id=820542

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE

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

[Bug 820544] Review Request: libguac-client-rdp - RDP support for guacd

2012-05-15 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=820544

Bug 820544 depends on bug 820542, which changed state.

Bug 820542 Summary: Review Request: libguac - The common library used by all C 
components of Guacamole
https://bugzilla.redhat.com/show_bug.cgi?id=820542

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE

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

[Bug 820561] Review Request: guacd - Proxy daemon for Guacamole

2012-05-15 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=820561

Bug 820561 depends on bug 820542, which changed state.

Bug 820542 Summary: Review Request: libguac - The common library used by all C 
components of Guacamole
https://bugzilla.redhat.com/show_bug.cgi?id=820542

   What|Old Value   |New Value

 Resolution||NEXTRELEASE
 Status|ASSIGNED|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 820542] Review Request: libguac - The common library used by all C components of Guacamole

2012-05-15 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=820542

Simone Caronni  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE
Last Closed||2012-05-15 09:43:24

-- 
Configure 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 808768] Review Request: annox - Java annotations in XML resources

2012-05-15 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=808768

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 820970] Review Request: perl-Task-Moose - Task::Moose install Moose and Moose extensions

2012-05-15 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=820970

Jitka Plesnikova  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2012-05-15 09:33:12

--- Comment #6 from Jitka Plesnikova  2012-05-15 09:33:12 
EDT ---
Thank you for the review and the repository.

-- 
Configure 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 820660] Review Request: FEniCS - tracker bug

2012-05-15 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=820660

Jonathan Underwood  changed:

   What|Removed |Added

 Depends on||807731

--- Comment #5 from Jonathan Underwood  
2012-05-15 09:29:25 EDT ---
EPEL6 also blocked by needing a later Swig package. BZ#807731

-- 
Configure 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 821732] New: Review Request: scantailor - recovering retired package

2012-05-15 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: scantailor - recovering retired package

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

   Summary: Review Request: scantailor - recovering retired
package
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: jho...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://xhorak.fedorapeople.org/scantailor.spec
SRPM URL: http://xhorak.fedorapeople.org/scantailor-0.9.11-1.fc16.src.rpm
Description: 
Scan Tailor is an interactive post-processing tool for scanned pages.
It performs operations such as page splitting, deskewing, adding/removing
borders, and others. You give it raw scans, and you get pages ready to be
printed or assembled into a PDF or DJVU file. Scanning, optical character
recognition, and assembling multi-page documents are out of scope of this
project.

NOTE: this is actually recover of retired package. I was the maintainer before.
I'm just following procedure from:
https://fedoraproject.org/wiki/Orphaned_package_that_need_new_maintainers
This package works just fine on f15 (see
http://koji.fedoraproject.org/koji/buildinfo?buildID=302121 ) so review
shouldn't be much complicated.

-- 
Configure 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 821727] Review Request: dolfin - the C++/Python interface of FEniCS

2012-05-15 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=821727

Jonathan Underwood  changed:

   What|Removed |Added

 Depends on||820659(python-ufc),
   ||693158(python-viper),
   ||693137(python-ffc),
   ||799702(python-ufl),
   ||483496(python-instant),
   ||483501(python-fiat)

-- 
Configure 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 820659] Review Request: python-ufc - Unified Form-assembly Code

2012-05-15 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=820659

Jonathan Underwood  changed:

   What|Removed |Added

 Blocks||821727(dolfin)

-- 
Configure 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 799702] Review Request: python-ufl - A compiler for finite element variational forms

2012-05-15 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=799702

Jonathan Underwood  changed:

   What|Removed |Added

 Blocks||821727(dolfin)

-- 
Configure 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 821727] Review Request: dolfin - the C++/Python interface of FEniCS

2012-05-15 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=821727

Jonathan Underwood  changed:

   What|Removed |Added

 Blocks||820660(FEniCS)

-- 
Configure 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 820660] Review Request: FEniCS - tracker bug

2012-05-15 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=820660

Jonathan Underwood  changed:

   What|Removed |Added

 Depends on||821727(dolfin)

-- 
Configure 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 821727] New: Review Request: dolfin - the C++/Python interface of FEniCS

2012-05-15 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: dolfin - the C++/Python interface of FEniCS

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

   Summary: Review Request: dolfin - the C++/Python interface of
FEniCS
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: jonathan.underw...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://jgu.fedorapeople.org/dolfin.spec
SRPM URL: http://jgu.fedorapeople.org/dolfin-1.0.0-0.1.el6.src.rpm
Description: DOLFIN is the C++/Python interface of FEniCS, providing a
consistent
PSE (Problem Solving Environment) for ordinary and partial differential
equations.

-- 
Configure 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 821727] Review Request: dolfin - the C++/Python interface of FEniCS

2012-05-15 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=821727

Jonathan Underwood  changed:

   What|Removed |Added

  Alias||dolfin

-- 
Configure 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 808768] Review Request: annox - Java annotations in XML resources

2012-05-15 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=808768

--- Comment #8 from Jon Ciesla  2012-05-15 09:03:33 EDT 
---
Git done (by process-git-requests).

-- 
Configure 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 808768] Review Request: annox - Java annotations in XML resources

2012-05-15 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=808768

gil cattaneo  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #7 from gil cattaneo  2012-05-15 08:48:11 EDT 
---
New Package SCM Request
===
Package Name: annox
Short Description: Java annotations in XML resources
Owners: gil
Branches: f16 f17
InitialCC: java-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 808768] Review Request: annox - Java annotations in XML resources

2012-05-15 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=808768

Mikolaj Izdebski  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #6 from Mikolaj Izdebski  2012-05-15 08:43:54 
EDT ---
Fixed issue:

[x]: MUST Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.

Checked now:

[x]: MUST Sources used to build the package match the upstream source, as
 provided in the spec URL.

Some minor problems not blocking the package:

[!]: SHOULD %check is present and all tests pass.

Tests are skipped, but a justification is given.

[!]: SHOULD Latest version is packaged.

A newer upstream version is available.

[!]: SHOULD If the source package does not include license text(s) as a
 separate file from upstream, the packager SHOULD query upstream to
 include it.

The maintainer hasn't asked upstream to include the license text.


Because the remaining issues are not very important, annox is:


*** APPROVED ***


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

[Bug 821511] Review Request: hornetq - High performance messaging system

2012-05-15 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=821511

--- Comment #1 from Marek Goldmann  2012-05-15 08:24:20 
EDT ---
Reverting to version 2.2.13, as the previous one (2.2.16) is NOT compatible
with AS7 (7.1.1.Final).

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

Spec URL:
http://goldmann.fedorapeople.org/package_review/hornetq/2.2.13-1/hornetq.spec
SRPM URL:
http://goldmann.fedorapeople.org/package_review/hornetq/2.2.13-1/hornetq-2.2.13-1.fc17.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 808768] Review Request: annox - Java annotations in XML resources

2012-05-15 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=808768

--- Comment #5 from gil cattaneo  2012-05-15 08:23:04 EDT 
---
Spec URL: http://gil.fedorapeople.org/annox/annox.spec
SRPM URL: http://gil.fedorapeople.org/annox/annox-0.5.0-2.fc16.src.rpm

tested on : http://koji.fedoraproject.org/koji/taskinfo?taskID=4078187

-- 
Configure 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 820845] Review Request: python-django-dajax - Library to create asynchronous presentation logic with Django and dajaxice

2012-05-15 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=820845

--- Comment #15 from Jon Ciesla  2012-05-15 08:18:37 EDT 
---
Git done (by process-git-requests).

-- 
Configure 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 820841] Review Request: python-django-dajaxice - Django AJAX library

2012-05-15 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=820841

--- Comment #15 from Jon Ciesla  2012-05-15 08:18:19 EDT 
---
Git done (by process-git-requests).

-- 
Configure 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 820970] Review Request: perl-Task-Moose - Task::Moose install Moose and Moose extensions

2012-05-15 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=820970

--- Comment #5 from Jon Ciesla  2012-05-15 08:19:05 EDT 
---
Git done (by process-git-requests).

-- 
Configure 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 815583] Review Request: ghc-edit-distance - Levenshtein and restricted Damerau-Levenshtein edit distances

2012-05-15 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=815583

--- Comment #9 from Jon Ciesla  2012-05-15 08:16:12 EDT 
---
Git done (by process-git-requests).

-- 
Configure 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 820542] Review Request: libguac - The common library used by all C components of Guacamole

2012-05-15 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=820542

--- Comment #14 from Jon Ciesla  2012-05-15 08:17:44 EDT 
---
Git done (by process-git-requests).

-- 
Configure 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 817984] Review Request: ghc-zlib-conduit - Conduits for (de)compression

2012-05-15 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=817984

--- Comment #4 from Jon Ciesla  2012-05-15 08:17:11 EDT 
---
Git done (by process-git-requests).

-- 
Configure 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 821712] New: Review Request: apache-commons-vfs - Commons Virtual File System

2012-05-15 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-vfs - Commons Virtual File System

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

   Summary: Review Request: apache-commons-vfs - Commons Virtual
File System
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: punto...@libero.it
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://gil.fedorapeople.org/apache-commons-vfs.spec
SRPM URL: http://gil.fedorapeople.org/apache-commons-vfs-2.0-1.fc16.src.rpm
Description: Commons VFS provides a single API for accessing various
different file systems. It presents a uniform view of the
files from various different sources, such as the files on
local disk, on an HTTP server, or inside a Zip archive.
Some of the features of Commons VFS are:
* A single consistent API for accessing files of different
 types.
* Support for numerous file system types.
* Caching of file information. Caches information in-JVM,
 and optionally can cache remote file information on the
 local file system.
* Event delivery.
* Support for logical file systems made up of files from
 various different file systems.
* Utilities for integrating Commons VFS into applications,
 such as a VFS-aware ClassLoader and URLStreamHandlerFactory.
* A set of VFS-enabled Ant tasks.

Tested on: http://koji.fedoraproject.org/koji/taskinfo?taskID=4078170

-- 
Configure 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 674008] Review Request: openrave - Open Robotics Automation Virtual Environment

2012-05-15 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=674008

--- Comment #91 from Tim Niemueller  2012-05-15 07:08:37 
EDT ---
(In reply to comment #87)
> hi tim,
> 
> unfortunately compatibility for sympy 0.7.1 is not that easy. like promised
> i'll be looking into compatibility for it shortly and within the week it 
> should
> be in trunk. but keep in mind it will be officially released in openrave 
> 0.8.0.
> for now, just package sympy 0.6.x, i'm sure fedora core supports offering rpms
> with different versions of libraries as long as they are named differently.
> it's not worth breaking openrave functionality.

Nope, it ain't that easy and it would mean I need to file another review
request. I have already done almost half a dozen for OpenRAVE, so I'd rather
wait for sympy 0.7 compatibility.

> also, just did a hot fix for openrave 0.6.4 for "openrave-core.pc.in", i 
> highly
> suggest you repackage the srpm with r3204 of
> 
> https://openrave.svn.sourceforge.net/svnroot/openrave/tags/0.6.4

I'll have a look.

> as for libfparser, the priority on it is really low. by the time openrave 
> 0.8.0
> releases, most likely fparser will have reached a different version ;0)

Well, for now I'm happy with the old fparser version. Let me know when I can
upgrade.

-- 
Configure 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 674008] Review Request: openrave - Open Robotics Automation Virtual Environment

2012-05-15 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=674008

--- Comment #92 from Tim Niemueller  2012-05-15 07:12:14 
EDT ---
> can you give us the current output and expected output? by openrave tools, do
> you mean the openrave-config program?

The output always includes "-L/usr/lib" which is wrong on x86_64. Also,
historically *-config had been used to know which flags were used to compile
that particular tool and not what flags should be used to link against it. That
has arguably change over the recent years, but I still prefer pkg-config
whenever available.

(In reply to comment #90)
> you should be able to do:
> 
> const Vector& trans;
> __env->plot3(&trans[0], ...)

Right, that'll be faster. I'll change it. Thanks for catching this. Is there a
specific reason why there is no convenience (overridden) method to directly
pass the transform?

-- 
Configure 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 808768] Review Request: annox - Java annotations in XML resources

2012-05-15 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=808768

--- Comment #4 from Mikolaj Izdebski  2012-05-15 07:00:54 
EDT ---
My question about the licensing situation on Fedora-legal-list:
http://lists.fedoraproject.org/pipermail/legal/2012-May/001907.html
and the answer:
http://lists.fedoraproject.org/pipermail/legal/2012-May/001908.html

I'm waiting for gil to take an appropriate action.

gil: The spec file you sent to me looked fine. Post here the updated SRPM and
spec please.

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

[Bug 794783] Review Request: xorg-x11-drv-modesetting - X.org modesetting fallback driver

2012-05-15 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=794783

Peter Lemenkov  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE
Last Closed||2012-05-15 07:02:26

--- Comment #12 from Peter Lemenkov  2012-05-15 07:02:26 
EDT ---
Ok, I'm closing this. I don't think that build failure on F-15 is relevant.

-- 
Configure 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 821247] Review Request: fcitx-table-other - Other tables for Fcitx

2012-05-15 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=821247

Parag AN(पराग)  changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |fcitx-table-other - Oher|fcitx-table-other - Other
   |tables for Fcitx|tables for Fcitx

-- 
Configure 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 814924] Review Request: jalv - a simple LV2 host

2012-05-15 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=814924

--- Comment #6 from Fedora Update System  2012-05-15 
05:54:22 EDT ---
jalv-1.0.0-2.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/jalv-1.0.0-2.fc17

-- 
Configure 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 814924] Review Request: jalv - a simple LV2 host

2012-05-15 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=814924

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 805587] Review Request: jquery - The jQuery JavaScript Library

2012-05-15 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=805587

Karel Klíč  changed:

   What|Removed |Added

 CC||kk...@redhat.com

--- Comment #3 from Karel Klíč  2012-05-15 05:39:50 EDT ---
How can I help to move this review forward?


Here is a jQuery package built from upstream source tree:
http://kklic.fedorapeople.org/jquery.spec
http://kklic.fedorapeople.org/jquery-1.7.2-1.fc17.src.rpm

The build does not contain a minified version.  I'd wait until nodejs is
packaged into Fedora and then start shipping the minified version the same way
as upstream does.

-- 
Configure 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   >