Hello community, here is the log from the commit of package libguestfs for openSUSE:Factory checked in at 2017-10-26 18:45:42 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/libguestfs (Old) and /work/SRC/openSUSE:Factory/.libguestfs.new (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "libguestfs" Thu Oct 26 18:45:42 2017 rev:57 rq:536717 version:1.36.5 Changes: -------- --- /work/SRC/openSUSE:Factory/libguestfs/libguestfs.changes 2017-10-19 19:34:16.374879679 +0200 +++ /work/SRC/openSUSE:Factory/.libguestfs.new/libguestfs.changes 2017-10-26 18:45:59.747437963 +0200 @@ -1,0 +2,10 @@ +Wed Oct 25 13:24:50 UTC 2017 - cbosdon...@suse.com + +- Move guestfs-data to libguestfs0. bsc#1064399 +- Force distro detection + Removed: + * use-idlike.patch + Added: + * 531316cc-build-improve-and-simplify-distro-detection.patch + +------------------------------------------------------------------- Old: ---- use-idlike.patch New: ---- 531316cc-build-improve-and-simplify-distro-detection.patch ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ libguestfs.spec ++++++ --- /var/tmp/diff_new_pack.I0S2o8/_old 2017-10-26 18:46:01.131373368 +0200 +++ /var/tmp/diff_new_pack.I0S2o8/_new 2017-10-26 18:46:01.131373368 +0200 @@ -143,8 +143,8 @@ # PATCH-FIX-UPSTREAM - python3 fixes Patch1: 9d25b4e5-python-add-simple-wrappers-for-PyObject-string-funct.patch Patch2: f3f99a09-python-use-right-func-when-PyString_FromStringAndSiz.patch -# PATCH-FIX-OPENSUSE - pending upstream review, fixing distro detection in OBS -Patch3: use-idlike.patch +# PATCH-FIX-UPSTREAM - add configure --with-distro parameter +Patch3: 531316cc-build-improve-and-simplify-distro-detection.patch Patch100: appliance.patch Source0: http://download.libguestfs.org/1.36-stable/libguestfs-%{version}.tar.gz @@ -195,7 +195,6 @@ Requires: perl(Win::Hivex::Regedit) %endif %endif -Requires: guestfs-data >= %{version} %if %{with fuse} Requires: fuse @@ -274,7 +273,6 @@ # Provides: libguestfs-perl = %{version} Obsoletes: libguestfs-perl < %{version} -Requires: guestfs-data >= %{version} Requires: perl(File::Temp) Requires: perl(Locale::TextDomain) %perl_requires @@ -289,7 +287,6 @@ License: GPL-2.0 Group: Development/Languages/Lua BuildRequires: lua-devel -Requires: guestfs-data >= %{version} %define _configure_lua --enable-lua # @@ -313,7 +310,6 @@ Obsoletes: libguestfs-python < %{version} Obsoletes: python-libguestfs < %{version} Provides: python-libguestfs = %{version} -Requires: guestfs-data >= %{version} %description -n python2-libguestfs Allows Python 2 scripts to directly use libguestfs. @@ -327,7 +323,6 @@ Obsoletes: libguestfs-python < %{version} Obsoletes: python-libguestfs < %{version} Provides: python-libguestfs = %{version} -Requires: guestfs-data >= %{version} %description -n python3-libguestfs Allows Python 3 scripts to directly use libguestfs. @@ -341,7 +336,6 @@ BuildRequires: ruby BuildRequires: ruby-devel BuildRequires: rubygem-rake -Requires: guestfs-data >= %{version} %define _configure_ruby --enable-ruby # @@ -495,6 +489,7 @@ License: LGPL-2.1 Group: System/Libraries Requires: %{kvm_binary} +Requires: guestfs-data >= %{version} Requires: qemu >= 2.0 Requires: qemu-tools Requires: supermin >= 5.1.6 @@ -599,7 +594,8 @@ %{_configure_python} \ %{_configure_ruby} \ --disable-rpath \ - --disable-static + --disable-static \ + --with-distro=SUSE #Workaround an autotools bug make -j1 -C builder index-parse.c # 'INSTALLDIRS' ensures that perl libs are installed in the vendor dir instead of the site dir ++++++ 531316cc-build-improve-and-simplify-distro-detection.patch ++++++ >From 531316cc3f25db22d94310334dc8b61596dc263f Mon Sep 17 00:00:00 2001 From: Pino Toscano <ptosc...@redhat.com> Date: Mon, 25 Sep 2017 14:29:47 +0200 Subject: [PATCH] build: improve and simplify distro detection Add a --with-distro=ID argument for configure, so it is possible to manually specify the distro to use for the packages (in case os-release does not provide ID=.., or the ID is not recognized yet). In the case when --with-distro is not set, keep doing the autodetection, but using os-release only, i.e. dropping the checks for all the other -release files -- since there is --with-distro, older distros with no os-release can still be used. RWMJ: Add documentation to guestfs-building(1). --- docs/guestfs-building.pod | 14 ++++++++++++++ m4/guestfs_appliance.m4 | 44 +++++++++++++++++++------------------------- 2 files changed, 33 insertions(+), 25 deletions(-) diff --git a/docs/guestfs-building.pod b/docs/guestfs-building.pod index ecc27f9d6..187da37be 100644 --- a/docs/guestfs-building.pod +++ b/docs/guestfs-building.pod @@ -648,6 +648,20 @@ Note that despite this setting, all backends are built into libguestfs, and you can override the backend at runtime by setting the C<$LIBGUESTFS_BACKEND> environment variable (or using API methods). +=item B<--with-distro=REDHAT|DEBIAN|...> + +Libguestfs needs to know which Linux distro is in use so it can choose +package names for the appliance correctly (see for example +F<appliance/packagelist.in>). It normally does this automatically. + +However if you can building or packaging libguestfs on a new distro +then you can use I<--with-distro> to specify that the distro is +similar to an existing one (eg. I<--with-distro=REDHAT> if the distro +is a new Red Hat or CentOS derivative). + +Note that if your distro is completely new then it may still require +upstream modifications. + =item B<--with-extra=">I<distroname>=I<version>,libvirt,...B<"> =item B<--with-extra="local"> diff --git a/m4/guestfs_appliance.m4 b/m4/guestfs_appliance.m4 index fbba3373f..4993f57a1 100644 --- a/m4/guestfs_appliance.m4 +++ b/m4/guestfs_appliance.m4 @@ -94,31 +94,25 @@ dnl names vary slightly across distros. (See dnl appliance/packagelist.in, appliance/excludefiles.in, dnl appliance/hostfiles.in) AC_MSG_CHECKING([which Linux distro for package names]) -if test -f /etc/os-release; then - ( . /etc/os-release && echo $ID | tr '@<:@:lower:@:>@' '@<:@:upper:@:>@' ) >&AS_MESSAGE_LOG_FD - DISTRO="`. /etc/os-release && echo $ID | tr '@<:@:lower:@:>@' '@<:@:upper:@:>@'`" - AS_CASE([$DISTRO], - [FEDORA | RHEL | CENTOS],[DISTRO=REDHAT], - [OPENSUSE | SLED | SLES],[DISTRO=SUSE], - [ARCH],[DISTRO=ARCHLINUX]) -elif test -f /etc/debian_version; then - DISTRO=DEBIAN - if grep -q 'DISTRIB_ID=Ubuntu' /etc/lsb-release 2>&AS_MESSAGE_LOG_FD; then - DISTRO=UBUNTU - fi -elif test -f /etc/arch-release; then - DISTRO=ARCHLINUX -elif test -f /etc/SuSE-release; then - DISTRO=SUSE -elif test -f /etc/frugalware-release; then - DISTRO=FRUGALWARE -elif test -f /etc/mageia-release; then - DISTRO=MAGEIA -else -dnl fallback option - DISTRO=REDHAT -fi -AC_MSG_RESULT([$DISTRO]) +AC_ARG_WITH([distro], + [AS_HELP_STRING([--with-distro="DISTRO_ID"], + [distro ID @<:@default=ID in /etc/os-release@:>@])],[ + DISTRO="$withval" + AC_MSG_RESULT([$DISTRO (manually specified)]) + ],[ + if test -f /etc/os-release; then + ( . /etc/os-release && echo $ID | tr '@<:@:lower:@:>@' '@<:@:upper:@:>@' ) >&AS_MESSAGE_LOG_FD + DISTRO="`. /etc/os-release && echo $ID | tr '@<:@:lower:@:>@' '@<:@:upper:@:>@'`" + AS_CASE([$DISTRO], + [FEDORA | RHEL | CENTOS],[DISTRO=REDHAT], + [OPENSUSE | SLED | SLES],[DISTRO=SUSE], + [ARCH],[DISTRO=ARCHLINUX]) + AC_MSG_RESULT([$DISTRO (from /etc/os-release)]) + else + AC_MSG_ERROR([/etc/os-release not available, please specify the distro using --with-distro=DISTRO]) + fi + ] +) AC_SUBST([DISTRO]) dnl Add extra packages to the appliance. -- 2.13.2