Move out building of the horrible ctx_request closure

Also move out the rest, while we're at it. This is intended to aid a subsequent
refactoring.

Remove some kane-isms that hurt my eyes

Run http-server.t tests on something other than the server root

Remove testsuite's assumptions about where it's mounted on a web server

Make http-server.t use Test::TCP

No dep, as it's author-only and pulled in by Plack already anyway.

Don't use the server.pl script in the live tests

Instead, just spin up a server with Plack::Loader. The script classes are
already tested separately, so we only need this test to make sure live tests
work.

This is intended to aid a subsequent refactoring.

.psgi stuff is now sorted out

::Wx is declared dead

lighttpd and iis6 now work

Reduce code repetition

Add legacy workaround for nginx PATH_INFO

Make legacy IIS env fixing work again

Make legacy lighttpd env fixing work again

Load middlewares where they're actually used

Make _wrapped_legacy_psgi_app do wrapping only

We now pass in the psgi app to wrap, which should make it quite a bit easier to
test. We'll need that to check if the various quirk fixes for different web
servers actually work.

Version 5.89002

The HTTP engine is no more, and was also never required for this test

Avoid loading actual engines in script tests

This way we can run these tests without having FCGI.pm installed.

PSGI, not Plack

Update Upgrading docs

Make getting to a PSGI app saner

All the user should ever need to call is ->psgi_app (previously raw_psgi_app),
to get an unwrapped psgi app as provided by the engine. This is what should be
used when writing .psgi files.

For all other ways of running catalyst apps there's no need to be able to get to
the psgi application that'll actually run, so here we're making everything
that's responsible for reading .psgi files to run them with Catalyst::Script::*
or for wrapping the plain psgi code ref with back-compat middleware private.

Unfortunately this removes some of the docs about the automatic .psgi
loading. These should be added again elsewhere. Also the Upgrading docs need
updating after this.

Version 5.89001

Stop psgi_file.t from recursing

psgi_app is *not* to be used in .psgi files. It's the compat layer that'll give
you either an app loaded from from a .psgi file, or the default PSGI app with
the reverse proxy middleware and stuff.

In .psgi files only raw_psgi_app should ever be used, as that'll return the bare
catalyst psgi app to then be wrapped in middlewares manually.

I see how the naming is shit and confusing though. Suggestions welcome.

Rename Engine::Loader to EngineLoader

It's not an engine, as those exist no more, so move it out of the ::Engine::
namespace.

Changelog updates

Merge branch 'master' into psgi

* master: (22 commits)
  Change repos metadata to git
  Version 5.80032
  Pass the extra restart options
  Fix test
  Changelog
  added myself (dd070) in contributors list
  removed duplicate parameters in return value of _restarter_args
  added more parameters in return value of _restarter_args
  Add a few tests
  Patch to make restarter class configurable / settable in prefs
  Apply more correct fix than 8df53b (I hope)
  Rename test
  Make tests more clear
  Merge revert 8df53bed
  Un-TODO abraxxa's tests
  This scares me, but it fixes stuf work work
  Fix 5.80 bug which causes slurp to fail if called multiple times
  Fix body predicate bug/back compat issue
  Version 5.80031
  added test for chained dispatcher fail on multiple CaptureArgs(0) parts introduced in 5.80030
  ...

Conflicts:
Changes
Makefile.PL
lib/Catalyst.pm
lib/Catalyst/Engine/FastCGI.pm
lib/Catalyst/Runtime.pm

Properly skip known broken tests

Faked passing tests are not cool, and nor are __END__ sections, as those will
break aggregation.

If we actually want to prevent the body of the tests from being compiled,
putting the C<< plan skip_all => ... >> in a BEGIN block is the right thing to
do.

Change repos metadata to git

Version 5.80032

Pass the extra restart options

Fix test

Changelog

Merge branch 'restarter_changeclass' into 5.80032_dev

* restarter_changeclass:
  added myself (dd070) in contributors list
  removed duplicate parameters in return value of _restarter_args
  added more parameters in return value of _restarter_args
  Add a few tests
  Patch to make restarter class configurable / settable in prefs

added myself (dd070) in contributors list

removed duplicate parameters in return value of _restarter_args

added more parameters in return value of _restarter_args

Add a few tests

Patch to make restarter class configurable / settable in prefs

Merge branch 'chained_fix' into 5.80032_dev

* chained_fix:
  Apply more correct fix than 8df53b (I hope)
  Rename test
  Make tests more clear
  Merge revert 8df53bed
  Un-TODO abraxxa's tests

Apply more correct fix than 8df53b (I hope)

Rename test

Make tests more clear

