Package Details: libdisplay-info-git 0.1.1.r24.gebee359-1

Git Clone URL: https://aur.archlinux.org/libdisplay-info-git.git (read-only, click to copy)
Package Base: libdisplay-info-git
Description: EDID and DisplayID library
Upstream URL: https://gitlab.freedesktop.org/emersion/libdisplay-info
Licenses: MIT
Conflicts: libdisplay-info
Provides: libdisplay-info, libdisplay-info.so
Submitter: 1ace
Maintainer: yrlf
Last Packager: yrlf
Votes: 1
Popularity: 0.000511
First Submitted: 2022-11-17 16:55 (UTC)
Last Updated: 2024-02-06 08:12 (UTC)

Dependencies (5)

Required by (57)

Sources (1)

Latest Comments

yrlf commented on 2023-12-13 15:23 (UTC) (edited on 2023-12-13 15:23 (UTC) by yrlf)

Here is a patch that adds the .so file to provides and updates the packaging to the VCS packaging guidelines like @MarsSeed suggested:

diff --git a/PKGBUILD b/PKGBUILD
index 6f1d84a..801b5f9 100644
--- a/PKGBUILD
+++ b/PKGBUILD
@@ -1,7 +1,7 @@
 # Maintainer: Eric Engestrom <aur [at] engestrom [dot] ch>

 pkgname=libdisplay-info-git
-pkgver=0.1.1+9.g83ae3f5515
+pkgver=0.1.1.r24.gebee359
 pkgrel=1
 pkgdesc='EDID and DisplayID library'
 url='https://gitlab.freedesktop.org/emersion/libdisplay-info'
@@ -11,10 +11,10 @@ source=("git+$url")
 sha1sums=('SKIP')
 makedepends=(git hwdata meson ninja python)
 conflicts=(libdisplay-info)
-provides=(libdisplay-info)
+provides=(libdisplay-info libdisplay-info.so)

 pkgver() {
-  git -C libdisplay-info describe --tags --abbrev=10 | sed 's/^v//; s/-/+/; s/-/./'
+  git -C libdisplay-info describe --long --tags --abbrev=7 | sed 's/\([^-]*-g\)/r\1/;s/-/./g'
 }

 build() {

Since the package has been flagged out of date for about half a year now with no comment by the maintainers, I think that the package maintainer(s) @1ace (and potentially @emersion if they feel responsible for this package) should think about whether this deviation from the packaging guidelines is intended and/or whether they want to continue maintaining this package.

I'd happily take up maintainership or co-maintainership if you don't have time or don't want to do it any more.

MarsSeed commented on 2023-07-16 11:22 (UTC)

Please add provides+=(libdisplay-info.so) as in repo package.

Also please change pkgver format to follow the {RELEASE_VERSION}.r{REVISION} scheme.

1ace commented on 2023-02-17 13:42 (UTC)

@emersion: thanks for the pointer! I've disabled the test for now (with the same comment you put on the non-git package) and made you a co-maintainer of this package so that you can fix it up without having to go through me next time ^^

emersion commented on 2023-02-13 17:23 (UTC)

Hi! gen-test-data is not a good command to run without manual checks, it'll override the expected test data, so the tests will always succeed. See also: https://gitlab.freedesktop.org/emersion/libdisplay-info/-/issues/35

1ace commented on 2023-02-04 12:08 (UTC)

@italoghost: thanks for the comment! you are correct on every point, and I have applied the fix (although I localized it to check() instead so that it doesn't affect builds with tests disabled)

Would you like to be added as a co-maintainer, so that you can make any change without going through me?

italoghost commented on 2022-12-30 22:51 (UTC) (edited on 2022-12-30 22:51 (UTC) by italoghost)

Hi! The package fails to compile, giving an error on the edid decoding of a specific monitor. Reading the libdisplay-info GitLab's page, it says that to re-generate the test data, it is needed to run ninja -C build/ gen-test-data.

I overwrote the line 29 of the PKGBUILD and it worked!

Could you change from

build() {
  arch-meson libdisplay-info build
  ninja -C build
}

to

build() {
  arch-meson libdisplay-info build
  ninja -C build/ gen-test-data
}

?

Thanks!