Bug 484485

Quote from a testrun on F10, after I have installed like this:

yum -y install --nogpgcheck ./reviewtool-0.04-1.fc10.noarch.rpm ./perl-Fedora-App-ReviewTool-0.04-1.fc10.noarch.rpm 



[root@pc64 ~]# reviewtool setup
Could not load class (MooseX::App::Cmd) because : Can't locate MooseX/App/Cmd.pm in @INC (@INC contains: /usr/bin/../lib /usr/lib/perl5/5.10.0/i386-linux-thread-multi /usr/lib/perl5/5.10.0 /usr/local/lib/perl5/site_perl/5.10.0/i386-linux-thread-multi /usr/local/lib/perl5/site_perl/5.10.0 /usr/lib/perl5/vendor_perl/5.10.0/i386-linux-thread-multi /usr/lib/perl5/vendor_perl/5.10.0 /usr/lib/perl5/vendor_perl /usr/local/lib/perl5/site_perl .) at /usr/lib/perl5/vendor_perl/5.10.0/i386-linux-thread-multi/Class/MOP.pm line 151.
 at /usr/lib/perl5/vendor_perl/5.10.0/i386-linux-thread-multi/Class/MOP.pm line 133
        Class::MOP::load_first_existing_class('MooseX::App::Cmd') called at /usr/lib/perl5/vendor_perl/5.10.0/i386-linux-thread-multi/Class/MOP.pm line 157
        Class::MOP::load_class('MooseX::App::Cmd') called at /usr/lib/perl5/vendor_perl/5.10.0/Moose.pm line 52
        Moose::extends('Fedora::App::ReviewTool', 'MooseX::App::Cmd') called at /usr/lib/perl5/vendor_perl/5.10.0/Moose/Exporter.pm line 201
        Moose::extends('MooseX::App::Cmd') called at /usr/lib/perl5/vendor_perl/5.10.0/Fedora/App/ReviewTool.pm line 34
        require Fedora/App/ReviewTool.pm called at /usr/bin/reviewtool line 30
        main::BEGIN() called at /usr/lib/perl5/vendor_perl/5.10.0/Fedora/App/ReviewTool.pm line 0
        eval {...} called at /usr/lib/perl5/vendor_perl/5.10.0/Fedora/App/ReviewTool.pm line 0
Compilation failed in require at /usr/bin/reviewtool line 30.
BEGIN failed--compilation aborted at /usr/bin/reviewtool line 30.

that mean spec need
Requires: perl (MooseX::App::Cmd)

Even after adding that command failed
need
Requires: perl (RPC::XML::Client)

RPM doesn't yet pick up Moose syntax, sadly :\

Spec URL: http://fedorapeople.org/~cweyl/review/perl-Fedora-App-ReviewTool.spec
SRPM URL: http://fedorapeople.org/~cweyl/review/perl-Fedora-App-ReviewTool-0.04-2.fc10.src.rpm

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

some deps are still missing:
# reviewtool setup

couldn't load Fedora::App::ReviewTool::Command::status: Could not load class (Fedora::App::ReviewTool::Bugzilla) because : Could not load class (Moose::Meta::Attribute::Custom::Trait::MooseX::MultiInitArg::Trait) because : Can't locate Moose/Meta/Attribute/Custom/Trait/MooseX/MultiInitArg/Trait.pm in @INC (@INC contains: /usr/bin/../lib /usr/lib/perl5/5.10.0/i386-linux-thread-multi /usr/lib/perl5/5.10.0 /usr/local/lib/perl5/site_perl/5.10.0/i386-linux-thread-multi /usr/local/lib/perl5/site_perl/5.10.0 /usr/lib/perl5/vendor_perl/5.10.0/i386-linux-thread-multi /usr/lib/perl5/vendor_perl/5.10.0 /usr/lib/perl5/vendor_perl /usr/local/lib/perl5/site_perl .) at /usr/lib/perl5/vendor_perl/5.10.0/i386-linux-thread-multi/Class/MOP.pm line 151.

