[Bug 852892] Review Request: ddccontrol-db - DDC/CI control database for ddccontrol

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=852892

Jan Vcelak  changed:

   What|Removed |Added

  Flags||fedora-review+

--- Comment #7 from Jan Vcelak  ---
All issues are fixed. Thank you.

fedora-review+

-- 
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 854176] Review Request: python-django-admin-honeypot - A fake Django admin login screen to notify admins of attempted unauthorized access

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=854176

Matthias Runge  changed:

   What|Removed |Added

  Flags|fedora-review?  |
  Flags||fedora-review+

--- Comment #25 from Matthias Runge  ---
Package Review
==

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



 Generic 
[x]: EXTRA Rpmlint is run on all installed packages.
 Note: There are rpmlint messages (see attachment).
[x]: EXTRA Spec file according to URL is the same as in SRPM.
[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.
[x]: 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 Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[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
[x]: MUST Macros in Summary, %description expandable at SRPM build time.
[-]: MUST Package contains desktop file if it is a GUI application.
[-]: MUST Development files must be in a -devel package
[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 Package complies to the Packaging Guidelines
[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.
[!]: MUST License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. No licenses
 found. Please check the source files for licenses manually.
[x]: MUST Package consistently uses macro is (instead of hard-coded directory
 names).
[x]: MUST Package is named using only allowed ascii characters.
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generate any conflict.
 Note: Package contains no Conflicts: tag(s)
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[x]: MUST If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[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 Package is not relocatable.
[x]: MUST Requires correct, justified where necessary.
[x]: MUST Rpmlint is run on all rpms the build produces.
 Note: No rpmlint messages.
[x]: MUST Sources used to build the package match the upstream source, as
 provided in the spec URL.
[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 systemd file(s) if in need.
[x]: MUST File names are valid UTF-8.
[x]: SHOULD Reviewer should test that the package builds in mock.
[x]: SHOULD Buildroot is not present
 Note: Unless packager wants to package for EPEL5 this is fine
[x]: SHOULD Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
 Note: Clean would be needed if support for EPEL5 is required
[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).
[?]: 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 Patches link to upstream bugs/comments/lists or are otherwise
 justified.
[x]: SHOULD SourceX tarball generation or download is documented.
[!]: SHOULD SourceX / PatchY prefixed with %{name}.
 Note: Patch1

[Bug 849545] Review Request: perl-Algorithm-NaiveBayes - Bayesian prediction of categories

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=849545

--- Comment #4 from Mathieu Bridon  ---
Thanks for the comments, this should address both issues.

Spec URL:
http://bochecha.fedorapeople.org/packages/perl-Algorithm-NaiveBayes.spec
SRPM URL:
http://bochecha.fedorapeople.org/packages/perl-Algorithm-NaiveBayes-0.04-2.fc18.src.rpm

-- 
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 854176] Review Request: python-django-admin-honeypot - A fake Django admin login screen to notify admins of attempted unauthorized access

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=854176

--- Comment #24 from Eduardo Echeverria  ---
Created attachment 611658
  --> https://bugzilla.redhat.com/attachment.cgi?id=611658&action=edit
Fix runtime requirements for rawhide

With Patch 0001-change-setup.py-requires-to-fix.patch
## Koji Build F18 ##
http://koji.fedoraproject.org/koji/taskinfo?taskID=4462885

With Patch 0002-change-setup.py-requires-to-fix.patch
## Koji Build Rawhide ##
http://koji.fedoraproject.org/koji/taskinfo?taskID=4473702

-- 
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 854176] Review Request: python-django-admin-honeypot - A fake Django admin login screen to notify admins of attempted unauthorized access

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=854176

--- Comment #23 from Eduardo Echeverria  ---
With pep8 1.3 in f18
Spec URL:
http://www.saef.com.ve/fedorarpm/python-django-admin-honeypot/python-django-admin-honeypot.spec
SRPM URL:
http://www.saef.com.ve/fedorarpm/python-django-admin-honeypot/python-django-admin-honeypot-0.2.3-4.fc17.src.rpm

With the pep8 1.3.3 in rawhide
http://echevemaster.fedorapeople.org/python-django-admin-honeypot/5/python-django-admin-honeypot-0.2.3-5.fc17.src.rpm
http://echevemaster.fedorapeople.org/python-django-admin-honeypot/5/python-django-admin-honeypot.spec

## Koji Build F18 ##
http://koji.fedoraproject.org/koji/taskinfo?taskID=4462885


## Koji Build Rawhide ##
http://koji.fedoraproject.org/koji/taskinfo?taskID=4473702

-- 
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 854837] inkscape-sozi - Inkscape extension for creating animated presentations

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=854837

--- Comment #17 from Eduardo Echeverria  ---
http://echevemaster.fedorapeople.org/inkscape-sozi/7/inkscape-sozi-12.05-7.fc17.src.rpm
http://echevemaster.fedorapeople.org/inkscape-sozi/7/inkscape-sozi.spec

$ rpmlint -i inkscape-sozi-12.05-7.fc17.noarch.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings

$ rpmlint -i inkscape-sozi-12.05-7.fc17.src.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

-- 
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 853463] Review Request: php-redis - Extension for communicating with the Redis key-value store

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=853463

Remi Collet  changed:

   What|Removed |Added

  Flags|needinfo?(fedora@famillecol |
   |let.com)|

--- Comment #5 from Remi Collet  ---
(In reply to comment #4)
> A couple questions before approval:
> 1) Is there a reason you used "%{_sysconfdir}/php.d" instead of
> "%{php_inidir}"?

%{php_inidir} is only define in Fedora (php 5.4), not in RHEL (php 5.3)

> 2) Is there a reason you left arrays.markdown out of the documentation (it
> looks like it may be useful to end users)

good catch (as for serialize.list)


https://github.com/remicollet/remirepo/commit/9010319213503293e6ff213d286baa4a6d0ddf27

New SRPM:
http://rpms.famillecollet.com/SRPMS/php-redis-2.2.2-5.git6f7087f.remi.src.rpm

-- 
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 854837] inkscape-sozi - Inkscape extension for creating animated presentations

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=854837

--- Comment #16 from Eduardo Echeverria  ---
Changes Ready Itamar: 

http://echevemaster.fedorapeople.org/inkscape-sozi/inkscape-sozi-12.05-6.fc17.src.rpm
http://echevemaster.fedorapeople.org/inkscape-sozi/inkscape-sozi.spec

$ rpmlint -i inkscape-sozi-12.05-6.fc17.noarch.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings

$ rpmlint -i inkscape-sozi-12.05-6.fc17.src.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

-- 
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 849618] Review Request: perl-AI-DecisionTree - Automatically Learns Decision Trees

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=849618

Mathieu Bridon  changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #5 from Mathieu Bridon  ---
New Package SCM Request
===
Package Name: perl-AI-DecisionTree
Short Description: Automatically Learns Decision Trees
Owners: bochecha
Branches: devel
InitialCC: perl-sig

-- 
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 855283] Review Request: pass - A unix password manager using standard tools

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855283

--- Comment #30 from Jason A. Donenfeld  ---
In response to some requests from some Fedora devs, version 1.2 has been
released, before QA on 1.1.4 has even completed. There are no build changes or
any other changes that will change the nature of the packaging, so a simple
version bump should do the trick.

-- 
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 856026] Review Request: kahadb - A file based persistence database

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856026

Matt Spaulding  changed:

   What|Removed |Added

 Blocks||856027

-- 
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 856025] Review Request: activemq-jaas - ActiveMQ Jaas

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856025

Matt Spaulding  changed:

   What|Removed |Added

 Blocks||856027

-- 
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 856024] Review Request: activemq-parent - ActiveMQ Parent POM

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856024

Matt Spaulding  changed:

   What|Removed |Added

 Blocks||856027

-- 
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 856027] Review Request: activemq-core - The most popular and powerful open source messaging and Integration Patterns server

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856027

Matt Spaulding  changed:

   What|Removed |Added

 Blocks||652183 (FE-JAVASIG)
 Depends On||856024, 856025, 856026

-- 
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 856027] New: Review Request: activemq-core - The most popular and powerful open source messaging and Integration Patterns server

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856027

Bug ID: 856027
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: activemq-core - The most popular and
powerful open source messaging and Integration
Patterns server
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: mspauldin...@gmail.com
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://madsa.fedorapeople.org/activemq-core.spec
SRPM URL: http://madsa.fedorapeople.org/activemq-core-5.6.0-1.fc18.src.rpm
Description:
The most popular and powerful open source messaging and Integration Patterns
server.
Fedora Account System Username: madsa

-- 
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 856024] Review Request: activemq-parent - ActiveMQ Parent POM

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856024

Matt Spaulding  changed:

   What|Removed |Added

 Blocks||856026

-- 
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 856026] Review Request: kahadb - A file based persistence database

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856026

Matt Spaulding  changed:

   What|Removed |Added

 Blocks||652183 (FE-JAVASIG)
 Depends On||856024

-- 
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 856026] New: Review Request: kahadb - A file based persistence database

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856026

Bug ID: 856026
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: kahadb - A file based persistence
database
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: mspauldin...@gmail.com
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://madsa.fedorapeople.org/kahadb.spec
SRPM URL: http://madsa.fedorapeople.org/kahadb-5.6.0-1.fc18.src.rpm
Description:
A file based persistence database that is local to the message broker that
is using it. It has been optimised for fast persistence and is the the default
storage mechanism from ActiveMQ 5.4 onwards. KahaDB uses less file descriptors
and provides faster recovery than its predecessor, the AMQ Message Store.

Fedora Account System Username: madsa

-- 
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 856024] Review Request: activemq-parent - ActiveMQ Parent POM

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856024

Matt Spaulding  changed:

   What|Removed |Added

 Blocks||652183 (FE-JAVASIG)

-- 
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 856024] Review Request: activemq-parent - ActiveMQ Parent POM

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856024

Matt Spaulding  changed:

   What|Removed |Added

 Blocks||856025

-- 
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 856025] Review Request: activemq-jaas - ActiveMQ Jaas

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856025

Matt Spaulding  changed:

   What|Removed |Added

 Blocks||652183 (FE-JAVASIG)
 Depends On||856024

-- 
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 856025] New: Review Request: activemq-jaas - ActiveMQ Jaas

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856025

Bug ID: 856025
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: activemq-jaas - ActiveMQ Jaas
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: mspauldin...@gmail.com
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://madsa.fedorapeople.org/activemq-jaas.spec
SRPM URL: http://madsa.fedorapeople.org/activemq-jaas-5.6.0-1.fc18.src.rpm
Description: ActiveMQ Jaas
Fedora Account System Username: madsa

-- 
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 856024] New: Review Request: activemq-parent - ActiveMQ Parent POM

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856024

Bug ID: 856024
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: activemq-parent - ActiveMQ Parent POM
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: mspauldin...@gmail.com
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://madsa.fedorapeople.org/activemq-parent.spec
SRPM URL: http://madsa.fedorapeople.org/activemq-parent-5.6.0-1.fc18.src.rpm
Description: ActiveMQ Parent POM
Fedora Account System Username: madsa

-- 
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 854837] inkscape-sozi - Inkscape extension for creating animated presentations

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=854837

--- Comment #15 from Itamar Reis Peixoto  ---
Eduardo, please look at koji build logs.

warning: File listed twice: /usr/share/inkscape/extensions/sozi.css
...
...
warning: File listed twice: /usr/share/inkscape/extensions/sozi_upgrade.pyo

adding the flowing line into %files section.

%{_datadir}/inkscape/extensions/sozi*

will include all files starting with sozi.

when you list the files starting with sozi again, gives the warning.

