#2 Change logic to allow release to allow installs on non-stream machines
Closed 2 years ago by sbonazzo. Opened 2 years ago by neil.
rpms/ neil/centos-release-advanced-virtualization c8-sig-virt-8  into  c8-sig-virt-8

@@ -1,7 +1,7 @@ 

  Summary: Advanced Virtualization packages from the CentOS Virtualization SIG repository

  Name: centos-release-advanced-virtualization

  Version: 1.0

- Release: 3%{?dist}

+ Release: 4%{?dist}

  License: GPLv2

  URL: http://wiki.centos.org/SpecialInterestGroup/Virtualization

  Source0: CentOS-Advanced-Virtualization.repo
@@ -25,7 +25,7 @@ 

  

  %post

  source /etc/os-release

- if [[ $ID == 'centos' && $NAME == *'Stream' ]] || [ $ID != 'centos' ]; then

+ if [[ $ID == 'centos' && $NAME == *'Stream' ]]; then

      echo "8-stream" > /etc/dnf/vars/avstream

      echo "advancedvirt-common" > /etc/dnf/vars/avdir

  else
@@ -46,6 +46,9 @@ 

  %license COPYING

  

  %changelog

+ * Tue Oct 26 2021 Neil Hanlon <neil@resf.org> - 1.0-4

+ - Fix logic for avstream and avdir to allow installation on non-Stream installs and derivatives

+ 

  * Wed Jun 09 2021 Sandro Bonazzola <sbonazzo@redhat.com> - 1.0-3

  - Add support for CentOS Stream 8 repos

  

Fixes https://pagure.io/centos-sig-virt/sig/issue/1

Tested logic on CentOS 8, CentOS Stream 8, Rocky Linux 8, Alma Linux 8, Oracle Linux 8 and all seem good.

For some reason I have no merge rights... opened https://pagure.io/centos-infra/issue/548.

looks like pagure's merge button is not really working. I'm going to try to push the patch directly within the repo.

ok, pushing the patch into the tree worked. closing the PR

Pull-Request has been closed by sbonazzo

2 years ago
Metadata