triggered from:

Class::MOP::load_first_existing_class('Moose::Meta::Attribute::Custom::Trait::MooseX::MultiInitArg::...', 'MooseX::MultiInitArg::Trait') called at /usr/lib/perl5/vendor_perl/5.10.0/Moose/Util.pm line 143
        Moose::Util::resolve_metaclass_alias('Attribute', 'MooseX::MultiInitArg::Trait', 'trait', 1) called at /usr/lib/perl5/vendor_perl/5.10.0/Moose/Util.pm line 123
        Moose::Util::resolve_metatrait_alias('Attribute', 'MooseX::MultiInitArg::Trait') called at /usr/lib/perl5/vendor_perl/5.10.0/Moose/Meta/Attribute.pm line 116
        Moose::Meta::Attribute::interpolate_class('Moose::Meta::Attribute', 'predicate', 'has_id', 'builder', '_build_id', 'isa', 'Int', 'clear_on', 'data', ...) called at /usr/lib/perl5/vendor_perl/5.10.0/Moose/Meta/Attribute.pm line 86
        Moose::Meta::Attribute::interpolate_class_and_new('Moose::Meta::Attribute', 'id', 'predicate', 'has_id', 'builder', '_build_id', 'isa', 'Int', 'clear_on', ...) called at /usr/lib/perl5/vendor_perl/5.10.0/Moose/Meta/Class.pm line 510

Looks like MooseX::MultiInitArg is a missing dep of Fedora::Bugzilla...  I'm really beginning to think the rpm/Perl autoprov/req scripts need an overhaul.

I'll get Fedora::Bugzilla rebuilt (after I deal with an expired cert) and post here when it's in rawhide.

Spec URL: http://fedorapeople.org/~cweyl/review/perl-Fedora-App-ReviewTool.spec
SRPM URL: http://fedorapeople.org/~cweyl/review/perl-Fedora-App-ReviewTool-0.04-3.fc10.src.rpm

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

