[Bug 110956] List of 19.20-812932 release mistakes

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=110956

            Bug ID: 110956
           Summary: List of 19.20-812932 release mistakes
           Product: DRI
           Version: unspecified
          Hardware: x86-64 (AMD64)
                OS: Linux (All)
            Status: NEW
          Severity: normal
          Priority: medium
         Component: DRM/AMDgpu-pro
          Assignee: dri-devel at lists.freedesktop.org
          Reporter: ashark at linuxcomp.ru

I was repacking amdgpu-pro (ubuntu archive) for Arch linux, and while doing
this, I have noticed many probable mistakes. Here is my list.

- wsa-amdgpu package has empty copyright file
- Release page says that it supports ubuntu x86_64, but actually it can be
installed on ubuntu x86. Is this an error?
- In clinfo package description Homepage link is broken (404 page)
- In alternative dependencies there are such occurances: libva1-amdgpu,
libva2-amdgpu, libvdpau-1-amdgpu. But these are not provided neither by ubuntu
repos, nor in bundled archive. So is it an error?
- Provided libdrm packages have MIT licence. But are they built completely from
open source? In the changelog I can see that it is some
amd-mainline-hybrid-master20190125. And actually if I omit these packages, then
proprietay libgl driver and clinfo utility crashes. I want to avoid needing of
installation of these libdrm packages (libdrm-amdgpu-amdgpu1,
libdrm-amdgpu-common, libdrm2-amdgpu). Is that possible?
- amdgpu-pro(-hwe) and amdgpu-pro-lib32 depend on amdgpu(-hwe), but actually
they should depend on amdgpu-lib(-hwe) (just like open variant packages).
Because of that even when running installer with --no-dkms, it still is in
packages list (because amdgpu(-hwe) depends on it). Is it intensional ar just a
mistake?
- In libgl1-amdgpu-mesa-dri in postinst script in Support I+A hybrid graphics
there is such condition:
if [ -f ... ] && [ "str1" != "str2" ]; then
You just compare two different strings? I guess you wanted to compare folder
contents. But now that condition will always be false.
in if [ "${f%%/*}"... there is extra percent symbol, you should remove it. Btw,
in rpm variants scriptlets, it is already fixed.
- In the documentation at readthedocs there is missing information about Open
Vulkan component
- There is actually no PX package in bundled archive, so maybe remove it
completely from installer script and from the documentation?
- Release page says that you need to install lunar_sdk for vulkan to work. Is
it really true? And even then, it says that you need version 1.1.106.0, but in
vulkan-amdgpu-pro inside json file we have 1.1.108. The documentation is
outdated?
- roct-amdgpu-pro and roct-amdgpu-pro-dev have MIT licence. Are they actually
open souce components? If yes, then why do they called with -pro suffix?
- Can you remove hardcoded declining of installation on unsupported ubuntu
release? Just leave a warning (as you let it for non-ubuntu systems), but let
the users to decide. Otherwise, they need to edit release version or modify
debian packages internals (I even did a special script for this and published
in this bugtracking system).

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20190621/094e3926/attachment.html>