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

Don't build the psgi app twice

Make the no-tabs test happy again

Clarify what we really must do in order to merge

Merge branch 'master' into psgi

master: (44 commits)
Fixed chained action order resolution to pass t/aggregate/live__component_controller_action_chained2.t and without break any other test.
Clarify end action after forward / detach
Fix passing file GLOBs to $c->res->body
Clarify docs
Added a --title parameter to fastcgi script, sets process title for FCGI::ProcManager
update MooseX module version due to deprecated warning
typo in docs fixed
Version 5.80029
Patch from the mailing list to clarify view warning
Avoid issues when just using CMOP::Class
Need to use _add_meta_method from new Moose onwards to avoid warnings
Actually bump versions, ACTUAL 5.80028, sorry
Version 5.80028
Bug fix for ctx_request, via matso++
Don't rely on qw() providing PAREN tokens
We use Class::MOP in Catalyst::Utils, so load it
Version 5.80027
Fix bug in the tests
Version 5.80026
Add doc patch for go and captures.
...

Conflicts:
lib/Catalyst/Engine/FastCGI.pm
lib/Catalyst/Script/FastCGI.pm
lib/Catalyst/ScriptRole.pm
lib/Catalyst/Test.pm
t/aggregate/live_component_controller_action_streaming.t

Fixed chained action order resolution to pass t/aggregate/live__component_controller_action_chained2.t and without break any other test.

Clarify end action after forward / detach

Fix passing file GLOBs to $c->res->body

Contrary to the documentation of Catalyst::Response, passing a
filehandle GLOB did not work, since the content-length header was not
computed correctly.

body checked if it was passed a blessed object and only then use stat
to get the file size. In all other cases the passed thing was considered
a string and length was used. Filehandle GLOBs are neither blessed nor
strings, so the content-length header always got set to 15 (length of a
stringified GLOB: "GLOB(0x123456)").

Clarify docs

Added a --title parameter to fastcgi script, sets process title for FCGI::ProcManager

update MooseX module version due to deprecated warning

typo in docs fixed

Version 5.80029

Patch from the mailing list to clarify view warning

Avoid issues when just using CMOP::Class

Need to use _add_meta_method from new Moose onwards to avoid warnings