Summary: | process-lto: bad grammar of the error message | ||
---|---|---|---|
Product: | Sisyphus | Reporter: | viy <viy> |
Component: | rpm-build | Assignee: | placeholder <placeholder> |
Status: | CLOSED FIXED | QA Contact: | qa-sisyphus |
Severity: | normal | ||
Priority: | P5 | CC: | arseny, glebfm, imz, ldv, placeholder, vt |
Version: | unstable | ||
Hardware: | x86_64 | ||
OS: | Linux |
Description
viy
2021-08-25 23:27:48 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". > или вообще переформулировать. > > В таком виде не хочется добавлять в логовед, > так как потом кто-то молча исправит и правило в логовед внезапно перестанет > работать. А там разве нельзя регексп какой-нибудь написать? (Ответ для Dmitry V. Levin на комментарий #1) Я же не знаю, что будет в финальной фразе. На каких-то вариантах regexp сработатет, на других нет. Лучше сразу поправить. Я можно фразу "Perhaps.*" вообще не учитывать? Это просто совет. Вдруг там будет "%{?optflags_lto:%global optflags_lto %optflags_lto -ffat-lto-objects}". Кстати да, я использую '^process-lto: ERROR: ' для сбора своей статистики. Там ещё может не быть слова "only" в "__gnu_lto_slim only". В 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" - а как понять, таки нужно, или не нужно? На мой взгляд, предложенный текст не правильный, на раз он может ещё меняться, то тем более постоянная часть текста ошибки тут только "process-lto: ERROR:", как и предложил Дмитрий. * Thu Sep 09 2021 Dmitry V. Levin <ldv@altlinux> 4.0.4.178-alt1 - process-lto: enhanced error diagnostics. |