Skip to content
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

bugfix for vendor autoload.php file path #250

Merged
merged 1 commit into from Jun 14, 2016

Conversation

ctrl-f5
Copy link
Contributor

@ctrl-f5 ctrl-f5 commented Jun 14, 2016

No description provided.

@GrahamCampbell
Copy link
Member

I don't think this is right?

@colinodell
Copy link
Member

I think it is right. When installed as a Composer dependency you end up with these files:

├── composer.json
└── vendor
    ├── autoload.php
    ├── bin
    │   └── commonmark -> ../league/commonmark/bin/commonmark
    └── league
        └── commonmark
            └── bin
                └── commonmark

To access autoload.php from vendor/league/commonmark/bin/commonmark, you need to back out three levels to where autoload.php exists.

I merged a similar fix for league/html-to-markdown but didn't apply that here as well.

Thanks for contributing the fix!

@colinodell colinodell added the bug Something isn't working right label Jun 14, 2016
@colinodell colinodell merged commit 92ab63b into thephpleague:master Jun 14, 2016
@GrahamCampbell
Copy link
Member

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants