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
Paperwork 2.0 #102235
Paperwork 2.0 #102235
Conversation
Result of 4 packages marked as broken and skipped:
10 packages built:
|
Result of 9 packages failed to build:
6 packages built:
|
0d16616
to
585116a
Compare
I'm surprised by the build failure no x86_64 because it works for me™ |
Sandbox issue I guess:
|
Huh, the sandbox contains /bin/sh, and I do have the sandbox enabled. |
It is ready now. The only thing not working as well as I would like is that ocr takes forever (several minutes) even for single page documents. But this is probably a tesseract-level problem. cc @Ekleog as a potential reviewer. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Haha, actually I was about to start packaging 2.0, thanks for being first :-)
Just commented on a few nitpicks, but I'm wondering whether it makes sense to just build everything within one package and provide the components via multiple outputs, since everything is based on a single source tree. I think there is no other package depending on some of the Paperwork components and if there would be it would still work by referring to eg. a .dev
output.
What do you think?
5318e0a
to
5858e1c
Compare
5858e1c
to
15e249e
Compare
Result of 4 packages marked as broken and skipped:
10 packages built:
|
15e249e
to
d55fe7b
Compare
@symphorien: Is there anything left to do here? |
I added release notes about the backward incompatible on-disk format and I think this is ready. |
Is this also still the case? |
No it's just surprisingly slow. All the waiting time is due to a tesseract child process. |
c1e3694
to
15d2faa
Compare
fixed usage of wrapGappsHook. paperwork was crashing when opening a gtk file chooser, but I had not caught that when running paperwork in xfce-terminal because then paperwork was indirectly wrapped by xfce-terminal's own wrapper. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 But of course see my earth-shattering nitpobjections :-)
746eb82
to
0c38b12
Compare
Motivation for this change
Update. Also bugs which are only fixed in 2.0.
This is still wip, I opened the PR to prevent potential duplicate work with co-maintainer @aszlig
Scanning looks buggy, I still have to investigate.
Backup your papers directory before reviewing.
The format changes with paperwork 2.0, and it is not possible to downgrade (at least, some information will be lost).
https://forum.openpaper.work/t/paperwork-2-0/112/5
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)