Merge revert 8df53bed

Un-TODO abraxxa's tests

This scares me, but it fixes stuf work work

Fix 5.80 bug which causes slurp to fail if called multiple times

Fix body predicate bug/back compat issue

Dink docs to be more clear on psgi files / converting

Recommending infinite loops in the docs is bad

Update changelog and upgrading

Add failing test of using a psgi file. Are you only meant to call ->raw_psgi_app, or?

Delete obslolete crap from ROADMAP, update version numbers

Tweaks

Fix Engine::Stomp with psgi

Version 5.80031

added test for chained dispatcher fail on multiple CaptureArgs(0) parts introduced in 5.80030

Fix undef warning in Engine::FastCGI

added Engine::PSGI instructions

fix =head levels for 'Upgrading to Catalyst 5.90'

Don't be dumb, auth breaks on new cat, not other way round

Small docs and fixes

Beat the args handling in Script::FastCGI into working condition.

Bump dep

Version 5.89000

This is a trial release from the psgi branch.

Merge branch 'master' into psgi

master:
Version 5.80030
Fix a warning with undef bodies
Changelog the Package::Stash bugfix
Changelog the resp->body improvements
Work with PP Package::Stash again now the incorrect assumption that broke with ::XS is fixed
Stop relying on Package::Stash PP bugs
Add conflict
Bad test failed to show bug.
Fix the case for body '0'
Make response body able to be undef to allow RenderView to see 'defined but empty' body, allowing X-Sendfile to work nicer. Also removes horrible modifier code. win/win if it doesn't break anything else.
Extra links in docs
Add test case for uri_for() with #fragment and query params, broken by 5.7008 when uri_for() was reimplemented without URI.pm.
Found a another fault in chained action dispatcher.

Conflicts:
lib/Catalyst/Engine/CGI.pm

Hit the docs some

Additional notes and cleanup

Version 5.80030

Fix a warning with undef bodies

Changelog the Package::Stash bugfix

Changelog the resp->body improvements

first go at upgrading docs

Work with PP Package::Stash again now the incorrect assumption that broke with ::XS is fixed

Stop relying on Package::Stash PP bugs

We want to know if $MyApp::_config contains something like a valid
configuration, not if the package symbol exists.

The pure-perl version Package::Stash has always been buggy in assuming that a
scalar symbol containing undef is a symbol that doesn't exist. This is to work
around the fact that getting the SCALAR slot from a glob in pure perl
autovivifies the scalar to undef. Package::Stash::XS doesn't have that
limitation and is used transparently by Package::Stash if it's installed. That
exposed this bug in our code.

Add conflict

Bad test failed to show bug.

Fix the case for body '0'

Make response body able to be undef to allow RenderView to see 'defined but empty' body, allowing X-Sendfile to work nicer. Also removes horrible modifier code. win/win if it doesn't break anything else.

Extra links in docs

Add test case for uri_for() with #fragment and query params,
broken by 5.7008 when uri_for() was reimplemented without URI.pm.

Found a another fault in chained action dispatcher.

Merge branch 'master' into psgi

master:
Improved test about chained actions and add note in Changes.

Don't build the full PSGI app more than once

Improved test about chained actions and add note in Changes.

TWMC issues were fixed, we need some more tests however

Fix local_request. TODO - tests for local_request, make ->psgi_app lazy class data again

Update TODO

After startup, tell where the server is listening

Set up MyApp->psgi_app lazily

This fixes the double inclusion through .psgi -> setup_psgi_app -> .psgi when
running with plackup.

Separate raw psgi app and wrapped psgi app

Fix merging mistake made in r13761

Engine::FastCGI is dead.

Merge remote branch 'svn/trunk' into psgi

svn/trunk:
Rename --title to --proc_title as that makes more sense. Remove -t option as it's confusing..
Update changelog

Conflicts:
lib/Catalyst/Engine/FastCGI.pm
lib/Catalyst/Script/FastCGI.pm
t/aggregate/unit_core_script_fastcgi.t

Rename --title to --proc_title as that makes more sense. Remove -t option as it's confusing..

Update changelog

Merge branch 'master' into psgi

master:
Stop passing along @INC
Use the proper way of setting @INC for tests
Aggregate http-server.t tests
Allow aggregating specific tests only
Use the harness instead of all of prove
Make the no-tabs test happy again

Stop passing along @INC

It's pointless - it's always the same anyway.

Use the proper way of setting @INC for tests

Aggregate http-server.t tests

With this, I get to run the full suite in under 2 minutes again, as opposed to
the >3 minutes it took before.

Allow aggregating specific tests only

Use the harness instead of all of prove