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

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

    <bug>
          <bug_id>33074</bug_id>
          
          <creation_ts>2017-02-03 10:25:19 +0300</creation_ts>
          <short_desc>После обновления до 0.5.15lorg2-alt56 LUA-скриптам не передаётся переменная script_slot</short_desc>
          <delta_ts>2017-07-05 17:59:46 +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>apt</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>
          
          <blocked>32941</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Chess">slchess</reporter>
          <assigned_to name="placeholder@altlinux.org">placeholder</assigned_to>
          <cc>boyarsh</cc>
    
    <cc>evg</cc>
    
    <cc>glebfm</cc>
    
    <cc>imz</cc>
    
    <cc>ldv</cc>
    
    <cc>placeholder</cc>
    
    <cc>vseleznv</cc>
    
    <cc>vt</cc>
          
          <qa_contact>qa-sisyphus</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>161698</commentid>
    <comment_count>0</comment_count>
    <who name="Chess">slchess</who>
    <bug_when>2017-02-03 10:25:19 +0300</bug_when>
    <thetext>Последнее сообщение в /var/log/apt.log

Upgrading apt 0.5.15lorg2-alt55 to 0.5.15lorg2-alt56</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>161700</commentid>
    <comment_count>1</comment_count>
    <who name="Evgenii Terechkov">evg</who>
    <bug_when>2017-02-03 11:22:41 +0300</bug_when>
    <thetext>Знаю. Похоже, проблема в том, что после rpm-4.3 не вызываются нужные хуки apt-а. Пока расследую этот вопрос...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>162269</commentid>
    <comment_count>2</comment_count>
    <who name="Evgenii Terechkov">evg</who>
    <bug_when>2017-03-05 14:21:20 +0300</bug_when>
    <thetext>Расследование показывает, что apt-log перестал работать после обновления alt55-&gt;alt56, потому что не знает, в каком режиме это делать (начало/окончание транзакции). А это определяется по значению переменной script_slot, которая теперь всегда не определена (nil).

Получается, добавление поддержки lua5.3 сломало передачу в скрипты script_slot (и других переменных?). Я проверил на парочке других скриптов из apt/contrib/ , использующих script_slot для показа встроенной помощи - эффект тот же, значение не определено.

Проверить очень просто: добавить
=8&lt;==========================================
print(script_slot)
return
=8&lt;==========================================
в начало lua-скрипта.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>164562</commentid>
    <comment_count>3</comment_count>
    <who name="Vladimir D. Seleznev">vseleznv</who>
    <bug_when>2017-07-04 19:04:36 +0300</bug_when>
    <thetext>Исправлено в
http://git.altlinux.org/tasks/archive/done/_180/185002/logs/events.2.2.log</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>164574</commentid>
    <comment_count>4</comment_count>
    <who name="Evgenii Terechkov">evg</who>
    <bug_when>2017-07-05 16:25:03 +0300</bug_when>
    <thetext>Стало лучше, но всё равно пока плохо:

=8&lt;==========================================================
root@thinkpad ~ #pkin xorg-conf-synaptics 
Чтение списков пакетов... Завершено
Построение дерева зависимостей... Завершено
Starting
Starting 2
Done
Следующие пакеты будут ОБНОВЛЕНЫ:
  xorg-conf-synaptics
1 будет обновлено, 0 новых установлено, 0 пакетов будет удалено и 1 не будет обновлено.
Необходимо получить 3105B архивов.
После распаковки будет освобождено 1B дискового пространства.
Получено: 1 http://ftp.altlinux.org noarch/classic xorg-conf-synaptics 1.0-alt1 [3105B]
Получено 3105B за 0s (12,9kB/s).     

