Skip to content

Added PHP 8.2.8 for Linux x64 #6

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

Merged
merged 2 commits into from
Jul 27, 2023
Merged

Added PHP 8.2.8 for Linux x64 #6

merged 2 commits into from
Jul 27, 2023

Conversation

kalizi
Copy link
Contributor

@kalizi kalizi commented Jul 27, 2023

Hi, as in the title, I added the binary for Linux x64 and tested it on Fedora Linux 38!

@simonhamp
Copy link
Member

Could you describe how you've built this executable?

Which extensions are included?

@kalizi
Copy link
Contributor Author

kalizi commented Jul 27, 2023

Sure, as in the issue https://github.com/NativePHP/php-bin/issues/2 I got the prebuilt binary from https://github.com/crazywhalecc/static-php-cli

The extensions contained (as reported from the doc are):
bcmath,bz2,calendar,ctype,curl,dom,exif,fileinfo,filter,ftp,gd,gmp,iconv,xml,mbstring,mbregex,mysqlnd,openssl,pcntl,pdo,pdo_mysql,pdo_sqlite,phar,posix,redis,session,simplexml,soap,sockets,sqlite3,tokenizer,xmlwriter,xmlreader,zlib,zip

Anyway to be sure, running ./php -m I get the following output:

[PHP Modules]
bcmath
bz2
calendar
Core
ctype
curl
date
dom
exif
fileinfo
filter
ftp
gd
gmp
hash
iconv
json
libxml
mbstring
mysqlnd
openssl
pcntl
pcre
PDO
pdo_mysql
pdo_sqlite
Phar
posix
random
redis
Reflection
session
SimpleXML
soap
sockets
SPL
sqlite3
standard
tokenizer
xml
xmlreader
xmlwriter
zip
zlib

[Zend Modules]

@mpociot
Copy link
Member

mpociot commented Jul 27, 2023

@kalizi thank you for the PR!
We currently use a trimmed down list of dependencies for the macOS builds in order to keep the binary smaller.

bcmath
Core
ctype
curl
date
dom
fileinfo
filter
hash
json
libxml
mbstring
openssl
pcre
PDO
pdo_sqlite
Reflection
session
SimpleXML
SPL
sqlite3
standard
tokenizer
xml
zlib

I think it makes sense to use the exact same set of extensions for Linux as well, so that we don't run into any cross-compilation issues.
Of course we can always rebuild the binaries and enable more extensions later on 👍

Could you rebuild the binary with only these extensions enabled?

@kalizi
Copy link
Contributor Author

kalizi commented Jul 27, 2023

Sure, should I open another PR to avoid embedding this binary into the commit tree?

@mpociot
Copy link
Member

mpociot commented Jul 27, 2023

I'll squash the commits anyway, so feel free to just push it to this PR 👍

@kalizi
Copy link
Contributor Author

kalizi commented Jul 27, 2023

I rebuilt it with the libs you provided, the current php -m is

bcmath
Core
ctype
curl
date
dom
fileinfo
filter
hash
json
libxml
mbstring
openssl
pcre
PDO
pdo_sqlite
random
Reflection
session
SimpleXML
SPL
sqlite3
standard
tokenizer
xml
zlib

@mpociot mpociot merged commit 36fec90 into NativePHP:main Jul 27, 2023
@mpociot
Copy link
Member

mpociot commented Jul 27, 2023

Great, thank you!

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

Successfully merging this pull request may close these issues.

3 participants