Skip to content

Conversation

@ferrine
Copy link

@ferrine ferrine commented Jan 5, 2026

fix #20

mixNixDeps = pkgs.callPackages ./deps.nix {
    rustlerPrecompiledOverrides = {
      vega_lite_convert = {
        env.RUSTY_V8_ARCHIVE = pkgs.fetchurl {
          url = "https://github.com/denoland/rusty_v8/releases/download/v0.105.1/librusty_v8_release_x86_64-unknown-linux-gnu.a.gz";
          hash = "sha256-f7aDA74Jn2h4rp9sACGHX4DBbN6yevgWCEKdfI1fJDU=";
        };
        postInstall = ''
          # this package does not expect lib prefix
          ln -s libex_vl_convert.so $out/lib/ex_vl_convert.so
        '';
      };
    };
  };

@camelpunch
Copy link
Contributor

Would you mind rebasing and resolving conflicts? Thanks for your contribution!

@ferrine
Copy link
Author

ferrine commented Jan 6, 2026

sure

refactor(deps_nix): nix formatting

Nix formatting in template
@ferrine ferrine force-pushed the rustlerPrecompiledOverrides branch from 3a8b66e to 3f9894c Compare January 6, 2026 20:34
@ferrine
Copy link
Author

ferrine commented Jan 6, 2026

just did that but the snippet concerning vega lite appeared to not just that... There is also a name of library mismatch. I do not know if it is very common, but the nif expects ex_vl_convert.so, not libex_vl_convert.so. It is possible to find a workaround using the proposed patch, but I suppose this might come from how build is organised

UPD:

I found this workaround makes the nif work

        postInstall = ''
          ln -s libex_vl_convert.so $out/lib/ex_vl_convert.so
        '';

@camelpunch
Copy link
Contributor

I don't understand. Is this PR ready, or is there an issue with it that requires a workaround? Or is the issue with the current main branch?

@ferrine
Copy link
Author

ferrine commented Jan 8, 2026

The pr is ready, the problem is package specific as far as I understand it now

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.

Add vega_lite_convert support

2 participants