mc-dev
mc-dev@conference.jabber.ru
[00:01:13] slavazanko/h вышел из конференции
[04:20:28] andrew_b зашёл в конференцию
[06:51:55] iliamaslakov зашёл в конференцию
[06:58:11] slavazanko/h зашёл в конференцию
[07:11:47] slavazanko/h вышел из конференции
[07:13:50] slavazanko/h зашёл в конференцию
[07:17:53] iliamaslakov вышел из конференции
[07:39:38] ASM вышел из конференции
[07:39:41] ASM зашёл в конференцию
[08:08:48] imho2 зашёл в конференцию
[08:34:53] slavazanko/h вышел из конференции
[09:11:01] iliamaslakov зашёл в конференцию
[09:14:52] <iliamaslakov> andrew_b: Кроме 2136.
почему?
[09:45:38] <andrew_b> Не хочу.
[10:05:46] iliamaslakov вышел из конференции
[10:08:25] <slavazanko> господа. а как вам такое предложение: если mc собирается с --enable-maintainer-mode, то при выполнении правила make install в корне проекта будет автоматически выполняться indent с нашими правилами (если indent установлен) и будет выполняться cppcheck (если установлен).
[10:10:18] <andrew_b> До сборки?
[10:10:27] <slavazanko> после
[10:10:41] <andrew_b> Не, после не интересно.
[10:11:19] <slavazanko> ну можно и до :) тогда на make all повесить можно хуку
[10:11:27] <andrew_b> Весь уимес в том, чтобы до.
[10:11:34] <andrew_b> цимес
[10:12:09] <slavazanko> просто cppcheck довольно долго отрабатывает по lib и по src
[10:12:47] <andrew_b> А...
[10:13:01] <andrew_b> А если в make check это делать? Отдельно.
[10:14:46] <slavazanko> цель check уже на тесты зарезервирована.
[10:15:32] <andrew_b> Если cppcheck долго отрабатывает, то его придётся ждать после каждой сборки.
[10:15:54] <andrew_b> Надо отдельную цель, что ли.
[10:15:59] <slavazanko> угу. А тесты я довольно часто запускаю
[10:16:23] <slavazanko> Да. пожалуй отдельная цель лучший вариант
[11:13:11] imho2 вышел из конференции
[12:22:16] <slavazanko> посмотрите предварительно 3051_cleanup.
Всё ли там нормально?
если всё. то я пройдусь по уборке варнингов cppcheck
[12:30:36] <andrew_b> $(CPPCHECK_CMD) --enable=all $(CPPCHECK_DIRS) 2>cppcheck-all.log.xml
[12:30:40] <andrew_b> Не.
[12:30:51] <slavazanko> ?
[12:31:12] <slavazanko> xml не нравится?
[12:31:26] <slavazanko> или all?
[12:31:54] <andrew_b> Надо 2>${BUILD_DIR}/cppcheck-all.log.xml
Или как там каталог сборки называется.
[12:32:09] <slavazanko> а. Да, справедливо
[12:32:14] <slavazanko> ща
[12:36:16] <andrew_b> +Before any commit, please run at the source root the command:
+make indent
Я бы сказал, что это написано по-русски английскими словами.
Please run the following command:
make indent
in the build root directory before each commit.
[12:36:28] <slavazanko> понял
[12:36:44] <andrew_b> Или every commit.
[12:36:57] <andrew_b> Я путаю each и every.
[12:38:08] <slavazanko> each - каждый из существующих
every - все из коммитов. не обязательно существующие
[12:39:38] <andrew_b> И это, давайте ошибки группами по коммитам.
Неинициализированные переменные -- один коммит.
Неиспользуемые -- другой.
Уменьшение области видимости -- третий.
Неизвестные дефайны из-за забытых инклюдов -- четвётртый.
И т. д.
[12:40:07] <slavazanko> да. так я и планирую
[12:40:18] <slavazanko> распараллелиться получится?
[12:40:45] <andrew_b> Думаю, да.
[12:41:23] <slavazanko> гуд. тогда расхватываем задачи.
[12:41:42] <andrew_b> Мне любую.
[12:41:53] <andrew_b> Область видимости -- самая большая.
[12:42:09] <andrew_b> Остальные мелкие.
[12:42:38] <slavazanko> ты - information
Я - style
Илья - остальные
[12:43:09] <slavazanko> конфликты разгребает последний закоммитившейся
[12:43:21] <andrew_b> На Илью надежды нет.
[12:43:56] <slavazanko> гм. тогда пока мы по information и по style пройдёмся. А там посмотрим, кто быстрее. тот остальные возьмёт
[12:47:25] <andrew_b> smbfs я бы не трогал. Пусть уж совсем протухает.
[12:47:55] <andrew_b> Лишние конфликты при рибейзе 1_smbfs.
[12:48:35] <slavazanko> нуу... ок. пусть будет нетронутой
[12:49:07] <slavazanko> пуллиться без конфликтов: git pull --rebase
[12:55:37] <andrew_b> Ну и вообще, xml неудобно глазами парсить. :)
[12:56:07] <andrew_b> Лучше так: http://code.haskell.org/~slyfox/cppc_mc.log
[12:57:25] <slavazanko> поменяй вывод :) Та в двух местах поменять. CPPCHECK_CMD и CPPCHECK_OUT_EXT
только запулься предвартельно
[13:11:51] <andrew_b> slavazanko:
mountlist.c:401]: (information) Unmatched suppression: syntaxError
И таких штук 20 в этом файле.
[13:11:57] <andrew_b> Твой коммит.
[13:12:20] <slavazanko> гм. У меня всё гуд
[13:12:45] <andrew_b> make cppcheck-information
[13:12:45] <slavazanko> cppcheck почему-то шизеет от того, что перед case идёт #define
[13:13:02] <andrew_b> $ cppcheck --version
Cppcheck 1.60.1
[13:13:13] <slavazanko> cppcheck --version
Cppcheck 1.61
[13:13:22] <slavazanko> блин
[13:24:17] andrew_b вышел из конференции
[16:05:08] andrew_b зашёл в конференцию
[16:09:40] slavazanko/h зашёл в конференцию
[16:54:18] monkey зашёл в конференцию
[16:55:39] iliamaslakov зашёл в конференцию
[17:06:40] slavazanko/h вышел из конференции
[17:07:12] slavazanko/h зашёл в конференцию
[17:21:06] Yury V. Zaytsev зашёл в конференцию
[17:21:22] <Yury V. Zaytsev> если хотите, я могу на ci-сервер cppcheck поставить
[17:22:05] <Yury V. Zaytsev> slavazanko: напиши мне тогда, если надо. и можно в job где clang, но он раз в неделю, либо после каждого коммита в мастер
[17:24:55] <andrew_b> Смысл в том, чтобы результат cppcheck был виден сразу, локально то бишь, до мержа в мастер.
[17:30:08] <Yury V. Zaytsev> andrew_b: а в чем проблема, можно джоб сделать который вообще все ветки подряд собирает
[17:30:40] <Yury V. Zaytsev> andrew_b: или по конкретному паттерну. типа все ветки на check...
[17:31:20] <andrew_b> У меня, например, куча веток в другом репо, чтобы основной и таймлайн не засорять.
[17:31:44] <Yury V. Zaytsev> andrew_b: ну можно сделать джобу с другим репо
[17:32:21] <Yury V. Zaytsev> andrew_b: или джобу где через параметр можно указывать репо. не уверен работают ли параметры в этом поле, правда.
[17:32:40] <andrew_b> Пока,как видно, есть некоторая несовместимость версий cppcheck.
[17:32:59] <Yury V. Zaytsev> andrew_b: я так понял вы стейбл упразднили же, можно тогда этот джоб грохнуть а создать другой. мне не жалко создать больше джобов, главное чтобы скрипты внутри вы сами писали, без меня, только никто ничего не говорит =)
[17:33:18] <Yury V. Zaytsev> andrew_b: ладно, мне надо домой идти, надумаете, пишите, ну и чат я читаю время от времени
[17:33:22] <andrew_b> Unmatched suppression: syntaxError у меня в 1.60.1
У Славика 1.61, этого сообщения нет.
[17:33:40] <Yury V. Zaytsev> у меня в fc18 1.61 будет
[17:34:00] Yury V. Zaytsev вышел из конференции
[17:34:23] <andrew_b> Завтра обсудим.
[17:39:41] iliamaslakov вышел из конференции
[17:40:06] iliamaslakov зашёл в конференцию
[18:16:32] <repolainen> а я злостно и git cppcheck ставлю
[18:16:45] <repolainen> из*
[18:16:47] <slavazanko/h> это что?
[18:18:22] andrew_b вышел из конференции
[18:21:12] <repolainen> [sf] ~:cppcheck --version
Cppcheck 1.62 dev
[20:04:23] monkey вышел из конференции
[20:52:38] ASM вышел из конференции
[20:52:40] ASM зашёл в конференцию
[23:22:36] slavazanko/h вышел из конференции
