Bug 40815 - process-lto: bad grammar of the error message
Summary: process-lto: bad grammar of the error message
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: rpm-build (show other bugs)
Version: unstable
Hardware: x86_64 Linux
: P5 normal
Assignee: placeholder@altlinux.org
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-08-25 23:27 MSK by viy
Modified: 2021-12-15 23:17 MSK (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description viy 2021-08-25 23:27:48 MSK
Грамматически неправильное сообщение в process-lto, не хватает глагола.
пример:
process-lto: ERROR: ./usr/lib64/libppl_c.a: contains __gnu_lto_slim only. Perhaps, you need to %global optflags_lto %optflags_lto -ffat-lto-objects

должно быть что-то вроде
Perhaps, you need to ADD %global optflags_lto %optflags_lto -ffat-lto-objects
Perhaps, you need to DEFINE %global optflags_lto %optflags_lto -ffat-lto-objects
или вообще переформулировать.

В таком виде не хочется добавлять в логовед,
так как потом кто-то молча исправит и правило в логовед внезапно перестанет работать.
Comment 1 Dmitry V. Levin 2021-08-25 23:34:12 MSK
(In reply to viy from comment #0)
> Грамматически неправильное сообщение в process-lto, не хватает глагола.
> пример:
> process-lto: ERROR: ./usr/lib64/libppl_c.a: contains __gnu_lto_slim only.
> Perhaps, you need to %global optflags_lto %optflags_lto -ffat-lto-objects
> 
> должно быть что-то вроде
> Perhaps, you need to ADD %global optflags_lto %optflags_lto -ffat-lto-objects
> Perhaps, you need to DEFINE %global optflags_lto %optflags_lto
> -ffat-lto-objects

Или "to use".

> или вообще переформулировать.
> 
> В таком виде не хочется добавлять в логовед,
> так как потом кто-то молча исправит и правило в логовед внезапно перестанет
> работать.

А там разве нельзя регексп какой-нибудь написать?
Comment 2 viy 2021-08-25 23:39:03 MSK
(Ответ для Dmitry V. Levin на комментарий #1)

Я же не знаю, что будет в финальной фразе. На каких-то вариантах regexp сработатет, на других нет.
Лучше сразу поправить.
Comment 3 Vitaly Chikunov 2021-08-25 23:42:40 MSK
Я можно фразу "Perhaps.*" вообще не учитывать? Это просто совет. Вдруг там будет "%{?optflags_lto:%global optflags_lto %optflags_lto -ffat-lto-objects}".
Comment 4 Dmitry V. Levin 2021-08-25 23:46:48 MSK
Кстати да, я использую '^process-lto: ERROR: ' для сбора своей статистики.
Comment 5 Vitaly Chikunov 2021-08-25 23:55:09 MSK
Там ещё может не быть слова "only" в "__gnu_lto_slim only".
Comment 6 viy 2021-08-29 09:41:30 MSK
В devel@ было хорошее замечание от asheplyakov@:

Тогда почему бы не написать об этом прямо, например:

process-lto: ERROR: ./usr/lib64/libfoo.a: contains GIMPLE bytecode only.
Bytecode should NOT be packaged since its format can change between GCC
+versions.
To package the machine code in static libraries use -ffat-lto-objects option:
%define optflags_lto %optflags_lto -ffat-lto-object
Alternatively you might want to stop packaging static libraries.

А не говорить загадками -- "contains __gnu_lto_slim only".
"Perhaps you need" - а как понять, таки нужно, или не нужно?
Comment 7 Vitaly Chikunov 2021-08-29 09:57:22 MSK
На мой взгляд, предложенный текст не правильный, на раз он может ещё меняться, то тем более постоянная часть текста ошибки тут только "process-lto: ERROR:", как и предложил Дмитрий.
Comment 8 Dmitry V. Levin 2021-12-15 23:17:25 MSK
* Thu Sep 09 2021 Dmitry V. Levin <ldv@altlinux> 4.0.4.178-alt1
- process-lto: enhanced error diagnostics.