Skip to content

The correct registration of binary script for composer #169

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 19, 2015
Merged

Conversation

hason
Copy link
Contributor

@hason hason commented Jul 17, 2015

No description provided.

if (!class_exists('\PHPCR\Shell\PhpcrShell')) {
if (is_file(__DIR__.'/../vendor/autoload.php')) {
require __DIR__.'/../vendor/autoload.php';
} elseif (is_file(__DIR__.'/../../../autoload.php')) {
Copy link
Member

Choose a reason for hiding this comment

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

this is the case of binaries installed in vendor/.composer/bin right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes. This is case of symlink to binary from COMPOSER_BIN_DIR (default vendor/composer/bin) to vendor/phpcr/phpcr-shell/bin.

@lsmith77 lsmith77 added review and removed wip/poc labels Jul 17, 2015
@dantleech
Copy link
Member

The build was failing ofr some random reason. Restarted now, if it is green feel free to merge if I havn't already done so (bad internet connectivity) @dbu

@dbu
Copy link
Member

dbu commented Jul 19, 2015

i can't see how the build failure would be related to the changes here. i restarted the last successful build on master, just to be sure: https://travis-ci.org/phpcr/phpcr-shell/builds/65419628

but i think its more that meanwhile there are too many symfony versions available and thus composer has too many options.

dbu added a commit that referenced this pull request Jul 19, 2015
The correct registration of binary script for composer
@dbu dbu merged commit e435a28 into phpcr:master Jul 19, 2015
@lsmith77 lsmith77 removed the review label Jul 19, 2015
@dbu
Copy link
Member

dbu commented Jul 19, 2015

master now fails as well, so this really is not related. thanks for the fix!

created #170 about the build failures.

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