%attr(644,root,root) %{_datadir}/inkscape/extensions/sozi_extras_upgrade.inx*
%attr(644,root,root) %{_datadir}/inkscape/extensions/sozi_extras_addvideo.js*
%attr(644,root,root) %{_datadir}/inkscape/extensions/sozi.inx*
%attr(644,root,root)
%{_datadir}/inkscape/extensions/sozi_extras_addvideo_upgrade.py*
%attr(644,root,root) %{_datadir}/inkscape/extensions/sozi.css*
%attr(644,root,root) %{_datadir}/inkscape/extensions/sozi_extras_addvideo.inx*
%attr(644,root,root) %{_datadir}/inkscape/extensions/sozi_upgrade.py*
%attr(644,root,root) %{_datadir}/inkscape/extensions/sozi.js*

look -> 

http://fedoraproject.org/wiki/How_to_create_an_RPM_package#.25files_section

%defattr(, , , )


what do you think about cleaning your %files section to something like this ->

%files
%defattr(0644,root,root,-)
%doc install-en.html GPL-license.txt install-fr.html MIT-license.txt
%{_datadir}/inkscape/extensions/sozi*

I think will produce same result, with installed files set as 0644, and no
warning about files listed twice in build log.

-- 
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 855588] Review Request: ghc-concrete-typerep - Provides Binary and Hashable instances for TypeRep.

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855588

--- Comment #1 from Jens Petersen  ---
(In reply to comment #0)
> For F18 and greater, we need to add configure flags to cabal. Link to spec
> file for f18 
> http://narasim.fedorapeople.org/package_reviews/ghc-concrete-typerep-f18.spec

Then please make it conditional.  One spec file should be able to build
for both f18 and earlier.

-- 
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 855561] Review Request: mediawiki119-RSS - Displays one or more RSS feeds on a wiki page

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855561

--- Comment #11 from Jon Ciesla  ---
Git done (by process-git-requests).

-- 
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 854062] Review Request: mediawiki119-intersection - Create a list of pages that are listed in a set of categories

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=854062

--- Comment #9 from Jon Ciesla  ---
Git done (by process-git-requests).

-- 
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 854176] Review Request: python-django-admin-honeypot - A fake Django admin login screen to notify admins of attempted unauthorized access

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=854176

--- Comment #22 from Eduardo Echeverria  ---
(In reply to comment #21)
> Eduardo, it looks like, it still requires pep-1.3.3 to build. I haven't
> looked into your patch, yet. I guess, that can be fixed really fast.

Matthias the package does not build on f17, basically what I did was follow
your tip:
- yum --enablerepo=rawhide install python-pep8
- Make the patch to change pep 1.3.3 to 1.3 (available in f18)
- Incorporate test suite
(https://github.com/dmpayton/django-admin-honeypot/tree/master/tests)
- rpmbuild

Today will incorporate the f19 , pep8 1.3.3 is in rawhide, BTW thanks Ian

-- 
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 853463] Review Request: php-redis - Extension for communicating with the Redis key-value store

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=853463

Shawn Iwinski  changed:

   What|Removed |Added

 CC||shawn.iwin...@gmail.com
  Flags||needinfo?(fedora@famillecol
   ||let.com)

--- Comment #4 from Shawn Iwinski  ---
A couple questions before approval:
1) Is there a reason you used "%{_sysconfdir}/php.d" instead of
"%{php_inidir}"?
2) Is there a reason you left arrays.markdown out of the documentation (it
looks like it may be useful to end users)

-- 
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 835170] Review Request: cgdcbxd - dcb network priority manager

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=835170

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |ERRATA
Last Closed||2012-09-10 18:28:24

--- Comment #16 from Fedora Update System  ---
cgdcbxd-1.0.1-1.fc17 has been pushed to the Fedora 17 stable repository.

-- 
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 851583] Review Request: gnome-abrt - System problems browser

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=851583

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #9 from Fedora Update System  ---
gnome-abrt-0.2-9.fc17 has been pushed to the Fedora 17 testing repository.

-- 
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 856002] New: Review Request: plug - Linux software for Fender Mustang amplifiers

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856002

Bug ID: 856002
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: plug - Linux software for Fender
Mustang amplifiers
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: jonathan.underw...@gmail.com
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://fedorapeople.org/~jgu/plug.spec
SRPM URL: http://fedorapeople.org/~jgu/plug-1.1-3.fc17.src.rpm
Description: Linux replacement for Fender FUSE software for Mustang amps
Fedora Account System Username: jgu

-- 
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 855561] Review Request: mediawiki119-RSS - Displays one or more RSS feeds on a wiki page

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855561

Patrick Uiterwijk  changed:

   What|Removed |Added

  Flags|fedora-cvs+ |
  Flags||fedora-cvs?

--- Comment #10 from Patrick Uiterwijk  ---
Package Change Request
==
Package Name: mediawiki119-RSS
New Branches: f18
Owners: puiterwijk

It has been decided to make mediawiki119* available on f18 too.

-- 
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 854062] Review Request: mediawiki119-intersection - Create a list of pages that are listed in a set of categories

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=854062

Patrick Uiterwijk  changed:

   What|Removed |Added

  Flags|fedora-cvs+ |
  Flags||fedora-cvs?

--- Comment #8 from Patrick Uiterwijk  ---
Package Change Request
==
Package Name: mediawiki119-intersection
New Branches: f18
Owners: puiterwijk

It has been decided to make mediawiki119* available on f18 too.

-- 
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 855561] Review Request: mediawiki119-RSS - Displays one or more RSS feeds on a wiki page

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855561

--- Comment #9 from Fedora Update System  ---
mediawiki119-RSS-2.16-3.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/mediawiki119-RSS-2.16-3.el6

-- 
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 855561] Review Request: mediawiki119-RSS - Displays one or more RSS feeds on a wiki page

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855561

Fedora Update System  changed:

   What|Removed |Added

 Status|NEW |MODIFIED

-- 
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 854764] Review Request:openshift-origin-port-proxy - Script to configure HAProxy to do port forwarding for OpenShift Origin

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=854764

--- Comment #5 from Adam Miller  ---
I was referencing a --nobase install via kickstart, a previous employer of mine
did all their installations like that to keep OS installs in the virtualization
farm as minimal as possible. Don't know how realistic that is of a goal to meet
but that was the motivation behind the explicit requires entries.

