[Bug 498147] Review Request: perl-App-Nopaste - Easy access to any pastebin

2009-06-02 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=498147





--- Comment #14 from Fedora Update System upda...@fedoraproject.org  
2009-06-02 10:18:07 EDT ---
perl-App-Nopaste-0.10-2.fc10 has been pushed to the Fedora 10 stable
repository.  If problems still persist, please make note of it in this bug
report.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 498147] Review Request: perl-App-Nopaste - Easy access to any pastebin

2009-06-02 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=498147


Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||0.10-2.fc10
 Resolution||NEXTRELEASE




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 498147] Review Request: perl-App-Nopaste - Easy access to any pastebin

2009-06-02 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=498147


Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

   Fixed In Version|0.10-2.fc10 |0.10-2.fc11




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 498147] Review Request: perl-App-Nopaste - Easy access to any pastebin

2009-05-25 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=498147


Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA




--- Comment #12 from Fedora Update System upda...@fedoraproject.org  
2009-05-25 17:09:05 EDT ---
perl-App-Nopaste-0.10-2.fc10 has been pushed to the Fedora 10 testing
repository.  If problems still persist, please make note of it in this bug
report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update perl-App-Nopaste'.  You can
provide feedback for this update here:
http://admin.fedoraproject.org/updates/F10/FEDORA-2009-5420

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 498147] Review Request: perl-App-Nopaste - Easy access to any pastebin

2009-05-25 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=498147





--- Comment #13 from Fedora Update System upda...@fedoraproject.org  
2009-05-25 17:14:15 EDT ---
perl-App-Nopaste-0.10-2.fc11 has been pushed to the Fedora 11 testing
repository.  If problems still persist, please make note of it in this bug
report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update perl-App-Nopaste'.  You can
provide feedback for this update here:
http://admin.fedoraproject.org/updates/F11/FEDORA-2009-5421

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 498147] Review Request: perl-App-Nopaste - Easy access to any pastebin

2009-05-22 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=498147





--- Comment #11 from Fedora Update System upda...@fedoraproject.org  
2009-05-22 03:02:43 EDT ---
perl-App-Nopaste-0.10-2.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/perl-App-Nopaste-0.10-2.fc10

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 498147] Review Request: perl-App-Nopaste - Easy access to any pastebin

2009-05-22 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=498147





--- Comment #10 from Fedora Update System upda...@fedoraproject.org  
2009-05-22 03:02:38 EDT ---
perl-App-Nopaste-0.10-2.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/perl-App-Nopaste-0.10-2.fc11

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 498147] Review Request: perl-App-Nopaste - Easy access to any pastebin

2009-05-03 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=498147


Iain Arnell iarn...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED




--- Comment #8 from Iain Arnell iarn...@gmail.com  2009-05-03 05:51:47 EDT ---
Aargh!  There's already a nopaste rpm (with /usr/bin/nopaste that conflicts
with this one).  Will leave this as a single RPM, and rename the installed
nopaste script and man page.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 498147] Review Request: perl-App-Nopaste - Easy access to any pastebin

2009-05-03 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=498147





--- Comment #9 from Chris Weyl cw...@alumni.drew.edu  2009-05-03 13:55:01 EDT 
---
Hmm... looks like a small ruby script hardcoded to use
http://www.rafb.net/paste.

Given that both of these packages provide similar base functionality (but this
one supproting significantly more), I wonder if it wouldn't be approprate to
just have perl-App-Nopaste explicitly Conflict: with nopaste; or perhaps
leverage the alternatives system (see, e.g. man update-alternatives).  I
admittedly know a very small amount about the alternatives system, but it would
seem to fit this situation...

Anyways, I've probably hijacked the review tix enough :)  Let me know if
there's anything I can do if you decide to go either of those routes.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 498147] Review Request: perl-App-Nopaste - Easy access to any pastebin

2009-04-30 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=498147





--- Comment #7 from Iain Arnell iarn...@gmail.com  2009-04-30 07:38:13 EDT ---
My first thought was that it could be a bit confusing for those coming from
CPAN to install perl-App-Foo and not find the corresponding foo binary
installed. But the more I think about it, I'm starting to like the idea of
giving foo a bit more prominence.

Since this was reviewed as is, I've checked it in without changes, but offer a
split nopaste and perl-App-Nopaste for further consideration before going ahead
and arbitrarily making the change.

Spec URL: http://iarnell.fedorapeople.org/review/perl-App-Nopaste.spec
SRPM URL:
http://iarnell.fedorapeople.org/review/perl-App-Nopaste-0.10-2.fc12.src.rpm

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

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 498147] Review Request: perl-App-Nopaste - Easy access to any pastebin

