Bug 40522 - ocaml-luv FTBFS
Summary: ocaml-luv FTBFS
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: ocaml-luv (show other bugs)
Version: unstable
Hardware: all Linux
: P5 normal
Assignee: Anton Farygin
QA Contact: qa-sisyphus
URL: https://lists.altlinux.org/pipermail/...
Keywords:
Depends on:
Blocks: 27222
  Show dependency tree
 
Reported: 2021-07-17 13:58 MSK by Dmitry V. Levin
Modified: 2021-07-29 11:47 MSK (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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 в котором заапдейтил версию в тестах, повешу что-то в апстрим по этому поводу и посмотрим на реакцию.