Bug 40522

Summary: ocaml-luv FTBFS
Product: Sisyphus Reporter: Dmitry V. Levin <ldv>
Component: ocaml-luvAssignee: 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 в котором заапдейтил версию в тестах, повешу что-то в апстрим по этому поводу и посмотрим на реакцию.