setup_kernels_arches should not call rpm on Debian/Ubuntu based distros #329

Activity

1. evelikov commented on Apr 19, 2023

   

   evelikov

   on Apr 19, 2023

   Last edited by evelikov

   Collaborator

   More actions

   Glancing at the two uses:

   • the ia32e in setup_kernels_arches was added ~15 years ago with 5a9c77b by @superchalupa.

   Michael do you recall what distro/version it's aimed at, is it still relevant?

   Edit: fwiw actual rpm invocations around ia32e were introduced by 3abe63b, although one can remove the whole block if the Arch is no longer supported/relevant.

   • while the do_uninstall reference is more recent 79fc855 by @vt-alt.

   What are the side-effects of removing the (empty) folder when it's owned by rpm? Should we do a similar check for other package manageds - yum, apt, pacman, etc? I don't think such a check scales well, plus my gut feeling is that this might be a workaround for broken package.
2. anbe42

   added a commit that references this issue on Apr 19, 2023

   do not perform rpm queries on Debian/Ubuntu/Arch based distributions

   ...

   This issue will close once commit 096979d is merged into the 'master' branch.

   096979d
3. anbe42 commented on Apr 19, 2023

   

   anbe42

   on Apr 19, 2023

   CollaboratorAuthor

   More actions

   a possible solution: anbe42@096979d
   this adds a case "$running_distribution" in around rpm usage but does not try to reimplement the rpm usage with dkms/pacman, not yet tested ;-)
4. evelikov commented on Apr 20, 2023

   

   evelikov

   on Apr 20, 2023

   Collaborator

   More actions

   Our ultimate goal is to remove all running_distribution instances. Iff needed a config toggle can be used to control the code flow. Considering the side effect is couple of empty directories, I'm not in favour of adding more.
5. evelikov commented on Aug 15, 2023

   

   evelikov

   on Aug 15, 2023

   Collaborator

   More actions

   I've added the required notes to the "move distro specifics into the config" task and I'll be closing this in favour of it.

   Hope that's OK, I see you've got a local workaround in Debian for the short term.
6. evelikov

   closed this as completedon Aug 15, 2023
7. anbe42

   added a commit that references this issue on Dec 6, 2023

   do not perform rpm queries on Debian/Ubuntu/Arch based distributions

   ...

   This issue will close once commit 4c8057c is merged into the 'master' branch.

   4c8057c
8. anbe42

   added a commit that references this issue on Feb 8, 2024

   do not perform rpm queries on Debian/Ubuntu/Arch based distributions

   ...

   This issue will close once commit 2315ab0 is merged into the 'master' branch.

   2315ab0
9. anbe42

   added 2 commits that reference this issue on Feb 1, 2025

   do not perform rpm queries on Debian/Ubuntu/Arch based distributions

   ...

   This issue will close once commit 276851d is merged into the 'master' branch.

   276851d

   dkms: do not perform rpm queries on Debian/Ubuntu/Arch based distribu…

   ...

   This issue will close once commit d3aedcd is merged into the 'master' branch.

   d3aedcd
10. scaronni

    added a commit that references this issue on Feb 4, 2025

    dkms: do not perform rpm queries on Debian/Ubuntu/Arch based distribu…

    ...

    This issue will close once commit 878c618 is merged into the 'main' branch.

    878c618
11. akorn

    added a commit that references this issue on Feb 12, 2025

    dkms: do not perform rpm queries on Debian/Ubuntu/Arch based distribu…

    ...

    This issue will close once commit c5ceed9 is merged into the 'master' branch.

    c5ceed9