-- 
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 854764] Review Request:openshift-origin-port-proxy - Script to configure HAProxy to do port forwarding for OpenShift Origin

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=854764

Rob Millner  changed:

   What|Removed |Added

 CC||rmill...@redhat.com

--- Comment #4 from Rob Millner  ---
I believe we can just have one copy of the configuration file and mark it as
%config(noreplace).

The configuration file changes as the set of local gears change and as the
cartridges embedded in a gear change.  

Either location: %{_sysconfdir}/stickshift or
%{_localstatedir}/lib/stickshift/.stickshift-proxy.d is acceptable.  The latter
is a convention also used by our per-gear http configuration.  

Perhaps the best choice is to use %{_sysconfdir}/stickshift.

-- 
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 854176] Review Request: python-django-admin-honeypot - A fake Django admin login screen to notify admins of attempted unauthorized access

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=854176

--- Comment #21 from Matthias Runge  ---
Eduardo, it looks like, it still requires pep-1.3.3 to build. I haven't looked
into your patch, yet. I guess, that can be fixed really fast.

-- 
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 850679] Review Request: rubygem-pdf-reader - Ruby library to parse PDF files

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=850679

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #16 from Fedora Update System  ---
rubygem-pdf-reader-1.1.1-6.el6 has been pushed to the Fedora EPEL 6 testing
repository.

-- 
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 852898] Review Request: CImg - C++ Template Image Processing Toolkit

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=852898

--- Comment #7 from Thibault North  ---
Ralf, were you about to review this package ?
If so, I'll step back from the review and the following can be considered
informal.

MUST: rpmlint must be run on the source rpm and all binary rpms the build
produces. The output should be posted in the review. OK
MUST: The package must be named according to the Package Naming Guidelines OK
MUST: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption. OK
MUST: The package must meet the Packaging Guidelines OK
MUST: The package must be licensed with a Fedora approved license and meet the
Licensing Guidelines OK
MUST: The License field in the package spec file must match the actual license.
OK
MUST: If (and only if) the source package includes the text of the license(s)
in its own file, then that file, containing the text of the license(s) for the
package must be included in %doc. OK
MUST: The spec file must be written in American English. OK
MUST: The spec file for the package MUST be legible. OK
MUST: The sources used to build the package must match the upstream source, as
provided in the spec URL. Reviewers should use sha256sum for this task as it is
used by the sources file once imported into git. If no upstream URL can be
specified for this package, please see the Source URL Guidelines for how to
deal with this. OK
MUST: The package MUST successfully compile and build into binary rpms on at
least one primary architecture. OK
MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line. N/A
MUST: All build dependencies must be listed in BuildRequires, except for any
that are listed in the exceptions section of the Packaging Guidelines ;
inclusion of those as BuildRequires is optional. Apply common sense. OK
MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden. N/A
MUST: Every binary RPM package (or subpackage) which stores shared library
files (not just symlinks) in any of the dynamic linker's default paths, must
call ldconfig in %post and %postun. N/A
MUST: Packages must NOT bundle copies of system libraries. OK
MUST: If the package is designed to be relocatable, the packager must state
this fact in the request for review, along with the rationalization for
relocation of that specific package. Without this, use of Prefix: /usr is
considered a blocker. N/A
MUST: A package must own all directories that it creates. If it does not create
a directory that it uses, then it should require a package which does create
that directory. OK
MUST: A Fedora package must not list a file more than once in the spec file's
%files listings. (Notable exception: license texts in specific situations) OK
MUST: Permissions on files must be set properly. Executables should be set with
executable permissions, for example. OK
MUST: Each package must consistently use macros. OK
MUST: The package must contain code, or permissable content. OK
MUST: Large documentation files must go in a -doc subpackage. (The definition
of large is left up to the packager's best judgement, but is not restricted to
size. Large can refer to either size or quantity). Doc can be considered small
at this point.
MUST: If a package includes something as %doc, it must not affect the runtime
of the application. To summarize: If it is in %doc, the program must run
properly if it is not present. OK
MUST: Static libraries must be in a -static package. N/A
MUST: Development files must be in a -devel package. Headers only, OK
MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency: Requires: %{name}%{?_isa} =
%{version}-%{release} N/A
MUST: Packages must NOT contain any .la libtool archives, these must be removed
in the spec if they are built. OK
MUST: Packages containing GUI applications must include a %{name}.desktop file,
and that file must be properly installed with desktop-file-install in the
%install section. If you feel that your packaged GUI application does not need
a .desktop file, you must put a comment in the spec file with your explanation.
N/A
MUST: Packages must not own files or directories already owned by other
packages. The rule of thumb here is that the first package to be installed
should own the files or directories that other packages may rely upon. This
means, for example, that no package in Fedora should ever share ownership with
any of the files or directories owned by the filesystem or man package. If you
feel that you have a good reason to own a file 

[Bug 854176] Review Request: python-django-admin-honeypot - A fake Django admin login screen to notify admins of attempted unauthorized access

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=854176

--- Comment #20 from Eduardo Echeverria  ---
(In reply to comment #19)
> Great. I'll do a full review later today.
> 
> Please note, I also lifted FE-NEEDSPONSOR.
> 
> Eduardo, please feel free, to contact me as well, if there are problems,
> questions, etc.

Thank you very much for your valuable comments Matthias, I hope your review.

-- 
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 841753] Review Request: ghc-IOSpec - A pure specification of the IO monad

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=841753

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #8 from Fedora Update System  ---
ghc-IOSpec-0.2.5-1.fc18 has been pushed to the Fedora 18 testing repository.

-- 
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 855283] Review Request: pass - A unix password manager using standard tools

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855283

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #29 from Fedora Update System  ---
pass-1.1.4-1.fc18 has been pushed to the Fedora 18 testing repository.

