Package Details: ledger-udev 1-6

Git Clone URL: https://aur.archlinux.org/ledger-udev.git (read-only)
Package Base: ledger-udev
Description: Udev rules to connect a ledger wallet to your linux box
Upstream URL: https://www.ledgerwallet.com
Licenses: GPL
Submitter: quest
Maintainer: quest
Last Packager: quest
Votes: 3
Popularity: 1.287555
First Submitted: 2018-07-26 08:07
Last Updated: 2019-06-04 02:53

Latest Comments

es20490446e commented on 2019-06-05 10:36

@quest This package only does one thing, installing a single file, file that should always work as provided upstream. Otherwise it's better to let them know.

The pkgver function relies on the GitHub interface itself, which has been also quite stable from day one.

The likelihood of the package suddenly stopping working is quite low. And even if that rare case happened, the consequence would simply be that the package no longer updates, and the package manager will clearly notice that to the user. The installed version will still work meanwhile.

I think that it's more productive only to update the PKGBUILD on error, versus doing on every software update. The likelihood of getting a malfunction due to an outdated package, with missing rules, is higher than due to automating updates.

https://bin.privacytools.io/?c286ecfa84da2af0#5Khi3wbmIwlzCTIR5Ew01s72HSM5PmrIaXuq5Ffic0g=

cuzyoo commented on 2019-06-05 10:16

Seems like the upstream repository has new rules, but the checksum hasn't been updated. ==> Validating source files with sha256sums... 20-hw1.rules ... FAILED ==> ERROR: One or more files did not pass the validity check!

quest commented on 2019-06-05 02:43

@es20490446e I would do this, but it seems the format of the file has changed dramatically before. For instance in your script you still have the sed call which is no longer needed. For now I actually think it is ok for this to break temporarily and require a fix to make sure it continues to operate correctly and doesn't have commands that do nothing (like the sed call).

es20490446e commented on 2019-06-04 08:38

@ quest In order for the package to be able to update properly it needs a pkgver function.

Take my provided PKGBUILD as example. We can change something if you are not convinced about it, it's rather a suggestion.

es20490446e commented on 2019-05-22 16:40

@pancakes I don't know what you mean. After building the package is binary itself, with all the files being inside it.

It's only before building that the files are remote, but that's the case to any AUR package.

pancakes commented on 2019-05-22 15:27

Could the rules be packaged locally at build-time instead? it doesn't seem right to load it remotely, as the behavior is less transparent.

Packaging the rule file in the AUR package would also show the diffs in an AUR manager. Either way the AUR package will need a version bump if the rule file changes, so why not be explicit?

es20490446e commented on 2019-05-22 10:37

I have enhanced the PKGBUILD to automatically update in case the rules get modified upstream:

https://bin.privacytools.io/?df36fb318a83bfee#8T7MQIrFPdPKeoZkwtLGqpNZyO3rbnqu7iU/u6tHGas=

es20490446e commented on 2019-05-22 08:40

Thank you ;)

quest commented on 2019-05-22 06:26

Good idea, we now pull the list from their GitHub. It does a little sed magic to set the MODE and the additional udev-acl tag.

es20490446e commented on 2019-05-21 11:14

Better to source the rules from upstream itself, so no further maintenance of the package is needed. Plus it's a warrant that no device will be missing at any time.

https://github.com/LedgerHQ/udev-rules/blob/master/20-hw1.rules