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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
mapserver: fix build with libxml2 2.12 #279418
Conversation
Can't you use this URL instead: https://github.com/MapServer/MapServer/commit/2c1d4f928b9657dbb4517ecbfc8cac30cc73ae9f.patch ? |
This was my first attempt, but
failed on
I am actually not sure why. |
Most probably that patches has been added in the meantime on the development branch. Anyway, your PR LGTM. |
This comment was marked as resolved.
This comment was marked as resolved.
@jtojnar , is it failing only on Darwin ? Bot Linux a Darwin builds of this PR are successful and the error message doesn't look related to this PR. |
IIRC, the prerequisite is Clang 16. You can try replacing |
Do you mean keep using clang 15 by changing
Sorry, I am not Darwin user. |
Actually, it might be that mapserver just has bad platform detection or something. I can reproduce the same error with |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/week-in-geospatial-team/37035/2 |
Result of 1 package built:
|
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/week-in-geospatial-team/37035/3 |
Description of changes
Fix mapserver build with libxml2 2.12.
This PR is based on MapServer/MapServer#6975 ,
which can't be applied without other patches.
Required for #275620
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 馃憤 reaction to pull requests you find important.