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

Error processing package with no composer.json file #107

Open
olegabr opened this issue Dec 21, 2020 · 1 comment
Open

Error processing package with no composer.json file #107

olegabr opened this issue Dec 21, 2020 · 1 comment

Comments

@olegabr
Copy link

olegabr commented Dec 21, 2020

Freemius/wordpress-sdk#463

It can not process freemius/wordpress-sdk because it prunes the composer.json file: https://github.com/Freemius/wordpress-sdk/blob/master/package.json#L12

> "vendor/bin/mozart" compose
PHP Warning:  file_get_contents(/home/USERNAME/PROJECTDIR/vendor/freemius/wordpress-sdk//composer.json): failed to open stream: No such file or directory in /home/USERNAME/PROJECTDIR/vendor/coenjacobs/mozart/src/Composer/Package.php on line 33
PHP Warning:  file_get_contents(/home/USERNAME/PROJECTDIR/vendor/freemius/wordpress-sdk/composer.json): failed to open stream: No such file or directory in /home/USERNAME/PROJECTDIR/vendor/coenjacobs/mozart/src/Console/Commands/Compose.php on line 171
PHP Notice:  Trying to get property 'name' of non-object in /home/USERNAME/PROJECTDIR/vendor/coenjacobs/mozart/src/Mover.php on line 103
@coenjacobs
Copy link
Owner

It shouldn't prune the composer.json file if the package is installable via Composer (which I am not sure it actually is, but I'm confident it can). Actually, does this even prune the file after you install it via Composer? The composer.json file is in the repository, so if you composer install it, it should be there still after installation, right?

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

No branches or pull requests

2 participants