[Bug 500959] Review Request: perl-Task-Padre-Plugin-Deps - Task::Padre::Plugin::Deps Perl module

2009-11-18 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=500959





--- Comment #17 from Marcela Mašláňová mmasl...@redhat.com  2009-11-18 
09:47:11 EDT ---
Ok, now I really applied correct patch.
http://koji.fedoraproject.org/koji/getfile?taskID=1814160name=build.log

http://mmaslano.fedorapeople.org/review/perl-Task-Padre-Plugin-Deps-0.12-4.fc12.src.rpm

But still there are problems in requirements, you might find in build log lines
like: Warning: prerequisite Acme::PlayCode 0 not found.

These ^ are problems of mentioned three packages, which I have to fix first.

-- 
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 500959] Review Request: perl-Task-Padre-Plugin-Deps - Task::Padre::Plugin::Deps Perl module

2009-11-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=500959


Bug 500959 depends on bug 497283, which changed state.

Bug 497283 Summary: Review Request: perl-Syntax-Highlight-Engine-Kate - Port to 
Perl of the syntax highlight engine of the Kate texteditor
https://bugzilla.redhat.com/show_bug.cgi?id=497283

   What|Old Value   |New Value

 Status|CLOSED  |ASSIGNED
 Resolution|RAWHIDE |



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

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


[Bug 500959] Review Request: perl-Task-Padre-Plugin-Deps - Task::Padre::Plugin::Deps Perl module

2009-11-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=500959


Bug 500959 depends on bug 497283, which changed state.

Bug 497283 Summary: Review Request: perl-Syntax-Highlight-Engine-Kate - Port to 
Perl of the syntax highlight engine of the Kate texteditor
https://bugzilla.redhat.com/show_bug.cgi?id=497283

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE



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

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


[Bug 500959] Review Request: perl-Task-Padre-Plugin-Deps - Task::Padre::Plugin::Deps Perl module

2009-11-01 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=500959





--- Comment #16 from Jan Klepek jan.kle...@hp.com  2009-11-01 17:50:21 EDT ---
still same issue, patch is still the same as before.

-- 
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 500959] Review Request: perl-Task-Padre-Plugin-Deps - Task::Padre::Plugin::Deps Perl module

2009-10-20 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=500959





--- Comment #15 from Marcela Mašláňová mmasl...@redhat.com  2009-10-20 
04:12:25 EDT ---
http://mmaslano.fedorapeople.org/review/perl-Task-Padre-Plugin-Deps-0.12-3.fc12.src.rpm
http://mmaslano.fedorapeople.org/review/perl-Task-Padre-Plugin-Deps.spec

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

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


[Bug 500959] Review Request: perl-Task-Padre-Plugin-Deps - Task::Padre::Plugin::Deps Perl module

2009-10-16 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=500959





--- Comment #12 from Marcela Mašláňová mmasl...@redhat.com  2009-10-16 
04:14:20 EDT ---
(In reply to comment #11)
 It is simply not permitted to download external content at build time; you
 don't even know that the builders have outside network access.  Anything 
 needed
 for the build must be installed, either as part of the default package set or
 as a consequence of a BuildRequires: line.  

No, I don't know about automagicall cpan and that's the reason why I switched
off automaticall installation by patch which is included in srpm /sarcasm

-- 
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 500959] Review Request: perl-Task-Padre-Plugin-Deps - Task::Padre::Plugin::Deps Perl module

2009-10-16 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=500959





