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

E2E tests fails #1901

Closed
maks-rafalko opened this issue Dec 6, 2023 · 5 comments
Closed

E2E tests fails #1901

maks-rafalko opened this issue Dec 6, 2023 · 5 comments

Comments

@maks-rafalko
Copy link
Member

maks-rafalko commented Dec 6, 2023

https://github.com/infection/infection/actions/runs/7120585613/job/19388267027#step:16:232

Box Requirements Checker
========================

> Using PHP 8.1.26
> PHP is using the following php.ini file:
  /etc/php/8.1/cli/php.ini

> Checking Box requirements:
  ......E

                                                                                
Error: ] Your system is not ready to run the application.                       
                                                                                

Fix the following mandatory requirements:
=========================================

 * The package "symfony/service-contracts" conflicts with the extension "psr".
   You need to disable it in order to run this application.

make[1]: Leaving directory '/home/runner/work/infection/infection'
make[1]: *** [Makefile:226: build/infection.phar] Error 1
make: *** [Makefile:57: compile] Error 2
Error: Process completed with exit code 2.

Extracted from #1900

@theofidry
Copy link
Member

Hm I think it's an issue on Box side. Looks like symfony/service-contracts conflicts with ext-psr: <1.1|>=2. Back when I implemented the requirement checker checking the extensions versions was not a thing. It looks like it is now.

Meanwhile I guess it's best to disable the requirement checker.

@maks-rafalko
Copy link
Member Author

Meanwhile I guess it's best to disable the requirement checker.

could you advice how to do it so we can have a temp workaround?

theofidry added a commit that referenced this issue Dec 7, 2023
As per #1901, the requirement checker needs to be updated. I opened box-project/box#1243 for this purpose. Meanwhile, to reduce the frictions in Infection, it is better to disable the requirement checker.
maks-rafalko pushed a commit that referenced this issue Dec 7, 2023
As per #1901, the requirement checker needs to be updated. I opened box-project/box#1243 for this purpose. Meanwhile, to reduce the frictions in Infection, it is better to disable the requirement checker.
@caendesilva
Copy link

Hey! I'm not familiar with this project, but just wanted to chime in and say that this was probably due to shivammathur/setup-php#798, which was fixed in shivammathur/php-ubuntu#3.

@sidz
Copy link
Member

sidz commented Dec 13, 2023

Seems like you are right @caendesilva. Thanks for letting know!

Fixed in #1906

@sidz sidz closed this as completed Dec 13, 2023
@caendesilva
Copy link

Seems like you are right @caendesilva. Thanks for letting know!

Fixed in #1906

No worries, glad I could help!

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

4 participants