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

PHAR minimal requirements #164

Closed
theofidry opened this issue Apr 25, 2018 · 2 comments
Closed

PHAR minimal requirements #164

theofidry opened this issue Apr 25, 2018 · 2 comments
Milestone

Comments

@theofidry
Copy link
Member

theofidry commented Apr 25, 2018

The extension ext-phar will always be necessary for the PHAR and as such should be included in the requirements whether or not it is declared in the composer.json require section.

If the PHAR is compressed, the zip extension is also required, but it is likely that PHP will not be able to even execute it without it anyway so to be check if it's worth it or not

@c33s
Copy link

c33s commented Apr 27, 2018

just a thought: isn't it also possible to keep the files uncompressed? so zip would be optional. not sure if an uncompressed phar can be executed without the zip extension

@theofidry
Copy link
Member Author

theofidry commented Apr 27, 2018 via email

theofidry added a commit to theofidry/box that referenced this issue Apr 28, 2018
When a PHAR is compressed, it also requires the PHP extension `zip`.

Closes box-project#164
theofidry added a commit that referenced this issue Apr 28, 2018
When a PHAR is compressed, it also requires the PHP extension `zip`.

Closes #164
@theofidry theofidry added this to the 3.0.0 milestone May 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants