Bug 28408 - [PATCH] исправленный debuginfo пакет для monit
Summary: [PATCH] исправленный debuginfo пакет для monit
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: monit (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: Michael Shigorin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-01-20 03:39 MSK by Igor Zubkov
Modified: 2014-03-18 14:45 MSK (History)
2 users (show)

See Also:


Attachments
Сам патч (3.18 KB, patch)
2013-01-20 03:40 MSK, Igor Zubkov
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Igor Zubkov 2013-01-20 03:39:09 MSK
Monit при сборке игнорирует флаги оптимизации которые выставляет rpm. Из-за этого, пропадает -g и -O2 меняется на -O3. Из-за этого debuginfo пакет бесполезен. -g он не должен вырезать, а -O3 ему не нужно. Приложенный патч это исправляет.

При выходе следующей версии, я обновлю патч если надо будет.
Comment 1 Igor Zubkov 2013-01-20 03:40:21 MSK
Created attachment 5712 [details]
Сам патч
Comment 2 Michael Shigorin 2014-03-17 19:37:51 MSK
Достаточно убрать из спека %{subst_enable optimized}, незачем ломать configure.

CFLAGS='%optflags' ломает сборку, т.к. тогда недостаёт как минимум -std=c99 (дублировать и затем отслеживать апстримную логику неохота).

Спасибо, сделано в 5.7-alt2 (решил не делать отдельный -alt3).
Comment 3 Repository Robot 2014-03-17 19:59:16 MSK
monit-5.7-alt2 -> sisyphus:

* Mon Mar 17 2014 Michael Shigorin <mike@altlinux> 5.7-alt2
- fixes/enhancements by icesik@:
  + converted spec to utf8 (closes: #28406)
  + added sample sysconfig file (closes: #28407)
  + dropped ancient style initscript (for ALM2.2 and before)
  + do not build with explicit optimization to fix debuginfo subpackage
    (closes: #28408)
Comment 4 Igor Zubkov 2014-03-18 14:45:52 MSK
(В ответ на комментарий №2)
> Достаточно убрать из спека %{subst_enable optimized}, незачем ломать configure.

Видимо уже что-то починили в самом monit. Когда я делал патч, малой кровью обойтись не получилось.

> CFLAGS='%optflags' ломает сборку, т.к. тогда недостаёт как минимум -std=c99
> (дублировать и затем отслеживать апстримную логику неохота).
> 
> Спасибо, сделано в 5.7-alt2 (решил не делать отдельный -alt3).

Так тоже работает. Я проверил лог и всё в порядке. Спасибо!