Overview

Request 883721 superseded

Add new package opensuse-signkey-cert-20210408 (bsc#1182641)


Andreas Jaeger's avatar

Why should this live in Base:System at all? If it's for kernel-modules, why not have it live there?


Joey Lee's avatar
author source maintainer

Thanks for review!

We want to install signkey package by pattern. So I created a independent package to help user to enroll key to MOK. It's only for openSUSE KMP on OBS.


Fabian Vogt's avatar
  • The check in %build shouldn't be needed. If the file is missing, it's going to break anyway
  • Calling rpm in scriptlets might deadlock
  • Scriptlets are also called on upgrades, but then most of them shouldn't do anything unless the file changes
  • Most of the scriptlets can be simplified by moving them inside if command -v mokutil

Joey Lee's avatar
author source maintainer

Thanks for your review!

. The check in %build shouldn't be needed. If the file is missing, it's going to break anyway

OK! I will remove the check.

. Calling rpm in scriptlets might deadlock

I will use "ls" to grab the certificate file name.

. Scriptlets are also called on upgrades, but then most of them shouldn't do anything unless the file changes

This RPM will not be rebuilt unless the openSUSE signkey be changed on OBS. When this RPM be upgraded, it means that the openSUSE signkey also be updated.

. Most of the scriptlets can be simplified by moving them inside if command -v mokutil

OK, I will move them inside ' if command -v mokutil '.


Fabian Vogt's avatar

This RPM will not be rebuilt unless the openSUSE signkey be changed on OBS. When this RPM be upgraded, it means that the openSUSE signkey also be updated.

There will be forced rebuilds without source changes, so the package has to take that into account.


Andreas Jaeger's avatar

Please name the package openSUSE-...

Check: rpmqpack |grep -i opensuse


Joey Lee's avatar
author source maintainer

Thanks! I will change the name.

Request History
Joey Lee's avatar

joeyli created request

Add new package opensuse-signkey-cert-20210408 (bsc#1182641)


Andreas Jaeger's avatar

a_jaeger declined request

Please update as Fabian suggested.


Joey Lee's avatar

joeyli superseded request

superseded by 886524

openSUSE Build Service is sponsored by