--- Comment #13 from Marcela Mašláňová mmasl...@redhat.com  2009-10-16 
04:30:59 EDT ---
(In reply to comment #10)
 it doesn't break build just because dependencies are downloaded automagically
 directly from CPAN, and I'm not ok with that, when mentioned packages exists 
 in
 fedora pkgdb.
 Please add them into BR.  

Ok, it looks like all BR are now in buildroot. I fixed all problems which you
mentioned.

Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1749448
SRPM:
http://mmaslano.fedorapeople.org/review/perl-Task-Padre-Plugin-Deps-0.12-2.fc12.src.rpm
Spec: http://mmaslano.fedorapeople.org/review/perl-Task-Padre-Plugin-Deps.spec

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

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


[Bug 500959] Review Request: perl-Task-Padre-Plugin-Deps - Task::Padre::Plugin::Deps Perl module

2009-10-16 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=500959





--- Comment #14 from Jan Klepek jan.kle...@hp.com  2009-10-16 15:59:40 EDT ---
(In reply to comment #12)
 (In reply to comment #11)
  It is simply not permitted to download external content at build time; you
  don't even know that the builders have outside network access.  Anything 
  needed
  for the build must be installed, either as part of the default package set 
  or
  as a consequence of a BuildRequires: line.  
 
 No, I don't know about automagicall cpan and that's the reason why I switched
 off automaticall installation by patch which is included in srpm /sarcasm  

Which doesn't work.
When I'm building this in mock, it definitely downloads missing BR/requirements
automatically.

You need to comment auto_install; to get rid of this behaviour.
Anyway auto_install is strongly discouraged. See CPAN page for Module::Install.
http://search.cpan.org/~adamk/Module-Install-0.91/lib/Module/Install.pod#Module::Install::AutoInstall.

fixed patch
http://hpejakle.fedorapeople.org/Task-Padre-Plugin-Deps-0.12-switch_off.patch

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

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


[Bug 500959] Review Request: perl-Task-Padre-Plugin-Deps - Task::Padre::Plugin::Deps Perl module

2009-10-15 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #10 from Jan Klepek jan.kle...@hp.com  2009-10-15 16:45:22 EDT ---
it doesn't break build just because dependencies are downloaded automagically
directly from CPAN, and I'm not ok with that, when mentioned packages exists in
fedora pkgdb.
Please add them into BR.

-- 
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 500959] Review Request: perl-Task-Padre-Plugin-Deps - Task::Padre::Plugin::Deps Perl module

2009-10-15 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #11 from Jason Tibbitts ti...@math.uh.edu  2009-10-15 18:58:24 
EDT ---
It is simply not permitted to download external content at build time; you
don't even know that the builders have outside network access.  Anything needed
for the build must be installed, either as part of the default package set or
as a consequence of a BuildRequires: line.

-- 
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 500959] Review Request: perl-Task-Padre-Plugin-Deps - Task::Padre::Plugin::Deps Perl module

2009-10-13 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=500959


Jan Klepek jan.kle...@hp.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
  Status Whiteboard|NotReady|




--- Comment #8 from Jan Klepek jan.kle...@hp.com  2009-10-13 05:27:52 EDT ---
BR missing:
perl-Acme-PlayCode
perl-WebService-Validator-CSS-W3C
perl-WebService-Validator-HTML-W3C

spec file:
if this BR is not needed, remove it at all
#BuildRequires:  perl(Audio::Beep)

- md5sum source ok
- tests ok
- rpmlint clean
- license ok

please fix specfile and BR.

-- 
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 500959] Review Request: perl-Task-Padre-Plugin-Deps - Task::Padre::Plugin::Deps Perl module

2009-10-13 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=500959





--- Comment #9 from Marcela Mašláňová mmasl...@redhat.com  2009-10-13 
07:33:42 EDT ---
BR weren't packaged and it should be partially working without them. If you
insist on packaging also them, we can postpone this review again.

I can add BR now because it doesn't brake build
http://koji.fedoraproject.org/koji/getfile?taskID=1743352name=build.log

-- 
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 500959] Review Request: perl-Task-Padre-Plugin-Deps - Task::Padre::Plugin::Deps Perl module

2009-10-12 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=500959


Jan Klepek jan.kle...@hp.com changed:

   What|Removed |Added

 CC||jan.kle...@hp.com
 AssignedTo|nob...@fedoraproject.org|jan.kle...@hp.com
   Flag||fedora-review?




--- Comment #6 from Jan Klepek jan.kle...@hp.com  2009-10-12 06:57:44 EDT ---
(In reply to comment #0)
 Spec URL:
 http://mmaslano.fedorapeople.org/review/perl-Task-Padre-Plugin-Deps.spec
 SRPM URL:
 http://mmaslano.fedorapeople.org/review/perl-Task-Padre-Plugin-Deps-0.12-1.fc11.src.rpm
not valid anymore
please provide current link.

-- 
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 500959] Review Request: perl-Task-Padre-Plugin-Deps - Task::Padre::Plugin::Deps Perl module

2009-10-12 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=500959





--- Comment #7 from Marcela Mašláňová mmasl...@redhat.com  2009-10-12 
07:37:06 EDT ---
Aha, I accidentally removed them as old. They are back and we have also passed
koji build http://koji.fedoraproject.org/koji/taskinfo?taskID=1741846

-- 
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 500959] Review Request: perl-Task-Padre-Plugin-Deps - Task::Padre::Plugin::Deps Perl module

2009-08-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=500959


Stepan Kasal ska...@redhat.com changed:

   What|Removed |Added

 Depends on||508496




