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

Missing components #12

Open
asklar opened this issue Aug 25, 2022 · 0 comments
Open

Missing components #12

asklar opened this issue Aug 25, 2022 · 0 comments

Comments

@asklar
Copy link

asklar commented Aug 25, 2022

Thanks for providing a prebuilt DLL. I tried using it and ran into some issues - I've been able to work around them but it would be better if these were solved centrally. Sharing the issues here for other people's benefit too:

  1. It would be better if the library - including headers, etc. - was made available as a NuGet package on nuget.org; this would make it super easy to consume. Happy to help onboard this if there is interest.
  2. The release is missing an import lib (libquickjs.lib) - this is needed in order to link the DLL. I generated it by dumping the exports in the DLL, cleaning it up, saving the result as a .def file, and then using the lib.exe tool to generate the import lib.
  3. The headers are missing some types - some types such as JSString and others, are defined in quickjs.c, which means that an app using the release from this repo (and the quickjs.h header) won't be able to use those types or any macros that depend on them.
    I'd be interested to hear back and if this project is still active and you are interested in help, let me know :)
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

No branches or pull requests

1 participant