Bug 110963 - Wrong condition and wrong variable substitution in libgl1-amdgpu-mesa-dri in postinst script
Summary: Wrong condition and wrong variable substitution in libgl1-amdgpu-mesa-dri in ...
Status: RESOLVED MOVED
Alias: None
Product: DRI
Classification: Unclassified
Component: DRM/AMDgpu-pro (show other bugs)
Version: unspecified
Hardware: Other All
: medium major
Assignee: Jeremy Newton
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 110956
  Show dependency treegraph
 
Reported: 2019-06-22 03:32 UTC by Andrew Shark
Modified: 2019-11-19 08:00 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments

Description Andrew Shark 2019-06-22 03:32:18 UTC
19.20-812932 release for Ubuntu

In libgl1-amdgpu-mesa-dri in postinst script there is such condition:
    # Support I+A hybrid graphics 
    if [ -f ... ] && [ "str1" != "str2" ]; then
You just compare two different strings? I guess you wanted to compare folder contents. But now that second condition will always be true.

Further there is:
    if [ "${f%%/*}" = ... ]; then
there is extra percent symbol, you should remove it. Btw, in rpm release variant in scriptlet there is no such mistake.
Comment 1 Eric Engestrom 2019-06-22 15:49:25 UTC
Ubuntu packaging issue; this needs to be reported to Ubuntu instead.
Comment 2 Eric Engestrom 2019-06-22 17:57:20 UTC
TIL AMD does its support via our bugzilla...
Comment 3 Jeremy Newton 2019-07-29 20:32:56 UTC
This should be fixed. I back ported it internally, but I'm not sure what build it'll be in.
Comment 4 Martin Peres 2019-11-19 08:00:22 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to freedesktop.org's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.freedesktop.org/drm/amd/issues/30.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.