[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#988968: unblock: e17/0.24.2-6



Control: tags -1 - moreinfo

On Tue, May 25, 2021 at 09:12:37AM +0200, Sebastian Ramacher wrote:
> Control: tags -1 moreinfo
> 
> On 2021-05-22 08:39:18 -0700, Ross Vandegrift wrote:
> > Control: tags -1 - moreinfo
> > 
> > Hello,
> > 
> > On Sat, May 22, 2021 at 09:53:26AM +0200, Sebastian Ramacher wrote:
> > > Control: tags -1 moreinfo
> > > 
> > > On 2021-05-21 22:23:20 -0700, Ross Vandegrift wrote:
> > > > Package: release.debian.org
> > > > Severity: normal
> > > > User: release.debian.org@packages.debian.org
> > > > Usertags: unblock
> > > > X-Debbugs-Cc: rvandegrift@debian.org
> > > > 
> > > > Please unblock package e17
> > > > 
> > > > [ Reason ]
> > > > 
> > > > 0.24.2-6 recommends libddcutil2, which has been replaced by libddcutil3.  
> > > > 
> > > > [ Impact ]
> > > > 
> > > > A non-existant package will be recommended.  Backlight controls for external monitors won't work unless the user tries libddcutil3.
> > > > 
> > > > [ Tests ]
> > > > 
> > > > There are no automated tests.  I have used libddcutil3 without regression
> > > > since uploading the change.
> > > 
> > > Are you sure?
> > > 
> > > /tmp/e17-0.24.2%% rgrep ddcutil\.so
> > > src/bin/system/e_system_ddc.c:   ddc_lib = dlopen("libddcutil.so.2", RTLD_NOW | RTLD_LOCAL);
> > > 
> > > I don't see libddcutil.so.3 used anywhere.
> > 
> > No, I'm not sure anymore - I must've messed up.  Apologies!
> > 
> > Upstream git after 0.24.2 has a patch to support libddcutil3.  There may not be
> > time for testing + migration before release.  But if there were, would the
> > below patch be acceptable during freeze?
> 
> Yes. Please remove the moreinfo tag once the version with that patch is
> available in unstable.

Thanks - I confirmed the fix works and 0.24.2-8 has been uploaded to unstable.

Thanks,
Ross

Attachment: signature.asc
Description: PGP signature


Reply to: