Bug 30713 - ini-config-set корёжит /etc/krb5.conf
Summary: ini-config-set корёжит /etc/krb5.conf
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: libshell (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: Alexey Gladkov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-02-09 12:18 MSK by Andrey Cherepanov
Modified: 2015-02-24 15:56 MSK (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Andrey Cherepanov 2015-02-09 12:18:13 MSK 
Имеем /etc/krb5.conf следующего содержимого:
[libdefaults]
dns_lookup_kdc = false
dns_lookup_realm = false
forwardable = true
[realms]
SCHOOL.ALT = {
kdc = SCHOOL.ALT
}

. shell-ini-config
ini_config_set /etc/krb5.conf libdefaults default_realm SCHOOL.ALT
[root@host-15 backend3]# cat /etc/krb5.conf
[libdefaults]
        dns_lookup_kdc = false
        dns_lookup_realm = false
        forwardable = true
        default_realm = SCHOOL.ALT
[realms]
        SCHOOL.ALT = {
        kdc = SCHOOL.ALT
        } = }

Последняя строка повреждена.
Comment 1 Michael Shigorin 2015-02-10 10:37:34 MSK 
Это не совсем ini, очевидно.
Comment 2 Alexey Gladkov 2015-02-10 12:41:18 MSK 
Миш, спасибо что напомнил про этот баг. Да, очевидно не совсем валидный ini, но думаю будет неплохо обращать внимание на строки без значений.

Проверьте 0.2.0-alt1-14-ge1b89ed пожалуйста.
Comment 3 Andrey Cherepanov 2015-02-13 18:00:43 MSK 
(В ответ на комментарий №2)
> Миш, спасибо что напомнил про этот баг. Да, очевидно не совсем валидный ini, но
> думаю будет неплохо обращать внимание на строки без значений.
> 
> Проверьте 0.2.0-alt1-14-ge1b89ed пожалуйста.
Работает, спасибо.
Comment 4 Repository Robot 2015-02-24 15:56:13 MSK 
libshell-0.3.0-alt1 -> sisyphus:

* Tue Feb 24 2015 Alexey Gladkov <legion@altlinux> 0.3.0-alt1
- New version (0.3.0).
- Fix bootstrap (ALT#29584).
- shell-ini-config changes:
  + Add ini_config_is_set() function.
  + Take care about lines without values (ALT#30713).
- shell-unittest changes:
  + Add TESTCASES variable to list individual testcases (ALT#27059).
  + Add TESTTRACE variable to run testcase in debug mode (ALT#27059).