Skip to content

Rebuild libcurl-static with libpsl-static - #172

Merged
h-vetinari merged 2 commits into
conda-forge:mainfrom
jjerphan:rebuild-libcurl-static-with-libpsl-static
Jul 30, 2026
Merged

Rebuild libcurl-static with libpsl-static#172
h-vetinari merged 2 commits into
conda-forge:mainfrom
jjerphan:rebuild-libcurl-static-with-libpsl-static

Conversation

@jjerphan

@jjerphan jjerphan commented Jul 30, 2026

Copy link
Copy Markdown
Member

Checklist

  • Used a personal fork of the feedstock to propose changes
  • Bumped the build number (if the version is unchanged)
  • Reset the build number to 0 (if the version changed)
  • Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
  • Ensured the license file is being packaged.

  • Build Windows libcurl-static against libpsl-static with -DPSL_STATIC, so object files reference undecorated psl_* symbols instead of DLL imports (__imp_psl_*).
  • Use a separate CMake build tree for the static pass and point LIBPSL_LIBRARY at libpsl.a.
  • Bump build number to 3.

This unblocks static micromamba linking on Windows (see mamba-org/mamba#4355 and conda-forge/micromamba-feedstock#304), where libcurl_a.lib currently fails with unresolved __imp_psl_* when linked with libpsl.a.

Define PSL_STATIC and link libpsl.a so static objects call psl_* instead
of DLL imports (__imp_psl_*), which broke micromamba static linking.
@jjerphan

Copy link
Copy Markdown
Member Author

@conda-forge-admin, please rerender

@conda-forge-admin

Copy link
Copy Markdown
Contributor

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

@jjerphan jjerphan changed the title Rebuild libcurl-static with libpsl-static on Windows Rebuild libcurl-static with libpsl-static Jul 30, 2026
@jjerphan
jjerphan marked this pull request as draft July 30, 2026 08:12
@jjerphan
jjerphan marked this pull request as ready for review July 30, 2026 08:24
@h-vetinari

Copy link
Copy Markdown
Member

LGTM, thanks

@h-vetinari
h-vetinari merged commit 073d3c2 into conda-forge:main Jul 30, 2026
10 checks passed
@jjerphan
jjerphan deleted the rebuild-libcurl-static-with-libpsl-static branch July 30, 2026 10:29
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