[Bug 857461] Review Request: perl-CDDB_get - Read the CDDB entry for an audio CD in your drive

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=857461

Petr Pisar  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||perl-CDDB_get-2.28-1.fc19
 Resolution|--- |RAWHIDE
Last Closed||2012-12-06 03:43:01

--- Comment #4 from Petr Pisar  ---
Thank you for the review and the 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 878243] Review Request: rubygem-compass-rails - Integrate Compass into Rails 2.3 and up

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=878243

--- Comment #5 from Vít Ondruch  ---
(In reply to comment #4)
> Spec URL: http://tdawson.fedorapeople.org/rubygems/rubygem-compass-rails.spec
> SRPM URL:
> http://tdawson.fedorapeople.org/rubygems/rubygem-compass-rails-1.0.3-2.fc18.
> src.rpm
> 
> License - Fixed
> Doc subpackage - moved all suggested files in there
> Requires: ruby - removed  (I'll have to start doing that from now on)

Thank you. Will check that later.

> Test Suite:
> I didn't notice that it had bypassed all the tests and returned with a
> positive outcome.  I wish it wouldn't do that.
> I tried with minitest, building and installing all the dependancies (compass
> > 0.12, and rainbow), and after all that, it didn't run any tests either.
> 
> Even without the tests, this package cannot be installed, because it
> requires rubygem-compass > 0.12.2.  It looks like mmorsi is trying to get it
> working but it's currently failling on F19 (while it build on F18 for me)
> 
> http://koji.fedoraproject.org/koji/packageinfo?packageID=10224

Yes, I noticed that as well. Are you in contact with him? I'll continue with
this review as soon as updated compass gets into Fedora.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 876403] Review Request: perl-ExtUtils-CppGuess - Guess C++ compiler and flags

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=876403

Jitka Plesnikova  changed:

   What|Removed |Added

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

--- Comment #7 from Jitka Plesnikova  ---
Spec file changes:

 BuildRequires:  perl(Module::Build) >= 0.36
 BuildRequires:  perl(Test::More)
 BuildRequires:  perl(Capture::Tiny)
+BuildRequires:  perl(ExtUtils::Manifest)
+BuildRequires:  perl(File::Path)
+BuildRequires:  perl(File::Spec::Functions)
+BuildRequires:  perl(Cwd)
+BuildRequires:  perl(Fatal)
+BuildRequires:  perl(Exporter)

> FIX: Add missing BR for tests (./t/lib/TestUtils.pm)
> perl(ExtUtils::Manifest)
> perl(File::Path)
> perl(File::Spec::Functions)
> perl(Cwd)
> perl(Fatal)
> perl(Exporter)
Ok

 %install
 ./Build install destdir=%{buildroot} create_packlist=0
 find %{buildroot} -type f -name '*.bs' -size 0 -exec rm -f {} \;
-find %{buildroot} -depth -type d -exec rmdir {} 2>/dev/null \;

> TODO: Remove the deleting empty directories in %install section. This is 
> default behavior.
Ok.


Package 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 880842] Review Request: pam_url - PAM module to authenticate with HTTP servers

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880842

--- Comment #13 from Fedora Update System  ---
pam_url-0.3.2-2.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/pam_url-0.3.2-2.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 880842] Review Request: pam_url - PAM module to authenticate with HTTP servers

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880842

--- Comment #14 from Fedora Update System  ---
pam_url-0.3.2-2.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/pam_url-0.3.2-2.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 884497] New: Review request: perl-Math-BigInt - Arbitrary size integer/float math package

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884497

Bug ID: 884497
   Summary: Review request: perl-Math-BigInt - Arbitrary size
integer/float math package
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: unspecified
  Priority: unspecified
  Reporter: mmasl...@redhat.com

SPEC: http://mmaslano.fedorapeople.org/review/perl-Math-BigInt.spec
SRPM:
http://mmaslano.fedorapeople.org/review/perl-Math-BigInt-1.997-1.fc18.src.rpm
Description: All operators (including basic math operations) are overloaded if
you
declare your big integers as documented. Operations with overloaded
operators preserve the arguments which is exactly what you expect.
Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=4762700

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 884497] Review request: perl-Math-BigInt - Arbitrary size integer/float math package

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884497

Petr Šabata  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||psab...@redhat.com
   Assignee|nob...@fedoraproject.org|psab...@redhat.com
  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 884604] New: Review Request: rubygem-riot - An extremely fast, expressive, and context-driven unit-testing framework

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884604

Bug ID: 884604
   Summary: Review Request: rubygem-riot - An extremely fast,
expressive, and context-driven unit-testing framework
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Reporter: jstri...@redhat.com

Spec URL: http://data-strzibny.rhcloud.com/rubygem-riot.spec
SRPM URL: http://data-strzibny.rhcloud.com/rubygem-riot-0.12.5-1.fc19.src.rpm
Description: An extremely fast, expressive, and context-driven unit-testing
framework. A replacement for all other testing frameworks. Protest the slow
test.
Fedora Account System Username: jstribny
Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=4762910

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 878501] Review Request: rubygem-ci_reporter - Generate XML for continuous integration systems

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=878501

--- Comment #5 from Miroslav Suchý  ---
This works for me:

