EasyCodingStandard - phar file

[zener05]

Hello, I tried to compile EasyCodingStandard as a phar file for using it standalone but it doesn't work.

I opened discussion on StackOverflow. There are also print-screens of my problem. Can you check it please? Thank you.

https://stackoverflow.com/questions/50052815/phar-doesnt-work-after-compiling

[TomasVotruba]

Hey, what issue do you try to solve?

Building phar is very complicated thing for Symfony-based applications, see very similar architecture at Rector:

• PHAR improvements rectorphp/rector#389
• PHAR alternative - build PrefixedRector rectorphp/rector#449
• Create PHAR with prefixes to avoid class and package version conflicts rectorphp/rector#177

[mssimi]

@TomasVotruba I think for ECS it might be actually easier, as I think there will be no exceptions with namespace prefixing. Still I think he should think of prefixing only as it is much easier than fixing PHAR incompatibilities.

[TomasVotruba]

I see no prefixing in original question.

[mssimi]

I assume he is creating PHAR coz of dependency conflicts.

[mssimi]

I think problem with $skip is caused by incompatibility of glob PHP method.

[zener05]

Thank you for replies guys. It seems that the problem is in symfony/dependency-injection. I'm gonna write them to the issues. Maybe they will fix it :-)

[zener05]

I found it!

Workaround is quite easy. This commit will fix the problem with PHAR. So you can just download the code via composer, then manually update the file based on that commit and that's all. After building PHAR (I recommend MacFJA/PharBuilder - it's awesome) it will works :-)

[TomasVotruba]

@zener05 Wow, that's awesome! 🚀

Thanks for sharing this with use, we got stuck on the very same problem

[TomasVotruba]

The PR in Symfony for reference: symfony/symfony#26845

[c33s]

@TomasVotruba would it be possible to add a prebuild phar file to the github release page of EasyCodingStandard (like in https://github.com/phpstan/phpstan/releases)?

[TomasVotruba]

Hi, why not!

Feel free to send PR. I'd be happy to merge it

[TomasVotruba]

Good news everyone, the ecs.phar is here:
#1734