From 5c4f1b44baa4c2c74dd8abeb61bf4bd4df0353db Mon Sep 17 00:00:00 2001 From: Zach van Rijn Date: Wed, 3 May 2023 15:22:09 -0500 Subject: user/libgit2: delete broken test. fixes #686 again. --- user/libgit2/APKBUILD | 4 +-- user/libgit2/remove-flaky-revparse-test.patch | 40 +++++++++++++++++++++++++++ user/libgit2/test-fix.patch | 27 ------------------ 3 files changed, 42 insertions(+), 29 deletions(-) create mode 100644 user/libgit2/remove-flaky-revparse-test.patch delete mode 100644 user/libgit2/test-fix.patch diff --git a/user/libgit2/APKBUILD b/user/libgit2/APKBUILD index 634b9ea9c..5026d5cb3 100644 --- a/user/libgit2/APKBUILD +++ b/user/libgit2/APKBUILD @@ -14,7 +14,7 @@ depends_dev="curl-dev libssh2-dev" makedepends="$depends_dev cmake http-parser-dev openssl-dev python3 zlib-dev" subpackages="$pkgname-dev" source="$pkgname-$pkgver.tar.gz::https://github.com/$pkgname/$pkgname/archive/v$pkgver.tar.gz - test-fix.patch + remove-flaky-revparse-test.patch " # secfixes: @@ -49,4 +49,4 @@ package() { } sha512sums="abfea885f46444b0304ae57c32c06f4252afb0093c924da5e1ba10aaed952824d1b84036adb79b5b8ad8bea56a6331a51c62f3b9839aead16c7b26cb4554b53a libgit2-0.28.5.tar.gz -0df17aeec1445cd88a4dc361e6f2c9451e1b547bc6d1d932a245382184a466422bb66be9e1bfc1aab9370de43b2f0c33a102bab710ba8e0fdb7fc8e8bd875546 test-fix.patch" +ac38da9da307a898e68d5b9c0a77136f82fd873daa13f903e67aa8b7274d781d6d917d97d1affcfd7fd24885ac4b27ab91900446c49f842cc39a9fd39e8ba43e remove-flaky-revparse-test.patch" diff --git a/user/libgit2/remove-flaky-revparse-test.patch b/user/libgit2/remove-flaky-revparse-test.patch new file mode 100644 index 000000000..94d6806b8 --- /dev/null +++ b/user/libgit2/remove-flaky-revparse-test.patch @@ -0,0 +1,40 @@ +This patch differs from upstream in the following way(s): + + * Changed paths + + * Changed "11 years ago" to "10 years ago" to match state + before libgit2 bdab22384cc61d315005a65456a9f9563bb27c8f + + * Changed a commit hash to 'NULL' to match source code + +From 7f53d7b600a838c19276e835fd0188c361a63801 Mon Sep 17 00:00:00 2001 +From: Carl Dong +Date: Mon, 9 May 2022 12:09:08 -0400 +Subject: [PATCH] revparse: Remove error-prone, redundant test + +Originally introduced in: 776a6a8e5f8e258d31aded73c0ce38df6ac7bdc4 + +This test case has recently been fixed in bdab22384cc61d315005a65456a9f9563bb27c8f, but that fix will only last for a year. Next year the same problem will crop up and the test will need to be re-edited. + +This is not ideal as +- This test case becomes an unnecessary burden for developers +- Downstream distros or even just users who want to build older versions of libgit2 are guaranteed to have this test fail + +Furthermore, this test case is entirely unnecessary, as the functionality that was originally (see 776a6a8e5f8e258d31aded73c0ce38df6ac7bdc4) intended to be tested is well-covered by subsequent tests which specify a date instead of a "x ago" specification. +--- + tests/libgit2/refs/revparse.c | 2 -- + 1 file changed, 2 deletions(-) + +diff --git a/tests/libgit2/refs/revparse.c b/tests/libgit2/refs/revparse.c +index 93d7ce6db81..56af3c939be 100644 +--- a/tests/refs/revparse.c ++++ b/tests/refs/revparse.c +@@ -399,8 +399,6 @@ void test_refs_revparse__date(void) + * a65fedf HEAD@{1335806603 -0900}: commit: + * be3563a HEAD@{1335806563 -0700}: clone: from /Users/ben/src/libgit2/tests/resour + */ +- test_object("HEAD@{10 years ago}", NULL); +- + test_object("HEAD@{1 second}", "a65fedf39aefe402d3bb6e24df4d4f5fe4547750"); + test_object("HEAD@{1 second ago}", "a65fedf39aefe402d3bb6e24df4d4f5fe4547750"); + test_object("HEAD@{2 days ago}", "a65fedf39aefe402d3bb6e24df4d4f5fe4547750"); diff --git a/user/libgit2/test-fix.patch b/user/libgit2/test-fix.patch deleted file mode 100644 index a6eed991a..000000000 --- a/user/libgit2/test-fix.patch +++ /dev/null @@ -1,27 +0,0 @@ -From bdab22384cc61d315005a65456a9f9563bb27c8f Mon Sep 17 00:00:00 2001 -From: =?UTF-8?q?Carlos=20Mart=C3=ADn=20Nieto?= -Date: Tue, 3 May 2022 09:28:43 +0200 -Subject: [PATCH] revparse: adjust reflog test as we move away from the dates - -The dates we use in `refs::revparse::date` has just passed the ten years, so -now everything is beyond ten years, leading to an unexpected commit being -returned. - -Adjust it to 11 years so it all looks fine again. ---- - tests/libgit2/refs/revparse.c | 2 +- - 1 file changed, 1 insertion(+), 1 deletion(-) - -diff --git a/tests/libgit2/refs/revparse.c b/tests/libgit2/refs/revparse.c -index 0bd2ae5bc2c..93d7ce6db81 100644 ---- a/tests/refs/revparse.c -+++ b/tests/refs/revparse.c -@@ -400,7 +400,7 @@ void test_refs_revparse__date(void) - * a65fedf HEAD@{1335806603 -0900}: commit: - * be3563a HEAD@{1335806563 -0700}: clone: from /Users/ben/src/libgit2/tests/resour - */ -- test_object("HEAD@{10 years ago}", NULL); -+ test_object("HEAD@{11 years ago}", NULL); - - test_object("HEAD@{1 second}", "a65fedf39aefe402d3bb6e24df4d4f5fe4547750"); - test_object("HEAD@{1 second ago}", "a65fedf39aefe402d3bb6e24df4d4f5fe4547750"); -- cgit v1.2.3-70-g09d2