Skip to content

ddcutil: add back depends on i2c-tools#39113

Merged
sgn merged 1 commit intovoid-linux:masterfrom
dkwo:addi2c
Sep 5, 2022
Merged

ddcutil: add back depends on i2c-tools#39113
sgn merged 1 commit intovoid-linux:masterfrom
dkwo:addi2c

Conversation

@dkwo
Copy link
Copy Markdown
Contributor

@dkwo dkwo commented Sep 5, 2022

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

@sgn sgn merged commit 3d9c6d8 into void-linux:master Sep 5, 2022
@lemmi
Copy link
Copy Markdown
Member

lemmi commented Sep 5, 2022

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

@dkwo
Copy link
Copy Markdown
Contributor Author

dkwo commented Sep 5, 2022 via email

@lemmi
Copy link
Copy Markdown
Member

lemmi commented Sep 5, 2022

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

@dkwo dkwo deleted the addi2c branch February 13, 2023 22:08
arp242 added a commit to arp242/void-packages that referenced this pull request Apr 17, 2026
It's not used; the configure script hasn't checked for it since 2022:
rockowitz/ddcutil@ec7377e7 It looks like the
code that uses it was removed around 2017.

This was "added back" in Void in void-linux#39113, but for unclear reason and
people are reporting it's not really needed there. Everything seems to
work as near as I can tell. I can't find any references in the code
beyond some commented out stuff.
arp242 added a commit to arp242/void-packages that referenced this pull request Apr 17, 2026
It's not used; the configure script hasn't checked for it since 2022:
rockowitz/ddcutil@ec7377e7 It looks like the
code that uses it was removed around 2017.

This was "added back" in Void in void-linux#39113, but for unclear reason and
people are reporting it's not really needed there too. Everything seems
to work as near as I can tell. I can't find any references in the code
beyond some commented out stuff.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants