Bug 25946

Summary: libshell/shell-ini-config: проблемы с ini_config_set [patch]
Product: Sisyphus Reporter: Vladislav Zavjalov <slazav>
Component: libshellAssignee: Alexey Gladkov <legion>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: normal    
Priority: P3 CC: legion
Version: unstable   
Hardware: all   
OS: Linux   

Description Vladislav Zavjalov 2011-07-23 15:05:58 MSK
1. Значения  могут добавляться в неправильные секции.

Тестовый пример:
   
. shell-ini-config
printf '[aaa]\n[bbb]\n' > test.cfg
ini_config_set test.cfg section key val
cat test.cfg
    
Результат:
    
[aaa]
    key = val
[bbb]

2. Значения не добавляются в последнюю секцию

Тестовый пример:
   
. shell-ini-config
printf '[section]\n' > test.cfg
ini_config_set test.cfg section key val
cat test.cfg
    
Результат:

[section]

Разлом произошел в 0.1.5-alt1
Исправления выложил к себе в git (два последних коммита).
Comment 1 Vladislav Zavjalov 2011-07-23 19:28:46 MSK
Добавил еще два коммита:

1. Добавление в ini_config_set новой секции, если запрошенного названия не существует. Мне кажется, что это имеет смысл.

2. Исправление форматирования комметариев (добавление tab'а), которое я случайно поломал.
Comment 2 Alexey Gladkov 2011-07-27 01:29:38 MSK
Спасибо. Всё выглядит логично.
Comment 3 Repository Robot 2011-07-27 02:32:06 MSK
libshell-0.1.6-alt1 -> sisyphus:

* Wed Jul 27 2011 Alexey Gladkov <legion@altlinux> 0.1.6-alt1
- shell-ini-config changes (ALT#25946):
  + Fix comment formatting (thx Vladislav Zavjalov);
  + Add section if it is not exists (thx Vladislav Zavjalov);
  + Fix error at adding values to the last section (thx Vladislav Zavjalov);
  + Fix error of adding values into incorrect sections (thx Vladislav Zavjalov).