[Bug 690728] Review Request: Nitrate - A test case management system written in Django

2011-08-07 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=690728

Yuguang Wang  changed:

   What|Removed |Added

 Depends on||728823, 728837, 728826

--- Comment #49 from Yuguang Wang  2011-08-08 01:43:06 EDT 
---
'django-tinymce', 'xml-dict', 'wadofstuff-django-serializers' filed, 
added them as dependencies.

I guess those js libraries such as prototype or livepipe or tablekit needn't be
packaged separately, right?

-- 
Configure 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 728837] Review Request: xml2dict - Use attributes of dictionary to access xml elements.

2011-08-07 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=728837

Yuguang Wang  changed:

   What|Removed |Added

 Blocks||690728

-- 
Configure 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 728823] Review Request: django-tinymce - TinyMCE form field editor for Django applications

2011-08-07 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=728823

Yuguang Wang  changed:

   What|Removed |Added

 Blocks||690728

-- 
Configure 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 728826] Review Request: wadofstuff-django-serializers - Extended Django Serializer Module

2011-08-07 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=728826

Yuguang Wang  changed:

   What|Removed |Added

 Blocks||690728

-- 
Configure 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 728837] Review Request: xml2dict - Use attributes of dictionary to access xml elements.

2011-08-07 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=728837

--- Comment #2 from Yuguang Wang  2011-08-08 01:36:44 EDT ---
Wow, I'd say the package from googlecode has no 'setup.py' thus unable to
install, I wrote one for it, and the source rpm is available here:
http://yuwang.fedorapeople.org/xml2dict-1.0-1.fc13.src.rpm

Running rpmlint ok.

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


[Bug 728837] Review Request: xml2dict - Use attributes of dictionary to access xml elements.

2011-08-07 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=728837

Yuguang Wang  changed:

   What|Removed |Added

 CC||d...@redhat.com,
   ||peter...@redhat.com,
   ||r...@redhat.com,
   ||vc...@redhat.com,
   ||yuw...@redhat.com
 Blocks||177841(FE-NEEDSPONSOR)

--- Comment #1 from Yuguang Wang  2011-08-08 01:31:46 EDT ---
rpmlint result:
[yuwang@yuwang xml2dict]$ rpmlint xml2dict.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

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


[Bug 728837] New: Review Request: xml2dict - Use attributes of dictionary to access xml elements.

2011-08-07 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: xml2dict - Use attributes of dictionary to access xml 
elements.

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

   Summary: Review Request: xml2dict - Use attributes of
dictionary to access xml elements.
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: yuw...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL: http://yuwang.fedorapeople.org/xml2dict.spec

Description: Use attributes of dictionary to access xml elements.

Example as below:
from xml2dict import XML2Dict

if __name__ == '__main__':
s = """

10
491691test
491692test2
503938hello, world
"""

xml = XML2Dict()
r = xml.fromstring(s)
from pprint import pprint
pprint(r)
print r.result.count.value
print r.result.count.n

for data in r.result.data:
print data.id, data.name 
pprint(xml.parse('a'))

-- 
Configure 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 728826] Review Request: wadofstuff-django-serializers - Extended Django Serializer Module

2011-08-07 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=728826

Yuguang Wang  changed:

   What|Removed |Added

 CC||d...@redhat.com,
   ||peter...@redhat.com,
   ||r...@redhat.com,
   ||vc...@redhat.com,
   ||yuw...@redhat.com
 Blocks||177841(FE-NEEDSPONSOR)

--- Comment #1 from Yuguang Wang  2011-08-07 23:24:26 EDT ---
[yuwang@yuwang wadofstuff-django-serializers-1.1.0]$ rpmlint
wadofstuff-django-serializers.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

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


[Bug 728826] New: Review Request: wadofstuff-django-serializers - Extended Django Serializer Module

2011-08-07 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: wadofstuff-django-serializers - Extended Django 
Serializer Module

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

   Summary: Review Request: wadofstuff-django-serializers -
Extended Django Serializer Module
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: yuw...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL: http://yuwang.fedorapeople.org/wadofstuff-django-serializers.spec

Description: The wadofstuff.django.serializers python module extends Django's
built-in serializers, adding 3 new capabilities inspired by the Ruby on Rails
JSON serializer. These parameters allow the developer more control over how
their models are serialized. The additional capabilities are:

excludes - a list of fields to be excluded from serialization. The excludes
list takes precedence over the fields argument.
extras - a list of non-model field properties or callables to be serialized.
relations - a list or dictionary of model related fields to be followed and
serialized.

-- 
Configure 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 728823] Review Request: django-tinymce - TinyMCE form field editor for Django applications

2011-08-07 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=728823

--- Comment #3 from Jens Petersen  2011-08-07 22:49:10 EDT 
---
(In reply to comment #2)
> Warning becoz the package from googlecode has no spec files in it, I created a
> spec file for it.

Erm that is normal. :)

> (In reply to comment #1)
> > rpmlint results:
> > 
> > $ rpmlint django-tinymce-1.5-1.fc13.src.rpm
> > django-tinymce.src: W: file-size-mismatch django-tinymce-1.5.tar.gz = 13732,
> > http://django-tinymce.googlecode.com/files/django-tinymce-1.5.tar.gz = 13396
> > 1 packages and 0 specfiles checked; 0 errors, 1 warnings.

Please don't do that.  There is no need to re-roll the upstream tarball.

-- 
Configure 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 728823] Review Request: django-tinymce - TinyMCE form field editor for Django applications

2011-08-07 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=728823

Yuguang Wang  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 728823] Review Request: django-tinymce - TinyMCE form field editor for Django applications

2011-08-07 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=728823

Yuguang Wang  changed:

   What|Removed |Added

 CC||peter...@redhat.com,
   ||r...@redhat.com,
   ||vc...@redhat.com

--- Comment #1 from Yuguang Wang  2011-08-07 22:39:03 EDT ---
rpmlint results:

$ rpmlint django-tinymce-1.5-1.fc13.src.rpm
django-tinymce.src: W: file-size-mismatch django-tinymce-1.5.tar.gz = 13732,
http://django-tinymce.googlecode.com/files/django-tinymce-1.5.tar.gz = 13396
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

$ rpmlint django-tinymce.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

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


[Bug 728823] Review Request: django-tinymce - TinyMCE form field editor for Django applications

2011-08-07 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=728823

