should we also detect the usage of braces in the glob, and fail loudly if trying to use such path repository on a platform not supporting it ?

I think what @stof suggests makes sense and would be a nice addition. Can you take that into account in this PR @staabm or should I just merge this and leave that open for someone else to pick up?

Not sure we can detect the braces, as the regex could also contain e.g. abc{1,3} couldnt it?

What do you mean? glob() does not handle regex AFAIK

Ohh .. you are right... mixed things up.

see https://github.com/symfony/symfony/blob/ce73293532f6df48b2cac2aad8d040817429cea3/src/Symfony/Component/Config/Resource/GlobResource.php#L96

Symfony uses false === strpos($this->pattern, '{') to detect cases not needing GLOB_BRACE and so being able to use native globs even when they don't support it.

If you could add that check, would very much appreciate it :-)

Here we go

LGTM. @stof ?

Thanks guys