magento-composer-installer icon indicating copy to clipboard operation
magento-composer-installer copied to clipboard

Published 20 hours ago • verified publisher icon magento

Laminas\Stdlib for glob compatibility

Open phoenix-bjoern opened this issue 2 years ago • 17 comments

On Linux distributions like Alpine GLOB_BRACE is not available, which is a known issue.

An elegant workaround is to use Laminas\Stdlib instead of PHP's native glob() function. Will create a PR which hopefully gets accepted to solve this annoying issue.

phoenix-bjoern avatar Jun 03 '22 12:06 phoenix-bjoern

Oh yes please. Can we have this merged and released.

gwharton avatar Jun 22 '22 00:06 gwharton

Is this going to get merged anytime soon?

linshin avatar Aug 12 '22 12:08 linshin

@magento-admin Please merge this. It now gives us an error whenever we want to do a composer install!

leonhelmus avatar Aug 17 '22 09:08 leonhelmus

Experiencing the same issue, would like to see this get merged and released.

woutk88 avatar Aug 17 '22 10:08 woutk88

@sidolov: maybe the open PR's in this repo should be taken a look at? It seems like they are being forgotten 🙂

hostep avatar Aug 17 '22 11:08 hostep

I have the same issue.

RPrudon avatar Aug 17 '22 14:08 RPrudon

Please merge this, we have the same issue.

Epo123 avatar Aug 17 '22 15:08 Epo123

@fascinosum can you chime in here?

phoenix-bjoern avatar Aug 17 '22 16:08 phoenix-bjoern

@xmav could you help with this?

fascinosum avatar Aug 18 '22 03:08 fascinosum

@phoenix-bjoern Could you please use https://github.com/magento/magento-composer-installer/tree/0.4.0-beta1 as base ? also would be good to use newer version in constrain (https://github.com/magento/magento2/blob/e334af14414cb763fa3083a72f08bd72de6224d0/composer.json#L63)

xmav avatar Aug 19 '22 22:08 xmav

@xmav updated PR has been created as requested.

phoenix-bjoern avatar Aug 24 '22 17:08 phoenix-bjoern

Any movement on this? Even a beta would be stellar at this point :P

karbowiak avatar Sep 27 '22 11:09 karbowiak

can this please be merged?

leonhelmus avatar Oct 06 '22 11:10 leonhelmus

@phoenix-bjoern Thank you fixing this issue.

It will be taken care into one of the Internal JIRA ticket. Once Internal PR will get merged, we will notify.

Thank you!

CC @Wojton

rimple-saini avatar Oct 18 '22 11:10 rimple-saini

Any news?

It's a hassle to work with Magento on alpine containers and the fix was submitted months ago.

guvra avatar Nov 22 '22 09:11 guvra

@guvra We have raised the Mainline PR for this issue. It is estimated to be merged soon. Once Internal PR gets merge, we will keep you posted on this.

rimple-saini avatar Nov 22 '22 11:11 rimple-saini

Hello,

As I can see this issue got fixed in the scope of the internal Jira ticket JIRA ticket by the internal team.

Based on the Jira ticket, the target version is 2.4.6.

Thanks

rimple-saini avatar Dec 01 '22 16:12 rimple-saini