--- Comment #2 from Yuguang Wang  2011-08-07 22:40:45 EDT ---
Warning becoz the package from googlecode has no spec files in it, I created a
spec file for it.
(In reply to comment #1)
> rpmlint results:
> 
> $ rpmlint django-tinymce-1.5-1.fc13.src.rpm
> django-tinymce.src: W: file-size-mismatch django-tinymce-1.5.tar.gz = 13732,
> http://django-tinymce.googlecode.com/files/django-tinymce-1.5.tar.gz = 13396
> 1 packages and 0 specfiles checked; 0 errors, 1 warnings.
> 
> $ rpmlint django-tinymce.spec 
> 0 packages and 1 specfiles checked; 0 errors, 0 warnings.

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


[Bug 728823] New: Review Request: django-tinymce - TinyMCE form field editor for Django applications

2011-08-07 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: django-tinymce - TinyMCE form field editor for Django 
applications

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

   Summary: Review Request: django-tinymce - TinyMCE form field
editor for Django applications
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: yuw...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL: http://yuwang.fedorapeople.org/django-tinymce.spec
SRPM URL: http://yuwang.fedorapeople.org/django-tinymce-1.5-1.fc13.src.rpm
Description: A Django application that contains a widget to render a form field
as a TinyMCE editor.

Features:

Use as a form widget or with a view.
Enhanced support for content languages.
Integration with the TinyMCE spellchecker.
Enables predefined link and image lists for dialogs.
Can compress the TinyMCE Javascript code.
Integration with django-filebrowser.

-- 
Configure 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 711547] Review Request: sketch - Free Graphics Software for the TeX, LaTeX, and PSTricks Community

2011-08-07 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=711547

Ryan H. Lewis (rhl)  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #26 from Ryan H. Lewis (rhl)  2011-08-07 
21:30:15 EDT ---
New Package SCM Request
===
Package Name: sketch
Short Description: A 3D Scene Description Translator. 
Owners: rhl
Branches: f14 f15 el4 el5 el6
InitialCC:

-- 
Configure 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 716700] Review Request: perl-FusionInventory-Agent-Task-OcsDeploy - OCS Inventory NG Software deployment support for FusionInventory Agent

2011-08-07 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=716700

Haïkel Guémar  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #2 from Haïkel Guémar  2011-08-07 20:11:18 
EDT ---
perl-FusionInventory-Agent-Task-OcsDeploy (Perl package)

MUST: rpmlint must be run on src.rpm and rpm.  OK
$ rpmlint -iv perl-FusionInventory-Agent-Task-OcsDeploy-1.1.0-3.fc17.src.rpm
perl-FusionInventory-Agent-Task-OcsDeploy.src: I: checking
perl-FusionInventory-Agent-Task-OcsDeploy.src: I: checking-url
http://forge.fusioninventory.org/projects/fusioninventory-agent-task-ocsdeploy
(timeout 10 seconds)
perl-FusionInventory-Agent-Task-OcsDeploy.src: I: checking-url
http://search.cpan.org/CPAN/authors/id/F/FU/FUSINV/FusionInventory-Agent-Task-OcsDeploy-1.1.0.tar.gz
(timeout 10 seconds)
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

$ rpmlint -iv perl-FusionInventory-Agent-Task-OcsDeploy-1.1.0-3.fc17.noarch.rpm 
perl-FusionInventory-Agent-Task-OcsDeploy.noarch: I: checking
perl-FusionInventory-Agent-Task-OcsDeploy.noarch: I: checking-url
http://forge.fusioninventory.org/projects/fusioninventory-agent-task-ocsdeploy
(timeout 10 seconds)
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

MUST: package named accordingly to package naming guidelines. OK 

MUST: spec file name match %{name}  OK

MUST: package meets packaging guidelines OK

MUST: package must be licensed under a fedora-compliant license OK (GPLv2+)

MUST: License field in package spec match actual license OK
License file included and distributed

MUST: spec in legible american english OK

MUST: sources provided match upstream's OK

provided sources sha1sum: e9f3c25e60542aef69b61318fb526b7b042099e6
upstream sources sha1sum: e9f3c25e60542aef69b61318fb526b7b042099e6

MUST: package successfully compiles on at least one primary architecture OK
(all of them)

MUST: all build dependencies are listed in BR OK

MUST: package must own all directories it creates OK

MUST: package does not list a file more than once in %files section OK

MUST: permissions are properly set OK

MUST: package consistenly use macros OK
Note: perl filters works with both RPM 4.8 and 4.9

MUST: package contains permissable content OK

MUST: package does not own directories owned by other packages OK

MUST: all filenames in package are valid UTF-8 OK

SHOULD: mock builds were done on branched f16 && rawhide on all primary
architectures (x86/x86_64) OK

SHOULD: man pages are provided OK

This package respects general & Perl Fedora Packaging guidelines, so i hereby
approve this package into Fedora Packages Collection

-- 
Configure 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 702987] Review Request: stdair - C++ Standard Airline IT Library

2011-08-07 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=702987

Denis Arnaud  changed:

   What|Removed |Added

 Blocks||728815

-- 
Configure 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 728815] Review Request: trademgen - C++ Simulated Travel Demand Generation Library

2011-08-07 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=728815

Denis Arnaud  changed:

   What|Removed |Added

 Depends on||702987

-- 
Configure 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 728815] New: Review Request: trademgen - C++ Simulated Travel Demand Generation Library

2011-08-07 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: trademgen -  C++ Simulated Travel Demand Generation 
Library

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

   Summary: Review Request: trademgen -  C++ Simulated Travel
Demand Generation Library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: denis.arnaud_fed...@m4x.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL:
http://denisarnaud.fedorapeople.org/sim/trademgen/trademgen-0.1.0-1.spec
SRPM URL:
https://downloads.sourceforge.net/project/trademgen/fedora_15/trademgen-0.1.0-1.fc15.src.rpm
Description:
aims at providing a clean API, and the corresponding C++
implementation, able to generate demand for travel solutions (e.g.,
from JFK to PEK on 25-05-2009) according to characteristics (e.g.,
Willingness-To-Pay, preferred airline, etc).

-- 
Configure 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 728737] Review Request: libinstpatch - MIDI instrument patch library

2011-08-07 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=728737

--- Comment #3 from Orcan Ogetbil  2011-08-07 18:36:09 
EDT ---
Thanks, updated the Requires, added the COPYING file to %doc, and emailed
upstream about the FSF address:
Spec URL: http://oget.fedorapeople.org/review/libinstpatch.spec
SRPM URL:
http://oget.fedorapeople.org/review/libinstpatch-1.0.0-3.20110806svn386.fc15.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 227061] Review Request: isorelax-0.1-0.20041111.2jpp - Public interfaces useful for applications to support RELAX Core

