PackageCompiler.jl icon indicating copy to clipboard operation
PackageCompiler.jl copied to clipboard

Incompatibility with Julia 1.11-rc1 and 1.11-rc2

Open PatrickHaecker opened this issue 1 year ago • 6 comments

Creating an app using Julia 1.11-rc1 (also the manifest files pointing to 1.11-rc1) fails with the error

error: too few arguments to function ‘jl_get_binding_wr’
  109 |     jl_binding_t *bp = jl_get_binding_wr(jl_base_module, var);

Replacing line 108 of embedding_wrapper.c by

#if JULIA_VERSION_MAJOR == 1 && JULIA_VERSION_MINOR >= 11
    jl_binding_t *bp = jl_get_binding_wr(jl_base_module, var, 1);
    jl_checked_assignment(bp, jl_base_module, var, firstarg);
#elif JULIA_VERSION_MAJOR == 1 && JULIA_VERSION_MINOR >= 10

lets the build finish successfully. This is a trivial combination of the existing code for 1.9 and 1.10.

PatrickHaecker avatar Jul 15 '24 15:07 PatrickHaecker

I stumbled on the same issue on Julia 1.11-rc1 with MacOS M1.

JanisErdmanis avatar Jul 17 '24 08:07 JanisErdmanis

I hope that we get an update with this simple fix, soon.

But until we get it, you can copy ~/.julia/packages/PackageCompiler/nT5sD/src/embedding_wrapper.c into your project, modify it accordingly and include the keyword argument c_driver_program = "path_to_modified_embedding_wrapper.c" into your create_app() call. Obviously, you need to revert that change after the fix is applied.

PatrickHaecker avatar Jul 25 '24 15:07 PatrickHaecker

I hope that we get an update with this simple fix, soon.

You could try make a PR with the fix. Might expedite things.

KristofferC avatar Jul 26 '24 11:07 KristofferC

Fair enough :-). I need to go through some process because Enterprise, but if it works out I'll make a PR.

PatrickHaecker avatar Jul 26 '24 14:07 PatrickHaecker

Same problem with Julia 1.11-rc2

ufechner7 avatar Jul 30 '24 04:07 ufechner7

#954 fixes the issue for Linux and Windows, but the tests for macOS still fail. Could someone with access to a Mac have a look at the error messages?

ufechner7 avatar Jul 30 '24 07:07 ufechner7