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

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

    <bug>
          <bug_id>30929</bug_id>
          
          <creation_ts>2015-04-08 18:05:11 +0300</creation_ts>
          <short_desc>Остался старый код после мерджинга изменений upstream</short_desc>
          <delta_ts>2015-05-18 08:31:55 +0300</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>4</classification_id>
          <classification>Development</classification>
          <product>Sisyphus</product>
          <component>python-module-salt</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></keywords>
          <priority>P3</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Владимир Диденко">vladimir.didenko</reporter>
          <assigned_to name="Valentin Rosavitskiy">valintinr</assigned_to>
          <cc>real.altlinux.org</cc>
          
          <qa_contact>qa-sisyphus</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>151101</commentid>
    <comment_count>0</comment_count>
    <who name="Владимир Диденко">vladimir.didenko</who>
    <bug_when>2015-04-08 18:05:11 +0300</bug_when>
    <thetext>После мерджинга изменений upstream для 2015.2, осталась часть изменений от 2014.7. Т.е. пакет собирается не из чистых исходников, а из 2015.2 + ошметки от 2014.7. Убедиться в этом достаточно просто. 

1. Клонируем salt

   git clone http://git.altlinux.org/gears/s/salt.git

2. Переходим на ветку sisyphus

   git checkout sisyphus

3. Смотрим diff 

   git diff v2015.2 sisyphus

4. Видим ошметки от старых версий. Например в файле salt/runners/state.py есть
   дублирующий import

   from __future__ import print_function 

   который приводит к ошибке во время запуска salt-master. 

Я для себя локально все не alt-специфичные файлы потер, оставив только одну правку в salt/modules/ssh.py, в которой правился путь к ssh ключам (/etc/ssh заменялся на /etc/openssl). Не стал разбираться это осталось с прошлых версий или нет. 

Вобщем, мои предложения оформлены в виде 2015.2-alt3

http://git.altlinux.org/people/cow/packages/salt.git

Если устроят, то могу отправить на сборку.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>151213</commentid>
    <comment_count>1</comment_count>
    <who name="Valentin Rosavitskiy">valintinr</who>
    <bug_when>2015-04-18 11:19:13 +0300</bug_when>
    <thetext>(В ответ на комментарий №0)
&gt; После мерджинга изменений upstream для 2015.2, осталась часть изменений от
&gt; 2014.7. Т.е. пакет собирается не из чистых исходников, а из 2015.2 + ошметки от
&gt; 2014.7. Убедиться в этом достаточно просто. 
&gt; 
&gt; 1. Клонируем salt
&gt; 
&gt;    git clone http://git.altlinux.org/gears/s/salt.git
&gt; 
&gt; 2. Переходим на ветку sisyphus
&gt; 
&gt;    git checkout sisyphus
&gt; 
&gt; 3. Смотрим diff 
&gt; 
&gt;    git diff v2015.2 sisyphus
&gt; 
&gt; 4. Видим ошметки от старых версий. Например в файле salt/runners/state.py есть
&gt;    дублирующий import
&gt; 
&gt;    from __future__ import print_function 
&gt; 
&gt;    который приводит к ошибке во время запуска salt-master. 
&gt; 
&gt; Я для себя локально все не alt-специфичные файлы потер, оставив только одну
&gt; правку в salt/modules/ssh.py, в которой правился путь к ssh ключам (/etc/ssh
&gt; заменялся на /etc/openssl). Не стал разбираться это осталось с прошлых версий
&gt; или нет. 
&gt; 
&gt; Вобщем, мои предложения оформлены в виде 2015.2-alt3
&gt; 
&gt; http://git.altlinux.org/people/cow/packages/salt.git
&gt; 
&gt; Если устроят, то могу отправить на сборку.
Да, есть такое, самому интересно как оно так получилось.
Отправил на сборку новую версию с исправлениями, сверялся с чистим апстримом.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>151214</commentid>
    <comment_count>2</comment_count>
    <who name="Repository Robot">repository-robot</who>
    <bug_when>2015-04-18 11:25:14 +0300</bug_when>
    <thetext>salt-2015.2-alt3 -&gt; sisyphus:

* Sat Apr 18 2015 Valentin Rosavitskiy &lt;valintinr@altlinux&gt; 2015.2-alt3
- Removed code from previous tag (ALT 30929)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>151216</commentid>
    <comment_count>3</comment_count>
    <who name="Владимир Диденко">vladimir.didenko</who>
    <bug_when>2015-04-18 11:54:18 +0300</bug_when>
    <thetext>(В ответ на комментарий №1)

&gt; Да, есть такое, самому интересно как оно так получилось.

Обычное дело, когда upstream в stable ветку коммитит изменения, которых нет в последующих ветках. Я на такие же грабли при обновлении mongo наступил. Возможно, корифеи знают, как такие ситуации получше разруливать - надо будет спросить. 

&gt; Отправил на сборку новую версию с исправлениями, сверялся с чистим апстримом.

Спасибо!</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>