2011-08-07 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=227061

Mat Booth  changed:

   What|Removed |Added

 CC||fed...@matbooth.co.uk
   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #14 from Mat Booth  2011-08-07 16:56:07 EDT 
---
Package Change Request
==
Package Name: junitperf
New Branches: el5 el6
Owners: mbooth

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


[Bug 728275] Review Request: perl-Data-Section-Simple - Read data from __DATA__

2011-08-07 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=728275

--- Comment #9 from Fedora Update System  2011-08-07 
16:51:23 EDT ---
perl-Data-Section-Simple-0.02-2.fc15 has been submitted as an update for Fedora
15.
https://admin.fedoraproject.org/updates/perl-Data-Section-Simple-0.02-2.fc15

-- 
Configure 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 728275] Review Request: perl-Data-Section-Simple - Read data from __DATA__

2011-08-07 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=728275

--- Comment #8 from Fedora Update System  2011-08-07 
16:51:13 EDT ---
perl-Data-Section-Simple-0.02-2.el6 has been submitted as an update for Fedora
EPEL 6.
https://admin.fedoraproject.org/updates/perl-Data-Section-Simple-0.02-2.el6

-- 
Configure 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 728275] Review Request: perl-Data-Section-Simple - Read data from __DATA__

2011-08-07 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=728275

--- Comment #12 from Fedora Update System  
2011-08-07 16:51:46 EDT ---
perl-Data-Section-Simple-0.02-2.el5 has been submitted as an update for Fedora
EPEL 5.
https://admin.fedoraproject.org/updates/perl-Data-Section-Simple-0.02-2.el5

-- 
Configure 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 728275] Review Request: perl-Data-Section-Simple - Read data from __DATA__

2011-08-07 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=728275

--- Comment #7 from Fedora Update System  2011-08-07 
16:51:05 EDT ---
perl-Data-Section-Simple-0.02-2.el4 has been submitted as an update for Fedora
EPEL 4.
https://admin.fedoraproject.org/updates/perl-Data-Section-Simple-0.02-2.el4

-- 
Configure 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 728275] Review Request: perl-Data-Section-Simple - Read data from __DATA__

2011-08-07 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=728275

--- Comment #10 from Fedora Update System  
2011-08-07 16:51:31 EDT ---
perl-Data-Section-Simple-0.02-2.fc16 has been submitted as an update for Fedora
16.
https://admin.fedoraproject.org/updates/perl-Data-Section-Simple-0.02-2.fc16

-- 
Configure 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 728275] Review Request: perl-Data-Section-Simple - Read data from __DATA__

2011-08-07 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=728275

--- Comment #11 from Fedora Update System  
2011-08-07 16:51:38 EDT ---
perl-Data-Section-Simple-0.02-2.fc14 has been submitted as an update for Fedora
14.
https://admin.fedoraproject.org/updates/perl-Data-Section-Simple-0.02-2.fc14

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


[Bug 728275] Review Request: perl-Data-Section-Simple - Read data from __DATA__

2011-08-07 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=728275

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 725200] Review Request: raptor2 - RDF Parser Toolkit for Redland

2011-08-07 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=725200

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |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 728178] Review Request: xfce4-wmdock-plugin - Compatibility layer for running dockapps in an Xfce panel

2011-08-07 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=728178

--- Comment #6 from Jon Ciesla  2011-08-07 15:18:23 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 728275] Review Request: perl-Data-Section-Simple - Read data from __DATA__

2011-08-07 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=728275

--- Comment #6 from Jon Ciesla  2011-08-07 15: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 716700] Review Request: perl-FusionInventory-Agent-Task-OcsDeploy - OCS Inventory NG Software deployment support for FusionInventory Agent

2011-08-07 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=716700

Haïkel Guémar  changed:

   What|Removed |Added

 CC||karlthe...@gmail.com
 AssignedTo|nob...@fedoraproject.org|karlthe...@gmail.com
   Flag||fedora-review?

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

[Bug 728178] Review Request: xfce4-wmdock-plugin - Compatibility layer for running dockapps in an Xfce panel

2011-08-07 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=728178

Martin Gieseking  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #5 from Martin Gieseking  2011-08-07 
14:10:56 EDT ---
Thanks again for the review!


New Package SCM Request
===
Package Name: xfce4-wmdock-plugin
Short Description: Compatibility layer for running dockapps in an Xfce panel
Owners: mgieseki
Branches: f15 f16
InitialCC:

-- 
Configure 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 720857] Review Request: datalog - A Lightweight Deductive Database using Datalog

2011-08-07 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=720857

Martin Gieseking  changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)

--- Comment #7 from Martin Gieseking  2011-08-07 
14:08:01 EDT ---
OK, thanks for the information. Since you're not sponsored yet, I add
FE-NEEDSPONSOR to this ticket as well. 
When you've solved the bundled Lua issue and packaged the new release, post a
link to it and we can continue.

-- 
Configure 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 728662] Review Request: python-pylibmc - Memcached client for Python

2011-08-07 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=728662

Ankur Sinha  changed:

   What|Removed |Added

 Blocks||728664
 Depends on|728664  |

-- 
Configure 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 728664] Review Request: django-pylibmc - Django cache backend using pylibmc

2011-08-07 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=728664

Ankur Sinha  changed:

   What|Removed |Added

 Blocks|728662  |
 Depends on||728662

-- 
Configure 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 728664] Review Request: django-pylibmc - Django cache backend using pylibmc

2011-08-07 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=728664

--- Comment #1 from Ankur Sinha  2011-08-07 14:02:52 
EDT ---
REVIEW:

+ OK
- NA
? ISSUE

+ Package meets naming and packaging guidelines
+ Spec file matches base package name.
+ Spec has consistant macro usage.
+ Meets Packaging Guidelines.
+ License
+ License field in spec matches
+ License file included in package
+ Spec in American English
+ Spec is legible.
+ Sources match upstream md5sum:
git checkout, confirmed using diff. 

- Package needs ExcludeArch
+ BuildRequires correct
- Spec handles locales/find_lang
- Package is relocatable and has a reason to be.
+ Package is code or permissible content.
- Doc subpackage needed/used.
+ Packages %doc files don't affect runtime.