rm Gemfile.lock
for i in spec/ci/reporter/*_spec.rb; do
  if [ $i = "spec/ci/reporter/report_manager_spec.rb" -o $i =
"spec/ci/reporter/test_unit_spec.rb" ]; then
continue
  fi
  rspec $i
done

And I had to add
  require 'test/unit/diff'
on top of 
  /usr/share/gems/gems/test-unit-2.4.5/lib/test/unit/ui/console/testrunner.rb


Please report it as bug to rubygem-test-unit and for add that above to %check
as comments plus add as comment link to that test-unit bz. Plus link to reports
to upstream that those two tests are failing (I think at least one is already
reported).

And of course you have to add build requires on hoe and diff-lcs, to make this
above work.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 815018] Review Request: nodejs - javascript fast build framework

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=815018

--- Comment #52 from Stephen Gallagher  ---
I realized I should probably update this bug with the latest developments. I
had a long discussion with upstream regarding the /usr/bin/node name (or
rename) and we finally came to the conclusion that renaming to /usr/bin/nodejs
would actually cause a significant number of issues with a lot of scripts out
in the wild.

I have been in contact with the group owning the ham-radio package 'node' and I
am attempting to convince them to rename in Fedora instead, given the relative
sizes of their communities. If they do not agree, I think I'm going to take the
formal Conflicts: route as described by
http://fedoraproject.org/wiki/Packaging:Conflicts#Incompatible_Binary_Files_with_Conflicting_Naming_.28and_stubborn_upstreams.29

tl;dr version of the above: We're going to stick with the /usr/bin/node name
for this package.



T.C., I approved your request for libuv comaintainership, and thank you!

As far as the npm stuff, I have yet to receive a formal answer from legal, but
the tentative answer I got was this: It's probably a valid free license, but it
is NOT GPL-compatible, so we have to watch out for that. They're going to come
up with a formal name for the license and add it to
https://fedoraproject.org/wiki/Licensing:Main


Also, upstream Node.js is going to be releasing 0.9.4 shortly which has
upstreamed all of the shared-library patches in my package, so I'll submit
another build for this then (just to simplify things).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 876403] Review Request: perl-ExtUtils-CppGuess - Guess C++ compiler and flags

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=876403

Miro Hrončok  changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #8 from Miro Hrončok  ---
New Package SCM Request
===
Package Name: perl-ExtUtils-CppGuess
Short Description: Guess C++ compiler and flags
Owners: churchyard
Branches: f17 f18

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 876403] Review Request: perl-ExtUtils-CppGuess - Guess C++ compiler and flags

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=876403

--- 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 876401] Review Request: perl-ExtUtils-ParseXS - Converts Perl XS code into C code

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=876401

--- Comment #4 from Miro Hrončok  ---
Spec URL:
https://github.com/hroncok/SPECS/raw/master/perl-ExtUtils-ParseXS.spec
SRPM URL:
https://github.com/downloads/hroncok/SPECS/perl-ExtUtils-ParseXS-3.18-2.fc17.src.rpm

- Added missing BR for tests
- Removed deleting empty dirs
- Replaced obsoleted PERL_INSTALL_ROOT with DESTDIR
- Updated %%files to prevent duplicity

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 878247] Review Request: perl-Lingua-EN-Numbers-Easy - Hash access to Lingua::EN::Numbers objects

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=878247

--- Comment #2 from Miro Hrončok  ---
Spec URL:
https://raw.github.com/hroncok/SPECS/master/perl-Lingua-EN-Numbers-Easy.spec
SRPM URL:
https://github.com/downloads/hroncok/SPECS/perl-Lingua-EN-Numbers-Easy-2009110701-3.fc17.src.rpm


- Removed deleting empty dirs
- Replaced obsoleted PERL_INSTALL_ROOT with DESTDIR
- Removed README (same as manpage)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 878247] Review Request: perl-Lingua-EN-Numbers-Easy - Hash access to Lingua::EN::Numbers objects

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=878247

Miro Hrončok  changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #3 from Miro Hrončok  ---
New Package SCM Request
===
Package Name: perl-Lingua-EN-Numbers-Easy
Short Description: Hash access to Lingua::EN::Numbers objects
Owners: churchyard
Branches: f17 f18

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 884041] Review Request: python-logutils - Logging utilities

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884041

Ralph Bean  changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #6 from Ralph Bean  ---
New Package SCM Request
===
Package Name: python-logutils
Short Description: Logging utilities
Owners: ralph
Branches: f18 f17 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 878247] Review Request: perl-Lingua-EN-Numbers-Easy - Hash access to Lingua::EN::Numbers objects

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=878247

--- Comment #4 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 884041] Review Request: python-logutils - Logging utilities

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884041

--- Comment #7 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 876401] Review Request: perl-ExtUtils-ParseXS - Converts Perl XS code into C code

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=876401

Jitka Plesnikova  changed:

   What|Removed |Added

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

--- Comment #5 from Jitka Plesnikova  ---
Spec file changes.

 BuildRequires:  perl(ExtUtils::CBuilder)
+BuildRequires:  perl(Carp)
+BuildRequires:  perl(Cwd)
+BuildRequires:  perl(Exporter)
+BuildRequires:  perl(File::Spec)
+BuildRequires:  perl(File::Temp)
+BuildRequires:  perl(lib)
> * FIX: Missing BR for tests
Ok

 %files
 %doc Changes META.json README
 #%%{perl_vendorlib}/auto/*
-%{perl_vendorlib}/ExtUtils*
+%{perl_vendorlib}/ExtUtils/ParseXS*
+%{perl_vendorlib}/ExtUtils/Typemaps*
> TODO: Update %files to prevent duplicity
Ok

 %install
-make pure_install PERL_INSTALL_ROOT=%{buildroot}
+make pure_install DESTDIR=%{buildroot}
> TODO: Replace variable PERL_INSTALL_ROOT obsoleted with DESTDIR name in 
> %install.
Ok

 find %{buildroot} -type f -name '*.bs' -size 0 -exec rm -f {} \;
-find %{buildroot} -depth -type d -exec rmdir {} 2>/dev/null \;
> TODO: Remove the deleting empty directories in %install section. This is 
> default behavior.
Ok

Package 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 876401] Review Request: perl-ExtUtils-ParseXS - Converts Perl XS code into C code

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=876401

Miro Hrončok  changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #6 from Miro Hrončok  ---
New Package SCM Request
===
Package Name: perl-ExtUtils-ParseXS
Short Description: Converts Perl XS code into C code
Owners: churchyard
Branches: f17 f18

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 884674] New: Review Request: kde-plasma-alsa-volume -- ALSA Volume Control plasmoid

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884674

Bug ID: 884674
   Summary: Review Request: kde-plasma-alsa-volume -- ALSA Volume
Control plasmoid
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Reporter: kaperan...@gmail.com

Spec URL:
https://github.com/F1ash/plasmaVolume/blob/master/kde-plasma-alsa-volume.spec
SRPM URL:
http://kojipkgs.fedoraproject.org//work/tasks/3303/4763303/kde-plasma-alsa-volume-0.41.2-1.fc18.src.rpm
Description:
kde-plasma-alsa-volume
ALSA Volume Control plasmoid

Fedora Account System Username: f1ash

Addition:
succesful build http://koji.fedoraproject.org/koji/taskinfo?taskID=4763303

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 847160] Review Request: eclipse-m2e-core - Maven integration for Eclipse

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=847160

--- Comment #9 from Gerard Ryan  ---
Spec URL:
http://galileo.fedorapeople.org/eclipse-m2e-core/1.1.0-3/eclipse-m2e-core.spec
SRPM URL:
http://galileo.fedorapeople.org/eclipse-m2e-core/1.1.0-3/eclipse-m2e-core-1.1.0-3.fc18.src.rpm

This is the latest that I'm working on. It has all of the changes that Roland
had to make in comment #5 (thanks Roland!), and some others that were necessary
also to get it that far.

Everything appears to work, but when trying to create a new maven project, an
error dialog occurs, caused by the ClassCastException mentioned by Roland
above. It looks like the project gets created fine even though the dialog
appears, but the problem should still not occur. I'll try to solve this today,
there's a chance that I introduced it in one of the patches that I made to get
it to build.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 876403] Review Request: perl-ExtUtils-CppGuess - Guess C++ compiler and flags

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=876403

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 876403] Review Request: perl-ExtUtils-CppGuess - Guess C++ compiler and flags

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=876403

--- Comment #10 from Fedora Update System  ---
perl-ExtUtils-CppGuess-0.07-3.fc17 has been submitted as an update for Fedora
17.
https://admin.fedoraproject.org/updates/perl-ExtUtils-CppGuess-0.07-3.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 884674] Review Request: kde-plasma-alsa-volume -- ALSA Volume Control plasmoid

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884674

Fl@sh  changed:

   What|Removed |Added

 Blocks||656997 (kde-reviews)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 876403] Review Request: perl-ExtUtils-CppGuess - Guess C++ compiler and flags

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=876403

--- Comment #11 from Fedora Update System  ---
perl-ExtUtils-CppGuess-0.07-3.fc18 has been submitted as an update for Fedora
18.
https://admin.fedoraproject.org/updates/perl-ExtUtils-CppGuess-0.07-3.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 884677] New: Review Request: rubygem-rabl - General Ruby templating with JSON, BSON, XML and MessagePack support

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884677

Bug ID: 884677
   Summary: Review Request: rubygem-rabl - General Ruby templating
with JSON, BSON, XML and MessagePack support
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Reporter: jstri...@redhat.com

Spec URL: http://data-strzibny.rhcloud.com/rubygem-rabl.spec
SRPM URL: http://data-strzibny.rhcloud.com/rubygem-rabl-0.7.9-1.fc19.src.rpm
Description: General Ruby templating with JSON, BSON, XML and MessagePack
support.
Fedora Account System Username: jstribny

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 876401] Review Request: perl-ExtUtils-ParseXS - Converts Perl XS code into C code

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=876401

--- Comment #7 from Jon Ciesla  ---
Already exists in Fedora.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 876401] Review Request: perl-ExtUtils-ParseXS - Converts Perl XS code into C code

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=876401

--- Comment #8 from Miro Hrončok  ---
Sorry for 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 876399] Review Request: perl-ExtUtils-Typemaps-Default - Set of useful typemaps

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=876399

Bug 876399 depends on bug 876401, which changed state.

Bug 876401 Summary: Review Request: perl-ExtUtils-ParseXS - Converts Perl XS 
code into C code
https://bugzilla.redhat.com/show_bug.cgi?id=876401

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |DUPLICATE

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 876399] Review Request: perl-ExtUtils-Typemaps-Default - Set of useful typemaps

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=876399

Miro Hrončok  changed:

   What|Removed |Added

 Depends On||584722

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 876401] Review Request: perl-ExtUtils-ParseXS - Converts Perl XS code into C code

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=876401

Miro Hrončok  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |DUPLICATE
Last Closed||2012-12-06 09:53:29

--- Comment #9 from Miro Hrončok  ---


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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 884041] Review Request: python-logutils - Logging utilities

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884041

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 884041] Review Request: python-logutils - Logging utilities

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884041

--- Comment #8 from Fedora Update System  ---
python-logutils-0.3.2-1.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/python-logutils-0.3.2-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 884041] Review Request: python-logutils - Logging utilities

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884041

--- Comment #9 from Fedora Update System  ---
python-logutils-0.3.2-1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/python-logutils-0.3.2-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 884041] Review Request: python-logutils - Logging utilities

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884041

--- Comment #10 from Fedora Update System  ---
python-logutils-0.3.2-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/python-logutils-0.3.2-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 884677] Review Request: rubygem-rabl - General Ruby templating with JSON, BSON, XML and MessagePack support

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884677

Josef Stribny  changed:

   What|Removed |Added

 Depends On||884604

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 884604] Review Request: rubygem-riot - An extremely fast, expressive, and context-driven unit-testing framework

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884604

Josef Stribny  changed:

   What|Removed |Added

 Blocks||884677

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 881903] Review Request: python-zope-interface4 - Forward compat package for Zope 3 Interface Infrastructure

2012-12-06 Thread bugzilla
Product: Fedora EPEL
https://bugzilla.redhat.com/show_bug.cgi?id=881903

--- Comment #8 from Fedora Update System  ---
python-zope-interface4-4.0.2-5.el6 has been submitted as an update for Fedora
EPEL 6.
https://admin.fedoraproject.org/updates/python-zope-interface4-4.0.2-5.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 880716] Review Request: knot - Authoritative DNS server

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880716

Paul Wouters  changed:

   What|Removed |Added

   Assignee|nob...@fedoraproject.org|puiterw...@gmail.com

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 884706] New: Review Request: plantuml - Program to generate UML diagram from a text description

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884706

Bug ID: 884706
   Summary: Review Request: plantuml - Program to generate UML
diagram from a text description
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Reporter: jsafr...@redhat.com

Spec URL: http://people.redhat.com/jsafrane/review/plantuml/plantuml.spec
SRPM URL:
http://people.redhat.com/jsafrane/review/plantuml/plantuml-7943-1.fc17.src.rpm
Description:

PlantUML is a program allowing to draw UML diagrams, using a simple
and human readable text description. It is extremely useful for code
documenting, sketching project architecture during team conversations
and so on.

Fedora Account System Username: jsafrane

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 884711] New: Review Request: sphinxcontrib-plantuml - PlantUML extension for Sphinx

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884711

Bug ID: 884711
   Summary: Review Request: sphinxcontrib-plantuml - PlantUML
extension for Sphinx
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Reporter: jsafr...@redhat.com

Spec URL:
http://people.redhat.com/jsafrane/review/sphinxcontrib-plantuml/sphinxcontrib-plantuml.spec
SRPM URL:
http://people.redhat.com/jsafrane/review/sphinxcontrib-plantuml/sphinxcontrib-plantuml-0.3-1.fc17.src.rpm
Description: Plugin to embed PlantUML diagrams in Sphinx, the Python
documentation generator.
Fedora Account System Username: jsafrane

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 815018] Review Request: nodejs - javascript fast build framework

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=815018

--- Comment #53 from Toshio Ernie Kuratomi  ---
(In reply to comment #52)
> 
> I have been in contact with the group owning the ham-radio package 'node'
> and I am attempting to convince them to rename in Fedora instead, given the
> relative sizes of their communities. If they do not agree, I think I'm going
> to take the formal Conflicts: route as described by
> http://fedoraproject.org/wiki/Packaging:
> Conflicts#Incompatible_Binary_Files_with_Conflicting_Naming_.
> 28and_stubborn_upstreams.29
> 
Unfortunately, that doesn't work in this situation because it doesn't satisfy
the "as long as there are no clear cases for both packages to be installed
simultaneously" requirement.  I can't remember the exact example that prompted
that wording but here's a contrived example: "Fedora has a package for udev.  I
want to package udev2 which is a forked version.  No Fedora system is going to
have two udev stacks installed simultaneously, so it's okay for the two
packages to Conflict."

It would seem that these conflicting names falls under:
http://fedoraproject.org/wiki/Packaging:Conflicts#Approaching_Upstream

"If neither upstream renames, we would then approach other distributions [...]
about renaming that can be done in all distros. [...] Once a decision is made,
we would rename the Fedora packages to match."

Since it seems you've looked at other distros and they've settled on renaming
to nodejs, that's probably what we'd want to do in Fedora as well.  it might
even be better to do that even if the amateur radio package were willing to
give up its claim to the name if all other Linux distros are performing the
rename already.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 845107] Review Request: rubygem-openshift-origin-msg-broker-mcollective - OpenShift Origin plugin for mcollective service

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=845107

--- Comment #30 from Fedora Update System  ---
rubygem-openshift-origin-msg-broker-mcollective-1.1.6-2.fc17 has been submitted
as an update for Fedora 17.
https://admin.fedoraproject.org/updates/rubygem-openshift-origin-msg-broker-mcollective-1.1.6-2.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 884497] Review request: perl-Math-BigInt - Arbitrary size integer/float math package

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884497

--- Comment #1 from Petr Šabata  ---
Issues:

1. This module is provided by the perl package and needs to be subpackaged
first to resolve the conflicts.

2. Either buildrequires perl(inc::Module::Install) or explicitly list inc/*
dependencies.

3. Buildrequire the package (lib/*) dependencies.  This is required since you
run the test suite.

4. The same applies to the test suite dependencies (t/*).

5. Require MODULE_COMPAT macro.

6. Your %__provides_exclude macro doesn't make much sense.  In case you wanted
to exlude unversioned Math::BigInt provides, change 'requires' further on the
line to 'provides' too.

7. Require undetected runtime deps (Carp, Exporter, File::Spec)

8. The description is an incomplete sentence.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 876401] Review Request: perl-ExtUtils-ParseXS - Converts Perl XS code into C code

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=876401

--- Comment #10 from Petr Pisar  ---
*** Bug 884686 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 884759] New: Review Request: toilet - colorful ASCII art generator

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884759

Bug ID: 884759
   Summary: Review Request: toilet - colorful ASCII art generator
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: unspecified
  Reporter: zidkova.zuz...@seznam.cz

Spec URL: http://jnovy.fedorapeople.org/toilet/toilet.spec
SRPM URL: http://jnovy.fedorapeople.org/toilet/toilet-0.2-1.fc18.src.rpm

Description: 

The TOIlet project attempts to create a free replacement for the FIGlet
utility. TOIlet stands for "The Other Implementation's letters", coined
after FIGlet's "Frank, Ian and Glen's letters".

TOIlet is in its very early development phase. It uses the powerful
libcaca library to achieve various text-based effects. TOIlet
implements or plans to implement the following features:

The ability to load FIGlet fonts
Support for Unicode input and output
Support for colour fonts
Support for colour output
Support for various output formats: HTML, IRC, ANSI... 

TOIlet also aims for full FIGlet compatibility. It is currently able to
load FIGlet fonts and perform horizontal smushing.
Fedora Account System Username:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 884807] New: Review Request: python-paste-deploy1.5 - Load, configure, and compose WSGI applications and servers

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884807

Bug ID: 884807
   Summary: Review Request: python-paste-deploy1.5 - Load,
configure, and compose WSGI applications and servers
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: unspecified
  Priority: unspecified
  Reporter: rb...@redhat.com


Spec URL: http://ralph.fedorapeople.org//python-paste-deploy1.5.spec
SRPM URL:
http://ralph.fedorapeople.org//python-paste-deploy1.5-1.5.0-1.fc18.src.rpm

Description:
This tool provides code to load WSGI applications and servers from
URIs; these URIs can refer to Python Eggs for INI-style configuration
files.  PasteScript provides commands to serve applications based on
this configuration file.

This is a forwards-compat package for 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 884807] Review Request: python-paste-deploy1.5 - Load, configure, and compose WSGI applications and servers

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884807

--- Comment #1 from Ralph Bean  ---
This package built on koji: 
http://koji.fedoraproject.org/koji/taskinfo?taskID=4764317

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 880716] Review Request: knot - Authoritative DNS server

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880716

Patrick Uiterwijk  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED

--- Comment #3 from Patrick Uiterwijk  ---
As already discussed, I will do this 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 884807] Review Request: python-paste-deploy1.5 - Load, configure, and compose WSGI applications and servers

2012-12-06 Thread bugzilla
Product: Fedora EPEL
https://bugzilla.redhat.com/show_bug.cgi?id=884807

Ralph Bean  changed:

   What|Removed |Added

  Component|Package Review  |Package Review
Version|rawhide |el6
Product|Fedora  |Fedora EPEL

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 815018] Review Request: nodejs - javascript fast build framework

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=815018

Kevin Fox  changed:

   What|Removed |Added

 CC||kevin@pnnl.gov

--- Comment #54 from Kevin Fox  ---
How about a compat package?

nodejs would provide /usr/bin/nodejs to match all the other distros. It
wouldn't conflict with ham-radio.

compat-nodejs or something could provide a symlink from /usr/bin/nodejs ->
/usr/bin/node and conflict with the ham-radio package to help with
transitioning older scripts that havent cought up with the distros?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 815018] Review Request: nodejs - javascript fast build framework

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=815018

--- Comment #55 from Stephen Gallagher  ---
Well, the problem is that the other distributions have split completely down
the middle on the issue. Ubuntu was until recently shipping /usr/bin/node, but
they switched to /usr/bin/nodejs... and have been suffering serious breakage
because of it. Nowadays, nearly all Ubuntu users of Node.js rely on a PPA that
provides /usr/bin/node again.

Furthermore, when I approached upstream, they are actively hostile to the
renaming and have asserted that they will not make any effort whatsoever to
support any issues caused by it. So it's really against our interests to use
/usr/bin/nodejs.

I'm still hopeful that the 'node' amateur radio team will agree to rename. They
seemed fairly receptive to the idea (please pardon the pun).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 882090] Review Request: python-nose-progressive - Nose plugin to show a progress bar and tracebacks during tests

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=882090

Mario Blättermann  changed:

   What|Removed |Added

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

--- Comment #5 from Mario Blättermann  ---
-
key:

[+] OK
[.] OK, not applicable
[X] needs work
-

[+] MUST: rpmlint must be run on the source rpm and all binary rpms the build
produces. The output should be posted in the review.
[+] MUST: The package must be named according to the Package Naming Guidelines.
[+] MUST: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption.
[+] MUST: The package must meet the Packaging Guidelines.
[+] MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines.
[+] MUST: The License field in the package spec file must match the actual
license.
GPLv2+
[.] 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.
[+] MUST: The spec file must be written in American English.
[+] MUST: The spec file for the package MUST be legible.
[+] 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.
$ sha256sum *
837ac8f9eb4be5b23a15c516bdd7047727a237832d741f8b61cd4a399f7cdba1 
nose-progressive-1.3.tar.gz
837ac8f9eb4be5b23a15c516bdd7047727a237832d741f8b61cd4a399f7cdba1 
nose-progressive-1.3.tar.gz.orig

[+] MUST: The package MUST successfully compile and build into binary rpms on
at least one primary architecture.
[.] 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.
[+] 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.
[.] MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.
[.] 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.
[+] MUST: Packages must NOT bundle copies of system libraries.
[.] 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.
[+] 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.
[+] 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)
[+] MUST: Permissions on files must be set properly. Executables should be set
with executable permissions, for example.
[+] MUST: Each package must consistently use macros.
[+] MUST: The package must contain code, or permissable content.
[.] 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).
[+] 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.
[.] MUST: Static libraries must be in a -static package.
[.] MUST: Development files must be in a -devel package.
[.] MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency: Requires: %{name}%{?_isa} =
%{version}-%{release}
[.] MUST: Packages must NOT contain any .la libtool archives, these must be
removed in the spec if they are built.
[.] 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 m

[Bug 815018] Review Request: nodejs - javascript fast build framework

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=815018

--- Comment #56 from Toshio Ernie Kuratomi  ---
  If distros aren't all on board with renaming and the amateur radio team
is amenable to renaming their package/binaries then that will work out fine.

If not, then we'll need to look into renaming something despite hostility from
upstream.  We can't please both upstreams in that case :-(

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 824949] Review Request: awesome - Highly configurable, framework window manager for X. Fast, light and extensible

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=824949

Rino Rondan  changed:

   What|Removed |Added

 CC||villadalm...@gmail.com

--- Comment #11 from Rino Rondan  ---
is this package completed ??

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 824949] Review Request: awesome - Highly configurable, framework window manager for X. Fast, light and extensible

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=824949

Peter Lemenkov  changed:

   What|Removed |Added

 CC||lemen...@gmail.com

--- Comment #12 from Peter Lemenkov  ---
(In reply to comment #11)
> is this package completed ??

Yes.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 760357] Review Request: rubygem-systemu - Universal capture of stdout and stderr and handling of child process pid

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=760357

--- Comment #14 from Fedora Update System  ---
Package rubygem-systemu-2.5.2-1.fc18:
* should fix your issue,
* was pushed to the Fedora 18 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing rubygem-systemu-2.5.2-1.fc18'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2012-19857/rubygem-systemu-2.5.2-1.fc18
then log in and leave karma (feedback).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 760357] Review Request: rubygem-systemu - Universal capture of stdout and stderr and handling of child process pid

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=760357

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 876403] Review Request: perl-ExtUtils-CppGuess - Guess C++ compiler and flags

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=876403

--- Comment #12 from Fedora Update System  ---
perl-ExtUtils-CppGuess-0.07-3.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 876403] Review Request: perl-ExtUtils-CppGuess - Guess C++ compiler and flags

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=876403

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 884674] Review Request: kde-plasma-alsa-volume -- ALSA Volume Control plasmoid

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884674

Kevin Kofler  changed:

   What|Removed |Added

 CC||ke...@tigcc.ticalc.org

--- Comment #1 from Kevin Kofler  ---
Uhm, you do know that KMix has an ALSA mode, too (KMIX_PULSEAUDIO_DISABLE=1)?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 878243] Review Request: rubygem-compass-rails - Integrate Compass into Rails 2.3 and up

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=878243

--- Comment #6 from Troy Dawson  ---
I sent mmorsi an email today.  We'll see how quick it can be updated.
His package builds on F18, but it fails on the tests in rawhide (F19)

I'll make some noise on this when rubygem-compass is updated.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 884855] New: Review Request: python-webtest1.3 - Helper to test WSGI applications

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884855

Bug ID: 884855
   Summary: Review Request: python-webtest1.3 - Helper to test
WSGI applications
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: unspecified
  Priority: unspecified
  Reporter: rb...@redhat.com


Spec URL: http://ralph.fedorapeople.org//python-webtest1.3.spec
SRPM URL: http://ralph.fedorapeople.org//python-webtest1.3-1.3.4-3.fc18.src.rpm

Description:
WebTest wraps any WSGI application and makes it easy to send test
requests to that application, without starting up an HTTP server.

This provides convenient full-stack testing of applications written
with any WSGI-compatible framework.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 884855] Review Request: python-webtest1.3 - Helper to test WSGI applications

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884855

--- Comment #1 from Ralph Bean  ---
This package built on koji: 
http://koji.fedoraproject.org/koji/taskinfo?taskID=4764796

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 864187] Review Request: openscad - The Programmers Solid 3D CAD Modeller

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=864187

--- Comment #23 from Mario Blättermann  ---
Scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=4764644

It fails with the following error message:

BuildError: mismatch when analyzing openscad-MCAD-2012.10.31-4.fc19.noarch.rpm,
rpmdiff output was:
removed REQUIRES openscad(x86-64) = 2012.10.31-4.fc19
added   REQUIRES openscad(x86-32) = 2012.10.31-4.fc19

This means, different output will be produced for the arches, although the
package is actually noarch. You could try to find out the reason for the
differences. But maybe there is another solution. You are using two source
packages for openscad, and as you already wrote, openscad doesn't need
openscad-MCAD in the same version (probably, but this is yours). What about to
build openscad-MCAD in an independent source rpm? Anyway, openscad-MCAD is not
in "Requires:" for openscad, so that could be a proper way to get rid of such
errors.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 884674] Review Request: kde-plasma-alsa-volume -- ALSA Volume Control plasmoid

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884674

--- Comment #2 from Fl@sh  ---
(In reply to comment #1)
> Uhm, you do know that KMix has an ALSA mode, too (KMIX_PULSEAUDIO_DISABLE=1)?

but this plasmoid is more convenient, because it allows each device to reflect
on the panel (or workspace) for control. (
http://kde-look.org/content/show.php/ALSA+Volume+Control+plasmoid?content=138990
)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 882701] Review Request: zathura-ps - PS support for zathura via libspectre

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=882701

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 882701] Review Request: zathura-ps - PS support for zathura via libspectre

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=882701

--- Comment #6 from Fedora Update System  ---
zathura-pdf-poppler-0.2.1-4.fc18,zathura-ps-0.2.0-2.fc18 has been submitted as
an update for Fedora 18.
https://admin.fedoraproject.org/updates/zathura-pdf-poppler-0.2.1-4.fc18,zathura-ps-0.2.0-2.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 881431] Review Request: zathura-pdf-poppler - PDF support for zathura via poppler

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=881431

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 881431] Review Request: zathura-pdf-poppler - PDF support for zathura via poppler

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=881431

--- Comment #23 from Fedora Update System  ---
zathura-pdf-poppler-0.2.1-4.fc18,zathura-ps-0.2.0-2.fc18 has been submitted as
an update for Fedora 18.
https://admin.fedoraproject.org/updates/zathura-pdf-poppler-0.2.1-4.fc18,zathura-ps-0.2.0-2.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 884855] Review Request: python-webtest1.3 - Helper to test WSGI applications

2012-12-06 Thread bugzilla
Product: Fedora EPEL
https://bugzilla.redhat.com/show_bug.cgi?id=884855

Ralph Bean  changed:

   What|Removed |Added

 Target Release||---
  Component|Package Review  |Package Review
Version|rawhide |el6
Product|Fedora  |Fedora EPEL

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 878644] Review Request: openshift-origin-broker-util -Utility scripts for the OpenShift Origin broker

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=878644

Michael Scherer  changed:

   What|Removed |Added

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

--- Comment #3 from Michael Scherer  ---
Package is good, except it doesn't fully work, see bug 884860 for issue in
dependant rpms. 

And it should be updated to the latest version.

But I consider this is not blocking, so I will approve it ( as the issues are
not in this package, but others one )

Package Review
==

Key:
[x] = Pass
[!] = Fail
[-] = Not applicable
[?] = Not evaluated
[ ] = Manual review needed


Issues:
===
[!]: Latest version is packaged.
[!]: Package functions as described.

= MUST items =

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exceptions section of Packaging Guidelines.
[x]: Package contains no bundled libraries.
[x]: Changelog in prescribed format.
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Sources contain only permissible code or content.
[x]: Each %files section contains %defattr if rpm < 4.4
[x]: Macros in Summary, %description expandable at SRPM build time.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package requires other packages for directories it uses.
[x]: Package uses nothing in %doc for runtime.
[x]: Package is not known to require ExcludeArch.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Fully versioned dependency in subpackages, if present.
[x]: Package complies to the Packaging Guidelines
[x]: Spec file lacks Packager, Vendor, PreReq tags.
[x]: 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.
[!]: 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]: Package consistently uses macro is (instead of hard-coded directory
 names).
[x]: Package is named using only allowed ASCII characters.
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
 Note: Package contains no Conflicts: tag(s)
[x]: Package do not use a name that already exist
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: Package is not relocatable.
[x]: Requires correct, justified where necessary.
[x]: Sources used to build the package match the upstream source, as provided
 in the spec URL.
[x]: Spec file is legible and written in American English.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[-]: Package contains systemd file(s) if in need.
[x]: File names are valid UTF-8.
[x]: Large documentation must go in a -doc subpackage.
 Note: Documentation size is 10240 bytes in 1 files.
[x]: Packages must not store files under /srv, /opt or /usr/local

= SHOULD items =

Generic:
[x]: Reviewer should test that the package builds in mock.
[x]: Buildroot is not present
[x]: Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
[-]: 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]: Dist tag is present.
[x]: No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x]: Final provides and requires are sane (rpm -q --provides and rpm -q
 --requires).
[!]: Package functions as described.
[!]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[x]: SourceX tarball generation or download is documented.
[x]: SourceX / PatchY prefixed with %{name}.
[x]: SourceX is a working URL.
[x]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: Package should compile and build into binary rpms on all supported
 architectures.
[-]: %check is present and all tests pass.
[x]: Packages should try to preserve timestamps of original installed files.
[x]: Spec use %global instead of %define.

= EXTRA items =

Generic:
[x]: Spec

[Bug 864187] Review Request: openscad - The Programmers Solid 3D CAD Modeller

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=864187

--- Comment #24 from Miro Hrončok  ---
The reason is obviously here (isa part):
Requires:   %{name}%{?_isa} = %{version}-%{release}

But as you say, separating this might be better.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 883514] Review Request: inception - A fireWire physical memory manipulation tool

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=883514

Michael Scherer  changed:

   What|Removed |Added

 CC||m...@zarb.org

--- Comment #1 from Michael Scherer  ---
Hi,

the tarball is not downloadable ( ie, if there is a commit, it will be a
different tarball )

Source0:https://github.com/carmaa/inception/archive/master.tar.gz

See https://fedoraproject.org/wiki/Packaging:SourceURL#Troublesome_URLs

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 883514] Review Request: inception - A fireWire physical memory manipulation tool

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=883514

Michael Scherer  changed:

   What|Removed |Added

   Assignee|nob...@fedoraproject.org|m...@zarb.org
  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 878644] Review Request: openshift-origin-broker-util -Utility scripts for the OpenShift Origin broker

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=878644

--- Comment #4 from Troy Dawson  ---
Thank you for the approval.

1.1.8 is the latest stable release.  The 1.2.x branch is still in
development/QA.

I will work on the bugzilla you mentioned.  Thank you for finding that 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 878644] Review Request: openshift-origin-broker-util -Utility scripts for the OpenShift Origin broker

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=878644

Troy Dawson  changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #5 from Troy Dawson  ---
New Package SCM Request
===
Package Name: openshift-origin-broker-util
Short Description: Utility scripts for the OpenShift Origin broker
Owners: tdawson maxamillion
Branches: f18 f17
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 883512] Review Request: libforensic1394 - A library for performing live memory forensics over firewire

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=883512

Michael Scherer  changed:

   What|Removed |Added

 CC||m...@zarb.org
 Blocks||883514 (inception)
 Depends On|883514 (inception)  |

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 883514] Review Request: inception - A fireWire physical memory manipulation tool

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=883514

Michael Scherer  changed:

   What|Removed |Added

 Blocks|883512 (libforensic1394)|
 Depends On||883512 (libforensic1394)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 878247] Review Request: perl-Lingua-EN-Numbers-Easy - Hash access to Lingua::EN::Numbers objects

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=878247

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 878247] Review Request: perl-Lingua-EN-Numbers-Easy - Hash access to Lingua::EN::Numbers objects

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=878247

--- Comment #5 from Fedora Update System  ---
perl-Lingua-EN-Numbers-Easy-2009110701-3.fc17 has been submitted as an update
for Fedora 17.
https://admin.fedoraproject.org/updates/perl-Lingua-EN-Numbers-Easy-2009110701-3.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 864187] Review Request: openscad - The Programmers Solid 3D CAD Modeller

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=864187

--- Comment #25 from Miro Hrončok  ---
Spec URL: https://raw.github.com/hroncok/SPECS/master/openscad.spec
SRPM URL:
https://github.com/downloads/hroncok/SPECS/openscad-2012.10.31-5.fc17.src.rpm

Removed MCAD from this source package

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 878247] Review Request: perl-Lingua-EN-Numbers-Easy - Hash access to Lingua::EN::Numbers objects

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=878247

--- Comment #6 from Fedora Update System  ---
perl-Lingua-EN-Numbers-Easy-2009110701-3.fc18 has been submitted as an update
for Fedora 18.
https://admin.fedoraproject.org/updates/perl-Lingua-EN-Numbers-Easy-2009110701-3.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 884876] New: Review Request: openscad-MCAD - OpenSCAD Parametric CAD Library

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884876

Bug ID: 884876
   Summary: Review Request: openscad-MCAD - OpenSCAD Parametric
CAD Library
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Reporter: m...@hroncok.cz

Spec URL: https://raw.github.com/hroncok/SPECS/master/openscad-MCAD.spec
SRPM URL:
https://github.com/downloads/hroncok/SPECS/openscad-MCAD-0.0-1.20121031git9af89906fa.fc17.src.rpm

Description:

This library contains components commonly used in designing and moching up
mechanical designs. It is currently unfinished and you can expect some API
changes, however many things are already working.

Fedora Account System Username: churchyard

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 884876] Review Request: openscad-MCAD - OpenSCAD Parametric CAD Library

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=884876

Miro Hrončok  changed:

   What|Removed |Added

 Depends On||864187

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 864187] Review Request: openscad - The Programmers Solid 3D CAD Modeller

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=864187

Miro Hrončok  changed:

   What|Removed |Added

 Blocks||884876

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 882090] Review Request: python-nose-progressive - Nose plugin to show a progress bar and tracebacks during tests

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=882090

--- Comment #6 from Dan Callaghan  ---
Thanks for the review, Mario. Unfortunately I can't add this package until its
dependency python-blessings (bug 882084) is also accepted. If you could take
that review as well I would appreciate it :-)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 883514] Review Request: inception - A fireWire physical memory manipulation tool

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=883514

--- Comment #2 from Michael Scherer  ---
It also need aply and modprobe 
https://github.com/carmaa/inception/blob/master/inception/sound.py
https://github.com/carmaa/inception/blob/master/inception/firewire.py

and libforensic1394 

( and I am not sure if that need some ExcludeArch, but that's a detail )

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 883514] Review Request: inception - A fireWire physical memory manipulation tool

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=883514

--- Comment #3 from Michael Scherer  ---

Package Review
==

Key:
[x] = Pass
[!] = Fail
[-] = Not applicable
[?] = Not evaluated
[ ] = Manual review needed


Issues:
===
[!]: Requires correct, justified where necessary.
[!]: If the source package does not include license text(s) as a separate file
 from upstream, the packager SHOULD query upstream to include it.
[!]: Package functions as described, was not able to test as i need first
the requires to be fixed.
[!]: License field in the package spec file matches the actual license.
license should be GPL v3+, not LGPL v3+

= MUST items =

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exceptions section of Packaging Guidelines.
[x]: Package contains no bundled libraries.
[x]: Changelog in prescribed format.
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Sources contain only permissible code or content.
[x]: Each %files section contains %defattr if rpm < 4.4
[x]: Macros in Summary, %description expandable at SRPM build time.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package requires other packages for directories it uses.
[x]: Package uses nothing in %doc for runtime.
[x]: Package is not known to require ExcludeArch.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Fully versioned dependency in subpackages, if present.
[x]: Package complies to the Packaging Guidelines
[x]: Spec file lacks Packager, Vendor, PreReq tags.
[!]: 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.
 Note: Cannot find licenses in rpm(s)
[!]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 "GPL (v3 or later)", "Unknown or generated". 2 files have unknown
 license. Detailed output of licensecheck in
 /home/misc/checkout/git/FedoraReview/883514-inception/licensecheck.txt
[x]: Package consistently uses macro is (instead of hard-coded directory
 names).
[x]: Package is named using only allowed ASCII characters.
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
 Note: Package contains no Conflicts: tag(s)
[x]: Package do not use a name that already exist
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: Package is not relocatable.
[!]: Requires correct, justified where necessary.
[x]: Sources used to build the package match the upstream source, as provided
 in the spec URL.
[x]: Spec file is legible and written in American English.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[-]: Package contains systemd file(s) if in need.
[x]: File names are valid UTF-8.
[x]: Large documentation must go in a -doc subpackage.
 Note: Documentation size is 61440 bytes in 3 files.
[x]: Packages must not store files under /srv, /opt or /usr/local

Python:
[x]: Package contains BR: python2-devel or python3-devel
[x]: Binary eggs must be removed in %prep
[x]: Python eggs must not download any dependencies during the build process.
[x]: A package which is used by another package via an egg interface should
 provide egg info.
[x]: Package meets the Packaging Guidelines::Python

= SHOULD items =

Generic:
[x]: Reviewer should test that the package builds in mock.
[x]: Buildroot is not present
[x]: Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
[!]: 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]: Dist tag is present.
[x]: No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x]: Final provides and requires are sane (rpm -q --provides and rpm -q
 --requires).
[!]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[x]: SourceX tarball generation or download is documented.
[!]: SourceX / PatchY prefixed with %{name}.
 Note: Source0 (master.tar.gz)
[x]: SourceX is a working URL.
[x]: Description and summary sections in the package spec file contains
 tran

[Bug 878501] Review Request: rubygem-ci_reporter - Generate XML for continuous integration systems

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=878501

--- Comment #6 from Troy Dawson  ---
Spec URL: http://tdawson.fedorapeople.org/rubygems/rubygem-ci_reporter.spec
SRPM URL:
http://tdawson.fedorapeople.org/rubygems/rubygem-ci_reporter-1.8.2-1.fc18.src.rpm

That was the fix we needed for the test.
If we add "require 'test/unit/diff'" to the top of all the tests, then they
wall work, even report_manager_spec.rb and test_unit_spec.rb.

I have informed upstream.
I have also tested it with a newer version of test-unit, and it works without
having to do that.  But for now, I have a sed statement that fixes up the
tests.  Once we have a newer version of test-unit in Fedora we can take that
out.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=g98Ng2atYk&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 880179] Review Request: libnetfilter_cthelper - User-space infrastructure for connection tracking helpers

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=880179

--- Comment #9 from Fedora Update System  ---
libnetfilter_cthelper-1.0.0-3.fc16 has been pushed to the Fedora 16 stable
repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=6yOVxVJjCO&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 861923] Review Request: ghc-hs-bibutils - Haskell binding to bibutils

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=861923

--- Comment #16 from Fedora Update System  ---
ghc-hs-bibutils-4.15-2.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.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=Z15ykJu6a4&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 861923] Review Request: ghc-hs-bibutils - Haskell binding to bibutils

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=861923

--- Comment #17 from Fedora Update System  ---
ghc-hs-bibutils-4.15-2.fc16 has been pushed to the Fedora 16 stable repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=Sd6lbeyyXl&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 878245] Review Request: perl-Lingua-EN-Numbers - Turn "407" into "four hundred and seven", etc

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=878245

--- Comment #14 from Fedora Update System  ---
perl-Lingua-EN-Numbers-1.04-2.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.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=k25i3QOHA6&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 868717] Review Request: libtvdb - Library to retrieve TV series information from web

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=868717

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |CURRENTRELEASE
Last Closed||2012-12-06 22:28:17

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=9wxNdYGCiW&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 868947] Review Request: nepomuktvnamer - Nepomuk TV shows manager

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=868947

Bug 868947 depends on bug 868717, which changed state.

Bug 868717 Summary: Review Request: libtvdb - Library to retrieve TV series 
information from web
https://bugzilla.redhat.com/show_bug.cgi?id=868717

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |CURRENTRELEASE

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=hxabMtVRCk&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 868717] Review Request: libtvdb - Library to retrieve TV series information from web

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=868717

--- Comment #15 from Fedora Update System  ---
libtvdb-0.3.0-3.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.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=ilHng9IEdx&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 851689] Review Request: mingw-libgsf - Library for reading and writing structured files

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=851689

Fedora Update System  changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |CURRENTRELEASE
Last Closed||2012-12-06 22:29:39

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=3twf8RZePR&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 851689] Review Request: mingw-libgsf - Library for reading and writing structured files

2012-12-06 Thread bugzilla
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=851689

--- Comment #9 from Fedora Update System  ---
mingw-libgsf-1.14.25-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.
Unsubscribe from this bug 
https://bugzilla.redhat.com/token.cgi?t=BKW8MHx2O5&a=cc_unsubscribe
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

  1   2   >