Discussion:
Bug#861617: RFS: ddcutil/0.8.0-6 [ITP] - control monitor settings
Sanford Rockowitz
2017-05-01 15:26:04 UTC
Permalink
Package: sponsorship-requests
Severity: normal

Dear mentors,

I am looking for a sponsor for my package "ddcutil"

* Package name : ddcutil
Version : 0.8.0-6
Upstream Author : Sanford Rockowitz <***@minsoft.com>
* URL : https://github.com/rockowitz/ddcutil
* License : GPL-2.0
Section : utils

It builds those binary packages:

ddcutil - Control monitor settings - Standalone command line version
libddcutil-dev - ddcutil - C development files
libddcutil0 - ddcutil - Shared libraries

To access further information about this package, please visit the
following URL:

https://mentors.debian.net/package/ddcutil


Alternatively, one can download the package with dget using this command:

dget -x
https://mentors.debian.net/debian/pool/main/d/ddcutil/ddcutil_0.8.0-6.dsc

More information about ddcutil can be obtained from
https://www.ddcutil.com

Changes since the last upload:

ddcutil (0.8.0-6) unstable; urgency=low

* Bump Debian release for debian-mentors

ddcutil is a utility for querying and changing monitor settings, such as
brightness and color levels.

ddcutil primarily uses DDC/CI (Display Data Channel Command Interface) to
communicate with monitors implementing MCCS (Monitor Control Command Set)
over I2C. Alternatively, there is support for monitors (such as Eizo
ColorEdge
displays) that implement MCCS using a USB connection.

Use cases for ddcutil include:
- To record and reset color related settings as part of color profile
management
- Switching monitor input
- Controlling brightness

The most closely comparable program is ddccontrol, which has not been
maintained
for several years. The design of ddcutil differs from ddccontrol in two
fundamental
ways that should make it more robust in the face of hardware changes and
monitor
variation:

1) It uses the userspace i2c-dev interface instead of providing its own
I2C drivers,
so is less sensitive to video driver implementation.

2) Instead of relying on a database of features by monitor model, which can
never hope to be up to date, ddcutil bases its feature interpretation on
a detailed understanding of the MCCS specification. The downside is that
can only give generic interpretation for MCCS Virtual Control Panel features
in the range reserved for manufacturer specific features.


Regards,
Sanford Rockowitz
Andrey Rahmatullin
2017-07-24 19:11:57 UTC
Permalink
Control: tags -1 + moreinfo

Please remove all unneeded files from debian/.
Please remove commented out sample lines from debian/rules.
You shouldn't "Bump Debian release for debian-mentors". Please remove all
changelog entries (both old upstream ones and recent bumps) and add oe
correct entry.
Please switch to the debhelper compat level 10.
B-D: libglib2.0-0 looks wrong.
Please update to the current Standards-Version version.
Please fix and uncomment Vcs-Browser.
Adding explicit Depends: on library packages is usually wrong.
Please remove unneeded comments from debian/control.
Please drop debian/menu.
Consider adding the symbols file.
Please don't ship .la.
I'd expect ddcutil to be linked against libddcutil0.
--
WBR, wRAR
Sanford Rockowitz
2017-07-24 21:49:46 UTC
Permalink
Andrey,

Thank you for your comments. This being my fist debian submission, I
want to make sure there are not too many cooks in the kitchen before
uploading a new version. Back at the beginning of May, Stephen Kitt had
offered to co-maintain and sponsor the project. However, nothing has
happened since. Stephen, what you you like me to do?

Assuming that I'm uploading a new version, some comments and questions.

1) I will create a debian directory that exists solely for the debian
submission (i.e. not PPA, not OBS). This should clean things up.
2) IIRC, I was forced to increment the Debian release by
debain-mentors. A failed submission "used up" a debian release number
and blocked a corrected submission with the same release number. Is
there some way around this?
3) I will eliminate packages libddcutil0 and libddcutil-dev. The shared
library does not (yet) implement all the functionality needed by the
command line application, which is why ddcutil is not linked against
libddcutil0, and it is subject to change from user feedback and as the
Python API is implemented. Those wishing to experiment the the API can
get the shared library from my site or OBS.

Sanford
Post by Andrey Rahmatullin
Control: tags -1 + moreinfo
Please remove all unneeded files from debian/.
Please remove commented out sample lines from debian/rules.
You shouldn't "Bump Debian release for debian-mentors". Please remove all
changelog entries (both old upstream ones and recent bumps) and add oe
correct entry.
Please switch to the debhelper compat level 10.
B-D: libglib2.0-0 looks wrong.
Please update to the current Standards-Version version.
Please fix and uncomment Vcs-Browser.
Adding explicit Depends: on library packages is usually wrong.
Please remove unneeded comments from debian/control.
Please drop debian/menu.
Consider adding the symbols file.
Please don't ship .la.
I'd expect ddcutil to be linked against libddcutil0.
Andrey Rahmatullin
2017-07-25 08:16:17 UTC
Permalink
2) IIRC, I was forced to increment the Debian release by debain-mentors. A
failed submission "used up" a debian release number and blocked a corrected
submission with the same release number. Is there some way around this?
mentors doesn't have such requirement.
You may mean that dput doesn't reupload the same file name because an
upload log exist locally. You may remove it or use dput -f.
--
WBR, wRAR
Sanford Rockowitz
2017-07-30 11:03:47 UTC
Permalink
Per Andrey Rahmatullin's comments. version 0.8.4-1 of ddcutil cleans up
the debian directory. It also eliminates the shared library version of
ddcutil, as the API is still subject to significant change.

The debian/watch file does not include a cryptographic signature check
as I was unable to get the pgpsigurlmangle option to work.

Sanford Rockowitz
Post by Andrey Rahmatullin
Control: tags -1 + moreinfo
Please remove all unneeded files from debian/.
Please remove commented out sample lines from debian/rules.
You shouldn't "Bump Debian release for debian-mentors". Please remove all
changelog entries (both old upstream ones and recent bumps) and add oe
correct entry.
Please switch to the debhelper compat level 10.
B-D: libglib2.0-0 looks wrong.
Please update to the current Standards-Version version.
Please fix and uncomment Vcs-Browser.
Adding explicit Depends: on library packages is usually wrong.
Please remove unneeded comments from debian/control.
Please drop debian/menu.
Consider adding the symbols file.
Please don't ship .la.
I'd expect ddcutil to be linked against libddcutil0.
Loading...