<?xml version="1.0" encoding="UTF-8" ?>

<bugzilla version="5.2"
          urlbase="https://bugzilla.altlinux.org/"
          
          maintainer="jenya@basealt.ru"
>

    <bug>
          <bug_id>24412</bug_id>
          
          <creation_ts>2010-10-25 11:09:15 +0400</creation_ts>
          <short_desc>Содержимое пакета не помечено как %config</short_desc>
          <delta_ts>2010-10-25 18:19:46 +0400</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>4</classification_id>
          <classification>Development</classification>
          <product>Sisyphus</product>
          <component>apache2-mod_php5-control</component>
          <version>unstable</version>
          <rep_platform>all</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>CLOSED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>NMU, patch</keywords>
          <priority>P3</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>24413</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="solo">solo</reporter>
          <assigned_to name="solo">solo</assigned_to>
          <cc>boyarsh</cc>
    
    <cc>kurakin</cc>
    
    <cc>rider</cc>
          
          <qa_contact>qa-sisyphus</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>114218</commentid>
    <comment_count>0</comment_count>
    <who name="solo">solo</who>
    <bug_when>2010-10-25 11:09:15 +0400</bug_when>
    <thetext>Файлы предоставляемые пакетом apache2-mod_php5-control-5.3.3.20100722-alt2.2 (%_sysconfdir/control.d/facilities/* и %php5_sysconfdir/%php5_sapi/control.d/*) не помечены макросом %config. =&gt; при обновлении пакета будут переписаны _без_ создания резервной копии, даже если они привились руками.

  Возможность сохранения ручных настроек важна для адаптацию под задачу и для сохранения оной при обновлениях.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>114219</commentid>
    <comment_count>1</comment_count>
    <who name="Anton Farygin">rider</who>
    <bug_when>2010-10-25 11:18:04 +0400</bug_when>
    <thetext>Это и не есть конфигурационные файлы, и их не надо править под задачу.

Если стоит задача делать что-то из control&apos;а, то надо просто сделать ещё один файл (пакет) с другим поведением.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>114222</commentid>
    <comment_count>2</comment_count>
    <who name="solo">solo</who>
    <bug_when>2010-10-25 11:52:21 +0400</bug_when>
    <thetext>  Почему? Чем они отличаются по сути?

  Да, создание кастомизированного конфига под задачу -- вариант более предпочтительный (более дистрибутивный, как минимум). Но вынуждение адменистратора помимо foo (дистрибутивный конфиг под задачу, для zabbix, например) иметь и его личный my_foo -- вариант не лучший:

1. Невидно когда его надо синхронизировать с дистрибутивным. (Если отредактирован foo отмеченный как %config -- признаком необходимости синхронизации служит появление *.rpm*.)

2. Молчаливая замена в /etc файлов, изменённых вручную -- достаточно часто ломает рабочую конфигурации при обновлении. Такие разломы как минимум неприятны (а иногда и не очевидны).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>114223</commentid>
    <comment_count>3</comment_count>
    <who name="solo">solo</who>
    <bug_when>2010-10-25 11:54:24 +0400</bug_when>
    <thetext>  Необоснованную _молчаливую_ замену ручных правок в /etc -- считаю ошибкой.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>114225</commentid>
    <comment_count>4</comment_count>
    <who name="Anton Farygin">rider</who>
    <bug_when>2010-10-25 12:20:30 +0400</bug_when>
    <thetext>/etc содержит множество файлов, не являющихся конфигурационными (исполняемыми скриптами). Т.к. в control нет поддержки никаких других каталогов, кроме /etc, то это ошибкой не является. Впрочем, всё равно решать мейнтейнеру - меня наличие control&apos;а не интересует.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>114233</commentid>
    <comment_count>5</comment_count>
    <who name="solo">solo</who>
    <bug_when>2010-10-25 13:08:32 +0400</bug_when>
    <thetext>&gt; /etc содержит множество файлов, не являющихся конфигурационными (исполняемыми
&gt; скриптами). Т.к. в control нет поддержки никаких других каталогов, кроме /etc,
&gt; то это ошибкой не является.

  В данном случаи, файлы %php5_sysconfdir/%php5_sapi/control.d/* конечно скрипты. Но по суте своей исполняемые конфиги, т. к. содержат практически только список подлежащих установки параметров с их значениями (вызовы вида php_rule $mode &quot;&lt;имя параметра&gt;&quot; &lt;значение&gt;). Управляющая логика в данных файлах отсутствует.

  В спеке control (см http://sisyphus.ru/ru/srpm/Sisyphus/control/spec) присутствует определение:

%config %_sysconfdir/control.d

  Т. е. всё содержимое %_sysconfdir/control.d считается конфигами (и меня это неоднократно выручало) =&gt; и %_sysconfdir/control.d/facilities/* предоставляемые данным пакетом стоит отметить аналогично.


&gt; Впрочем, всё равно решать мейнтейнеру - меня
&gt; наличие control&apos;а не интересует.

  В spec`е мантейнер не указан. По ACL -- мантейнер пакета rider@ ;-)

$ ssh git.alt acl sisyphus apache2-mod_php5 show
apache2-mod_php5        rider @everybody

  По git`у, автор обсуждаемого пакета apache2-mod_php5-control -- Sergey Kurakin &lt;kurakin@...&gt;. Я подпасал его на данную багу.

PS: Я готовлю NMU с данными правками.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>114241</commentid>
    <comment_count>6</comment_count>
    <who name="solo">solo</who>
    <bug_when>2010-10-25 14:20:51 +0400</bug_when>
    <thetext>Предлагаемое исправление: http://git.altlinux.org/people/solo/packages/apache2-mod_php5.git?p=apache2-mod_php5.git;a=commit;h=688234db996e2fcb80a2ca48b573c37c557b1758</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>114257</commentid>
    <comment_count>7</comment_count>
    <who name="Repository Robot">repository-robot</who>
    <bug_when>2010-10-25 18:19:46 +0400</bug_when>
    <thetext>apache2-mod_php5-5.3.3.20100722-alt2.3 -&gt; sisyphus:

* Mon Oct 25 2010 Aleksey Avdeev &lt;solo@altlinux&gt; 5.3.3.20100722-alt2.3
- control(8) fix:
  + backup files (~|\.rpm(save|new)) are excluded from processing
    (Closes: #24413)
  + use %config for files subpackage %name-control (Closes: #24412)</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>