-- 
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 772989] Review Request: globus-gram-job-manager-sge - Globus Toolkit - Grid Engine Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772989

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #15 from Fedora Update System  ---
globus-gram-job-manager-sge-1.5-1.fc18, globus-gram-job-manager-pbs-1.5-1.fc18,
globus-gram-job-manager-fork-1.5-1.fc18 has been pushed to the Fedora 18
testing repository.

-- 
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 772988] Review Request: globus-gram-job-manager-pbs - Globus Toolkit - PBS Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772988

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #15 from Fedora Update System  ---
globus-gram-job-manager-sge-1.5-1.fc18, globus-gram-job-manager-pbs-1.5-1.fc18,
globus-gram-job-manager-fork-1.5-1.fc18 has been pushed to the Fedora 18
testing repository.

-- 
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 772986] Review Request: globus-gram-job-manager-fork - Globus Toolkit - Fork Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772986

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #15 from Fedora Update System  ---
globus-gram-job-manager-sge-1.5-1.fc18, globus-gram-job-manager-pbs-1.5-1.fc18,
globus-gram-job-manager-fork-1.5-1.fc18 has been pushed to the Fedora 18
testing repository.

-- 
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 850646] Review Request: ghc-blaze-textual - Haskell blaze-textual library

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=850646

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #9 from Fedora Update System  ---
ghc-blaze-textual-0.2.0.6-2.fc18 has been pushed to the Fedora 18 testing
repository.

-- 
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 851583] Review Request: gnome-abrt - System problems browser

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=851583

--- Comment #8 from Fedora Update System  ---
gnome-abrt-0.2-9.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/gnome-abrt-0.2-9.fc17

-- 
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 851583] Review Request: gnome-abrt - System problems browser

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=851583

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
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 849618] Review Request: perl-AI-DecisionTree - Automatically Learns Decision Trees

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=849618

Michael Scherer  changed:

   What|Removed |Added

  Flags|fedora-review?  |
  Flags||fedora-review+

--- Comment #4 from Michael Scherer  ---
Errors are fixed, so approved.

-- 
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 843775] Review Request: rubygem-temple - Template compilation framework in Ruby

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=843775

Vít Ondruch  changed:

   What|Removed |Added

 CC||mza...@redhat.com

--- Comment #5 from Vít Ondruch  ---
*** Bug 839654 has been marked as a duplicate of this bug. ***

-- 
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 839654] Review Request: rubygem-temple - Template compilation framework in Ruby

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=839654

Vít Ondruch  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 CC||vondr...@redhat.com
 Resolution|--- |DUPLICATE
Last Closed||2012-09-10 11:47:09

--- Comment #3 from Vít Ondruch  ---
Good new, this is already in Fedora. Apologize for not noticing this review
when I did the review bug 843775 :/

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

-- 
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 855561] Review Request: mediawiki119-RSS - Displays one or more RSS feeds on a wiki page

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855561

--- Comment #8 from Jon Ciesla  ---
Git done (by process-git-requests).

-- 
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 855561] Review Request: mediawiki119-RSS - Displays one or more RSS feeds on a wiki page

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855561

Patrick Uiterwijk  changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #7 from Patrick Uiterwijk  ---
New Package SCM Request
===
Package Name: mediawiki119-RSS
Short Description: Displays one or more RSS feeds on a wiki page
Owners: puiterwijk
Branches: el6
InitialCC:

-- 
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 855561] Review Request: mediawiki119-RSS - Displays one or more RSS feeds on a wiki page

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855561

Paul Wouters  changed:

   What|Removed |Added

  Flags||fedora-review+

--- Comment #6 from Paul Wouters  ---
indeed, as per https://fedoraproject.org/wiki/Packaging:UnownedDirectories

APPROVED

-- 
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 855561] Review Request: mediawiki119-RSS - Displays one or more RSS feeds on a wiki page

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855561

--- Comment #5 from Paul Wouters  ---
So who owns %{_datadir}/mediawiki119/extensions/RSS  ?

I thought this package needed to own that?

-- 
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 851721] Review Request: perl-Net-Nessus-XMLRPC - Communicate with Nessus scanner(v4.2+) via XMLRPC

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=851721

--- Comment #2 from Olivier Bilodeau  ---
Updates:

- whitespace cleanup
- global instead of define
- removed useless Provides:...
- Source URL now uses real_name global

I plan to propose this to EPEL5 also.

Successful koji build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=4472101

The SPEC and SRPM URLs are still valid.

Let me know about anything else I can do to move this forward.

-- 
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 852503] Review Request: perl-Net-Radius - Object-oriented Perl interface to RADIUS

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=852503

--- Comment #2 from Olivier Bilodeau  ---
Changes to package based on comments:

- whitespace cleanup
- global instead of define
- removed duplicated Provides:...
- Source URL now uses real_name global

Notes:

I do plan to propose this in EPEL5 so I haven't implemented the defattr and "rm
-rf" changes. 

Successful koji build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=4472073

The SPEC and SRPM URLs are still valid.

Let me know about anything else I can do to move this forward.

-- 
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 854609] Review Request: python-moksha-hub - Hub components for Moksha

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=854609

Ralph Bean  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |ERRATA
Last Closed||2012-09-10 09:42:50

--- Comment #7 from Ralph Bean  ---
Updates -> https://admin.fedoraproject.org/updates/python-moksha-hub

-- 
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 854608] Review Request: python-moksha-wsgi - WSGI components for Moksha

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=854608

Ralph Bean  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |ERRATA
Last Closed||2012-09-10 09:42:14

--- Comment #7 from Ralph Bean  ---
Updates -> https://admin.fedoraproject.org/updates/python-moksha-wsgi

-- 
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 855829] Review Request: python-autopep8 - formats Python code based on the output of pep8

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855829

--- Comment #6 from Jon Ciesla  ---
Git done (by process-git-requests).

-- 
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 853553] Review Request: guayadeque - Audio player and organizer

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=853553

--- Comment #7 from MartinKG  ---
Spec URL:
https://www.disk.dsl.o2online.de/FclyPlh/RPMS/guayadeque/guayadeque-0.3.6-5.svn1830/guayadeque.spec?a=416FXyjkgDc

