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

Provide all the libzim javascript bindings #11

Open
mossroy opened this issue Nov 19, 2022 · 4 comments
Open

Provide all the libzim javascript bindings #11

mossroy opened this issue Nov 19, 2022 · 4 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@mossroy
Copy link

mossroy commented Nov 19, 2022

(transferred from kiwix/kiwix-js#768)

Like it is done for python : https://github.com/openzim/python-libzim/
Apart from generating the libzim.a file (kiwix/kiwix-build#548), the CI should also provide all the javascript bindings that allow to call it.

A few bindings are already in the demo_file_api.cpp file of this repo.
This file should be renamed, and completed with all the libzim APIs.
It should also be compiled in the CI, IMHO.

It would allow the libzim to be used in different javascript projects (kiwix-js of course, but maybe mwoffliner or some other ones)

@mossroy mossroy changed the title Provide all the libzim js bindings Provide all the libzim javascript bindings Nov 19, 2022
@mossroy
Copy link
Author

mossroy commented Nov 19, 2022

The easiest way (to begin with) would be to generate the wasm file (with its prejs/postjs contents), ready to be used in a webworker context (like it's already done here).

I don't know if it's necessary (and possible) that this repo only generates a wasm file (without the prejs, postjs and webworker context), that could be used in different contexts. I did not investigate that

@stale
Copy link

stale bot commented May 26, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be now be reviewed manually. Thank you for your contributions.

@stale stale bot added the stale label May 26, 2023
@Jaifroid Jaifroid removed the stale label Jun 12, 2023
@kelson42
Copy link

@Jaifroid This an issue, I though this has finally be solved?!

@Jaifroid
Copy link
Collaborator

It's still an issue: the only bindings ever done were the basic ones. It's something that I would need help with, as I don't know C`++ or the libzim APIs. What we currently have is getting content from the ZIM by path, doing an article count, and doing Xapian search, if I remember rightly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants