Skip to content

Conversation

@igorw
Copy link
Contributor

@igorw igorw commented Sep 22, 2013

If you don't like all of this commit, you can apply a subset of the commits. But this is what I would recommend. If you merge this, please also submit it to Packagist.

@RobLoach
Copy link

Looks good.

fcgiget.php Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not acceptable.

Do you know, lot of people still don't use composer ?
So the fcgiget command need to work without composer.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if (file_exists('vendor/autoload.php')
require 'vendor/autoload.php';
else
require 'Adoy/FastCGI/Client.php';

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed.

@igorw
Copy link
Contributor Author

igorw commented Oct 9, 2013

Ping?

adoy added a commit that referenced this pull request Oct 31, 2013
Composer, namespacing, PSR-0 autoloading
@adoy adoy merged commit bb0e722 into adoy:master Oct 31, 2013
@adoy
Copy link
Owner

adoy commented Oct 31, 2013

Sorry about the delay :) I was really busy those last days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants