openzim / openzim/node-libzim

Add support for file descriptors since nodejs supports those.

Open
#172 1 comment 1 reaction 1 assignee View on GitHub

@kelvinhammond is already working on this.

Since Dec 9, 2025.

enhancement
Dominant language
C++
Stars
36
Forks
14
Avg merge
9d 1h
Merged PRs (30d)
1

Description

Add support for file descriptors since nodejs supports those.

We currently don't support Archive(int fd) (fd == 'file descriptor') and similar constructors.
Nodejs supports file descriptors.
Add support for it.

Possibly Helpful Resources

See diff below for include/zim//archive.h
Diff: https://github.com/openzim/libzim/compare/9.3.0...9.4.0?diff=unified&w=#diff-b803fcb7f17ed9235f1e5cb1fcd2f5d3b2838429d4368ae4c57ce4436577f03f

Or just look in include/zim/archive.h

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.