Package Details: check-broken-packages-pacman-hook-git r92.ae70a79-1

Git Clone URL: https://aur.archlinux.org/check-broken-packages-pacman-hook-git.git (read-only, click to copy)
Package Base: check-broken-packages-pacman-hook-git
Description: Pacman hook to check for broken packages
Upstream URL: https://github.com/desbma/pacman-hooks
Licenses: GPL3
Conflicts: pacman-hooks-desbma-git
Submitter: desbma
Maintainer: desbma
Last Packager: desbma
Votes: 13
Popularity: 0.73
First Submitted: 2021-05-14 22:30 (UTC)
Last Updated: 2023-12-30 21:04 (UTC)

Dependencies (3)

Required by (0)

Sources (1)

Latest Comments

MarsSeed commented on 2023-12-30 22:49 (UTC)

Hi,

Thank you very much for maintaining this very useful package.

But if you let me share my advice and polite request, it would be best if you did not do empty pkgver bump commits in the future.

The purpose of a '-git' VCS package is that Arch build tools like makepkg and devtools, as well as AUR helpers already pick up changes in the upstream git repository when checking for updates. Therefore, users automatically get to compile the latest version when they initiate the build.

Also there are AUR support tools that periodically check for updates - the properly implemented ones do pick up upstream git repo changes and notify users, even without you pushing this bump in pkgver to the PKGBUILD & .SRCINFO.

Although pushing such empty pkgver bumps might not have outright harmful impact, but, depending on the AUR build tool used, it might cause someone who already built the latest commit version get a notification upon your version bump push that there is a new update, and it might trigger a rebuild of the same version - which then the AUR helper tool will not install due to same version being already installed, or it might still reinstall it, depending on the configuration. In addition, AUR users who are subscribed to package update notifications of this one will get an email about this pkgver change that is essentially meaningless to them.

Thank you in advance if you heed my advice. And again let me express my gratitude for this superb package that is truly a must-have one for me!

thiagowfx commented on 2021-07-19 01:21 (UTC)

@desbma Thanks for cycling back. Re: First comment, you're right, it was a problem with my own setup (~/.gitconfig), so sorry for the noise.

Re: Second comment, will do.

yochananmarqos commented on 2021-06-30 22:52 (UTC)

@thiagowfx: It's a false positive because it's an Electron package. 1password ships with it's own copy of Electron which includes it's own libraries in a non-standard location.

desbma commented on 2021-06-30 20:02 (UTC)

@thiagowfx From https://github.com/rust-lang/cargo/issues/3381 it seems your issue is due to your own git config file Adding CARGO_NET_GIT_FETCH_WITH_CLI=1 would make sense, but this is not specific to this package and should go in https://wiki.archlinux.org/title/Rust_package_guidelines before I consider adding it, because I'm not sure if does not have other side effects.

Regarding the libffmpeg.so dependency, please open an issue at https://github.com/desbma/pacman-hooks/issues since this is related to the program, not the package.

thiagowfx commented on 2021-06-30 03:19 (UTC)

One more thing:

% check-broken-packages
File "/usr/bin/1password" from package "1password" is missing dependency "libffmpeg.so"

Is there any chance this could be a false positive? It seems odd that a password manager would have ffmpeg as dependency.

thiagowfx commented on 2021-06-30 03:17 (UTC)

The following error occurs:

==> Updated version: check-broken-packages-pacman-hook-git r59.b62c0f4-1
==> Starting build()...
    Updating crates.io index
error: failed to get `ansi_term` as a dependency of package `check-broken-packages v1.0.0 (/home/typhoon/aur/check-broken-packages-pacman-hook-git/src/pacman-hooks/check-broken-packages)`

Caused by:
  failed to fetch `https://github.com/rust-lang/crates.io-index`

Caused by:
  failed to authenticate when downloading repository: git@github.com:rust-lang/crates.io-index

  * attempted ssh-agent authentication, but no usernames succeeded: `git`

  if the git CLI succeeds then `net.git-fetch-with-cli` may help here
  https://doc.rust-lang.org/cargo/reference/config.html#netgit-fetch-with-cli

Caused by:
  error authenticating: no auth sock variable; class=Ssh (23)
==> ERROR: A failure occurred in build().
    Aborting...

Apparently it works if I do this first:

export CARGO_NET_GIT_FETCH_WITH_CLI=true

I am not much familiar with Rust, maybe you could change the PKGBUILD or your Cargo config to include that setting by default?

yochananmarqos commented on 2021-05-15 00:23 (UTC)

@desbma: Done.

desbma commented on 2021-05-14 23:42 (UTC)

@yochananmarqos I'm quite confident these are not false positives. Please open an issue at https://github.com/desbma/pacman-hooks/issues if you want me to look at it.

yochananmarqos commented on 2021-05-14 23:16 (UTC)

Sounds like a great idea, but there are false positives:

File '/usr/lib/ImageMagick-6.9.12/modules-Q16HDRI/coders/dng.so' from package 'libmagick6' is missing dependency 'libraw_r.so.20'
File '/usr/lib/ImageMagick-6.9.12/modules-Q16HDRI/coders/wmf.so' from package 'libmagick6' is missing dependency 'libwmflite-0.2.so.7'
File '/usr/bin/pangolin-desktop' from package 'pangolin-desktop-bin' is missing dependency 'libflutter_linux_gtk.so'
Systemd enabled service has broken link in '/etc/systemd/system/multi-user.target.wants/org.cups.cupsd.path'
Systemd enabled service has broken link in '/etc/systemd/system/multi-user.target.wants/snapd.service'
Systemd enabled service has broken link in '/etc/systemd/system/multi-user.target.wants/snapd.apparmor.service'
Systemd enabled service has broken link in '/etc/systemd/system/multi-user.target.wants/clightd.service'
Systemd enabled service has broken link in '/etc/systemd/system/printer.target.wants/org.cups.cupsd.service'
Systemd enabled service has broken link in '/etc/systemd/system/sockets.target.wants/org.cups.cupsd.socket'