Перенастроить нашего buildbot на использование CMake

den-run-ai / mipt-course

Automatically exported from code.google.com/p/mipt-course

BSD 3-Clause "New" or "Revised" License

0 stars 0 forks source link

Перенастроить нашего buildbot на использование CMake #19

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

http://code.google.com/p/mipt-course/wiki/BuildbotSetup
+
см. issue 1 - добавление CMake

После этого надо будет удалить "старый" 
Makefile.

Ставлю высокий приоритет, т.к. до 
завершения миграции Makefile -> CMake придётся 
поддерживать оба способа сборки, что есть 
нехорошо.  (Александр, beware!)

Original issue reported on code.google.com by timurrrr on 22 Oct 2011 at 8:54

GoogleCodeExporter commented 9 years ago

Обещался взяться Ростислав

Original comment by timurrrr on 29 Oct 2011 at 10:03

GoogleCodeExporter commented 9 years ago

Если можно, включите меня в СС. Я хотел бы 
непосредственное участив этом issue и в 
настройке buildbot принимать (в смысле +email и 
+queue)

Original comment by djsa...@gmail.com on 30 Oct 2011 at 7:26

GoogleCodeExporter commented 9 years ago

Собственно, давно уже работаю над этим issue,
на днях планирую завершить работу.

Original comment by Rostisla...@gmail.com on 31 Oct 2011 at 5:38

GoogleCodeExporter commented 9 years ago

Original comment by timurrrr on 1 Nov 2011 at 6:31

Changed state: Started

GoogleCodeExporter commented 9 years ago

Пока выдает ошибку на этапе cd

То что я сделал - это попытался "перенести" 
код отсюда
http://code.google.com/p/mipt-course/wiki/Building
в master.cfg

Original comment by Rostisla...@gmail.com on 4 Nov 2011 at 12:37

Attachments:

GoogleCodeExporter commented 9 years ago

вместо cd используй параметр workdir
http://buildbot.net/buildbot/docs/0.8.4/Using-ShellCommands.html#Using-ShellComm
ands

Original comment by timurrrr on 4 Nov 2011 at 1:33

GoogleCodeExporter commented 9 years ago

Спасибо.
Так - работает.
Что дальше?

Original comment by Rostisla...@gmail.com on 4 Nov 2011 at 3:17

Attachments:

master.cfg

GoogleCodeExporter commented 9 years ago

http://code.google.com/p/mipt-course/wiki/CodeReview

Original comment by timurrrr on 4 Nov 2011 at 4:02

GoogleCodeExporter commented 9 years ago

Ping?
Я жду патча на codereview.

Original comment by timurrrr on 8 Nov 2011 at 11:08

GoogleCodeExporter commented 9 years ago

Ping

Original comment by timurrrr on 14 Nov 2011 at 7:06

GoogleCodeExporter commented 9 years ago

http://codereview.appspot.com/5410042

Original comment by Rostisla...@gmail.com on 17 Nov 2011 at 4:46

GoogleCodeExporter commented 9 years ago

rietveld сейчас на maintenance, поэтому там комменты 
написать не получается.

Напишу здесь, чтобы время не терялось:
а) сделай svn up, у тебя старая версия (см. дифф 
- он удаляет половину нового кода)
б) сделай Configure вместо ShellCommand; у них немного 
разный смысл и default логика обработки ошибок
в) не трогай отступ USE_VALGRIND - Valgrind надо 
запускать только на Debug билде!
г) локально проверял?
д) почему поменял forceBuild в диффе?

Original comment by timurrrr on 17 Nov 2011 at 5:37

GoogleCodeExporter commented 9 years ago

е) не добавляй отступы перед )) у Compile -> там 
специально ) под (

Original comment by timurrrr on 17 Nov 2011 at 5:37

GoogleCodeExporter commented 9 years ago

Вас понял.
г) локально - проверял

Original comment by Rostisla...@gmail.com on 17 Nov 2011 at 5:45

GoogleCodeExporter commented 9 years ago

http://codereview.appspot.com/5414049

Original comment by Rostisla...@gmail.com on 18 Nov 2011 at 6:45

GoogleCodeExporter commented 9 years ago

Посылай е-мейл с сайта кодревью (Publish + Mail 
comments), а не ссылкой здесь.
Еще - лучше использовать "старый" review, чем 
создавать каждый раз новый. (флаг -i, см wiki 
CodeReview)

Original comment by timurrrr on 18 Nov 2011 at 8:48

GoogleCodeExporter commented 9 years ago

This issue was closed by revision r298.

Original comment by timurrrr on 2 Dec 2011 at 4:43

Changed state: Fixed