Ralf Lang
Ralf Lang
Update: I did run this against the phar release of composer 2.4.0 and it did work just as well as with composer 2.3.10. How to reproduce: Have a root project...
Ultimately I wanted it to work with composer 2.3/2.4 and then got to play around a bit. Abandoning satis' own Application kernel in favor of hooking into composer's runtime helped...
Yes, that is easily possible. The current PR creates one disruptive change on purpose: - If satis is not the root project, composer will not be installed as a dependency....
@alcohol I have re-added composer/composer to require section. In my tests it now works in CLI mode both as a root package and as a dependency.
@alcohol Is there anything left to do?
Closing as the RFC resulted in a merge.
Yes, I have merged the conflicts. Meanwhile PHPStan has some complaints and I think a re-test with different composer versions is due after work today. I will highlight you once...
@alcohol Test went through In Standalone mode with "lowest" and "latest" dependencies allowed by constraints: ``` ./bin/satis ./bin/satis init ./bin/satis add --type vcs --name "horde/text_diff" https://github.com/maintaina-com/text_diff ./bin/satis build satis.json /tmp/composer/...
I think I fixed 2.3 and 2.4 compat in #700 - unfortunately that is not fit for simply being a pull request.
One could make this work again by dropping/faking the name attribute when pushing the config array into the composer instance. However I would prefer if both formats would be more...