Bug 33799 - lxc 2.0.8 не собирается lcc из-за опечатки
Summary: lxc 2.0.8 не собирается lcc из-за опечатки
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: lxc (show other bugs)
Version: unstable
Hardware: e2k Linux
: P3 minor
Assignee: Alexey Shabalin
QA Contact: qa-sisyphus
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2017-08-23 14:02 MSK by Michael Shigorin
Modified: 2017-08-29 17:13 MSK (History)
2 users (show)

See Also:

Attachments
0001-conf-non-functional-fixup.patch (1.33 KB, patch)
2017-08-23 14:02 MSK, Michael Shigorin 		no flags Details | Diff
View All 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 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