Arch Linux User Repository

Source checksum is not matching.

LedgerHQ fixed bug: Error when building from release archive without git repo (#1719). This package build from release archive again!

@intelfx Fixed git folder.

About _check_git_rev, if I specify commit to build - it makes possible build reverted release: e0e5bb6ac602e121d3b02494323ddbe680e13bf7. I don't want install reverted released. I started worry about 'force push' in future. I added _check_git_rev function to protect from 'force push'. I want manually check 'force push' or 'delete' tag cases.

Thanks!

@HD_hirku I added comment to git log

You should have put your pinned comment message https://aur.archlinux.org/packages/ledger-live#comment-893072 into the git log for 1b63bdcdc18c. Otherwise it looks like stupid nonsense.

Shame on Ledger.

_gitdir=${pkgname}-${pkgver}-git
source=("${_gitdir}::git+${url}.git#tag=@ledgerhq/live-desktop@${pkgver}") 

Why are you doing this? You are causing a completely new git repository to be cloned with each new release, especially making life miserable for people with shared $SRCDEST. The whole point of git is to not have to do that, so please don't.

The _check_git_rev dance also seems useless. Either specify a #commit=<output of git rev-parse <tag>^0> instead of #tag=<tag>, or don't.

Build temporary fails for v2.51.0!

I updated to v2.51.0 but LedgerHQ reverted release and deleted tag.

You should manually set pkgver=2.50.1 and set _gitrev=c5aba7d153e07729c478b159a895f3bc51d89e1d, can check git hash with command: git rev-parse @ledgerhq/live-desktop@2.50.1

@phnx47 that assumes people aren't building in a chroot, which is really what they should be doing with AUR packages.

I think the statement that "fnm is better than nvm" is a little contrived and subjective; I don't use JS with any sort of regularity (SRE; primarily golang and rust), but some cursory searching doesn't indicate that fnm has supplanted nvm as "the default".

That said, this is your package. You do with it what you want, and have no need to make adjustments to it unless you think it is logical and needed.

Note, however, that it is typically recommended to read and follow the packaging guidelines for packages you want to maintain. For this, we have the NodeJS Package Guidelines. These are not rules, simply guidelines.

I tried to improve build with fnm, but this create a lot of discussions. Let's explain again.

To build ledger-live you need exactly NodeJS 16: https://github.com/LedgerHQ/ledger-live/tree/%40ledgerhq/live-desktop%402.49.2/apps/ledger-live-desktop#requirements.

I can just pin nodejs-lts-gallium, many packages do same thing, for example code: https://github.com/archlinux/svntogit-community/blob/packages/code/trunk/PKGBUILD. It is ok for Arch Packages, but for AUR create nodejs versions conflicts with every upgrade packages. Check vscodium: https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=vscodium. There is nvm dependency. It makes possible build package without nodejs versions conflicts. fnm is better alternative for nvm. One more code-git: https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=code-git, there is nodejs-lts-fermium, they forgot to update nodejs version and build failes!