Bug 40352 - Сломалась сборка audiofile
Summary: Сломалась сборка audiofile
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: audiofile (show other bugs)
Version: unstable
Hardware: all Linux
: P5 critical
Assignee: Yuri N. Sedunov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-07-02 13:25 MSK by AEN
Modified: 2021-07-05 14:37 MSK (History)
3 users (show)

See Also:


Attachments
патчи из debian libaudiofile1 0.3.6-5 (8.28 KB, application/x-compressed-tar)
2021-07-02 14:15 MSK, Michael Shigorin
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description AEN 2021-07-02 13:25:29 MSK
Прошу исправить.
Comment 1 Dmitry V. Levin 2021-07-02 14:10:10 MSK
Для начала VERBOSE=1 бы make check'у передать, чтобы было видно, что именно сломалось.
Comment 2 Michael Shigorin 2021-07-02 14:12:57 MSK
Согласно http://altlinux.org/Bug_Severity_Policy это всё же normal. :)

Юр, можешь глянуть?
Comment 3 AEN 2021-07-02 14:14:50 MSK
critical потому, что блокирует бранчевание.
Comment 4 Michael Shigorin 2021-07-02 14:15:05 MSK
Created attachment 9471 [details]
патчи из debian libaudiofile1 0.3.6-5

На всякий прилагаю дебиановский набор патчей -- на первый взгляд он полнее федориного, оба включают исправления переполнений и CVE.
Comment 5 Yuri N. Sedunov 2021-07-02 14:31:04 MSK
(Ответ для Michael Shigorin на комментарий #2)
> Согласно http://altlinux.org/Bug_Severity_Policy это всё же normal. :)
> 
> Юр, можешь глянуть?

Гляну.
Comment 6 Yuri N. Sedunov 2021-07-02 15:25:32 MSK
(Ответ для Dmitry V. Levin на комментарий #1)
> Для начала VERBOSE=1 бы make check'у передать, чтобы было видно, что именно
> сломалось.

...
FAIL: FLAC

* Sat Jun 26 2021 Dmitry V. Levin <ldv@altlinux.org> 1.3.3.0.79.37d1-alt1
- 1.3.3-62-gce6dd6b5 -> 1.3.3-79-g37d1a620.

Что будем чинить?
Comment 7 Dmitry V. Levin 2021-07-02 15:31:17 MSK
(In reply to Yuri N. Sedunov from comment #6)
> (Ответ для Dmitry V. Levin на комментарий #1)
> > Для начала VERBOSE=1 бы make check'у передать, чтобы было видно, что именно
> > сломалось.
> 
> ...
> FAIL: FLAC
> 
> * Sat Jun 26 2021 Dmitry V. Levin <ldv@altlinux.org> 1.3.3.0.79.37d1-alt1
> - 1.3.3-62-gce6dd6b5 -> 1.3.3-79-g37d1a620.
> 
> Что будем чинить?

Для начала VERBOSE=1 бы make check'у передать, чтобы было видно, что именно сломалось.
Comment 8 Yuri N. Sedunov 2021-07-02 15:43:15 MSK
#276770 BUILDING #1 [locked] [test-only] sisyphus srpm=audiofile-0.3.6-alt4.src.rpm

Логи очень большие, я не смог прочитать.
$ l ../BUILD/audiofile-0.3.6/test/FLAC.log 
-rw-r--r-- 1 builder builder 1777959404 Jul  2 12:14 ../BUILD/audiofile-0.3.6/test/FLAC.log
Comment 9 Dmitry V. Levin 2021-07-02 15:52:12 MSK
(In reply to Yuri N. Sedunov from comment #8)
> #276770 BUILDING #1 [locked] [test-only] sisyphus
> srpm=audiofile-0.3.6-alt4.src.rpm
> 
> Логи очень большие, я не смог прочитать.
> $ l ../BUILD/audiofile-0.3.6/test/FLAC.log 
> -rw-r--r-- 1 builder builder 1777959404 Jul  2 12:14
> ../BUILD/audiofile-0.3.6/test/FLAC.log

Урезанный лог:
http://git.altlinux.org/tasks/276770/build/100/i586/log

[00:05:36] FAIL: FLAC
[00:05:36] ==========
[00:05:36] 
[00:05:36] [==========] Running 5 tests from 1 test case.
[00:05:36] [----------] Global test environment set-up.
[00:05:36] [----------] 5 tests from FLAC
[00:05:36] [ RUN      ] FLAC.FLAC_16
[00:05:36] Audio File Library: could not seek to frame 0 [error 47]
[00:05:36] Lossless.h:108: Failure
[00:05:36] Value of: readData[i*channelCount + c]
[00:05:36]   Actual: 0
[00:05:36] Expected: data[i*channelCount + c]
[00:05:36] Which is: 1
[00:05:36] failed at 0
[00:05:36] Google Test trace:
[00:05:36] Lossless.h:69: 1
[00:05:36] Lossless.h:154: Sequential
[00:05:36] Lossless.h:108: Failure
и далее аналогично.
Comment 10 Dmitry V. Levin 2021-07-02 19:15:21 MSK
Откат коммита
https://github.com/xiph/flac/commit/159cd6c41a6ec17b36d74043c45a3aa64de90d5e
восстанавливает проходимость теста в audiofile, но этот коммит выглядит правдоподобно, и непонятно, может быть, тест делает что-то странное?
Comment 11 Repository Robot 2021-07-05 14:37:49 MSK
flac-1.3.3.0.79.37d1-alt2 -> sisyphus:

 Fri Jul 02 2021 Dmitry V. Levin <ldv@altlinux> 1.3.3.0.79.37d1-alt2
 - Revert upstream commit 1.3.3-64-g159cd6c4 to make audiofile FLAC test pass
   (closes: #40352).