Jan Tojnar 2020-02-08 01:49:06 +01:00 committed by worldofpeace
parent c55c3b9c3c
commit 26a06b7917
4 changed files with 79 additions and 44 deletions

View file

@ -1,4 +1,4 @@
From a9a62d7c7adf6fc0237c7d04937b538a4cea87ad Mon Sep 17 00:00:00 2001 From 977fdfad2ceba7232b4f78144b20640d7fd0aedb Mon Sep 17 00:00:00 2001
From: Colin Walters <walters@verbum.org> From: Colin Walters <walters@verbum.org>
Date: Tue, 19 Jun 2018 09:34:18 -0400 Date: Tue, 19 Jun 2018 09:34:18 -0400
Subject: [PATCH] Drop "ostree trivial-httpd" CLI, move to tests directory Subject: [PATCH] Drop "ostree trivial-httpd" CLI, move to tests directory
@ -13,22 +13,22 @@ Also at this point nothing should depend on `ostree trivial-httpd`.
Makefile-man.am | 6 -- Makefile-man.am | 6 --
Makefile-ostree.am | 7 --- Makefile-ostree.am | 7 ---
Makefile-tests.am | 7 +++ Makefile-tests.am | 7 +++
configure.ac | 10 --- configure.ac | 9 ---
man/ostree-trivial-httpd.xml | 118 ----------------------------------- man/ostree-trivial-httpd.xml | 118 -----------------------------------
src/ostree/main.c | 5 -- src/ostree/main.c | 5 --
tests/libtest.sh | 13 ++-- tests/libtest.sh | 13 ++--
7 files changed, 12 insertions(+), 154 deletions(-) 7 files changed, 12 insertions(+), 153 deletions(-)
delete mode 100644 man/ostree-trivial-httpd.xml delete mode 100644 man/ostree-trivial-httpd.xml
diff --git a/Makefile-man.am b/Makefile-man.am diff --git a/Makefile-man.am b/Makefile-man.am
index 8ccbba8c..d204aa3e 100644 index bc58103b..bcfde285 100644
--- a/Makefile-man.am --- a/Makefile-man.am
+++ b/Makefile-man.am +++ b/Makefile-man.am
@@ -34,12 +34,6 @@ ostree-init.1 ostree-log.1 ostree-ls.1 ostree-prune.1 ostree-pull-local.1 \ @@ -34,12 +34,6 @@ ostree-init.1 ostree-log.1 ostree-ls.1 ostree-prune.1 ostree-pull-local.1 \
ostree-pull.1 ostree-refs.1 ostree-remote.1 ostree-reset.1 \ ostree-pull.1 ostree-refs.1 ostree-remote.1 ostree-reset.1 \
ostree-rev-parse.1 ostree-show.1 ostree-summary.1 \ ostree-rev-parse.1 ostree-show.1 ostree-summary.1 \
ostree-static-delta.1 ostree-static-delta.1
-if BUILDOPT_TRIVIAL_HTTPD -if USE_LIBSOUP
-man1_files += ostree-trivial-httpd.1 -man1_files += ostree-trivial-httpd.1
-else -else
-# We still want to distribute the source, even if we are not building it -# We still want to distribute the source, even if we are not building it
@ -38,10 +38,10 @@ index 8ccbba8c..d204aa3e 100644
if BUILDOPT_FUSE if BUILDOPT_FUSE
man1_files += rofiles-fuse.1 man1_files += rofiles-fuse.1
diff --git a/Makefile-ostree.am b/Makefile-ostree.am diff --git a/Makefile-ostree.am b/Makefile-ostree.am
index 8d352e38..1471b3e5 100644 index f861afe4..497d99b0 100644
--- a/Makefile-ostree.am --- a/Makefile-ostree.am
+++ b/Makefile-ostree.am +++ b/Makefile-ostree.am
@@ -133,13 +133,6 @@ ostree_SOURCES += src/ostree/ot-builtin-pull.c @@ -144,13 +144,6 @@ ostree_SOURCES += src/ostree/ot-builtin-pull.c
endif endif
if USE_LIBSOUP if USE_LIBSOUP
@ -56,12 +56,12 @@ index 8d352e38..1471b3e5 100644
# This is necessary for the cookie jar bits # This is necessary for the cookie jar bits
ostree_CFLAGS += $(OT_INTERNAL_SOUP_CFLAGS) ostree_CFLAGS += $(OT_INTERNAL_SOUP_CFLAGS)
diff --git a/Makefile-tests.am b/Makefile-tests.am diff --git a/Makefile-tests.am b/Makefile-tests.am
index 2c0916f6..b11fde89 100644 index fc2f2d91..7343b63f 100644
--- a/Makefile-tests.am --- a/Makefile-tests.am
+++ b/Makefile-tests.am +++ b/Makefile-tests.am
@@ -248,6 +248,13 @@ _installed_or_uninstalled_test_programs = tests/test-varint tests/test-ot-unix-u @@ -263,6 +263,13 @@ _installed_or_uninstalled_test_programs += \
tests/test-gpg-verify-result tests/test-checksum tests/test-lzma tests/test-rollsum \ $(NULL)
tests/test-basic-c tests/test-sysroot-c tests/test-pull-c tests/test-repo tests/test-include-ostree-h endif
+if USE_LIBSOUP +if USE_LIBSOUP
+test_extra_programs += ostree-trivial-httpd +test_extra_programs += ostree-trivial-httpd
@ -74,10 +74,10 @@ index 2c0916f6..b11fde89 100644
test_programs += tests/test-repo-finder-avahi test_programs += tests/test-repo-finder-avahi
endif endif
diff --git a/configure.ac b/configure.ac diff --git a/configure.ac b/configure.ac
index e6e145db..1e36e6a0 100644 index 46a900f5..2f91cdec 100644
--- a/configure.ac --- a/configure.ac
+++ b/configure.ac +++ b/configure.ac
@@ -187,15 +187,6 @@ if test x$with_soup != xno; then OSTREE_FEATURES="$OSTREE_FEATURES libsoup"; fi @@ -190,14 +190,6 @@ if test x$with_soup != xno; then OSTREE_FEATURES="$OSTREE_FEATURES libsoup"; fi
AM_CONDITIONAL(USE_LIBSOUP, test x$with_soup != xno) AM_CONDITIONAL(USE_LIBSOUP, test x$with_soup != xno)
AM_CONDITIONAL(HAVE_LIBSOUP_CLIENT_CERTS, test x$have_libsoup_client_certs = xyes) AM_CONDITIONAL(HAVE_LIBSOUP_CLIENT_CERTS, test x$have_libsoup_client_certs = xyes)
@ -85,15 +85,14 @@ index e6e145db..1e36e6a0 100644
- [AS_HELP_STRING([--enable-trivial-httpd-cmdline], - [AS_HELP_STRING([--enable-trivial-httpd-cmdline],
- [Continue to support "ostree trivial-httpd" [default=no]])],, - [Continue to support "ostree trivial-httpd" [default=no]])],,
- enable_trivial_httpd_cmdline=no) - enable_trivial_httpd_cmdline=no)
-AM_CONDITIONAL(BUILDOPT_TRIVIAL_HTTPD, test x$enable_trivial_httpd_cmdline = xyes) -AS_IF([test x$enable_trivial_httpd_cmdline = xyes],
-AM_COND_IF(BUILDOPT_TRIVIAL_HTTPD,
- [AC_DEFINE([BUILDOPT_ENABLE_TRIVIAL_HTTPD_CMDLINE], 1, [Define if we are enabling ostree trivial-httpd entrypoint])] - [AC_DEFINE([BUILDOPT_ENABLE_TRIVIAL_HTTPD_CMDLINE], 1, [Define if we are enabling ostree trivial-httpd entrypoint])]
-) -)
- -
AS_IF([test x$with_curl = xyes && test x$with_soup = xno], [ AS_IF([test x$with_curl = xyes && test x$with_soup = xno], [
AC_MSG_WARN([Curl enabled, but libsoup is not; libsoup is needed for tests (make check, etc.)]) AC_MSG_WARN([Curl enabled, but libsoup is not; libsoup is needed for tests (make check, etc.)])
]) ])
@@ -602,7 +593,6 @@ echo " @@ -617,7 +609,6 @@ echo "
Rust (internal oxidation): $rust_debug_release Rust (internal oxidation): $rust_debug_release
rofiles-fuse: $enable_rofiles_fuse rofiles-fuse: $enable_rofiles_fuse
HTTP backend: $fetcher_backend HTTP backend: $fetcher_backend
@ -226,10 +225,10 @@ index d03c12be..00000000
- </refsect1> - </refsect1>
-</refentry> -</refentry>
diff --git a/src/ostree/main.c b/src/ostree/main.c diff --git a/src/ostree/main.c b/src/ostree/main.c
index c5b45012..6478a62b 100644 index a523ff9a..61ea742d 100644
--- a/src/ostree/main.c --- a/src/ostree/main.c
+++ b/src/ostree/main.c +++ b/src/ostree/main.c
@@ -116,11 +116,6 @@ static OstreeCommand commands[] = { @@ -118,11 +118,6 @@ static OstreeCommand commands[] = {
{ "summary", OSTREE_BUILTIN_FLAG_NONE, { "summary", OSTREE_BUILTIN_FLAG_NONE,
ostree_builtin_summary, ostree_builtin_summary,
"Manage summary metadata" }, "Manage summary metadata" },
@ -242,10 +241,10 @@ index c5b45012..6478a62b 100644
}; };
diff --git a/tests/libtest.sh b/tests/libtest.sh diff --git a/tests/libtest.sh b/tests/libtest.sh
index e0022512..b07dc962 100755 index 3f5fd931..eacd96de 100755
--- a/tests/libtest.sh --- a/tests/libtest.sh
+++ b/tests/libtest.sh +++ b/tests/libtest.sh
@@ -149,15 +149,12 @@ fi @@ -160,15 +160,12 @@ fi
if test -n "${OSTREE_UNINSTALLED:-}"; then if test -n "${OSTREE_UNINSTALLED:-}"; then
OSTREE_HTTPD=${OSTREE_UNINSTALLED}/ostree-trivial-httpd OSTREE_HTTPD=${OSTREE_UNINSTALLED}/ostree-trivial-httpd
else else
@ -267,5 +266,5 @@ index e0022512..b07dc962 100755
files_are_hardlinked() { files_are_hardlinked() {
-- --
2.22.0 2.25.0

View file

@ -1,6 +1,7 @@
{ stdenv { stdenv
, fetchurl , fetchurl
, fetchpatch , fetchpatch
, substituteAll
, pkgconfig , pkgconfig
, gtk-doc , gtk-doc
, gobject-introspection , gobject-introspection
@ -13,6 +14,7 @@
, libsoup , libsoup
, gpgme , gpgme
, which , which
, makeWrapper
, autoconf , autoconf
, automake , automake
, libtool , libtool
@ -29,28 +31,41 @@
, python3 , python3
}: }:
stdenv.mkDerivation rec { let
testPython = (python3.withPackages (p: with p; [
pyyaml
]));
in stdenv.mkDerivation rec {
pname = "ostree"; pname = "ostree";
version = "2019.2"; version = "2019.6";
outputs = [ "out" "dev" "man" "installedTests" ]; outputs = [ "out" "dev" "man" "installedTests" ];
src = fetchurl { src = fetchurl {
url = "https://github.com/ostreedev/ostree/releases/download/v${version}/libostree-${version}.tar.xz"; url = "https://github.com/ostreedev/ostree/releases/download/v${version}/libostree-${version}.tar.xz";
sha256 = "0nbbrz3p4ms6vpl272q6fimqvizryw2a8mnfqcn69xf03sz5204y"; sha256 = "1bhrfbjna3rnymijxagzkdq2zl74g71s2xmimihjhvcw2zybi0jl";
}; };
patches = [ patches = [
# Tests access the helper using relative path
# https://github.com/ostreedev/ostree/issues/1593
# Patch from https://github.com/ostreedev/ostree/pull/1633
./01-Drop-ostree-trivial-httpd-CLI-move-to-tests-director.patch
# Fix tests running in Catalan instead of C locale.
(fetchpatch {
url = "https://github.com/ostreedev/ostree/commit/5135a1e58ade2bfafc8c1fda359540eafd72531e.patch";
sha256 = "1crzaagw1zzx8v6rsnxb9jnc3ij9hlpvdl91w3skqdm28adx7yx8";
})
# Workarounds for https://github.com/ostreedev/ostree/issues/1592 # Workarounds for https://github.com/ostreedev/ostree/issues/1592
./fix-1592.patch ./fix-1592.patch
# Disable test-gpg-verify-result.test, # Hard-code paths in tests
# https://github.com/ostreedev/ostree/issues/1634 (substituteAll {
./disable-test-gpg-verify-result.patch src = ./fix-test-paths.patch;
python3 = testPython.interpreter;
# Tests access the helper using relative path })
# https://github.com/ostreedev/ostree/issues/1593
./01-Drop-ostree-trivial-httpd-CLI-move-to-tests-director.patch
]; ];
nativeBuildInputs = [ nativeBuildInputs = [
@ -61,6 +76,7 @@ stdenv.mkDerivation rec {
gtk-doc gtk-doc
gobject-introspection gobject-introspection
which which
makeWrapper
yacc yacc
libxslt libxslt
docbook_xsl docbook_xsl
@ -82,7 +98,7 @@ stdenv.mkDerivation rec {
utillinuxMinimal # for libmount utillinuxMinimal # for libmount
# for installed tests # for installed tests
(python3.withPackages (p: with p; [ pyyaml ])) testPython
gjs gjs
]; ];
@ -103,6 +119,12 @@ stdenv.mkDerivation rec {
"installed_test_metadir=${placeholder "installedTests"}/share/installed-tests/libostree" "installed_test_metadir=${placeholder "installedTests"}/share/installed-tests/libostree"
]; ];
postFixup = ''
for test in $installedTests/libexec/installed-tests/libostree/*.js; do
wrapProgram "$test" --prefix GI_TYPELIB_PATH : "$out/lib/girepository-1.0"
done
'';
passthru = { passthru = {
tests = { tests = {
installedTests = nixosTests.installed-tests.ostree; installedTests = nixosTests.installed-tests.ostree;

View file

@ -1,12 +0,0 @@
diff --git a/Makefile-tests.am b/Makefile-tests.am
index b11fde89..82937a67 100644
--- a/Makefile-tests.am
+++ b/Makefile-tests.am
@@ -245,7 +245,6 @@ endif
_installed_or_uninstalled_test_programs = tests/test-varint tests/test-ot-unix-utils tests/test-bsdiff tests/test-mutable-tree \
tests/test-keyfile-utils tests/test-ot-opt-utils tests/test-ot-tool-util \
- tests/test-gpg-verify-result tests/test-checksum tests/test-lzma tests/test-rollsum \
tests/test-basic-c tests/test-sysroot-c tests/test-pull-c tests/test-repo tests/test-include-ostree-h
if USE_LIBSOUP

View file

@ -0,0 +1,26 @@
diff --git a/tests/test-basic-user-only.sh b/tests/test-basic-user-only.sh
index f65094fd..105be893 100755
--- a/tests/test-basic-user-only.sh
+++ b/tests/test-basic-user-only.sh
@@ -29,7 +29,7 @@ extra_basic_tests=5
. $(dirname $0)/basic-test.sh
$CMD_PREFIX ostree --version > version.yaml
-python3 -c 'import yaml; yaml.safe_load(open("version.yaml"))'
+@python3@ -c 'import yaml; yaml.safe_load(open("version.yaml"))'
echo "ok yaml version"
# Reset things so we don't inherit a lot of state from earlier tests
diff --git a/tests/test-remote-headers.sh b/tests/test-remote-headers.sh
index a41d087a..77b34c90 100755
--- a/tests/test-remote-headers.sh
+++ b/tests/test-remote-headers.sh
@@ -26,7 +26,7 @@ echo '1..2'
. $(dirname $0)/libtest.sh
V=$($CMD_PREFIX ostree --version | \
- python3 -c 'import sys, yaml; print(yaml.safe_load(sys.stdin)["libostree"]["Version"])')
+ @python3@ -c 'import sys, yaml; print(yaml.safe_load(sys.stdin)["libostree"]["Version"])')
setup_fake_remote_repo1 "archive" "" \
--expected-header foo=bar \