2009-04-29 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=498147


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

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




--- Comment #1 from Parag AN(पराग) panem...@gmail.com  2009-04-29 02:02:09 
EDT ---
Review:
+ package builds in mock (rawhide i586).
koji Build = http://koji.fedoraproject.org/koji/taskinfo?taskID=1327573
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
78d560ad4fb67cb8ceb0d84c4de40ed0  App-Nopaste-0.10.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ make test gave
All tests successful.
Files=1, Tests=8,  0 wallclock secs ( 0.04 usr  0.01 sys +  0.17 cusr  0.01
csys =  0.23 CPU)
+ Package perl-App-Nopaste-0.10-1.fc12.noarch =
  Provides: perl(App::Nopaste) = 0.10 perl(App::Nopaste::Command)
perl(App::Nopaste::Service) perl(App::Nopaste::Service::Gist)
perl(App::Nopaste::Service::Husk) perl(App::Nopaste::Service::Mathbin)
perl(App::Nopaste::Service::PastebinCom) perl(App::Nopaste::Service::Pastie)
perl(App::Nopaste::Service::Rafb) perl(App::Nopaste::Service::Shadowcat)
perl(App::Nopaste::Service::Snitch)
Requires: /usr/bin/env perl(App::Nopaste) perl(App::Nopaste::Command)
perl(Module::Pluggable) perl(Moose) perl(WWW::Mechanize) perl(base)
perl(strict) perl(warnings)

Suggestion:
1) you don't need following
# autoreq doesn't catch this
Requires:   perl(MooseX::Getopt)
# necessary for optional modules
Requires:   perl(Clipboard)
Requires:   perl(Config::INI::Reader)
Requires:   perl(Git)
Requires:   perl(WWW::Pastebin::PastebinCom::Create)
Requires:   perl(WWW::Pastebin::RafbNet::Create)

Remove them and then import new SRPM. I see yum auto processed all these
Requires

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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 498147] Review Request: perl-App-Nopaste - Easy access to any pastebin

2009-04-29 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=498147





--- Comment #2 from Iain Arnell iarn...@gmail.com  2009-04-29 02:39:52 EDT ---
???!!?!?!

Without those explicit requires in the spec, I get an rpm that doesn't require
any of those packages and results in a broken installation - MooseX-Getopt is
absolutely required - the nopaste command definitely doesn't work without it.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 498147] Review Request: perl-App-Nopaste - Easy access to any pastebin

2009-04-29 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=498147





--- Comment #3 from Parag AN(पराग) panem...@gmail.com  2009-04-29 03:35:13 
EDT ---
Argh! my bad. I forgot to remove them from SPEC while testing and installing
this package locally.

Please add Requires: back.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 498147] Review Request: perl-App-Nopaste - Easy access to any pastebin

2009-04-29 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=498147


Iain Arnell iarn...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 498147] Review Request: perl-App-Nopaste - Easy access to any pastebin

2009-04-29 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=498147





--- Comment #4 from Iain Arnell iarn...@gmail.com  2009-04-29 03:39:50 EDT ---
New Package CVS Request
===
Package Name: perl-App-Nopaste
Short Description: Easy access to any pastebin
Owners: iarnell
Branches: F-10 F-11
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 498147] Review Request: perl-App-Nopaste - Easy access to any pastebin

2009-04-29 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=498147


Chris Weyl cw...@alumni.drew.edu changed:

   What|Removed |Added

 CC||cw...@alumni.drew.edu




--- Comment #5 from Chris Weyl cw...@alumni.drew.edu  2009-04-29 14:48:14 EDT 
---
Just as an aside; I've been trying to split packages in the App::* namespace
into two: the main perl-App-Fizz-Bin and a subpackage of fizz-bin.  To me,
this helps make it clearer to the end user that this isn't just another perl
module package but has an actual executable assoicated with it... especially
when the sub-package is included in comps.xml somewhere.  See, e.g.,
perl-App-Asciio/asciio, perl-Git-CPAN-Patch/git-cpan, perltidy (though perltidy
goes too far in the other direction, ditching a perl-Perl-Tidy at all).

It's certainly not mandated :), but I do think it helps overall adoption of the
programs in question, so I offer it up for consideration.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 498147] Review Request: perl-App-Nopaste - Easy access to any pastebin

2009-04-29 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=498147


Kevin Fenzi ke...@tummy.com changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #6 from Kevin Fenzi ke...@tummy.com  2009-04-30 01:00:49 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review