diff mbox series

automation: update openSUSE Tumbleweed building dependencies

Message ID 158524374263.32448.13098482060302794023.stgit@Palanthas (mailing list archive)
State New, archived
Headers show
Series automation: update openSUSE Tumbleweed building dependencies | expand

Commit Message

Dario Faggioli March 26, 2020, 5:29 p.m. UTC
We need python3 (and the respective -devel package), these days.

Signed-off-by: Dario Faggioli <dfaggioli@suse.com>
---
Cc: Doug Goldstein <cardoe@cardoe.com>
---
 .../build/suse/opensuse-tumbleweed.dockerfile      |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Andrew Cooper March 27, 2020, 1:50 p.m. UTC | #1
On 26/03/2020 17:29, Dario Faggioli wrote:
> We need python3 (and the respective -devel package), these days.
>
> Signed-off-by: Dario Faggioli <dfaggioli@suse.com>
> ---
> Cc: Doug Goldstein <cardoe@cardoe.com>
> ---
>  .../build/suse/opensuse-tumbleweed.dockerfile      |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/automation/build/suse/opensuse-tumbleweed.dockerfile b/automation/build/suse/opensuse-tumbleweed.dockerfile
> index 2676a87c85..e80d773a79 100644
> --- a/automation/build/suse/opensuse-tumbleweed.dockerfile
> +++ b/automation/build/suse/opensuse-tumbleweed.dockerfile
> @@ -54,8 +54,8 @@ RUN zypper install -y --no-recommends \
>          pandoc \
>          patch \
>          pkg-config \
> -        python \
> -        python-devel \
> +        python3 \
> +        python3-devel \

These containers get used for older Xen branches as well, so you
generally can't take deps out like this (Until we stop building the
oldest branches which need the dependency).

Will tumbleweed cope with both packages installed concurrently?

~Andrew
Dario Faggioli March 30, 2020, 10:32 a.m. UTC | #2
On Fri, 2020-03-27 at 13:50 +0000, Andrew Cooper wrote:
> On 26/03/2020 17:29, Dario Faggioli wrote:
> > 
> > diff --git a/automation/build/suse/opensuse-tumbleweed.dockerfile
> > b/automation/build/suse/opensuse-tumbleweed.dockerfile
> > index 2676a87c85..e80d773a79 100644
> > --- a/automation/build/suse/opensuse-tumbleweed.dockerfile
> > +++ b/automation/build/suse/opensuse-tumbleweed.dockerfile
> > @@ -54,8 +54,8 @@ RUN zypper install -y --no-recommends \
> >          pandoc \
> >          patch \
> >          pkg-config \
> > -        python \
> > -        python-devel \
> > +        python3 \
> > +        python3-devel \
> 
> These containers get used for older Xen branches as well, so you
> generally can't take deps out like this (Until we stop building the
> oldest branches which need the dependency).
>
Ah, ok, didn't think about that. Makes sense.

> Will tumbleweed cope with both packages installed concurrently?
> 
It does. I'll test building in a container that has both and (if it
works) submit v2 of this patch.

Thanks and Regards
diff mbox series

Patch

diff --git a/automation/build/suse/opensuse-tumbleweed.dockerfile b/automation/build/suse/opensuse-tumbleweed.dockerfile
index 2676a87c85..e80d773a79 100644
--- a/automation/build/suse/opensuse-tumbleweed.dockerfile
+++ b/automation/build/suse/opensuse-tumbleweed.dockerfile
@@ -54,8 +54,8 @@  RUN zypper install -y --no-recommends \
         pandoc \
         patch \
         pkg-config \
-        python \
-        python-devel \
+        python3 \
+        python3-devel \
         systemd-devel \
         tar \
         transfig \