SRPM URL:
https://www.disk.dsl.o2online.de/FclyPlh/RPMS/guayadeque/guayadeque-0.3.6-5.svn1830/guayadeque-0.3.6-5.svn1830.fc18.src.rpm?a=curGqhWziec

%changelog
* Mon Sep 10 2012 Martin Gansser  - 0.3.6-5.svn1830
- added BSD license for HMAC-SHA implementation

-- 
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 772989] Review Request: globus-gram-job-manager-sge - Globus Toolkit - Grid Engine Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772989

--- Comment #14 from Fedora Update System  ---
globus-gram-job-manager-sge-1.5-1.fc17,globus-gram-job-manager-pbs-1.5-1.fc17,globus-gram-job-manager-fork-1.5-1.fc17
has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/globus-gram-job-manager-sge-1.5-1.fc17,globus-gram-job-manager-pbs-1.5-1.fc17,globus-gram-job-manager-fork-1.5-1.fc17

-- 
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 772988] Review Request: globus-gram-job-manager-pbs - Globus Toolkit - PBS Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772988

--- Comment #14 from Fedora Update System  ---
globus-gram-job-manager-sge-1.5-1.fc17,globus-gram-job-manager-pbs-1.5-1.fc17,globus-gram-job-manager-fork-1.5-1.fc17
has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/globus-gram-job-manager-sge-1.5-1.fc17,globus-gram-job-manager-pbs-1.5-1.fc17,globus-gram-job-manager-fork-1.5-1.fc17

-- 
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 772986] Review Request: globus-gram-job-manager-fork - Globus Toolkit - Fork Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772986

--- Comment #14 from Fedora Update System  ---
globus-gram-job-manager-sge-1.5-1.fc17,globus-gram-job-manager-pbs-1.5-1.fc17,globus-gram-job-manager-fork-1.5-1.fc17
has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/globus-gram-job-manager-sge-1.5-1.fc17,globus-gram-job-manager-pbs-1.5-1.fc17,globus-gram-job-manager-fork-1.5-1.fc17

-- 
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 772989] Review Request: globus-gram-job-manager-sge - Globus Toolkit - Grid Engine Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772989

--- Comment #13 from Fedora Update System  ---
globus-gram-job-manager-sge-1.5-1.el5,globus-gram-job-manager-pbs-1.5-1.el5,globus-gram-job-manager-fork-1.5-1.el5
has been submitted as an update for Fedora EPEL 5.
https://admin.fedoraproject.org/updates/globus-gram-job-manager-sge-1.5-1.el5,globus-gram-job-manager-pbs-1.5-1.el5,globus-gram-job-manager-fork-1.5-1.el5

-- 
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 772988] Review Request: globus-gram-job-manager-pbs - Globus Toolkit - PBS Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772988

--- Comment #13 from Fedora Update System  ---
globus-gram-job-manager-sge-1.5-1.el5,globus-gram-job-manager-pbs-1.5-1.el5,globus-gram-job-manager-fork-1.5-1.el5
has been submitted as an update for Fedora EPEL 5.
https://admin.fedoraproject.org/updates/globus-gram-job-manager-sge-1.5-1.el5,globus-gram-job-manager-pbs-1.5-1.el5,globus-gram-job-manager-fork-1.5-1.el5

-- 
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 772986] Review Request: globus-gram-job-manager-fork - Globus Toolkit - Fork Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772986

--- Comment #13 from Fedora Update System  ---
globus-gram-job-manager-sge-1.5-1.el5,globus-gram-job-manager-pbs-1.5-1.el5,globus-gram-job-manager-fork-1.5-1.el5
has been submitted as an update for Fedora EPEL 5.
https://admin.fedoraproject.org/updates/globus-gram-job-manager-sge-1.5-1.el5,globus-gram-job-manager-pbs-1.5-1.el5,globus-gram-job-manager-fork-1.5-1.el5

-- 
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 772989] Review Request: globus-gram-job-manager-sge - Globus Toolkit - Grid Engine Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772989

--- Comment #12 from Fedora Update System  ---
globus-gram-job-manager-sge-1.5-1.fc16,globus-gram-job-manager-pbs-1.5-1.fc16,globus-gram-job-manager-fork-1.5-1.fc16
has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/globus-gram-job-manager-sge-1.5-1.fc16,globus-gram-job-manager-pbs-1.5-1.fc16,globus-gram-job-manager-fork-1.5-1.fc16

-- 
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 772988] Review Request: globus-gram-job-manager-pbs - Globus Toolkit - PBS Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772988

--- Comment #12 from Fedora Update System  ---
globus-gram-job-manager-sge-1.5-1.fc16,globus-gram-job-manager-pbs-1.5-1.fc16,globus-gram-job-manager-fork-1.5-1.fc16
has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/globus-gram-job-manager-sge-1.5-1.fc16,globus-gram-job-manager-pbs-1.5-1.fc16,globus-gram-job-manager-fork-1.5-1.fc16

-- 
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 772986] Review Request: globus-gram-job-manager-fork - Globus Toolkit - Fork Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772986

--- Comment #12 from Fedora Update System  ---
globus-gram-job-manager-sge-1.5-1.fc16,globus-gram-job-manager-pbs-1.5-1.fc16,globus-gram-job-manager-fork-1.5-1.fc16
has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/globus-gram-job-manager-sge-1.5-1.fc16,globus-gram-job-manager-pbs-1.5-1.fc16,globus-gram-job-manager-fork-1.5-1.fc16

-- 
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 772989] Review Request: globus-gram-job-manager-sge - Globus Toolkit - Grid Engine Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772989

--- Comment #11 from Fedora Update System  ---
globus-gram-job-manager-sge-1.5-1.el6,globus-gram-job-manager-pbs-1.5-1.el6,globus-gram-job-manager-fork-1.5-1.el6
has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/globus-gram-job-manager-sge-1.5-1.el6,globus-gram-job-manager-pbs-1.5-1.el6,globus-gram-job-manager-fork-1.5-1.el6

-- 
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 772988] Review Request: globus-gram-job-manager-pbs - Globus Toolkit - PBS Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772988

