-
Notifications
You must be signed in to change notification settings - Fork 18
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
php-master: init at php-8.4.0.snapshot.c803402-20230907122631 #104
base: master
Are you sure you want to change the base?
Conversation
The current issue: https://dpaste.org/wDL4A |
I'm trying to get it to compile and while doing things here and there, I notice something. In here: https://github.com/fossar/nix-phps/blob/php-master/pkgs/phps.nix#L438, the extension When I try to build Then I check the logs: I have the feeling that the configure flags coming from enabled extensions are not going through. |
The curl thing is the same when building regular PHP ( Would you be able to compare the source trees between git and archive? |
Weird, but ok. I succeeded to build PHP, but it fails now when trying to compile the extensions:
What's the battle plan ? What are we trying to figure out? |
84b0878
to
7d8a440
Compare
The reason the extensions fail is this line: The tarball has all files inside |
I also have this issue in the
I found those links that might be relevant: One possible solution would be to replace |
I think this might be due to
notice the empty space after https://github.com/php/php-src/blob/9e74e58bcf7e31aef563f1ee00b7f0bc17fe3ef2/Zend/Makefile.frag#L16 This is probably the same reason PDO fails:
You can build with
And then look at the generated I guess the issue is we are relying on But phpize’s generated https://github.com/php/php-src/blob/9e74e58bcf7e31aef563f1ee00b7f0bc17fe3ef2/configure.ac#L164-L165 https://github.com/php/php-src/blob/9e74e58bcf7e31aef563f1ee00b7f0bc17fe3ef2/build/php.m4#L1771 |
Wow. Impressive debug. How do you think we should fix that ? |
Maybe insert the |
6d8aca1
to
ce5896f
Compare
d4ed6e8
to
7965948
Compare
What's the battle plan on this one? Could be a very nice to have :) |
Still stuck on the flex/yacc issue. |
7cb1948
to
29afaf2
Compare
I can build PHP without extensions: but as soon as I enabled the build log of php-tokenizer extensions
|
I've checked the I've compared the
correspond to this line:
Another thing that I've notice during my research is that when compiling At this point, we are blocked, I don't know what I could do to get this working. Do you have any clue? |
ff129ac
to
62010d7
Compare
3e6c133
to
7b17864
Compare
The solution to this PR was the script This PR fix it: NixOS/nixpkgs#254001, follow it at https://nixpk.gs/pr-tracker.html?pr=254001 |
cd68108
to
764970c
Compare
Co-Authored-By: Pol Dellaiera <[email protected]>
Currently does not build.
Fixes: #35