-
Notifications
You must be signed in to change notification settings - Fork 23
fix: fix builds on darwin for PHP <= 7.3 #203
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
Conversation
It's failing... I don't get it. @HelloWorld017 any clue? |
pkgs/phps.nix
Outdated
"--with-libxml-dir=${prev.libxml2.dev}" | ||
] | ||
++ prev.lib.optionals (prev.lib.versionAtLeast args.version "7.3") [ | ||
"--with-pcre-regex=${prev.pcre2.dev}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
checking for PCRE headers location... configure: error: Could not find pcre.h in /nix/store/r2cgszbs5mpjbrj0k28i9qa7wi7lqg15-pcre2-10.42-dev
Might be that older versions expect ${prev.pcre2.dev}/include
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same issue
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We will need to look at the configure.ac
or m4
files, then.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Then I'll happily leave this job to someone else, it's way beyond my knowledge at this moment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, then it is looking for pcre2-config
program so it likely expects the bin
directory.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As the --with-external-pcre
is not recognized, the --with-pcre-regex
is also added. So the error message is not relevant.
I'll take a look at the error next morning.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have succeeded this PR in #204 with fixes for this issue. Please take a look at it
f739a8c
to
39f9f88
Compare
Context: #201 (comment) Related PR: #201
39f9f88
to
42eabd8
Compare
Context: #201 (comment)
Related PR: #201