--- Comment #11 from Fedora Update System  ---
globus-gram-job-manager-sge-1.5-1.el6,globus-gram-job-manager-pbs-1.5-1.el6,globus-gram-job-manager-fork-1.5-1.el6
has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/globus-gram-job-manager-sge-1.5-1.el6,globus-gram-job-manager-pbs-1.5-1.el6,globus-gram-job-manager-fork-1.5-1.el6

-- 
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 772986] Review Request: globus-gram-job-manager-fork - Globus Toolkit - Fork Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772986

--- Comment #11 from Fedora Update System  ---
globus-gram-job-manager-sge-1.5-1.el6,globus-gram-job-manager-pbs-1.5-1.el6,globus-gram-job-manager-fork-1.5-1.el6
has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/globus-gram-job-manager-sge-1.5-1.el6,globus-gram-job-manager-pbs-1.5-1.el6,globus-gram-job-manager-fork-1.5-1.el6

-- 
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 772989] Review Request: globus-gram-job-manager-sge - Globus Toolkit - Grid Engine Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772989

--- Comment #10 from Fedora Update System  ---
globus-gram-job-manager-sge-1.5-1.fc18,globus-gram-job-manager-pbs-1.5-1.fc18,globus-gram-job-manager-fork-1.5-1.fc18
has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/globus-gram-job-manager-sge-1.5-1.fc18,globus-gram-job-manager-pbs-1.5-1.fc18,globus-gram-job-manager-fork-1.5-1.fc18

-- 
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 772988] Review Request: globus-gram-job-manager-pbs - Globus Toolkit - PBS Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772988

--- Comment #10 from Fedora Update System  ---
globus-gram-job-manager-sge-1.5-1.fc18,globus-gram-job-manager-pbs-1.5-1.fc18,globus-gram-job-manager-fork-1.5-1.fc18
has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/globus-gram-job-manager-sge-1.5-1.fc18,globus-gram-job-manager-pbs-1.5-1.fc18,globus-gram-job-manager-fork-1.5-1.fc18

-- 
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 855561] Review Request: mediawiki119-RSS - Displays one or more RSS feeds on a wiki page

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855561

--- Comment #4 from Patrick Uiterwijk  ---
I have removed the EL5 parts and added the license.

The %dir is only required for root directories which the package should own,
but should not own all the files inside.
For example %{_libdir}/clean for the clean package: it does have folders that
it should own, but other packages like clean-mersennetwister also add their own
files to that directory.

New Spec URL:
http://puiterwijk.fedorapeople.org/packages/mediawiki-RSS/mediawiki119-RSS.spec
New SRPM URL:
http://puiterwijk.fedorapeople.org/packages/mediawiki-RSS/mediawiki119-RSS-2.16-3.fc17.src.rpm

-- 
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 772986] Review Request: globus-gram-job-manager-fork - Globus Toolkit - Fork Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772986

--- Comment #10 from Fedora Update System  ---
globus-gram-job-manager-sge-1.5-1.fc18,globus-gram-job-manager-pbs-1.5-1.fc18,globus-gram-job-manager-fork-1.5-1.fc18
has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/globus-gram-job-manager-sge-1.5-1.fc18,globus-gram-job-manager-pbs-1.5-1.fc18,globus-gram-job-manager-fork-1.5-1.fc18

-- 
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 772989] Review Request: globus-gram-job-manager-sge - Globus Toolkit - Grid Engine Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772989

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
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 772988] Review Request: globus-gram-job-manager-pbs - Globus Toolkit - PBS Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772988

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
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 772986] Review Request: globus-gram-job-manager-fork - Globus Toolkit - Fork Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772986

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
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 855829] Review Request: python-autopep8 - formats Python code based on the output of pep8

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855829

Nikola Dipanov  changed:

   What|Removed |Added

  Flags||fedora-cvs?

-- 
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 855829] Review Request: python-autopep8 - formats Python code based on the output of pep8

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855829

--- Comment #5 from Nikola Dipanov  ---
New Package SCM Request
===
Package Name: python-autopep8
Short Description: formats Python code based on the output of pep8
Owners: ndipanov mrunge
Branches: f18
InitialCC:

-- 
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 855829] Review Request: python-autopep8 - formats Python code based on the output of pep8

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855829

--- Comment #4 from Nikola Dipanov  ---
New Package SCM Request
===
Package Name: python-autopep8
Short Description: python-autopep8 - formats Python code based on the output of
pep8
Owners: ndipanov mrunge
Branches: f18
InitialCC:

-- 
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 855829] Review Request: python-autopep8 - formats Python code based on the output of pep8

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855829

--- Comment #3 from Nikola Dipanov  ---
New Package SCM Request
===
Package Name: python-autopep8
Short Description: autopep8 formats Python code based on the output of the pep8
utility.
Owners: ndipanov mrunge
Branches: f18
InitialCC:

-- 
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 855843] New: Review Request: mina-ftpserver - A 100% pure Java FTP server

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855843

Bug ID: 855843
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: mina-ftpserver - A 100% pure Java FTP
server
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: punto...@libero.it
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://gil.fedorapeople.org/mina-ftpserver.spec
SRPM URL: http://gil.fedorapeople.org/mina-ftpserver-1.0.6-1.fc16.src.rpm
Description: The Apache FtpServer is a 100% pure Java FTP server. It's
designed to be a complete and portable FTP server engine
solution based on currently available open protocols.
FtpServer can be run standalone as a Windows service or
Unix/Linux daemon, or embedded into a Java application.
We also provide support for integration within Spring
applications and provide our releases as OSGi bundles.

The default network support is based on Apache MINA, a
high performance asynchronous IO library. Using MINA,
FtpServer can scale to a large number of concurrent users.

It is also an FTP application platform. We have developed
a Java API to let you write Java code to process FTP event
notifications that we call the Ftplet API. Apache FtpServer
provides an implementation of an FTP server to support this API.
Fedora Account System Username: gil

-- 
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 855829] Review Request: python-autopep8 - formats Python code based on the output of pep8

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855829