--- Comment #5 from Stepan Kasal ska...@redhat.com  2009-08-25 09:31:03 EDT 
---
(In reply to comment #4)
 Wx::Perl::DataWalker .../usr/bin/perl: symbol lookup error:
 /usr/lib/perl5/vendor_perl/5.10.0/ppc-linux-thread-multi/auto/Wx/Wx.so:
 undefined symbol: Perl_Guse_safe_putenv_ptr

This is bug #508496.

-- 
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 500959] Review Request: perl-Task-Padre-Plugin-Deps - Task::Padre::Plugin::Deps Perl module

2009-06-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=500959


Marcela Maslanova mmasl...@redhat.com changed:

   What|Removed |Added

  Status Whiteboard||NotReady




--- Comment #4 from Marcela Maslanova mmasl...@redhat.com  2009-06-22 
08:00:01 EDT ---
http://koji.fedoraproject.org/koji/getfile?taskID=1429237name=build.log

Wx::Perl::DataWalker .../usr/bin/perl: symbol lookup error:
/usr/lib/perl5/vendor_perl/5.10.0/ppc-linux-thread-multi/auto/Wx/Wx.so:
undefined symbol: Perl_Guse_safe_putenv_ptr

This could be (maybe) fixed with update of threads in perl core.

-- 
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 500959] Review Request: perl-Task-Padre-Plugin-Deps - Task::Padre::Plugin::Deps Perl module

2009-06-19 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=500959


Stepan Kasal ska...@redhat.com changed:

   What|Removed |Added

 CC||ska...@redhat.com




--- Comment #2 from Stepan Kasal ska...@redhat.com  2009-06-19 07:22:04 EDT 
---
One nit in %description Simply - Simple

-- 
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 500959] Review Request: perl-Task-Padre-Plugin-Deps - Task::Padre::Plugin::Deps Perl module

2009-06-19 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=500959


Stepan Kasal ska...@redhat.com changed:

   What|Removed |Added

  Status Whiteboard|NotReady|




--- Comment #3 from Stepan Kasal ska...@redhat.com  2009-06-19 13:14:54 EDT 
---
The build failed because of an unresolved symbol in Class/XSAccessor.so.

To fix that, I updated perl-AutoXS-Header to 1.02-1.fc12 and then rebuild
perl-Class-XSAccessor-1.03-2.fc12

So an hour after now, when the repo catches the *XSAccessor-*-2.fc12 rpm, there
is a chance that the reviewed rpm could build OK.

I'm in a hurry right now, could you please try to build the rpm yourself?
(If it still fails, then please accept my apology and return the NotReady to
whiteboard.)

-- 
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 500959] Review Request: perl-Task-Padre-Plugin-Deps - Task::Padre::Plugin::Deps Perl module

2009-06-10 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=500959


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

  Status Whiteboard||NotReady




--- Comment #1 from Jason Tibbitts ti...@math.uh.edu  2009-06-10 19:14:01 EDT 
---
Since the blockers are all closed, I tried building this again but it fails in
rawhide due to lack of File::Spec = 3.2701.  I guess the core Perl package
needs an update.

Please clear the whiteboard when this package becomes buildable.

-- 
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 500959] Review Request: perl-Task-Padre-Plugin-Deps - Task::Padre::Plugin::Deps Perl module

2009-06-04 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=500959


Bug 500959 depends on bug 498189, which changed state.

Bug 498189 Summary: Review Request: perl-JavaScript-Minifier-XS - XS based 
JavaScript minifier
https://bugzilla.redhat.com/show_bug.cgi?id=498189

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE



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

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


[Bug 500959] Review Request: perl-Task-Padre-Plugin-Deps - Task::Padre::Plugin::Deps Perl module

2009-05-18 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=500959


Bug 500959 depends on bug 500958, which changed state.

Bug 500958 Summary: Review Request: perl-PAR-Packer - PAR Packager
https://bugzilla.redhat.com/show_bug.cgi?id=500958

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE



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

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


[Bug 500959] Review Request: perl-Task-Padre-Plugin-Deps - Task::Padre::Plugin::Deps Perl module

2009-05-17 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=500959


Bug 500959 depends on bug 497283, which changed state.

Bug 497283 Summary: Review Request: perl-Syntax-Highlight-Engine-Kate - Port to 
Perl of the syntax highlight engine of the Kate texteditor
https://bugzilla.redhat.com/show_bug.cgi?id=497283

   What|Old Value   |New Value

 Resolution||RAWHIDE
 Status|ASSIGNED|CLOSED



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

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


[Bug 500959] Review Request: perl-Task-Padre-Plugin-Deps - Task::Padre::Plugin::Deps Perl module

2009-05-15 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Marcela Maslanova mmasl...@redhat.com changed:

   What|Removed |Added

 Depends on||498189, 500958, 497283




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