Bug 110963

Summary: Wrong condition and wrong variable substitution in libgl1-amdgpu-mesa-dri in postinst script
Product: DRI Reporter: Andrew Shark <ashark>
Component: DRM/AMDgpu-proAssignee: Jeremy Newton <Jeremy.Newton>
Status: RESOLVED MOVED QA Contact:
Severity: major    
Priority: medium    
Version: unspecified   
Hardware: Other   
OS: All   
Whiteboard:
i915 platform: i915 features:
Bug Depends on:    
Bug Blocks: 110956    

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.