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

Switch to zimdump from zim-tools #66

Closed
4 of 6 tasks
lidel opened this issue Nov 8, 2019 · 1 comment · Fixed by #77
Closed
4 of 6 tasks

Switch to zimdump from zim-tools #66

lidel opened this issue Nov 8, 2019 · 1 comment · Fixed by #77
Labels
need/analysis Needs further analysis before proceeding P1 High: Likely tackled by core team if no one steps up snapshots issues related to snapshot creation and updates

Comments

@lidel
Copy link
Member

lidel commented Nov 8, 2019

Motivation

Creating a new snapshot requires unpacking data from ZIM archive.

Legacy process relied on a customized extract_zim tool which unfortunately is no longer able to unpack latest snapshots (#60 (comment)).

Good news: we now have upstream openzim/zim-tools which not only unpacks archives without a problem, but removes maintenance burden from the mirror project

Prerequisites

@kelson42 I took a look at output of zimdump v1.0.5 and believe we could switch to this tool when below issues are addressed:

Nice-to-haves

Not blockers, but things to consider in the future:

  • Prebuilt binaries for other platforms than 64bit Linux
    While folks on Windows and MacOS could run this in a VM, would be really nice if the pipeline for building a new snapshot worked on all three platforms.

  • Ability to skip processing of Xapian index
    (Not a hard blocker, but perhaps could speed up the build even further, as we don't use it atm?)

@kelson42
Copy link

kelson42 commented Nov 9, 2019

@lidel Thank you for having explained and detailed which problem we face here. We will do our best to fix the 4 tickets you have reported before the end of next January.

I have created two tickets to get binaries for:

@lidel lidel pinned this issue Jan 25, 2021
@lidel lidel added P1 High: Likely tackled by core team if no one steps up need/analysis Needs further analysis before proceeding labels Jan 25, 2021
@lidel lidel closed this as completed in #77 Feb 15, 2021
@lidel lidel unpinned this issue Feb 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
need/analysis Needs further analysis before proceeding P1 High: Likely tackled by core team if no one steps up snapshots issues related to snapshot creation and updates
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants