search

Perl-Evozon / PearlBee

116 stars 44 forks source link

Changed perlblog to PearlBee #3

Closed ghost closed 10 years ago

ghost commented 10 years ago

I think this should resolve issue #1

Take it with a grain of salt. I am getting an issue, but I don't think it is related to this change:

ctreptow@vm-1:~/Blog$ make test                                                                                                                                         
PERL_DL_NONLAZY=1 /usr/bin/perl "-MExtUtils::Command::MM" "-MTest::Harness" "-e" "undef *Test::Harness::Switches; test_harness(0, 'blib/lib', 'blib/arch')" t/*.
t
t/001_base.t ......... ok   
t/002_index_route.t .. isa check for "exception" failed: DBIx::Class::Storage::DBI::catch {...} (): DBI Connection failed: DBI connect('PearlBee','root',...) fa
iled: Access denied for user 'root'@'localhost' (using password: YES) at /usr/local/share/perl/5.14.2/DBIx/Class/Storage/DBI.pm line 1483. at /home/ctreptow/Blo
g/blib/lib/PearlBee.pm line 39
 is not a string! at (eval 1080) line 95.
        Dancer2::Core::Error::new("Dancer2::Core::Error", "context", Dancer2::Core::Context=HASH(0x43ebba0), "status", 500, "exception", DBIx::Class::Exception=
HASH(0x4f9d6e0)) called at /usr/local/share/perl/5.14.2/Dancer2/Core/Dispatcher.pm line 178
        Dancer2::Core::Dispatcher::response_internal_error(Dancer2::Core::Dispatcher=HASH(0x4384ef8), Dancer2::Core::Context=HASH(0x43ebba0), DBIx::Class::Excep
tion=HASH(0x4f9d6e0)) called at /usr/local/share/perl/5.14.2/Dancer2/Core/Dispatcher.pm line 130
        Dancer2::Core::Dispatcher::_dispatch_route(Dancer2::Core::Dispatcher=HASH(0x4384ef8), Dancer2::Core::Route=HASH(0x4348f88), Dancer2::Core::Context=HASH(
0x43ebba0)) called at /usr/local/share/perl/5.14.2/Dancer2/Core/Dispatcher.pm line 77
        Dancer2::Core::Dispatcher::__ANON__(CODE(0x4416108)) called at /usr/local/share/perl/5.14.2/Return/MultiLevel.pm line 49
        Return::MultiLevel::__ANON__() called at /usr/local/share/perl/5.14.2/Return/MultiLevel.pm line 54
        Return::MultiLevel::__ANON__(CODE(0x4401ff8)) called at /usr/local/share/perl/5.14.2/Dancer2/Core/Dispatcher.pm line 78
        Dancer2::Core::Dispatcher::dispatch(Dancer2::Core::Dispatcher=HASH(0x4384ef8), HASH(0x436b828), Dancer2::Core::Request=HASH(0x4395b28), undef) called at
 /usr/local/share/perl/5.14.2/Dancer2/Core/Dispatcher.pm line 165
        Dancer2::Core::Dispatcher::__ANON__(CODE(0x3306110), Dancer2::Core::Dispatcher=HASH(0x4384ef8), HASH(0x436b828), Dancer2::Core::Request=HASH(0x4395b28))
 called at (eval 166) line 1
        Dancer2::Core::Dispatcher::__ANON__(Dancer2::Core::Dispatcher=HASH(0x4384ef8), HASH(0x436b828), Dancer2::Core::Request=HASH(0x4395b28)) called at (eval 
168) line 2
        Dancer2::Core::Dispatcher::dispatch(Dancer2::Core::Dispatcher=HASH(0x4384ef8), HASH(0x436b828), Dancer2::Core::Request=HASH(0x4395b28)) called at /usr/l
ocal/share/perl/5.14.2/Dancer2/Test.pm line 71
        Dancer2::Test::dancer_response(ARRAY(0x203a998)) called at /usr/local/share/perl/5.14.2/Dancer2/Test.pm line 209
        Dancer2::Test::response_status_is(ARRAY(0x203a998), 200, "a route handler is defined for /") called at /usr/local/share/perl/5.14.2/Dancer2/Test.pm line
 218
        Dancer2::Test::route_exists(ARRAY(0x203a998), "a route handler is defined for /") called at t/002_index_route.t line 9
# Looks like your test exited with 255 before it could output anything.
t/002_index_route.t .. Dubious, test returned 255 (wstat 65280, 0xff00)
Failed 2/2 subtests 

Test Summary Report
-------------------
t/002_index_route.t (Wstat: 65280 Tests: 0 Failed: 0)
  Non-zero exit status: 255
  Parse errors: Bad plan.  You planned 2 tests but ran 0.
Files=2, Tests=1,  2 wallclock secs ( 0.03 usr  0.01 sys +  1.59 cusr  0.32 csys =  1.95 CPU)
Result: FAIL
Failed 1/2 test programs. 0/1 subtests failed.
make: *** [test_dynamic] Error 255
ghost commented 10 years ago

I get there errors I show with your original version (none of my changes) as well. So, I'm going to go to bed tonight thinking it is unrelated. :)

ghost commented 10 years ago

It really helps if you install ALL the dependencies. :)

ctreptow@vm-1:~/Blog$ make test
PERL_DL_NONLAZY=1 /usr/bin/perl "-MExtUtils::Command::MM" "-MTest::Harness" "-e" "undef *Test::Harness::Switches; test_harness(0, 'blib/lib', 'blib/arch')" t/*.
t
t/001_base.t ......... ok   
t/002_index_route.t .. ok   
All tests successful.
Files=2, Tests=3,  2 wallclock secs ( 0.03 usr  0.01 sys +  1.66 cusr  0.28 csys =  1.98 CPU)
Result: PASS
ctreptow@vm-1:~/Blog$
xsawyerx commented 10 years ago

There are a few more things missing. Just noticed:

Sorry for that. It would appear those are the last remains requiring change. I'm waiting to see if this PR is approved, and then rebase from it a commit I want to submit.

Perl-Evozon commented 10 years ago

Thanks a lot Sawyer... really appreciate your help in making this better.

Cheers, Mircea

On Feb 4, 2014, at 12:18 AM, Sawyer X notifications@github.com wrote:

There are a few more things missing. Just noticed:

The db_patches directory includes files which generate tables in a perlblog database. The bin/generate_schema.sh creates the perlblog database mentioned. You removed the default password in config.yml, but it's actually used in bin/generate_schema.sh. bin/app.pl loads and uses perlblog instead of PearlBee. Makefile.PL has more usage of perlblog. Sorry for that. It would appear those are the last remains requiring change. I'm waiting to see if this PR is approved, and then rebase from it a commit I want to submit.

— Reply to this email directly or view it on GitHub.

ghost commented 10 years ago

Thanks! I probably won't get back on this for another 4 or 5 hours at the earliest. On Feb 3, 2014 4:18 PM, "Sawyer X" notifications@github.com wrote:

There are a few more things missing. Just noticed:

  • The db_patches directory includes files which generate tables in a perlblog database.
  • The bin/generate_schema.sh creates the perlblog database mentioned.
  • You removed the default password in config.yml, but it's actually used in bin/generate_schema.sh.
  • bin/app.pl loads and uses perlblog instead of PearlBee.
  • Makefile.PL has more usage of perlblog.

Sorry for that. It would appear those are the last remains requiring change. I'm waiting to see if this PR is approved, and then rebase from it a commit I want to submit.

— Reply to this email directly or view it on GitHubhttps://github.com/Perl-Evozon/Blog/pull/3#issuecomment-34007244 .

xsawyerx commented 10 years ago

This time I'm just complaining from the sidelines, @ctreptow is doing the tedious work. Credit goes to him. :)

ghost commented 10 years ago

Ok, my wife went to the store and the kids are occupied for a bit. Let's see if this last update is the charm!

xsawyerx commented 10 years ago

Looks good to me. I'll submit my small clean up PR soon.

andrei-cacio commented 10 years ago

Thanks guys for the updates!