Still no go :(

[root@pc64 ~]# reviewtool status
Retrieving status from bugzilla....

What's your bugzilla login email addy? [nobody] wolfy
Thanks, wolfy. You can set this permanently by running setup.
Please enter your bugzilla password: ***********
1 bugs found.

Fetching bugzilla data:   0% [                                                                                              ]Attribute(reporter) does not pass the type constraint (Fedora::Bugzilla::Types::EmailAddress) with undef at /usr/lib/perl5/vendor_perl/5.10.0/Fedora/App/ReviewTool/Command/status.pm line 97
        Fedora::App::ReviewTool::Command::status::run('Fedora::App::ReviewTool::Command::status=HASH(0xa0668ec)', 'HASH(0x89538dc)', 'ARRAY(0x8aa3034)') called at /usr/lib/perl5/vendor_perl/5.10.0/App/Cmd.pm line 360
        App::Cmd::execute_command('Fedora::App::ReviewTool=HASH(0x89f68e4)', 'Fedora::App::ReviewTool::Command::status=HASH(0xa0668ec)', 'HASH(0x89538dc)') called at /usr/lib/perl5/vendor_perl/5.10.0/App/Cmd.pm line 256
        App::Cmd::run('Fedora::App::ReviewTool') called at /usr/bin/reviewtool line 32

quote from tail /var/log/yum.log
Feb 19 20:45:05 Installed: perl-MooseX-MultiInitArg-0.01-2.fc10.noarch
Feb 19 20:45:05 Updated: perl-Fedora-App-ReviewTool-0.04-3.fc11.noarch
Feb 19 20:45:05 Updated: reviewtool-0.04-3.fc11.noarch

[root@pc64 ~]# rpm -qa \*Bugz\*
perl-Fedora-Bugzilla-0.04-1.fc10.noarch
[root@pc64 ~]# yum update perl*
Setting up Update Process
Examining perl-Fedora-App-ReviewTool-0.04-2.fc10.noarch.rpm: perl-Fedora-App-ReviewTool-0.04-2.fc10.noarch
Package perl-Fedora-App-ReviewTool not installed, cannot update it. Run yum install to install it instead.
Examining perl-Fedora-App-ReviewTool-0.04-3.fc11.noarch.rpm: perl-Fedora-App-ReviewTool-0.04-3.fc11.noarch
Package perl-Fedora-App-ReviewTool not installed, cannot update it. Run yum install to install it instead.
No Packages marked for Update

should I try in rawhide ?

scratch the yum update perl* from above, I've done it correctly now. but the behaviour is the same
and it does not matter if I use the correct password or not, I have the exact same output...
[root@pc64 ~]# reviewtool status
Retrieving status from bugzilla....

What's your bugzilla login email addy? [nobody] wolfy
Thanks, wolfy. You can set this permanently by running setup.
Please enter your bugzilla password: *********** <-------- WRONG PASSWORD HERE
1 bugs found.

Fetching bugzilla data:   0% [                                                                                              ]Attribute(reporter) does not pass the type constraint (Fedora::Bugzilla::Types::EmailAddress) with undef at /usr/lib/perl5/vendor_perl/5.10.0/Fedora/App/ReviewTool/Command/status.pm line 97
        Fedora::App::ReviewTool::Command::status::run('Fedora::App::ReviewTool::Command::status=HASH(0xa2952d4)', 'HASH(0x8b788ac)', 'ARRAY(0x8cc806c)') called at /usr/lib/perl5/vendor_perl/5.10.0/App/Cmd.pm line 360
        App::Cmd::execute_command('Fedora::App::ReviewTool=HASH(0x8c22bb4)', 'Fedora::App::ReviewTool::Command::status=HASH(0xa2952d4)', 'HASH(0x8b788ac)') called at /usr/lib/perl5/vendor_perl/5.10.0/App/Cmd.pm line 256
        App::Cmd::run('Fedora::App::ReviewTool') called at /usr/bin/reviewtool line 32

Spec URL: http://fedorapeople.org/~cweyl/review/perl-Fedora-App-ReviewTool.spec
SRPM URL: http://fedorapeople.org/~cweyl/review/perl-Fedora-App-ReviewTool-0.05-1.fc10.src.rpm

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

(In reply to comment #9)
> scratch the yum update perl* from above, I've done it correctly now. but the
> behaviour is the same
> and it does not matter if I use the correct password or not, I have the exact
> same output...
> [root@pc64 ~]# reviewtool status
> Retrieving status from bugzilla....

This was being caused by a non-authenticated query _not_ returning the email address...  So the typing/coercion of that attribute was causing that issue.

I've updated to just force a bugzilla login for now; the above packages were used for submit/update in a mock rawhide chroot and should work fine now.  (Not sure about F-10, but if it's an issue I'll push out a new Fedora::Bugzilla to F-10 faster than normal.

Can I test the package in a F10 VM or do I have to update it to rawhide ?

(In reply to comment #10)
> Spec URL: http://fedorapeople.org/~cweyl/review/perl-Fedora-App-ReviewTool.spec
> SRPM URL:
> http://fedorapeople.org/~cweyl/review/perl-Fedora-App-ReviewTool-0.05-1.fc10.src.rpm
> 
> Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1206919  

http://koji.fedoraproject.org/koji/taskinfo?taskID=1206919 says " BuildError: error building package (arch noarch), mock exited with status 1; see build.log for more information "

Getting odder... http://koji.fedoraproject.org/koji/taskinfo?taskID=1225331 ( the result of koji build --scratch --arch-override=x86_64 dist-f10 /tmp/perl-Fedora-App-ReviewTool-0.05-1.fc10.src.rpm )

Hmm.

It's bombing out on the external bin requires of koji, which also needs to be a buildrequires:

"Locating required external dependency bin:koji...NA: Unable to build distribution on this platform."

Spec URL: http://fedorapeople.org/~cweyl/review/perl-Fedora-App-ReviewTool.spec
SRPM URL: http://fedorapeople.org/~cweyl/review/perl-Fedora-App-ReviewTool-0.06-1.fc10.src.rpm

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

Sorry for the delay, rough week at work.
Unfortunately it still does not work:

[root@pc64 ~]# reviewtool status
Retrieving status from bugzilla....

What's your bugzilla login email addy? [nobody] wolfy
Thanks, wolfy. You can set this permanently by running setup.
Please enter your bugzilla password: ***********
1 bugs found.


Fetching bugzilla data:   0% [                                                                           ]Attribute (reporter) does not pass the type constraint (Fedora::Bugzilla::Types::EmailAddress) with undef at /usr/lib/perl5/vendor_perl/5.10.0/Fedora/App/ReviewTool/Command/status.pm line 97
        Fedora::App::ReviewTool::Command::status::run('Fedora::App::ReviewTool::Command::status=HASH(0xbe07524)', 'HASH(0xa6ea9bc)', 'ARRAY(0xa83a034)') called at /usr/lib/perl5/vendor_perl/5.10.0/App/Cmd.pm line 360
        App::Cmd::execute_command('Fedora::App::ReviewTool=HASH(0xa794b04)', 'Fedora::App::ReviewTool::Command::status=HASH(0xbe07524)', 'HASH(0xa6ea9bc)') called at /usr/lib/perl5/vendor_perl/5.10.0/App/Cmd.pm line 256
        App::Cmd::run('Fedora::App::ReviewTool') called at /usr/bin/reviewtool line 32


We have two issues here
a) "1 bug found" is printed independent of using a correct or incorrect password
b) it bombs out, once again independent of using a correct or incorrect password

Apologies for the slow pace of this review on my side, too...  Sadly, you're not the only one with a rough workweek :)

Alright, so. The "Attribute (reporter) does not pass the type constraint..." error is generally a good indication that we're not logged into bugzilla yet.  Since 0.05 we should be forcing logins, so this shouldn't be happening.  I'll pull everything into a fresh mock root and see what's going on...

Two new deps up for review; will build 0.08 as soon as those are in rawhide.

0.09 is out at http://fedorapeople.org/~cweyl/reviewtool/.  It needs Fedora::Bugzilla 0.10 (currently making its way through testing) as well as Archive::RPM and MooseX::Role::XMLRPC::Client (both up for review, and out there as well).

We should be correctly pulling in IPC::System::Simple correctly now, as well.

any progress here?

I am still waiting for a "go ahead" and retry building. My last attempt was exactly a week ago and there were still some missing deps which stopped mock build.

Chris, could you please upload perl-Fedora-App-ReviewTool-0.10-1.fc10.src.rpm  ? I see only the binaries at http://fedorapeople.org/~cweyl/reviewtool/

Spec URL: http://fedorapeople.org/~cweyl/review/perl-Fedora-App-ReviewTool.spec
SRPM URL: http://fedorapeople.org/~cweyl/review/perl-Fedora-App-ReviewTool-0.10-1.fc10.src.rpm

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

Apologies for the massive delays with everything as of late: $life and $work are conspiring against free time.

There's a scratch build of the latest above; though I see I need to bump the version of Module::Install (to avoid the CPAN spew) and make sure Fedora::Bugzilla is at 0.10.

I apologize but personal issues that need solving occupy a lot of the time which I would have liked to allocate to package review. Since I want to see this package in a faster line, I am retiring myself from reviewing it, hoping that someone (Jussi ? Parag ?) can take care of it in a quicker pace.

No problem, I've hardly been handling this at the speed I'd like either.

Work/Life are going to continue to essentially dominate my time for the next couple weeks, so I'm going to mark this on the whiteboard as "NotReady" and come back to it in a little bit.

Hi Chris, any progress?

Hi Chris, any progress?

Half a year without progress and multiple pings with no response; I'll close this out.