- Headers/static libs in -devel subpackage.
- Spec has needed ldconfig in post and postun
- .pc files in -devel subpackage/requires pkgconfig
- .so files in -devel subpackage.
- -devel package Requires: %{name} = %{version}-%{release}
- .la files are removed.

- Package is a GUI app and has a .desktop file

+ Package compiles and builds on at least one arch.
+ Package has no duplicate files in %files.
+ Package doesn't own any directories other packages own.
+ Package owns all the directories it creates.
+ No rpmlint output.
^^
[ankur@ankur SRPMS]$ rpmlint /var/lib/mock/fedora-rawhide-i386/result/*.rpm
../SPECS/django-pylibmc.spec
django-pylibmc-0.2.1-1.20110806gitb56e74.fc15.src.rpm 
django-pylibmc.noarch: W: spelling-error Summary(en_US) backend -> backed, back
end, back-end
django-pylibmc.noarch: W: spelling-error %description -l en_US memcached ->
schemed
django-pylibmc.noarch: W: spelling-error %description -l en_US backend ->
backed, back end, back-end
django-pylibmc.src: W: spelling-error Summary(en_US) backend -> backed, back
end, back-end
django-pylibmc.src: W: spelling-error %description -l en_US memcached ->
schemed
django-pylibmc.src: W: spelling-error %description -l en_US backend -> backed,
back end, back-end
django-pylibmc.src: W: invalid-url Source0:
django-pylibmc-0.2.1.20110806gitb56e74.tar.xz
../SPECS/django-pylibmc.spec: W: invalid-url Source0:
django-pylibmc-0.2.1.20110806gitb56e74.tar.xz
django-pylibmc.src: W: spelling-error Summary(en_US) backend -> backed, back
end, back-end
django-pylibmc.src: W: spelling-error %description -l en_US memcached ->
schemed
django-pylibmc.src: W: spelling-error %description -l en_US backend -> backed,
back end, back-end
django-pylibmc.src: W: invalid-url Source0:
django-pylibmc-0.2.1.20110806gitb56e74.tar.xz
3 packages and 1 specfiles checked; 0 errors, 12 warnings.

ONLY MINOR SPELLING SUGGESTIONS. OK

+ final provides and requires are sane:
[ankur@ankur result]$ review-req-check 
== django-pylibmc-0.2.1-1.20110806gitb56e74.fc17.noarch.rpm ==
Provides:
django-pylibmc = 0.2.1-1.20110806gitb56e74.fc17

Requires:
python(abi) = 2.7

== django-pylibmc-0.2.1-1.20110806gitb56e74.fc17.src.rpm ==
Provides:

Requires:
python-devel  
python-setuptools  
python-pylibmc  
Django  


SHOULD Items:

+ Should build in mock.
+ Should build on all supported archs
- Should function as described.
- Should have sane scriptlets.
- Should have subpackages require base package with fully versioned depend.
+ Should have dist tag
+ Should package latest version
- check for outstanding bugs on package. (For core merge reviews)

Issues:

1. Again, only a cosmetic suggestion for the setup macro directory name. Can be
a simple %{name}. 

I'll approve this as soon as python-pylibmc is approved and built for rawhide.
(It's *blocking* this one)


Thanks,
Ankur

-- 
Configure 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 728662] Review Request: python-pylibmc - Memcached client for Python

2011-08-07 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=728662

--- Comment #2 from Ankur Sinha  2011-08-07 13:54:24 
EDT ---
One suggestion though: In the setup macro, you needn't use such a complex name.
You're creating the directory to extract into, even just %{name} will suffice.
Only a cosmetic suggestion.

-- 
Configure 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 728662] Review Request: python-pylibmc - Memcached client for Python

2011-08-07 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=728662

--- Comment #1 from Ankur Sinha  2011-08-07 13:49:01 
EDT ---
Review:

+ OK
- NA
? ISSUE

+ Package meets naming and packaging guidelines
+ Spec file matches base package name.
+ Spec has consistant macro usage.
+ Meets Packaging Guidelines.
+ License
+ License field in spec matches
+ License file included in package
+ Spec in American English
+ Spec is legible.
+ Sources match upstream md5sum:
^^ Git check out, checked using diff.

- Package needs ExcludeArch
+ BuildRequires correct
- Spec handles locales/find_lang
- Package is relocatable and has a reason to be.
+ Package is code or permissible content.
- Doc subpackage needed/used.
+ Packages %doc files don't affect runtime.

- Headers/static libs in -devel subpackage.
- Spec has needed ldconfig in post and postun
- .pc files in -devel subpackage/requires pkgconfig
- .so files in -devel subpackage.
- -devel package Requires: %{name} = %{version}-%{release}
- .la files are removed.

- Package is a GUI app and has a .desktop file

+ Package compiles and builds on at least one arch.
+ Package has no duplicate files in %files.
+ Package doesn't own any directories other packages own.
+ Package owns all the directories it creates.
+ No rpmlint output.
^^
[ankur@ankur SRPMS]$ rpmlint /var/lib/mock/fedora-rawhide-i386/result/*.rpm
../SPECS/python-pylibmc.spec
python-pylibmc-1.2.0-1.20110805gitf01c31.fc15.src.rpm 
python-pylibmc.noarch: W: spelling-error Summary(en_US) Memcached -> Schemed
python-pylibmc.noarch: W: spelling-error %description -l en_US memcached ->
schemed
python-pylibmc.noarch: W: spelling-error %description -l en_US libmemcached ->
semidetached
python-pylibmc.src: W: spelling-error Summary(en_US) Memcached -> Schemed
python-pylibmc.src: W: spelling-error %description -l en_US memcached ->
schemed
python-pylibmc.src: W: spelling-error %description -l en_US libmemcached ->
semidetached
python-pylibmc.src: W: invalid-url Source0:
python-pylibmc-1.2.0.20110805gitf01c31.tar.xz
../SPECS/python-pylibmc.spec: W: invalid-url Source0:
python-pylibmc-1.2.0.20110805gitf01c31.tar.xz
python-pylibmc.src: W: spelling-error Summary(en_US) Memcached -> Schemed
python-pylibmc.src: W: spelling-error %description -l en_US memcached ->
schemed
python-pylibmc.src: W: spelling-error %description -l en_US libmemcached ->
semidetached
python-pylibmc.src: W: invalid-url Source0:
python-pylibmc-1.2.0.20110805gitf01c31.tar.xz
3 packages and 1 specfiles checked; 0 errors, 12 warnings.


MINOR SPELLING ERRORS ONLY. OK.

+ final provides and requires are sane:
[ankur@ankur result]$ review-req-check 
== python-pylibmc-1.2.0-1.20110805gitf01c31.fc17.noarch.rpm ==
Provides:
python-pylibmc = 1.2.0-1.20110805gitf01c31.fc17

Requires:
python(abi) = 2.7

== python-pylibmc-1.2.0-1.20110805gitf01c31.fc17.src.rpm ==
Provides:

Requires:
python-devel  
python-setuptools  
libmemcached-devel  
zlib-devel  


SHOULD Items:

+ Should build in mock.
+ Should build on all supported archs
- Should function as described.
- Should have sane scriptlets.
- Should have subpackages require base package with fully versioned depend.
+ Should have dist tag
+ Should package latest version
- check for outstanding bugs on package. (For core merge reviews)

Issues:

1. It looks like a good package. I see you've removed some sonames. Why is
this? Please clarify and I shall approve the package :D

Thanks,
Ankur

-- 
Configure 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 728664] Review Request: django-pylibmc - Django cache backend using pylibmc

2011-08-07 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=728664

Ankur Sinha  changed:

   What|Removed |Added

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

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


[Bug 728662] Review Request: python-pylibmc - Memcached client for Python

2011-08-07 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=728662

Ankur Sinha  changed:

   What|Removed |Added

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

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


[Bug 728664] Review Request: django-pylibmc - Django cache backend using pylibmc

2011-08-07 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=728664

Ankur Sinha  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 728662] Review Request: python-pylibmc - Memcached client for Python

2011-08-07 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=728662

Ankur Sinha  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 463764] Package review: perl-Module-Manifest

2011-08-07 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=463764

Paul Howarth  changed:

   What|Removed |Added

 CC||p...@city-fan.org

--- Comment #5 from Paul Howarth  2011-08-07 12:55:36 EDT ---
Could we have branches of this package for all currently-supported EPEL
releases
please? This module works perfectly well all the way back to RHEL-3.

I would be happy to maintain the older branches if you're not interested in
them. I ask this so that I don't need to conditionalize inclusion of
perl(Test::DistManifest) in my own packages, where I try to remain compatible
with older release where possible.

-- 
Configure 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 672543] Review Request: perl-Test-DistManifest - Author test that validates a package MANIFEST

2011-08-07 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=672543

Paul Howarth  changed:

   What|Removed |Added

 CC||p...@city-fan.org

--- Comment #7 from Paul Howarth  2011-08-07 12:51:15 EDT ---
Could we have branches of this package for all currently-supported releases
please (including EPEL)? If the ExtUtils::MakeMaker 6.31 requirement is patched
out of Makefile.PL and the Test::More version requirement is ignored, this
module works perfectly well all the way back to RHEL-3.

I would be happy to maintain the older branches if you're not interested in
them.
I ask this so that I don't need to conditionalize inclusion of this package in
my own packages, where I try to remain compatible with older release where
possible.

-- 
Configure 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 728275] Review Request: perl-Data-Section-Simple - Read data from __DATA__

2011-08-07 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=728275

Paul Howarth  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #5 from Paul Howarth  2011-08-07 11:22:41 EDT ---
New Package SCM Request
===
Package Name: perl-Data-Section-Simple
Short Description: Read data from __DATA__
Owners: pghmcfc
Branches: EL-4 EL-5 EL-6 F-14 F-15 F-16
InitialCC: perl-sig

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


[Bug 724942] Review Request: libmodbus - A Modbus library written in C

2011-08-07 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=724942

--- Comment #22 from Stéphane Raimbault  
2011-08-07 10:31:16 EDT ---
I've tried to build my package for f17 but the build fails with this error:

+ autoreconf
src/Makefile.am:1: Libtool library used but `LIBTOOL' is undefined
src/Makefile.am:1:   The usual way to define `LIBTOOL' is to add
`AC_PROG_LIBTOOL'
src/Makefile.am:1:   to `configure.ac' and run `aclocal' and `autoconf' again.
src/Makefile.am:1:   If `AC_PROG_LIBTOOL' is in `configure.ac', make sure
src/Makefile.am:1:   its definition is in aclocal's search path.
autoreconf: automake failed with exit status: 1

I don't use AC_PROG_LIBTOOL in my configure.ac because these options are
deprecated
http://www.gnu.org/software/libtool/manual/html_node/LT_005fINIT.html

Do I need to check against the version of libtool LT_PREREQ([2.4]) or to use
older options for backward compatibility?

-- 
Configure 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 716701] Review Request: perl-FusionInventory-Agent-Task-SNMPQuery - SNMP Query support for FusionInventory Agent

2011-08-07 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=716701

Haïkel Guémar  changed:

   What|Removed |Added

 CC||karlthe...@gmail.com
   Flag||fedora-review+

--- Comment #1 from Haïkel Guémar  2011-08-07 10:09:27 
EDT ---
Another trivial review
perl-FusionInventory-Agent-Task-SNMPQuery (Perl package)

MUST: rpmlint must be run on src.rpm and rpm.  OK
$ rpmlint -iv perl-FusionInventory-Agent-Task-SNMPQuery-1.3-1.fc17.src.rpm 
perl-FusionInventory-Agent-Task-SNMPQuery.src: I: checking
perl-FusionInventory-Agent-Task-SNMPQuery.src: I: checking-url
http://forge.fusioninventory.org/projects/fusioninventory-agent-task-snmpquery
(timeout 10 seconds)
perl-FusionInventory-Agent-Task-SNMPQuery.src: I: checking-url
http://search.cpan.org/CPAN/authors/id/F/FU/FUSINV/FusionInventory-Agent-Task-SNMPQuery-1.3.tar.gz
(timeout 10 seconds)
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

$ rpmlint -iv perl-FusionInventory-Agent-Task-SNMPQuery-1.3-1.fc17.noarch.rpm
perl-FusionInventory-Agent-Task-SNMPQuery.noarch: I: checking
perl-FusionInventory-Agent-Task-SNMPQuery.noarch: I: checking-url
http://forge.fusioninventory.org/projects/fusioninventory-agent-task-snmpquery
(timeout 10 seconds)
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

MUST: package named accordingly to package naming guidelines. OK 

MUST: spec file name match %{name}  OK

MUST: package meets packaging guidelines OK

MUST: package must be licensed under a fedora-compliant license OK (GPLv2+)

MUST: License field in package spec match actual license OK
License file included and distributed

MUST: spec in legible american english OK

MUST: sources provided match upstream's OK

provided sources sha1sum: 2fe44c328879f55f5c49c2a69749d35a77bd404a
upstream sources sha1sum: 2fe44c328879f55f5c49c2a69749d35a77bd404a

MUST: package successfully compiles on at least one primary architecture OK
(all of them)

MUST: all build dependencies are listed in BR OK

MUST: package must own all directories it creates OK

MUST: package does not list a file more than once in %files section OK

MUST: permissions are properly set OK

MUST: package consistenly use macros OK

MUST: package contains permissable content OK

MUST: package does not own directories owned by other packages OK

MUST: all filenames in package are valid UTF-8 OK

SHOULD: mock builds were done on rawhide on all primary architectures
(x86/x86_64) OK


SHOULD: man pages are provided OK

This package respects general & Perl Fedora Packaging guidelines, so i hereby
approve this package into Fedora Packages Collection

-- 
Configure 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 716698] Review Request: perl-FusionInventory-Agent-Task-NetDiscovery - Network discovery support for FusionInventory Agent

2011-08-07 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=716698

Haïkel Guémar  changed:

   What|Removed |Added

 CC||karlthe...@gmail.com
 AssignedTo|nob...@fedoraproject.org|karlthe...@gmail.com
   Flag||fedora-review+

--- Comment #1 from Haïkel Guémar  2011-08-07 09:52:32 
EDT ---
quite a trivial review

perl-FusionInventory-Agent-Task-NetDiscovery (Perl package)

MUST: rpmlint must be run on src.rpm and rpm.  OK
$ rpmlint -iv perl-FusionInventory-Agent-Task-NetDiscovery-1.5-2.fc17.src.rpm  
perl-FusionInventory-Agent-Task-NetDiscovery.src: I: checking
perl-FusionInventory-Agent-Task-NetDiscovery.src: I: checking-url
http://forge.fusioninventory.org/projects/fusioninventory-agent-task-netdiscovery
(timeout 10 seconds)
perl-FusionInventory-Agent-Task-NetDiscovery.src: I: checking-url
http://search.cpan.org/CPAN/authors/id/F/FU/FUSINV/FusionInventory-Agent-Task-NetDiscovery-1.5.tar.gz
(timeout 10 seconds)
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

$ rpmlint -iv
perl-FusionInventory-Agent-Task-NetDiscovery-1.5-2.fc17.noarch.rpm
perl-FusionInventory-Agent-Task-NetDiscovery.noarch: I: checking
perl-FusionInventory-Agent-Task-NetDiscovery.noarch: I: checking-url
http://forge.fusioninventory.org/projects/fusioninventory-agent-task-netdiscovery
(timeout 10 seconds)
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

MUST: package named accordingly to package naming guidelines. OK 

MUST: spec file name match %{name}  OK

MUST: package meets packaging guidelines OK

MUST: package must be licensed under a fedora-compliant license OK (GPLv2+)

MUST: License field in package spec match actual license OK
License file included and distributed

MUST: spec in legible american english OK

MUST: sources provided match upstream's OK

provided sources sha1sum: e66c6bcdb77f8a4b8f9cc0927dc247dcb90b6888
upstream sources sha1sum: e66c6bcdb77f8a4b8f9cc0927dc247dcb90b6888

MUST: package successfully compiles on at least one primary architecture OK
(all of them)

MUST: all build dependencies are listed in BR OK

MUST: package must own all directories it creates OK

MUST: package does not list a file more than once in %files section OK

MUST: permissions are properly set OK

MUST: package consistenly use macros OK

MUST: package contains permissable content OK

MUST: package does not own directories owned by other packages OK

MUST: all filenames in package are valid UTF-8 OK

SHOULD: mock builds were done on rawhide on all primary architectures
(x86/x86_64) OK


SHOULD: man pages are provided OK

This package respects general & Perl Fedora Packaging guidelines, so i hereby
approve this package into Fedora Packages Collection

-- 
Configure 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 709180] Review Request: jackctlmmc - control JACK transport via MIDI

2011-08-07 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=709180

--- Comment #10 from Jon Ciesla  2011-08-07 09:49: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 727646] Review Request: link-grammar - A Grammar Checking library

2011-08-07 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=727646

--- Comment #7 from Jon Ciesla  2011-08-07 09:49:36 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 716697] Review Request: perl-FusionInventory-Agent-Task-ESX - vCenter/ESX/ESXi remote inventory for FusionInventory Agent

2011-08-07 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=716697

Haïkel Guémar  changed:

   What|Removed |Added

   Flag||fedora-review?

--- Comment #2 from Haïkel Guémar  2011-08-07 09:40:27 
EDT ---
perl-FusionInventory-Agent-Task-ESX (Perl package)

MUST: rpmlint must be run on src.rpm and rpm.  OK
$ rpmlint perl-FusionInventory-Agent-Task-ESX-1.1.2-1.fc17.src.rpm 
perl-FusionInventory-Agent-Task-ESX.src: W: spelling-error Summary(en_US)
vCenter -> v Center, center, cementer
perl-FusionInventory-Agent-Task-ESX.src: W: summary-not-capitalized C
vCenter/ESX/ESXi remote inventory for FusionInventory Agent
perl-FusionInventory-Agent-Task-ESX.src: W: spelling-error %description -l
en_US vCenter -> v Center, center, cementer
perl-FusionInventory-Agent-Task-ESX.src: W: spelling-error %description -l
en_US ocsinventory -> inventory
1 packages and 0 specfiles checked; 0 errors, 4 warnings.

$ rpmlint -iv perl-FusionInventory-Agent-Task-ESX-1.1.2-1.fc17.noarch.rpm  
   
 (0)
perl-FusionInventory-Agent-Task-ESX.noarch: I: checking
perl-FusionInventory-Agent-Task-ESX.noarch: W: spelling-error Summary(en_US)
vCenter -> v Center, center, cementer
The value of this tag appears to be misspelled. Please double-check.

perl-FusionInventory-Agent-Task-ESX.noarch: W: summary-not-capitalized C
vCenter/ESX/ESXi remote inventory for FusionInventory Agent
Summary doesn't begin with a capital letter.

perl-FusionInventory-Agent-Task-ESX.noarch: W: spelling-error %description -l
en_US vCenter -> v Center, center, cementer
The value of this tag appears to be misspelled. Please double-check.

perl-FusionInventory-Agent-Task-ESX.noarch: W: spelling-error %description -l
en_US ocsinventory -> inventory
The value of this tag appears to be misspelled. Please double-check.

perl-FusionInventory-Agent-Task-ESX.noarch: I: checking-url
http://forge.fusioninventory.org/projects/fusioninventory-agent-task-esx
(timeout 10 seconds)
1 packages and 0 specfiles checked; 0 errors, 4 warnings.

The non-capitalized summary is ok, since it would alter the proper spelling of
vCenter.

MUST: package named accordingly to package naming guidelines. OK 

MUST: spec file name match %{name}  OK

MUST: package meets packaging guidelines OK

MUST: package must be licensed under a fedora-compliant license OK (GPLv2+)

MUST: License field in package spec match actual license OK
License file included and distributed

MUST: spec in legible american english OK

MUST: sources provided match upstream's OK

provided sources sha1sum: dc5b24fe1e9f0f7d288448cc34435363a42566f9
upstream sources sha1sum: dc5b24fe1e9f0f7d288448cc34435363a42566f9

MUST: package successfully compiles on at least one primary architecture OK
(all of them)

MUST: all build dependencies are listed in BR KO

MUST: package must own all directories it creates OK

MUST: package does not list a file more than once in %files section OK

MUST: permissions are properly set OK

MUST: package consistenly use macros OK

MUST: package contains permissable content OK

MUST: package does not own directories owned by other packages OK

MUST: all filenames in package are valid UTF-8 OK

SHOULD: mock builds were done on rawhide on all primary architectures
(x86/x86_64) KO
tests fail due to missing BR: perl(LWP::Protocol::https)
I added it and mock builds went fine

SHOULD: man pages are provided OK

This package respects general & Perl Fedora Packaging guidelines, as soon as
the missing BR will be added, it will be 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 728275] Review Request: perl-Data-Section-Simple - Read data from __DATA__

2011-08-07 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=728275

Mario Blättermann  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #4 from Mario Blättermann  2011-08-07 09:32:15 
EDT ---
OK.



PACKAGE 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 716697] Review Request: perl-FusionInventory-Agent-Task-ESX - vCenter/ESX/ESXi remote inventory for FusionInventory Agent

2011-08-07 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=716697

Haïkel Guémar  changed:

   What|Removed |Added

 CC||karlthe...@gmail.com
 AssignedTo|nob...@fedoraproject.org|karlthe...@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 728407] Review Request: xqilla - XQuery and XPath 2.0 library, built on top of Xerces-C

2011-08-07 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=728407

--- Comment #4 from Haïkel Guémar  2011-08-07 08:55:37 
EDT ---
spec and src.rpm above were updated
changes summary:
* commented up the gcc46 patch line
* use %{?_isa} for arch-dependent requires
* use -p to install (the shell variable should be set at configure time or it
won't be used)
* add LICENSE file to -doc subpackage
* for the exit issue, it's flex C generated code, so using C++ exceptions
instead of aborting requires either editing generated code (ugly and
error-prone) or rewriting the command-line parser.

-- 
Configure 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 715498] Review Request: elmer-hutiter - Open Source Finite Element Software for Multiphysical Problems

2011-08-07 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=715498

--- Comment #2 from Ankur Sinha  2011-08-07 08:19:41 
EDT ---
http://ankursinha.fedorapeople.org/elmer/hutiter/elmer-hutiter.spec

http://ankursinha.fedorapeople.org/elmer/hutiter/elmer-hutiter-0-0.1.20110807.svn5280.fc15.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 715618] Review Request: elmer-eio - Open Source Finite Element Software for Multiphysical Problems

2011-08-07 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=715618

--- Comment #2 from Ankur Sinha  2011-08-07 08:08:43 
EDT ---
http://ankursinha.fedorapeople.org/elmer/eio/elmer-eio.spec

http://ankursinha.fedorapeople.org/elmer/eio/elmer-eio-0-0.1.20110807.svn5280.fc15.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 715503] Review Request: elmer-meshgen2d - Open Source Finite Element Software for Multiphysical Problems

2011-08-07 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=715503

--- Comment #2 from Ankur Sinha  2011-08-07 07:56:34 
EDT ---
http://ankursinha.fedorapeople.org/elmer/meshgen2d/elmer-meshgen2d.spec

http://ankursinha.fedorapeople.org/elmer/meshgen2d/elmer-meshgen2d-0-0.1.20110807.svn5280.fc15.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 705426] Review Request: blender25 - 3D modeling, animation, rendering and post-production

2011-08-07 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=705426

--- Comment #13 from Richard Shaw  2011-08-07 07:51:20 
EDT ---
Here's my package

SPEC: http://hobbes1069.fedorapeople.org/blender/blender.spec
SRPM: http://hobbes1069.fedorapeople.org/blender/blender-2.58-1.fc15.src.rpm

It included the new cycles rendering engine (requires OpenImageIO which just
made it to stable) and openCOLLADA support.

-- 
Configure 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 715619] Review Request: elmer-elmergrid - Open Source Finite Element Software for Multiphysical Problems

2011-08-07 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=715619

--- Comment #1 from Ankur Sinha  2011-08-07 07:39:26 
EDT ---
Updated packages:

http://ankursinha.fedorapeople.org/elmer/elmergrid/elmer-elmergrid.spec

http://ankursinha.fedorapeople.org/elmer/elmergrid/elmer-elmergrid-0-0.1.20110807.svn5280.fc15.src.rpm

These packages are without metis support. :)

Thanks,
Ankur

-- 
Configure 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 728737] Review Request: libinstpatch - MIDI instrument patch library

2011-08-07 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=728737

Brendan Jones  changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|brendan.jones...@gmail.com
   Flag||fedora-review?

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


[Bug 709180] Review Request: jackctlmmc - control JACK transport via MIDI

2011-08-07 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=709180

Brendan Jones  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #9 from Brendan Jones  2011-08-07 
07:00:36 EDT ---
New Package SCM Request
===
Package Name: jackctlmmc
Short Description: Allows control over JACK transport via Midi
Owners: bsjones
Branches: f14 f15 f16
InitialCC:

-- 
Configure 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 715496] Review Request: elmer-matc - Open Source Finite Element Software for Multiphysical Problems

2011-08-07 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=715496

--- Comment #2 from Ankur Sinha  2011-08-07 06:49:53 
EDT ---
Updated packages:

http://ankursinha.fedorapeople.org/elmer/matc/elmer-matc-0-0.1.20110807.svn5280.fc15.src.rpm

http://ankursinha.fedorapeople.org/elmer/matc/elmer-matc.spec

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


[Bug 728403] Review Request: libepc - Easy Publish and Consume library

2011-08-07 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=728403

--- Comment #6 from Haïkel Guémar  2011-08-07 05:26:19 
EDT ---
Package Change Request
==
Package Name: libepc
New Branches: devel
Owners: hguemar
InitialCC:

-- 
Configure 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 728403] Review Request: libepc - Easy Publish and Consume library

2011-08-07 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=728403

--- Comment #5 from Haïkel Guémar  2011-08-07 05:24:36 
EDT ---
@jon: the scm request went fine for f16 but devel branch still is deprecated
(for usual request, it is implict, but it may be different for deprecated
packages ?). Could you fix that or should i open a ticket at fedora-instracture
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 728757] Review Request: gnumed - The gnumed client

2011-08-07 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=728757

--- Comment #1 from Ankur Sinha  2011-08-07 04:55:47 
EDT ---
http://koji.fedoraproject.org/koji/taskinfo?taskID=3256619

Koji scratch build.

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


[Bug 714327] Review Request: libtpcimgio - Turku PET Centre for image file input and output procedures

2011-08-07 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=714327

Ankur Sinha  changed:

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution||CURRENTRELEASE
Last Closed||2011-08-07 04:49:58

--- Comment #12 from Ankur Sinha  2011-08-07 04:49:58 
EDT ---
Built and pushed to repos. Closing.

-- 
Configure 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 714328] Review Request: xmedcon - A medical image conversion utility and library

2011-08-07 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=714328

Bug 714328 depends on bug 714327, which changed state.

Bug 714327 Summary: Review Request: libtpcimgio - Turku PET Centre for image 
file input and output procedures
https://bugzilla.redhat.com/show_bug.cgi?id=714327

   What|Old Value   |New Value

 Status|ASSIGNED|MODIFIED
 Resolution||CURRENTRELEASE
 Status|MODIFIED|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 715180] Review Request: FreeMat - A free environment for rapid engineering, scientific prototyping and data processing

2011-08-07 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=715180

Ankur Sinha  changed:

   What|Removed |Added

 Blocks||673841(fedora-medical)

-- 
Configure 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 714326] Review Request: libtpcmisc - Miscellaneous PET functions

2011-08-07 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=714326

Ankur Sinha  changed:

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution||CURRENTRELEASE
Last Closed||2011-08-07 04:50:11

--- Comment #14 from Ankur Sinha  2011-08-07 04:50:11 
EDT ---
Built and pushed to repos. Closing.

-- 
Configure 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 714328] Review Request: xmedcon - A medical image conversion utility and library

2011-08-07 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=714328

Bug 714328 depends on bug 714326, which changed state.

Bug 714326 Summary: Review Request: libtpcmisc - Miscellaneous PET functions
https://bugzilla.redhat.com/show_bug.cgi?id=714326

   What|Old Value   |New Value

 Resolution||CURRENTRELEASE
 Status|MODIFIED|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 714327] Review Request: libtpcimgio - Turku PET Centre for image file input and output procedures

2011-08-07 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=714327

Bug 714327 depends on bug 714326, which changed state.

Bug 714326 Summary: Review Request: libtpcmisc - Miscellaneous PET functions
https://bugzilla.redhat.com/show_bug.cgi?id=714326

   What|Old Value   |New Value

 Resolution||CURRENTRELEASE
 Status|MODIFIED|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 728757] Review Request: gnumed - The gnumed client

2011-08-07 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=728757

Ankur Sinha  changed:

   What|Removed |Added

 CC||thinklinux@gmail.com
 Blocks||673841(fedora-medical)
  Alias||gnumed

-- 
Configure 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 728757] New: Review Request: gnumed - The gnumed client

2011-08-07 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: gnumed - The gnumed client

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

   Summary: Review Request: gnumed - The gnumed client
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: sanjay.an...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL: http://ankursinha.fedorapeople.org/gnumed/gnumed.spec
SRPM URL: http://ankursinha.fedorapeople.org/gnumed/gnumed-0.9.9-1.fc15.src.rpm

Description: 
This is the GNUmed Electronic Medical Record. 
Its purpose is to enable doctors to keep a medically sound record 
on their patients' health. Currently it is not fully featured. The 
features provided are, however, tested, in use, and considered 
stable. This package does NOT yet provide functionality for billing 
and stock keeping.

While the GNUmed team has taken the utmost care to make sure the 
medical records are safe at all times you still need to make sure 
you are taking appropriate steps to backup the medical data to a 
safe place at appropriate intervals. Do test your backup and 
disaster recovery procedures, too !

Protect your data! GNUmed itself comes without any warranty 
whatsoever. You have been warned.

Homepage: http://%{name}.org/

This package contains the wxpython client. 

Authors:

Sebastian Hilbert 
Karsten Hilbert 
GNUmed team

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


[Bug 728737] Review Request: libinstpatch - MIDI instrument patch library

2011-08-07 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=728737

--- Comment #2 from Brendan Jones  2011-08-07 
03:13:15 EDT ---
Hi Orcan,

The base package requirement in the subpackages needs to be in the format
"%{name}%{?_isa} = %{version}-%{release}"
(http://fedoraproject.org/wiki/Packaging:Guidelines#Requiring_Base_Package).


Not a blocker but noted use of old FSF address. Most source headers are OK.

bsjones@f15laptop ~$rpmlint /home/rawhide/rpmbuild/SRPMS/libinstpatch*.rpm
libinstpatch.src: W: spelling-error %description -l en_US wavetable -> wave
table, wave-table, vegetable
libinstpatch.src: W: invalid-url Source0: libinstpatch-1.0.0-svn386.tar.bz2
1 packages and 0 specfiles checked; 0 errors, 2 warnings.

bsjones@f15laptop ~$rpmlint
/home/rawhide/rpmbuild/RPMS/x86_64/libinstpatch*.rpm
libinstpatch.x86_64: W: spelling-error %description -l en_US wavetable -> wave
table, wave-table, vegetable
libinstpatch-devel.x86_64: E: incorrect-fsf-address
/usr/include/libinstpatch-1.0/libinstpatch/version.h
libinstpatch-devel.x86_64: E: incorrect-fsf-address
/usr/include/libinstpatch-1.0/libinstpatch/libinstpatch.h
3 packages and 0 specfiles checked; 2 errors, 1 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