INSTALL | 5 ----- config/Tools.mk.in | 6 ------ config/Toplevel.mk.in | 1 - configure | 27 --------------------------- configure.ac | 1 - tools/configure | 27 --------------------------- tools/configure.ac | 1 - 7 files changed, 68 deletions(-)
Following Demi's work to use HTTPS everywhere, all users of GIT_HTTP have
been removed from the build system. Drop the configure knob.
Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
---
CC: Anthony PERARD <anthony.perard@citrix.com>
CC: Demi Marie Obenour <demi@invisiblethingslab.com>
CC: George Dunlap <George.Dunlap@eu.citrix.com>
CC: Jan Beulich <JBeulich@suse.com>
CC: Stefano Stabellini <sstabellini@kernel.org>
CC: Wei Liu <wl@xen.org>
CC: Julien Grall <julien@xen.org>
---
INSTALL | 5 -----
config/Tools.mk.in | 6 ------
config/Toplevel.mk.in | 1 -
configure | 27 ---------------------------
configure.ac | 1 -
tools/configure | 27 ---------------------------
tools/configure.ac | 1 -
7 files changed, 68 deletions(-)
diff --git a/INSTALL b/INSTALL
index 0d3eb89f0298..3816c17dcde8 100644
--- a/INSTALL
+++ b/INSTALL
@@ -89,11 +89,6 @@ from a wrong location. Compiling the tools with rpath will force the
linker to look in the correct location.
--enable-rpath
-During build in a git checkout the buildsystem needs to download
-additional tools such as qemu. This is done with either the native git
-protocol, or via http if this option is enabled.
- --enable-githttp
-
Disable xenstat and xentop monitoring tools.
--disable-monitors
diff --git a/config/Tools.mk.in b/config/Tools.mk.in
index d0d460f922d8..6abb377564db 100644
--- a/config/Tools.mk.in
+++ b/config/Tools.mk.in
@@ -37,12 +37,6 @@ LIBNL3_LIBS := @LIBNL3_LIBS@
LIBNL3_CFLAGS := @LIBNL3_CFLAGS@
XEN_TOOLS_RPATH := @rpath@
-# Download GIT repositories via HTTP or GIT's own protocol?
-# GIT's protocol is faster and more robust, when it works at all (firewalls
-# may block it). We make it the default, but if your GIT repository downloads
-# fail or hang, please pass --enable-githttp to configure.
-GIT_HTTP ?= @githttp@
-
# Optional components
XENSTAT_XENTOP := @monitors@
OCAML_TOOLS := @ocamltools@
diff --git a/config/Toplevel.mk.in b/config/Toplevel.mk.in
index 4ecacbb37d68..4db7eafcab5d 100644
--- a/config/Toplevel.mk.in
+++ b/config/Toplevel.mk.in
@@ -1,2 +1 @@
SUBSYSTEMS := @SUBSYSTEMS@
-GIT_HTTP ?= @githttp@
diff --git a/configure b/configure
index f5cd3c286b55..99f8434cbf64 100755
--- a/configure
+++ b/configure
@@ -594,7 +594,6 @@ stubdom
tools
xen
subdirs
-githttp
DEBUG_DIR
XEN_DUMP_DIR
XEN_PAGING_DIR
@@ -673,7 +672,6 @@ with_xen_scriptdir
with_xen_dumpdir
with_rundir
with_debugdir
-enable_githttp
enable_xen
enable_tools
enable_stubdom
@@ -1309,8 +1307,6 @@ Optional Features:
--disable-option-checking ignore unrecognized --enable/--with options
--disable-FEATURE do not include FEATURE (same as --enable-FEATURE=no)
--enable-FEATURE[=ARG] include FEATURE [ARG=yes]
- --enable-githttp Download GIT repositories via HTTP (default is
- DISABLED)
--disable-xen Disable build and install of xen
--disable-tools Disable build and install of tools
--enable-stubdom Enable build and install of stubdom
@@ -2124,29 +2120,6 @@ DEBUG_DIR=$debugdir_path
-# Check whether --enable-githttp was given.
-if test "${enable_githttp+set}" = set; then :
- enableval=$enable_githttp;
-fi
-
-
-if test "x$enable_githttp" = "xno"; then :
-
- ax_cv_githttp="n"
-
-elif test "x$enable_githttp" = "xyes"; then :
-
- ax_cv_githttp="y"
-
-elif test -z $ax_cv_githttp; then :
-
- ax_cv_githttp="n"
-
-fi
-githttp=$ax_cv_githttp
-
-
-
case "$host_cpu" in
i[3456]86|x86_64)
arch_enable_stubdom=y
diff --git a/configure.ac b/configure.ac
index 3aea40715307..19d9311c2ae4 100644
--- a/configure.ac
+++ b/configure.ac
@@ -17,7 +17,6 @@ m4_include([m4/subsystem.m4])
m4_include([m4/paths.m4])
AX_XEN_EXPAND_CONFIG()
-AX_ARG_DEFAULT_DISABLE([githttp], [Download GIT repositories via HTTP])
dnl mini-os is only ported to certain platforms
case "$host_cpu" in
diff --git a/tools/configure b/tools/configure
index dae377c98252..bb5b1ae45067 100755
--- a/tools/configure
+++ b/tools/configure
@@ -714,7 +714,6 @@ ovmf
xsmpolicy
ocamltools
monitors
-githttp
rpath
werror
DEBUG_DIR
@@ -807,7 +806,6 @@ with_rundir
with_debugdir
enable_werror
enable_rpath
-enable_githttp
enable_monitors
enable_ocamltools
enable_xsmpolicy
@@ -1494,8 +1492,6 @@ Optional Features:
--disable-werror Build tools without -Werror (default is ENABLED)
--enable-rpath Build tools with -Wl,-rpath,LIBDIR (default is
DISABLED)
- --enable-githttp Download GIT repositories via HTTP (default is
- DISABLED)
--disable-monitors Disable xenstat and xentop monitoring tools (default
is ENABLED)
--disable-ocamltools Disable Ocaml tools (default is ENABLED)
@@ -4156,29 +4152,6 @@ rpath=$ax_cv_rpath
-# Check whether --enable-githttp was given.
-if test "${enable_githttp+set}" = set; then :
- enableval=$enable_githttp;
-fi
-
-
-if test "x$enable_githttp" = "xno"; then :
-
- ax_cv_githttp="n"
-
-elif test "x$enable_githttp" = "xyes"; then :
-
- ax_cv_githttp="y"
-
-elif test -z $ax_cv_githttp; then :
-
- ax_cv_githttp="n"
-
-fi
-githttp=$ax_cv_githttp
-
-
-
# Check whether --enable-monitors was given.
if test "${enable_monitors+set}" = set; then :
enableval=$enable_monitors;
diff --git a/tools/configure.ac b/tools/configure.ac
index 3a2f6a2da9e0..9bcf42f233f1 100644
--- a/tools/configure.ac
+++ b/tools/configure.ac
@@ -84,7 +84,6 @@ AX_XEN_EXPAND_CONFIG()
# Enable/disable options
AX_ARG_DEFAULT_ENABLE([werror], [Build tools without -Werror])
AX_ARG_DEFAULT_DISABLE([rpath], [Build tools with -Wl,-rpath,LIBDIR])
-AX_ARG_DEFAULT_DISABLE([githttp], [Download GIT repositories via HTTP])
AX_ARG_DEFAULT_ENABLE([monitors], [Disable xenstat and xentop monitoring tools])
AX_ARG_DEFAULT_ENABLE([ocamltools], [Disable Ocaml tools])
AX_ARG_DEFAULT_ENABLE([xsmpolicy], [Disable XSM policy compilation])
--
2.30.2
On Fri, Mar 24, 2023 at 08:14:04PM +0000, Andrew Cooper wrote: > Following Demi's work to use HTTPS everywhere, all users of GIT_HTTP have > been removed from the build system. Drop the configure knob. > > Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com> Do we need a changelog entry about these changes? (That git clone are now default over https and --enable-githttp / GIT_HTTP are ignored.) In any case: Acked-by: Anthony PERARD <anthony.perard@citrix.com> Thanks, -- Anthony PERARD
On 27/03/2023 3:00 pm, Anthony PERARD wrote: > On Fri, Mar 24, 2023 at 08:14:04PM +0000, Andrew Cooper wrote: >> Following Demi's work to use HTTPS everywhere, all users of GIT_HTTP have >> been removed from the build system. Drop the configure knob. >> >> Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com> > Do we need a changelog entry about these changes? (That git clone are > now default over https and --enable-githttp / GIT_HTTP are ignored.) Yeah, I was going to write one after everything has gone on, because its more than just this one issue. > In any case: > Acked-by: Anthony PERARD <anthony.perard@citrix.com> Thanks. ~Andrew
On Fri, Mar 24, 2023 at 08:14:04PM +0000, Andrew Cooper wrote: > Following Demi's work to use HTTPS everywhere, all users of GIT_HTTP have > been removed from the build system. Drop the configure knob. > > Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com> Reviewed-by: Demi Marie Obenour <demi@invisiblethingslab.com> > --- > CC: Anthony PERARD <anthony.perard@citrix.com> > CC: Demi Marie Obenour <demi@invisiblethingslab.com> > CC: George Dunlap <George.Dunlap@eu.citrix.com> > CC: Jan Beulich <JBeulich@suse.com> > CC: Stefano Stabellini <sstabellini@kernel.org> > CC: Wei Liu <wl@xen.org> > CC: Julien Grall <julien@xen.org> > --- > INSTALL | 5 ----- > config/Tools.mk.in | 6 ------ > config/Toplevel.mk.in | 1 - > configure | 27 --------------------------- > configure.ac | 1 - > tools/configure | 27 --------------------------- > tools/configure.ac | 1 - > 7 files changed, 68 deletions(-) > > diff --git a/INSTALL b/INSTALL > index 0d3eb89f0298..3816c17dcde8 100644 > --- a/INSTALL > +++ b/INSTALL > @@ -89,11 +89,6 @@ from a wrong location. Compiling the tools with rpath will force the > linker to look in the correct location. > --enable-rpath > > -During build in a git checkout the buildsystem needs to download > -additional tools such as qemu. This is done with either the native git > -protocol, or via http if this option is enabled. > - --enable-githttp > - > Disable xenstat and xentop monitoring tools. > --disable-monitors > > diff --git a/config/Tools.mk.in b/config/Tools.mk.in > index d0d460f922d8..6abb377564db 100644 > --- a/config/Tools.mk.in > +++ b/config/Tools.mk.in > @@ -37,12 +37,6 @@ LIBNL3_LIBS := @LIBNL3_LIBS@ > LIBNL3_CFLAGS := @LIBNL3_CFLAGS@ > XEN_TOOLS_RPATH := @rpath@ > > -# Download GIT repositories via HTTP or GIT's own protocol? > -# GIT's protocol is faster and more robust, when it works at all (firewalls > -# may block it). We make it the default, but if your GIT repository downloads > -# fail or hang, please pass --enable-githttp to configure. > -GIT_HTTP ?= @githttp@ > - > # Optional components > XENSTAT_XENTOP := @monitors@ > OCAML_TOOLS := @ocamltools@ > diff --git a/config/Toplevel.mk.in b/config/Toplevel.mk.in > index 4ecacbb37d68..4db7eafcab5d 100644 > --- a/config/Toplevel.mk.in > +++ b/config/Toplevel.mk.in > @@ -1,2 +1 @@ > SUBSYSTEMS := @SUBSYSTEMS@ > -GIT_HTTP ?= @githttp@ > diff --git a/configure b/configure > index f5cd3c286b55..99f8434cbf64 100755 > --- a/configure > +++ b/configure > @@ -594,7 +594,6 @@ stubdom > tools > xen > subdirs > -githttp > DEBUG_DIR > XEN_DUMP_DIR > XEN_PAGING_DIR > @@ -673,7 +672,6 @@ with_xen_scriptdir > with_xen_dumpdir > with_rundir > with_debugdir > -enable_githttp > enable_xen > enable_tools > enable_stubdom > @@ -1309,8 +1307,6 @@ Optional Features: > --disable-option-checking ignore unrecognized --enable/--with options > --disable-FEATURE do not include FEATURE (same as --enable-FEATURE=no) > --enable-FEATURE[=ARG] include FEATURE [ARG=yes] > - --enable-githttp Download GIT repositories via HTTP (default is > - DISABLED) > --disable-xen Disable build and install of xen > --disable-tools Disable build and install of tools > --enable-stubdom Enable build and install of stubdom > @@ -2124,29 +2120,6 @@ DEBUG_DIR=$debugdir_path > > > > -# Check whether --enable-githttp was given. > -if test "${enable_githttp+set}" = set; then : > - enableval=$enable_githttp; > -fi > - > - > -if test "x$enable_githttp" = "xno"; then : > - > - ax_cv_githttp="n" > - > -elif test "x$enable_githttp" = "xyes"; then : > - > - ax_cv_githttp="y" > - > -elif test -z $ax_cv_githttp; then : > - > - ax_cv_githttp="n" > - > -fi > -githttp=$ax_cv_githttp > - > - > - > case "$host_cpu" in > i[3456]86|x86_64) > arch_enable_stubdom=y > diff --git a/configure.ac b/configure.ac > index 3aea40715307..19d9311c2ae4 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -17,7 +17,6 @@ m4_include([m4/subsystem.m4]) > m4_include([m4/paths.m4]) > > AX_XEN_EXPAND_CONFIG() > -AX_ARG_DEFAULT_DISABLE([githttp], [Download GIT repositories via HTTP]) > > dnl mini-os is only ported to certain platforms > case "$host_cpu" in > diff --git a/tools/configure b/tools/configure > index dae377c98252..bb5b1ae45067 100755 > --- a/tools/configure > +++ b/tools/configure > @@ -714,7 +714,6 @@ ovmf > xsmpolicy > ocamltools > monitors > -githttp > rpath > werror > DEBUG_DIR > @@ -807,7 +806,6 @@ with_rundir > with_debugdir > enable_werror > enable_rpath > -enable_githttp > enable_monitors > enable_ocamltools > enable_xsmpolicy > @@ -1494,8 +1492,6 @@ Optional Features: > --disable-werror Build tools without -Werror (default is ENABLED) > --enable-rpath Build tools with -Wl,-rpath,LIBDIR (default is > DISABLED) > - --enable-githttp Download GIT repositories via HTTP (default is > - DISABLED) > --disable-monitors Disable xenstat and xentop monitoring tools (default > is ENABLED) > --disable-ocamltools Disable Ocaml tools (default is ENABLED) > @@ -4156,29 +4152,6 @@ rpath=$ax_cv_rpath > > > > -# Check whether --enable-githttp was given. > -if test "${enable_githttp+set}" = set; then : > - enableval=$enable_githttp; > -fi > - > - > -if test "x$enable_githttp" = "xno"; then : > - > - ax_cv_githttp="n" > - > -elif test "x$enable_githttp" = "xyes"; then : > - > - ax_cv_githttp="y" > - > -elif test -z $ax_cv_githttp; then : > - > - ax_cv_githttp="n" > - > -fi > -githttp=$ax_cv_githttp > - > - > - > # Check whether --enable-monitors was given. > if test "${enable_monitors+set}" = set; then : > enableval=$enable_monitors; > diff --git a/tools/configure.ac b/tools/configure.ac > index 3a2f6a2da9e0..9bcf42f233f1 100644 > --- a/tools/configure.ac > +++ b/tools/configure.ac > @@ -84,7 +84,6 @@ AX_XEN_EXPAND_CONFIG() > # Enable/disable options > AX_ARG_DEFAULT_ENABLE([werror], [Build tools without -Werror]) > AX_ARG_DEFAULT_DISABLE([rpath], [Build tools with -Wl,-rpath,LIBDIR]) > -AX_ARG_DEFAULT_DISABLE([githttp], [Download GIT repositories via HTTP]) > AX_ARG_DEFAULT_ENABLE([monitors], [Disable xenstat and xentop monitoring tools]) > AX_ARG_DEFAULT_ENABLE([ocamltools], [Disable Ocaml tools]) > AX_ARG_DEFAULT_ENABLE([xsmpolicy], [Disable XSM policy compilation]) > -- > 2.30.2 > -- Sincerely, Demi Marie Obenour (she/her/hers) Invisible Things Lab
© 2016 - 2024 Red Hat, Inc.