Package Details: klipper-git r4940.e2d7c5981-1

Git Clone URL: https://aur.archlinux.org/klipper-git.git (read-only, click to copy)
Package Base: klipper-git
Description: 3D printer firmware with motion planning on the host
Upstream URL: https://www.klipper3d.org/
Licenses: GPL3
Conflicts: klipper
Provides: klipper
Submitter: jade1
Maintainer: dack
Last Packager: dack
Votes: 4
Popularity: 0.000000
First Submitted: 2018-10-19 05:10 (UTC)
Last Updated: 2023-03-21 07:03 (UTC)

Dependencies (19)

Required by (1)

Sources (4)

Pinned Comments

dack commented on 2023-03-21 07:13 (UTC)

I have updated this package to use python 3. If you are currently using klipper-py3-git, please switch to using this package instead. It's now fully up to date. All future updates will go here and I will be removing klipper-py3-git at some point.

Latest Comments

« First ‹ Previous 1 2 3 4 Next › Last »

qubidt commented on 2022-01-07 20:46 (UTC)

sorry for the very late update but the PKGBUILD has been updated w/ the latest data dict file used in the tests

@wlhlm I addressed your comments. Fixed the permissions in the tmpfiles config. Hopefully that works, my rasbpi/printer isn't working ATM so I can't test it but it should work.

if anyone is confused about the permissions, I set them to 2775 because klipper (as part of normal operation) writes files directly to its source/config direectories (ugh) as part of normal operation (perhaps that's an indication that we should have this package in /opt/klipper). the 0775 permissions I inherited from the original AUR package allow anyone in the klipper group to write those files, but they would be inaccessible to other users. the setgid bit makes sure any generated files are also owned by the klipper group so other users in that group can likewise write to them (e.g. the klipper user itself)

wlhlm commented on 2021-12-31 10:19 (UTC) (edited on 2021-12-31 10:19 (UTC) by wlhlm)

There currently is a mismatch of directory permissions between the package() script and the tmpfiles configuration installed with this package. package() sets permissions on /etc/klipper to 2775, however the tmpfiles configuration sets it to 0755 every time on boot.

One way this surfaces is when upgrading as pacman will show the following:

:: Processing package changes...
(1/1) upgrading klipper-py3-git                                                                        [############################################################] 100%
warning: directory permissions differ on /etc/klipper/
filesystem: 755  package: 2775
warning: directory permissions differ on /var/lib/klipper/
filesystem: 755  package: 2755

wlhlm commented on 2021-12-25 01:21 (UTC) (edited on 2022-01-01 22:00 (UTC) by wlhlm)

Thank you for bringing the Klipper package forward into the Python 3 era.

Few minor suggestions:

  • Since dfu-util is also in popular use for flashing STM32 MCUs, I think adding it to the optional dependencies makes sense.
  • As noted in fluidd-git, adding git as a dependency for AUR packages is superfluous as base-devel is assumed to be installed already.
    • EDIT: I was mistaken, git is not part of base-devel. Apologies.

bobobo1618 commented on 2021-04-04 08:51 (UTC) (edited on 2021-04-04 08:52 (UTC) by bobobo1618)

A workaround for missing python2-pyserial:

curl -O https://raw.githubusercontent.com/archlinux/svntogit-community/fa67d039a4f5c7462af38e62ceec14249f56685b/python-pyserial/trunk/PKGBUILD
makepkg
pacman -U python2-pyserial-3.5-1-any.pkg.*

wlhlm commented on 2021-04-03 20:13 (UTC)

Small suggestions: Klipper needs numpy and matplotlib for resonance measurements 1. python2-numpy and python2-matplotlib could be added as optional dependencies.

patlefort commented on 2021-03-29 10:09 (UTC)

Dependency no longer exists: python2-pyserial.

novenary commented on 2021-03-28 14:28 (UTC)

The AVR toolchain, libusb and ncurses should all be optional dependencies as they are only necessary for building and flashing the MCU firmware, and are not required during normal operation. stm32flash is also not x86_64-specific, the PKGBUILD includes ARM in its arch array.

celogeek commented on 2020-01-24 18:13 (UTC)

hi, thanks for the patch

also I have dig more, and compile all have alert on build directory that appear in the binary package.

I have fix this way (also the way to bundle python package):

-    python2 -m compileall klippy
-    python2 klippy/chelper/__init__.py
+       python2 -m compileall -d /opt/klipper/klippy klippy
+       python2 -O -m compileall -d /opt/klipper/klippy klippy
+       python2 klippy/chelper/__init__.py

So I add an optimize compilation after the usual one, and include the final destination in the compile process.

loredan13 commented on 2020-01-21 08:12 (UTC)

Okay, thank you everyone, everything you said seems valid, I will include your changes in the package.

dack, about the logs, I removed that option with the intention of letting systemd-journal take care of it, but I wasn't aware of thread handling in klipper. I'll revert it to log file

dack commented on 2020-01-21 07:38 (UTC)

Apologies for repeating myself - I accidentally deleted my previous comment.

I think it would be better to go back to logging to a file (via the -l option), rather than letting systemd/journald do the logging. When logging to stdout (journald), klipper warns "No log file specified! Severe timing issues may result!". From looking at their code, logging to a file is handled completely differently. It uses a separate thread to log asynchronously to file. This would prevent log buffer issues from affecting time critical tasks. Logging to stdout is done in the main thread, which has the potential to cause issues.

« First ‹ Previous 1 2 3 4 Next › Last »