Skip to content
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

Build the libpoplar wrapper in CI #48

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Conversation

giordano
Copy link
Collaborator

@giordano giordano commented Mar 15, 2024

Continuation of #37.

Proof of concept for #18.

Still lots to do:

  • the resulting library is only 21 kB, it's definitely wrong, it should be over ~40 MB, investigate what's the problem (resolved: I missed to pass the input file template.cpp to the compiler, so it was building an empty library. Now I have some compilation errors, but it's good, it means it's trying to actually compile the right file)
  • fix compilation errors emerging after fixing the above point (was missing -I{includedir}/julia
  • link libjulia and libcxxwrap
  • augment platform to build different wrappers for each SDK version, and the julia version
  • provide method to build wrapper locally
  • remove from project of the main package redundant dependencies that are needed only to build the wrapper
  • require julia v1.9
  • write documentation

@giordano giordano added the build Related to the build process label Mar 15, 2024
@giordano giordano marked this pull request as draft March 15, 2024 11:22
@giordano giordano force-pushed the mg/build-libpoplar branch 3 times, most recently from 6dfd196 to da6f8ed Compare April 9, 2024 11:05
@giordano giordano force-pushed the mg/build-libpoplar branch from da6f8ed to 3b4b1c5 Compare April 9, 2024 11:17
@giordano giordano force-pushed the mg/build-libpoplar branch from 3b4b1c5 to fc0ba90 Compare April 9, 2024 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Related to the build process
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant