[PR PATCH] ddcutil: add back depends on i2c-tools

[PR PATCH] ddcutil: add back depends on i2c-tools

[dkwo]

[-- Attachment #1: Type: text/plain, Size: 458 bytes --]

There is a new pull request by dkwo against master on the void-packages repository

https://github.com/dkwo/void-packages addi2c
https://github.com/void-linux/void-packages/pull/39113

ddcutil: add back depends on i2c-tools
Recently, this was removed from depends, and added to makedepends,
but (at least on x86_64-musl) it is not picked up automatically.
 @lemmi 

A patch file from https://github.com/void-linux/void-packages/pull/39113.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-addi2c-39113.patch --]
[-- Type: text/x-diff, Size: 1286 bytes --]

From ede5ba0e202722a2a395b93f8c0d1db6f1a1ecb8 Mon Sep 17 00:00:00 2001
From: dkwo <nicolopiazzalunga@gmail.com>
Date: Mon, 5 Sep 2022 08:26:23 -0400
Subject: [PATCH] ddcutil: add back depends on i2c-tools

---
 srcpkgs/ddcutil/template | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/srcpkgs/ddcutil/template b/srcpkgs/ddcutil/template
index 919a3ccb42d8..170bcdae5cc0 100644
--- a/srcpkgs/ddcutil/template
+++ b/srcpkgs/ddcutil/template
@@ -1,14 +1,16 @@
 # Template file for 'ddcutil'
 pkgname=ddcutil
 version=1.3.0
-revision=1
+revision=2
 build_style=gnu-configure
 hostmakedepends="automake libtool pkg-config"
-makedepends="i2c-tools-devel libdrm-devel libgudev-devel libkmod-devel libusb-devel libXrandr-devel"
+makedepends="i2c-tools-devel libdrm-devel libgudev-devel libkmod-devel
+ libusb-devel libXrandr-devel"
+depends="i2c-tools"
 short_desc="Query and change monitor settings using DDC/CI and USB"
 maintainer="lemmi <lemmi@nerd2nerd.org>"
 license="GPL-2.0-or-later"
-homepage="http://www.ddcutil.com"
+homepage="https://www.ddcutil.com"
 changelog="https://www.ddcutil.com/release_notes/"
 distfiles="https://github.com/rockowitz/ddcutil/archive/v${version}.tar.gz"
 checksum=fa439f031c9372360e8d266f4294937dd3ee0a1c812d41e596b5e75a6d73d6d3

Re: [PR PATCH] [Merged]: ddcutil: add back depends on i2c-tools

[sgn]

[-- Attachment #1: Type: text/plain, Size: 314 bytes --]

There's a merged pull request on the void-packages repository

ddcutil: add back depends on i2c-tools
https://github.com/void-linux/void-packages/pull/39113

Description:
Recently, this was removed from depends, and added to makedepends,
but (at least on x86_64-musl) it is not picked up automatically.
 @lemmi 

Re: ddcutil: add back depends on i2c-tools

[lemmi]

[-- Attachment #1: Type: text/plain, Size: 198 bytes --]

New comment by lemmi on void-packages repository

https://github.com/void-linux/void-packages/pull/39113#issuecomment-1237279115

Comment:
Is `i2c-tools` needed? For me `ddcutil` works without it. 

Re: ddcutil: add back depends on i2c-tools

[dkwo]

[-- Attachment #1: Type: text/plain, Size: 257 bytes --]

New comment by dkwo on void-packages repository

https://github.com/void-linux/void-packages/pull/39113#issuecomment-1237339929

Comment:
I believe it's required e.g. to control brightness
of an external monitor,
see also https://www.ddcutil.com/building/

Re: ddcutil: add back depends on i2c-tools

[lemmi]

[-- Attachment #1: Type: text/plain, Size: 227 bytes --]

New comment by lemmi on void-packages repository

https://github.com/void-linux/void-packages/pull/39113#issuecomment-1237400532

Comment:
No, I'm using it to control the brightness of my monitor and I did not need it so far.