Bug 33799

Summary: lxc 2.0.8 не собирается lcc из-за опечатки
Product: Sisyphus Reporter: Michael Shigorin <mike>
Component: lxcAssignee: Alexey Shabalin <shaba>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: minor    
Priority: P3 CC: dans, shaba
Version: unstableKeywords: patch
Hardware: e2k   
OS: Linux   
Attachments:
Description Flags
0001-conf-non-functional-fixup.patch none

Description Michael Shigorin 2017-08-23 14:02:55 MSK
Created attachment 7194 [details]
0001-conf-non-functional-fixup.patch

При сборке lxc-2.0.8-alt1 на e2k наблюдаю следующее:

lcc: "conf.c", line 1514: error: unrecognized character escape sequence
          [-Werror]
        DEBUG("created directory for console and tty devices at \%s\"", path);
                                                                ^
 in expansion of macro "DEBUG" at line 1514

Убунтушный ляп, оставшийся незамеченным при -Wall -Werror, однако.

Отправил патч автору ляпа, в спеке сделал пока так:

+# E2K: lcc: "conf.c", line 1514: error: unrecognized character escape sequence
+#        DEBUG("created directory for console and tty devices at \%s\"", path);
+#                                                                ^
+sed -i 's,\\%s,\\"%s,' src/lxc/conf.c
Comment 1 Denis Pynkin 2017-08-24 18:10:02 MSK
А ты не хочешь сделать pull-request в апстрим: https://github.com/lxc/lxc ?
Comment 2 Denis Pynkin 2017-08-24 18:19:36 MSK
(В ответ на комментарий №1)
> А ты не хочешь сделать pull-request в апстрим: https://github.com/lxc/lxc ?

A! прочитал сообщение от Christian Brauner -- про https://github.com/lxc/lxc/pull/1757/commits/4742cd9a30eadcf3ef637e99e7a357b91f58b41d

Подожду пару дней, чтобы апстрим забрал, потом сделаю cherry-pick, чтобы не плодить сущности.
Comment 3 Michael Shigorin 2017-08-25 10:37:00 MSK
(В ответ на комментарий №2)
> Подожду пару дней, чтобы апстрим забрал, потом сделаю cherry-pick,
> чтобы не плодить сущности.

Да, конечно.
Comment 4 Michael Shigorin 2017-08-29 17:13:23 MSK
* Mon Aug 28 2017 Denis Pynkin <dans@altlinux> 2.0.8-alt2
- Fixes #33814
- Fixes #33799