Skip to content
This repository has been archived by the owner on Aug 9, 2019. It is now read-only.

Use pkgquery -Sif "%N" instead of pkgquery -S --qdepends -f "%n" #203

Merged
merged 4 commits into from
Jan 16, 2016

Conversation

larchunix
Copy link
Collaborator

I didn't see any regression regarding #39.

@f2404
Copy link
Collaborator

f2404 commented Jan 4, 2016

@larchunix I'm sorry but this doesn't seem to resolve the ffmpeg upgrade issue I've mentioned in #39 - the command produced empty output for libvidstab.so which means that the "required by -" message will appear:
$ package-query -Sif %N libvidstab.so

@larchunix
Copy link
Collaborator Author

@f2404 In the code, the name of the package is used so you have to use vid.stab instead of libvidstab.so:

$ package-query -S --qdepends -f "%n" vid.stab
openshot
$ package-query -Sif "%N" vid.stab
ffmpeg openshot

@f2404
Copy link
Collaborator

f2404 commented Jan 4, 2016

You are right, my bad :(

@@ -214,21 +214,11 @@ format_update() {
pkgver=$rversion
local requiredbypkg=""

local pkg_dep_on=( $(pkgquery -S --qdepends -f "%n" "$pkgname") )
local pkg_dep_on=( $(pkgquery -Sif "%N" "$pkgname") )
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggest adding the -1 option - this will reduce the execution time.

Skunnyk added a commit that referenced this pull request Jan 16, 2016
Use `pkgquery -Sif "%N"` instead of `pkgquery -S --qdepends -f "%n"`
@Skunnyk Skunnyk merged commit 972f4e8 into archlinuxfr:master Jan 16, 2016
@larchunix larchunix deleted the opti branch January 16, 2016 11:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants