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

Unable to generate prefixed PHAR #1507

Closed
maks-rafalko opened this issue Apr 25, 2021 · 2 comments
Closed

Unable to generate prefixed PHAR #1507

maks-rafalko opened this issue Apr 25, 2021 · 2 comments
Labels

Comments

@maks-rafalko
Copy link
Member

maks-rafalko commented Apr 25, 2021

Seems like after upgrading ocramius/package-versions to 1.11.0 in #1496, it's now not possible to generate prefixed PHAR (make compile).

When I use Box 3.10.0, the PHAR is generated, but when it's run, I get the following error:

PHP Fatal error:  Uncaught Error: Class "_HumbugBoxb4b9e03217d0\Composer\InstalledVersions" not found in phar:///home/maksrafalko/apps/infection/build/infection.phar/vendor/ocramius/package-versions/src/PackageVersions/Versions.php:28
Stack trace:
#0 phar:///home/maksrafalko/apps/infection/build/infection.phar/src/Console/Application.php(36): _HumbugBoxb4b9e03217d0\PackageVersions\Versions::getVersion()
#1 phar:///home/maksrafalko/apps/infection/build/infection.phar/bin/infection(71): _HumbugBoxb4b9e03217d0\Infection\Console\Application->__construct()
#2 /home/maksrafalko/apps/infection/build/infection.phar(14): require('phar:///home/ma...')
#3 {main}
  thrown in phar:///home/maksrafalko/apps/infection/build/infection.phar/vendor/ocramius/package-versions/src/PackageVersions/Versions.php on line 28

When I try to upgrade Box to the latest version, I got the following error (reported to Box project): box-project/box#549

So I'm not sure if this is an issue on our side with misconfiguration or the issue on incompatibility between ocramius/package-versions and 3.10.0 Box version.

@theofidry probably you can help here

@sidz
Copy link
Member

sidz commented Jul 6, 2021

@maks-rafalko shouldn't this issue be closed? As I can see you have already updated Box in cb9e8a7.

Also did a quick test locally and I am able to compile Infection with ocramius/package-versions 1.11.0 or 2.1.0

@maks-rafalko
Copy link
Member Author

Also did a quick test locally and I am able to compile Infection with ocramius/package-versions 1.11.0 or 2.1.0

then let's close it

I think we need at some point to look at #1508 to make sure it always compiles

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants