[Config] Disable default alphabet sorting in glob function due of unstable sort

[hurricane-voronin]

…table sort

Q	A
Branch?	3.4
Bug fix?	yes
New feature?	no
Deprecations?	no
Tickets	Fix #33990
License	MIT
Doc PR	no

\Symfony\Component\Config\Resource\GlobResource::getIterator loads files using glob not it the stable sorting, e.g several files: doctrine.yml and doctrine_mongodb.yaml in config/packages folder.
On requests these files come(randomly) in a different order, which leads to reinitialization of symfony kernel in dev environment. It's a little bit annoying and takes a lot of time in a common :(

[stof]

isn't disabled sorting even more likely to produce different orders ?

[hurricane-voronin]

isn't disabled sorting even more likely to produce different orders ?

Not able to reproduce issues with disabled sort.

[nicolas-grekas]

Would you be able to debug two different lists when sorting is enabled? Maybe that will allow us to better understand what's going on.

[hurricane-voronin]

Would you be able to debug two different lists when sorting is enabled? Maybe that will allow us to better understand what's going on.

Well, what exactly do you need? May I miss some information to provide?

[nicolas-grekas]

Two differently ordered list of files while the same input is provided would be nice.

[hurricane-voronin]

Two differently ordered list of files while the same input is provided would be nice.

Oh, sure, with the order I've got randomly 2 lists:

Array( [0] => /my_path/symfony/config/packages/doctrine_mongodb.yaml [1] => /my_path/symfony/config/packages/doctrine.yaml [2] => /my_path/symfony/config/packages/framework.yaml [3] => /my_path/symfony/config/packages/httplug.yaml [4] => /my_path/symfony/config/packages/jms_serializer.yaml [5] => /my_path/symfony/config/packages/knp_snappy.yaml [6] => /my_path/symfony/config/packages/monolog.yaml [7] => /my_path/symfony/config/packages/rabbitmq.yaml [8] => /my_path/symfony/config/packages/routing.yaml [9] => /my_path/symfony/config/packages/security.yaml [10] => /my_path/symfony/config/packages/sensio_framework_extra.yaml [11] => /my_path/symfony/config/packages/sentry.yaml [12] => /my_path/symfony/config/packages/static_content.yaml [13] => /my_path/symfony/config/packages/twig.yaml [14] => /my_path/symfony/config/packages/workflow.yaml )

and

Array( [0] => /my_path/symfony/config/packages/doctrine.yaml [1] => /my_path/symfony/config/packages/doctrine_mongodb.yaml [2] => /my_path/symfony/config/packages/framework.yaml [3] => /my_path/symfony/config/packages/httplug.yaml [4] => /my_path/symfony/config/packages/jms_serializer.yaml [5] => /my_path/symfony/config/packages/knp_snappy.yaml [6] => /my_path/symfony/config/packages/monolog.yaml [7] => /my_path/symfony/config/packages/rabbitmq.yaml [8] => /my_path/symfony/config/packages/routing.yaml [9] => /my_path/symfony/config/packages/security.yaml [10] => /my_path/symfony/config/packages/sensio_framework_extra.yaml [11] => /my_path/symfony/config/packages/sentry.yaml [12] => /my_path/symfony/config/packages/static_content.yaml [13] => /my_path/symfony/config/packages/twig.yaml [14] => /my_path/symfony/config/packages/workflow.yaml )

I guess that the problem is with an underscore in doctrine yamls.

[nicolas-grekas]

ok thanks
can you dump setlocale(LC_COLLATE, '0') in both cases, please?

[hurricane-voronin]

Got "en_US.UTF-8" and "C" for first and second lists appropriate

[nicolas-grekas]

Thanks, that's why.
Then I'd suggest to always sort() the array returned by glob() and always pass GLOB_NOSORT. Btw, no need for the "defined" check for it (GLOB_BRACE is special).
Can you please survey if other places in the code could need the same kind of patch?

[nicolas-grekas]

Thank you @hurricane-voronin.