Совершаем изменения...
Подготовка...                                                                       #################################################################################################### [100%]
Обновление / установка...
1: xorg-conf-synaptics-1.0-alt1                                                     #################################################################################################### [ 50%]
Очистка / удаление... 
2: xorg-conf-synaptics-0.3-alt1                                                     предупреждение: /etc/X11/xorg.conf.d/20-synaptics.conf сохранен как /etc/X11/xorg.conf.d/20-synaptics.conf.rpmsave
#################################################################################################### [100%]
Завершено.
W: Error running script: /usr/share/apt/scripts/log.lua:63: attempt to call a nil value (field &apos;foreach&apos;)
root@thinkpad ~ #tail /var/log/apt.log   
Transaction succeeded at Ср 05 июл 2017 08:13:08
Transaction starting at Ср 05 июл 2017 08:12:56
Transaction succeeded at Ср 05 июл 2017 08:18:11
Transaction starting at Ср 05 июл 2017 08:18:11
=8&lt;==========================================================

т.е. script_slot выдаётся скрипту не в том порядке, сначала Scripts::PM::Post, потом Scripts::PM::Pre, хотя должно быть наоборот.
И получается результат работы функции pkglist - пуст, в результате ничего не протоколируется.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>164576</commentid>
    <comment_count>5</comment_count>
    <who name="Gleb F-Malinovskiy">glebfm</who>
    <bug_when>2017-07-05 17:19:23 +0300</bug_when>
    <thetext>*** Bug 32941 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>164578</commentid>
    <comment_count>6</comment_count>
    <who name="Gleb F-Malinovskiy">glebfm</who>
    <bug_when>2017-07-05 17:37:24 +0300</bug_when>
    <thetext>(In reply to comment #4)
&gt; Стало лучше, но всё равно пока плохо:
...
&gt; W: Error running script: /usr/share/apt/scripts/log.lua:63: attempt to call a
&gt; nil value (field &apos;foreach&apos;)

Всё дело вот в этом месте.  Оно сделало file:write, а file:close не сделало, вот они и поменялись местами.

&gt; root@thinkpad ~ #tail /var/log/apt.log   
&gt; Transaction succeeded at Ср 05 июл 2017 08:13:08
&gt; Transaction starting at Ср 05 июл 2017 08:12:56
&gt; Transaction succeeded at Ср 05 июл 2017 08:18:11
&gt; Transaction starting at Ср 05 июл 2017 08:18:11
&gt; =8&lt;==========================================================
&gt; 
&gt; т.е. script_slot выдаётся скрипту не в том порядке, сначала Scripts::PM::Post,
&gt; потом Scripts::PM::Pre, хотя должно быть наоборот.
&gt; И получается результат работы функции pkglist - пуст, в результате ничего не
&gt; протоколируется.

script_slot точно правильно выдаётся, просто код на lua слишком старый.
Я не силён в lua, но вот так работает:

diff --git a/apt/contrib/log/log.lua b/apt/contrib/log/log.lua
index ab7bef3..a2c296d 100644
--- a/apt/contrib/log/log.lua
+++ b/apt/contrib/log/log.lua
@@ -57,14 +57,16 @@ if script_slot == &quot;Scripts::PM::Pre&quot; then
         end
     end
     file:write(&quot;Transaction starting at &quot;, os.date(), &quot;\n&quot;)
-    local function write(index, str)
-        file:write(str)
+    local function write(list)
+        for _,n in ipairs(list) do
+            file:write(n)
+        end
     end
-    table.foreach(removing, write)
-    table.foreach(installing, write)
-    table.foreach(reinstalling, write)
-    table.foreach(downgrading, write)
-    table.foreach(upgrading, write)
+    write(removing)
+    write(installing)
+    write(reinstalling)
+    write(downgrading)
+    write(upgrading)
     file:close()
 elseif script_slot == &quot;Scripts::PM::Post&quot; then
     if transaction_success then</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>164579</commentid>
    <comment_count>7</comment_count>
    <who name="Evgenii Terechkov">evg</who>
    <bug_when>2017-07-05 17:59:46 +0300</bug_when>
    <thetext>Огромное спасибо, с этим патчем действительно работает.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>