Search Criteria
Package Details: walc 0.2.4-2
Package Actions
Git Clone URL: | https://aur.archlinux.org/walc.git (read-only, click to copy) |
---|---|
Package Base: | walc |
Description: | An unofficial WhatsApp Desktop client for linux systems. |
Upstream URL: | https://github.com/WAClient/WALC |
Keywords: | internet messaging whatsapp whatsapp-web |
Licenses: | GPL3 |
Conflicts: | walc |
Provides: | walc |
Submitter: | YoungFellow |
Maintainer: | YoungFellow |
Last Packager: | YoungFellow |
Votes: | 0 |
Popularity: | 0.000000 |
First Submitted: | 2022-05-28 15:52 (UTC) |
Last Updated: | 2022-11-21 08:53 (UTC) |
Latest Comments
YoungFellow commented on 2022-06-25 06:13 (UTC) (edited on 2022-06-25 06:14 (UTC) by YoungFellow)
Thanks a lot for the support @opensorcerer!
Regarding
namcap
, this is what I get:So I adjusted the PKGBUILD according to your suggestions:
1.
arch=('x86_64')
This is correct I presume...as arch officially supports only 64 bit?2.
license=('GPL3')
3.
depends=('nodejs-lts-gallium' 'npm' 'gtk3' 'alsa-lib' 'nss')
Do you think I should push
walc-0.2.2-3
though?Or shall I wait until the next version is out?
Thanks again!
PS: I'll see what I can do regarding the
npm
logs, I might ask the project maintainer, but for the time being, I'm trying to get my pentesting certification over with, so I'll be a bit short on "research" time.opensorcerer commented on 2022-06-24 21:08 (UTC)
Excellent work on your first PKGBUILD!
One more thing: When you create your package, make sure you test both the PKGBUILD and the resulting package with namcap. namcap notifies you of common mistakes in packaging. There are error, warning and info levels of mistakes of which the errors are most important to fix.
Current walc-0.2.2-2 has a number of these errors you should turn your attention to:
There is rather long output on the bundled node-modules and shared libraries. I wonder if it's possible to reduce their existence there so far as to try and use the system-provided ones. A long list of warnings suggest they're not used, anyway. But that's for another time and if you've got the time. :)
Thank you for the package, and packaging.
YoungFellow commented on 2022-06-24 05:27 (UTC) (edited on 2022-06-24 05:28 (UTC) by YoungFellow)
Hey @opensorcerer, thank you for pointing that out to me, as this is my first PKGBUILD I just wrote everything into the package() function as if it were a bash script.
As per your advice, I split the process into the correct functions, and used
gendesk
for creating the desktop entry.Thanks a lot.
opensorcerer commented on 2022-06-23 18:11 (UTC) (edited on 2022-06-23 18:14 (UTC) by opensorcerer)
The build process (npm install, npm run) should be in build(), everything before that in prepare(). package() should only be used for putting what needs to be in the .zst into $pkg, see the manual.
Also, small typo in L29:
You could consider using gendesk for Desktop entry generation, see the wiki.
See also the Node.js packaging guidelines for more ideas for optimization.