Arch Linux User Repository

Added the missing mkdir, sorry for that, had no time to test it this afternoon.

It compiles fine, but:

error: failed to commit transaction (conflicting files) xournalpp-git: /usr/share/thumbnailers exists in filesystem

OK Youduda. It's all yours. Once this all settles down and things are working normally again, I don't mind taking over again. I really like this software and use it weekly for math tutoring.

@youduda Your PKGBUILD works perfectly. Even better that it's using poppler 0.69.0, which has less security issues than 0.61.1. Go ahead and use yours in the actual AUR package. EDIT: Glad you're using it! This was one of the main reasons I could switch to Linux (because a possible OneNote alternative existed).

@youduda Whoops, didn't notice your comment. Sorry about that. Trying it right now. EDIT: Yes, that would be great. Please take over.

Here's a pkgbuild that works for now. https://github.com/siliconninja/xournalpp-git

You will need to change the .SRCINFO before submitting to the AUR (because the Git commit checksum can change after more recent commits or builds, as that's part of the package). I have changed fixpoppler.patch to build poppler 0.61.1 from source and the PKGBUILD itself. Keep xournalpp.install (unchanged) because it's still used.

I can't maintain it but I give you my permission to incorporate these changes (please credit me by keeping my name, siliconninja, at the top of the PKGBUILD file: https://wiki.archlinux.org/index.php/Arch_User_Repository#Rules_of_submission).

Xournalpp in the future might change their root directory's CMakeLists.txt features (the xournalpp repo/CMakeLists.txt one, I don't mean the literal root directory on Linux, there are just so many CMakeLists.txt files, so I say "root directory" to mean the root of the xournalpp repo).

For example, they might remove the feature to build an old Poppler version manually in the future, which would make this package not build properly. It has poppler 0.61.1 which has security issues according to their site but it works for now.

I'm using xournalpp for a year now and will continue for the next few years, so if you want I can take over maintaining the PKGBUILD. @siliconninja did you try my PKGBUILD file? It should work fine with the gtk changes.

That's OK. Unfortunately, I don't have time to take over starting this weekend due to school. I've been working on contributing to the xournalpp project and I've learned a lot about how their build process in CMake works. If someone else could take over, that would be great.

I plan to use xournalpp for my classes to see how Linux can hold up for my daily use in the future, and I'm building manually using set (BUILD_POPPLER ON) in line 113 of the CMakeLists.txt in the root directory of xournalpp (after a git clone) so I can get it to work so I have a solution before the PKGBUILD is fixed.

Hope the info I gave (and the info on issue #292 on github, https://github.com/xournalpp/xournalpp/issues/292) can help future contributors to the PKGBUILD.

I'm sorry. This is a bit beyond me and I don't have the time right now to learn how all of this fits together. I'd like to disown this package for now if someone is willing to take over. All I ask is that the MATHTEX option be left on.

After a few recent commits to change the CMakeLists.txt (to get GTK3 to be more compatible with the CMake build scripts), I get a patch merge error. This is because of this line in the PKGBUILD. https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=xournalpp-git#n29

==> Starting prepare()... patching file CMakeLists.txt Hunk #1 FAILED at 322. 1 out of 1 hunk FAILED -- saving rejects to file CMakeLists.txt.rej ==> ERROR: A failure occurred in prepare(). Aborting... Error making: xournalpp-git

You might want to change the patch. https://aur.archlinux.org/cgit/aur.git/tree/fixpoppler.patch?h=xournalpp-git

This is because Poppler 0.61.1 from Poppler's git sources is now being built if you say set (BUILD_POPPLER ON) after line 112 in the CMakeLists.txt in the root directory. (edit: not by default). https://github.com/xournalpp/xournalpp/blob/master/CMakeLists.txt#L120

https://github.com/xournalpp/xournalpp/blob/master/src/pdf/popplerdirect/workaround/CMakeLists.txt

You could change this to 0.67.0 or 0.69.0 if someone can redo the GooLikely workaround (https://github.com/swiftgeek/xournalpp/commit/12a409c8afc3c3ea8c8afebb67377c29e3d90035.patch) in https://github.com/xournalpp/xournalpp/blob/master/src/control/settings/MetadataManager.cpp

(Note that src/pdf/popplerdirect/workaround/ directory has no more 0.64.0 folder, and it's building from git instead of locally.)