Skip to content

bump alpines packages #578

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sebastian-toepfer
Copy link

change version of alpine package where older version no longer be available

will fix one of my build issues from #577

@sebastian-toepfer
Copy link
Author

can somebody explain why thsi build is failed. i#m not able to understand the issue :(.

@tailhook
Copy link
Owner

Basically we write tests based on the real dependencies. And sometimes those things got deprecated:

Composer 2.3.0 dropped support for PHP <7.2.5 and you are running 5.6.27, please upgrade PHP or use Composer 2.2 LTS via "composer self-update --2.2". Aborting.

So we need to keep vagga's tests updated constantly, which I'm struggling to do.

This is not your fault.

@sebastian-toepfer
Copy link
Author

sebastian-toepfer commented Jun 18, 2022

@tailhook : any way how I can reproduce it? and provide a fix for it.

@tailhook
Copy link
Owner

@tailhook : any way how I can reproduce it? and provide a fix for it.

Just run tests on your machine. I.e. vagga test. Sometimes you may need to clean local cache first:

rm -rf tmp/
rm -rf .vagga
vagga test

@sebastian-toepfer
Copy link
Author

I'm so stupid, of course, vagga for all. Eat your own dogfood. And this one is tasty :)

@tailhook: Would this fix from ebf60eb be ok? I know is not the primary problem, but not on my machine it was an error.

@sebastian-toepfer
Copy link
Author

so the problem is, that we install an old php version with composer 2.3.x. but i do not found a way to install 2.2 :(
src/builder/commands/composer is the right place? and vagga make && vagga test should build a new version and use it to run the tests 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

Successfully merging this pull request may close these issues.

2 participants