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

Deprecations & more php version cleanups #10428

Merged
merged 3 commits into from Jan 4, 2022
Merged

Conversation

Seldaek
Copy link
Member

@Seldaek Seldaek commented Jan 4, 2022

@Seldaek Seldaek added this to the 2.3 milestone Jan 4, 2022
@Seldaek Seldaek merged commit 404c58a into composer:main Jan 4, 2022
@Seldaek Seldaek deleted the deprecations branch January 4, 2022 15:32
@@ -981,7 +980,7 @@ public static function getLoader()

$file .= <<<CLASSLOADER_INIT
spl_autoload_register(array('ComposerAutoloaderInit$suffix', 'loadClassLoader'), true, $prependAutoloader);
self::\$loader = \$loader = new \\Composer\\Autoload\\ClassLoader(\\dirname(\\dirname(__FILE__)));
Copy link

@Chrico Chrico Mar 30, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any reason why dirname(dirname(__FILE__)) was replaced by just dirname(__DIR__) ? It looks like this is breaking with version 2.3.0 of Composer for autoload.files for us. Currently investigating - will add a ticket as soon as i got more information, but curious about that change 🤔

@Seldaek


Edit: Looks like related to #9433

Copy link
Member Author

@Seldaek Seldaek Mar 30, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because dirname(__FILE__) is the equivalent of __DIR__? So this should not be an issue. Most likely you'ae having issues due to #10617 perhaps? Check if you can repro with 2.2.10 as well, and if 2.2.7 fixes it?

composer self-update 2.2.7 for example to downgrade to a specific version.

Copy link

@Chrico Chrico Mar 30, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the very fast reply! 💪

It looks like that not this is the problem. Also, with Composer 2.2.10 it is working again.

The problem for us is, that we're using our custom Composer Plugin https://github.com/inpsyde/vip-composer-plugin , which replaces the paths in those autoload_*.php files. But it seems those files are not used anymore in Composer v2.3.?


inpsyde/vip-composer-plugin#7

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

Successfully merging this pull request may close these issues.

End of php52 supporting
3 participants