Bug 40522

Summary:	ocaml-luv FTBFS
Product:	Sisyphus	Reporter:	Dmitry V. Levin <ldv>
Component:	ocaml-luv	Assignee:	Anton Farygin <rider>
Status:	CLOSED FIXED	QA Contact:	qa-sisyphus
Severity:	normal
Priority:	P5	CC:	aen, lav, rider
Version:	unstable
Hardware:	all
OS:	Linux
URL:	https://lists.altlinux.org/pipermail/sisyphus-incominger/2021-July/622431.html
Bug Depends on:
Bug Blocks:	27222

Description Dmitry V. Levin 2021-07-17 13:58:16 MSK

http://git.altlinux.org/tasks/279417/logs/events.1.1.log:
#100 build 0.5.8-alt1 from /gears/o/ocaml-luv.git fetched at 2021-Jul-17 10:23:15 from sisyphus
...
[x86_64] 4 failures! in 4.684s. 410 tests run.
[aarch64] 4 failures! in 5.363s. 410 tests run.
[ppc64le] 4 failures! in 4.981s. 410 tests run.
[i586] 4 failures! in 5.024s. 410 tests run.
[armh] 4 failures! in 5.333s. 410 tests run.

This is result of the following libuv update:
https://lists.altlinux.org/pipermail/sisyphus-incominger/2021-July/622256.html

proscriptions$ grep ocaml-luv dist.src_ftbfs
ocaml-luv	 alt-server-v-x86_64

Comment 1 Vitaly Lipatov 2021-07-17 23:00:10 MSK

потому что апстрим тестирует версию libuv, что несколько странно:

commit 2f66e155f4917f75a7116d2513e622b53309b592
Author: Anton Bachin <antonbachin@yahoo.com>
Date:   Mon Feb 15 20:22:33 2021 +0300

    Upgrade libuv to 1.41.0

diff --git a/test/version.ml b/test/version.ml
index e703837..57ec8e4 100644
--- a/test/version.ml
+++ b/test/version.ml
@@ -9,7 +9,7 @@ let tests = [
       Alcotest.(check int) "number" 1 Luv.Version.major);
 
     "minor", `Quick, (fun () ->
-      Alcotest.(check int) "number" 40 Luv.Version.minor);
+      Alcotest.(check int) "number" 41 Luv.Version.minor);
 
     "patch", `Quick, (fun () ->
       Alcotest.(check int) "number" 0 Luv.Version.patch);
@@ -21,12 +21,12 @@ let tests = [
       Alcotest.(check string) "suffix" "" Luv.Version.suffix);
 
     "hex", `Quick, (fun () ->
-      Alcotest.(check int) "number" 0x012800 Luv.Version.hex);
+      Alcotest.(check int) "number" 0x012900 Luv.Version.hex);
 
     "version", `Quick, (fun () ->
-      Alcotest.(check int) "number" 0x012800 (Luv.Version.version ()));
+      Alcotest.(check int) "number" 0x012900 (Luv.Version.version ()));
 
     "string", `Quick, (fun () ->
-      Alcotest.(check string) "value" "1.40.0" (Luv.Version.string ()));
+      Alcotest.(check string) "value" "1.41.0" (Luv.Version.string ()));
   ]
 ]

Comment 2 Vitaly Lipatov 2021-07-18 14:12:38 MSK

В таком случае надо либо ставить жёсткую зависимость на версию libuv (желательно и в бинарный пакет, тогда нельзя будет обновить libuv по ошибке), либо оторвать это тестирование версии.
Со своей стороны в связи с высокой стабильностью API у libuv я не вижу причин для жёсткой привязки.

Comment 3 Dmitry V. Levin 2021-07-28 19:50:09 MSK

ping

Comment 4 Anton Farygin 2021-07-29 08:22:47 MSK

pong

Comment 5 Anton Farygin 2021-07-29 11:47:39 MSK

(Ответ для Vitaly Lipatov на комментарий #2)
> В таком случае надо либо ставить жёсткую зависимость на версию libuv
> (желательно и в бинарный пакет, тогда нельзя будет обновить libuv по
> ошибке), либо оторвать это тестирование версии.
> Со своей стороны в связи с высокой стабильностью API у libuv я не вижу
> причин для жёсткой привязки.

На самом деле привязка осмысленна, но возможно не такая жёсткая.

ocaml-luv новой версии требует уже libluv-1.42.0 и тесты на нашей версии падают.

В общем я отправил в репозиторий ocaml-luv в котором заапдейтил версию в тестах, повешу что-то в апстрим по этому поводу и посмотрим на реакцию.