Matthias Runge  changed:

   What|Removed |Added

  Flags|fedora-review?  |
  Flags||fedora-review+

-- 
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 855829] Review Request: python-autopep8 - formats Python code based on the output of pep8

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855829

--- Comment #2 from Matthias Runge  ---
Package Review
==

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



 Generic 
[x]: EXTRA Rpmlint is run on all installed packages.
 Note: There are rpmlint messages (see attachment).
[x]: EXTRA Spec file according to URL is the same as in SRPM.
[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.
[x]: 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 Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[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
[x]: MUST Macros in Summary, %description expandable at SRPM build time.
[x]: MUST Package contains desktop file if it is a GUI application.
[-]: MUST Development files must be in a -devel package
[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 Package complies to the Packaging Guidelines
[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.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 "MIT/X11 (BSD like)" For detailed output of licensecheck see file:
 /home/mrunge/review/855829-python-autopep8/licensecheck.txt
[x]: MUST Package consistently uses macro is (instead of hard-coded directory
 names).
[x]: MUST Package is named using only allowed ascii characters.
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generate any conflict.
 Note: Package contains no Conflicts: tag(s)
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[x]: MUST If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[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 Package is not relocatable.
[x]: MUST Requires correct, justified where necessary.
[x]: MUST Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: MUST Sources used to build the package match the upstream source, as
 provided in the spec URL.
[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 systemd file(s) if in need.
[x]: MUST File names are valid UTF-8.
[x]: SHOULD Reviewer should test that the package builds in mock.
[x]: SHOULD Buildroot is not present
 Note: Unless packager wants to package for EPEL5 this is fine
[x]: SHOULD Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
 Note: Clean would be needed if support for EPEL5 is required
[!]: 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.
[!]: 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.
[x]: SHOULD SourceX tarball generation or download is documented.
[!]: SHOULD SourceX / PatchY prefixed with %{name}.
 Note: Source0 (autopep8-0.8.tar.gz)
[x]: SHOULD SourceX is a working URL.
[x]: 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 bina

[Bug 855829] Review Request: python-autopep8 - formats Python code based on the output of pep8

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855829

Matthias Runge  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||mru...@redhat.com
   Assignee|nob...@fedoraproject.org|mru...@redhat.com
  Flags||fedora-review?

--- Comment #1 from Matthias Runge  ---
I'll do a review, and I will also act as Nikolas sponsor.

-- 
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 855829] New: Review Request: python-autopep8 - formats Python code based on the output of pep8

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855829

Bug ID: 855829
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: unspecified
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: python-autopep8 - formats Python code
based on the output of pep8
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: ndipa...@redhat.com
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://dl.dropbox.com/u/29545457/python-autopep8.spec
SRPM URL: http://dl.dropbox.com/u/29545457/python-autopep8-0.8-1.src.rpm
Description: autopep8 formats Python code based on the output of the pep8
utility.
Fedora Account System Username: ndipanov

-- 
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 772989] Review Request: globus-gram-job-manager-sge - Globus Toolkit - Grid Engine Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772989

--- Comment #9 from Jon Ciesla  ---
Git done (by process-git-requests).

-- 
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 772988] Review Request: globus-gram-job-manager-pbs - Globus Toolkit - PBS Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772988

--- Comment #9 from Jon Ciesla  ---
Git done (by process-git-requests).

-- 
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 772986] Review Request: globus-gram-job-manager-fork - Globus Toolkit - Fork Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772986

--- Comment #9 from Jon Ciesla  ---
Git done (by process-git-requests).

f19==devel

-- 
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 849618] Review Request: perl-AI-DecisionTree - Automatically Learns Decision Trees

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=849618

--- Comment #3 from Mathieu Bridon  ---
Thanks for the comments, they should all be fixed with this new package.

Spec URL: http://bochecha.fedorapeople.org/packages/perl-AI-DecisionTree.spec
SRPM URL:
http://bochecha.fedorapeople.org/packages/perl-AI-DecisionTree-0.11-2.fc18.src.rpm

-- 
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 855780] Review Request: apacheds-daemon - Reusable Daemon Framework

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

gil cattaneo  changed:

   What|Removed |Added

 Blocks||652183 (FE-JAVASIG)

-- 
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 855780] New: Review Request: apacheds-daemon - Reusable Daemon Framework

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855780

Bug ID: 855780
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: apacheds-daemon - Reusable Daemon
Framework
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: punto...@libero.it
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://gil.fedorapeople.org/apacheds-daemon.spec
SRPM URL: http://gil.fedorapeople.org/apacheds-daemon-1.1.8-1.fc16.src.rpm
Description: Reusable framework for daemon applications based on Commons Daemon
Jsvc and Procrun. A small installation layout pattern combined with
some utility classes allows applications to be come UNIX daemons or
Windows NT services. Reusable bootstrappers along with an installer
plugin allow for the rapid creation of standalone daemon applications.
Fedora Account System Username: gil
ApacheDS/Studio build/requires

-- 
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 855283] Review Request: pass - A unix password manager using standard tools

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855283

--- Comment #28 from Fedora Update System  ---
pass-1.1.4-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/pass-1.1.4-1.el6

-- 
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 772989] Review Request: globus-gram-job-manager-sge - Globus Toolkit - Grid Engine Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772989

Mattias Ellert  changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #8 from Mattias Ellert  ---
Many thanks for the review.

New Package SCM Request
===
Package Name: globus-gram-job-manager-sge
Short Description: Globus Toolkit - Grid Engine Job Manager Support
Owners: ellert
Branches: f16 f17 f18 f19 el5 el6
InitialCC:

-- 
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 772986] Review Request: globus-gram-job-manager-fork - Globus Toolkit - Fork Job Manager Support

2012-09-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=772986

Mattias Ellert  changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #8 from Mattias Ellert  ---
Many thanks for the review.

New Package SCM Request
===
Package Name: globus-gram-job-manager-fork
Short Description: Globus Toolkit - Fork Job Manager Support
Owners: ellert
Branches: f16 f17 f18 f19 el5 el6
InitialCC:

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