xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] [RAISIN] Add installation support for CentOS
@ 2017-03-07  7:47 Géza Gémes
  2017-03-07  7:47 ` [PATCH] Allow installation on CentOS Géza Gémes
  0 siblings, 1 reply; 3+ messages in thread
From: Géza Gémes @ 2017-03-07  7:47 UTC (permalink / raw)
  To: xen-devel; +Cc: sstabellini, george.dunlap, Géza Gémes

Extend the existing Fedora case with CentOS in the install_package function

Géza Gémes (1):
  Allow installation on CentOS

 lib/common-functions.sh | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

-- 
2.7.4


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH] Allow installation on CentOS
  2017-03-07  7:47 [PATCH] [RAISIN] Add installation support for CentOS Géza Gémes
@ 2017-03-07  7:47 ` Géza Gémes
  2017-03-07 18:17   ` Stefano Stabellini
  0 siblings, 1 reply; 3+ messages in thread
From: Géza Gémes @ 2017-03-07  7:47 UTC (permalink / raw)
  To: xen-devel; +Cc: sstabellini, george.dunlap, Géza Gémes

Add $DISTRO = "CentOS"
to the rpm installation targets
complementing Fedora

Signed-off-by: Géza Gémes <geza.gemes@gmail.com>
---
 lib/common-functions.sh | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/common-functions.sh b/lib/common-functions.sh
index 19434c2..d4476f3 100644
--- a/lib/common-functions.sh
+++ b/lib/common-functions.sh
@@ -420,7 +420,7 @@ function install_package() {
     if [[ $DISTRO = "Debian" ]]
     then
         $SUDO dpkg -i "$1".deb
-    elif [[  $DISTRO = "Fedora" ]]
+    elif [[  $DISTRO = "Fedora" || $DISTRO = "CentOS" ]]
     then
         $SUDO rpm -i --force "$1"-`git show --oneline | head -1 | cut -d " " -f 1`-0.$RAISIN_ARCH.rpm
     else
@@ -432,7 +432,7 @@ function uninstall_package() {
     if [[ $DISTRO = "Debian" ]]
     then
         $SUDO dpkg -r "$1"
-    elif [[ $DISTRO = "Fedora" ]]
+    elif [[ $DISTRO = "Fedora" || $DISTRO = "CentOS" ]]
     then
         $SUDO rpm -e "$1"
     else
-- 
2.7.4


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] Allow installation on CentOS
  2017-03-07  7:47 ` [PATCH] Allow installation on CentOS Géza Gémes
@ 2017-03-07 18:17   ` Stefano Stabellini
  0 siblings, 0 replies; 3+ messages in thread
From: Stefano Stabellini @ 2017-03-07 18:17 UTC (permalink / raw)
  To: Géza Gémes; +Cc: xen-devel, sstabellini, george.dunlap

[-- Attachment #1: Type: TEXT/PLAIN, Size: 1165 bytes --]

On Tue, 7 Mar 2017, Géza Gémes wrote:
> Add $DISTRO = "CentOS"
> to the rpm installation targets
> complementing Fedora
> 
> Signed-off-by: Géza Gémes <geza.gemes@gmail.com>

Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>

Thanks for the patch!

>  lib/common-functions.sh | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/lib/common-functions.sh b/lib/common-functions.sh
> index 19434c2..d4476f3 100644
> --- a/lib/common-functions.sh
> +++ b/lib/common-functions.sh
> @@ -420,7 +420,7 @@ function install_package() {
>      if [[ $DISTRO = "Debian" ]]
>      then
>          $SUDO dpkg -i "$1".deb
> -    elif [[  $DISTRO = "Fedora" ]]
> +    elif [[  $DISTRO = "Fedora" || $DISTRO = "CentOS" ]]
>      then
>          $SUDO rpm -i --force "$1"-`git show --oneline | head -1 | cut -d " " -f 1`-0.$RAISIN_ARCH.rpm
>      else
> @@ -432,7 +432,7 @@ function uninstall_package() {
>      if [[ $DISTRO = "Debian" ]]
>      then
>          $SUDO dpkg -r "$1"
> -    elif [[ $DISTRO = "Fedora" ]]
> +    elif [[ $DISTRO = "Fedora" || $DISTRO = "CentOS" ]]
>      then
>          $SUDO rpm -e "$1"
>      else
> -- 
> 2.7.4
> 

[-- Attachment #2: Type: text/plain, Size: 127 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2017-03-07 18:17 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-03-07  7:47 [PATCH] [RAISIN] Add installation support for CentOS Géza Gémes
2017-03-07  7:47 ` [PATCH] Allow installation on CentOS Géza Gémes
2017-03-07 18:17   ` Stefano Stabellini

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).