{"action":"created","issue":{"url":"https://api.github.com/repos/mapsme/omim/issues/294","labels_url":"https://api.github.com/repos/mapsme/omim/issues/294/labels{/name}","comments_url":"https://api.github.com/repos/mapsme/omim/issues/294/comments","events_url":"https://api.github.com/repos/mapsme/omim/issues/294/events","html_url":"https://github.com/mapsme/omim/pull/294","id":112221465,"number":294,"title":"[DO NOT MERGE][build] Rewrite build_omim to fix osrm-backend building","user":{"login":"Zverik","id":766031,"avatar_url":"https://avatars.githubusercontent.com/u/766031?v=3","gravatar_id":"","url":"https://api.github.com/users/Zverik","html_url":"https://github.com/Zverik","followers_url":"https://api.github.com/users/Zverik/followers","following_url":"https://api.github.com/users/Zverik/following{/other_user}","gists_url":"https://api.github.com/users/Zverik/gists{/gist_id}","starred_url":"https://api.github.com/users/Zverik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Zverik/subscriptions","organizations_url":"https://api.github.com/users/Zverik/orgs","repos_url":"https://api.github.com/users/Zverik/repos","events_url":"https://api.github.com/users/Zverik/events{/privacy}","received_events_url":"https://api.github.com/users/Zverik/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-10-19T20:02:19Z","updated_at":"2015-10-19T20:05:00Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/mapsme/omim/pulls/294","html_url":"https://github.com/mapsme/omim/pull/294","diff_url":"https://github.com/mapsme/omim/pull/294.diff","patch_url":"https://github.com/mapsme/omim/pull/294.patch"},"body":"Как в рассылке заметили, с недавних пор, когда заменили spec на linux-clang-libc++, у нас на линуксовых машинах перестал собираться osrm-backend. Этот PR, прежде всего, фиксит эту несправедливость.\r\n\r\nА теперь подробнее:\r\n\r\n* Убрал вывод даты в UTC при неудачном мёрдже костлайнов (откуда он там вообще).\r\n* Скрипт тестирования планеты не отлавливал сообщение Segmentation fault.\r\n\r\nСекунду, мы о том пул-реквесте? Конечно, я просто решил под шумок пронести эти два мелких фикса. Извините. Ладно, теперь к сборке.\r\n\r\n* Добавил в `omim.pro` таргет `osrm`. Он собирает библиотеки, которые требуют osrm-routed и osrm-mapsme.\r\n* Поправил `CMakeLists.txt` бэкенда, чтобы можно было переопределить пути к собранному omim. Также теперь дебаг и релиз можно собрать отдельно.\r\n* По умолчанию `build_omim.sh` собирает только maps.me в конфигурации release.\r\n* А под путём `TARGET` не создаются `omim-build-*`: там сразу out. Нечего мусорить каталогами. Правда, не знаю, кто вообще использовал `TARGET`.\r\n* Корректно определяем на маке число процессоров для сборки omim.\r\n* OSRM-Backend теперь по умолчанию собирается не в репозиторий, а наружу, в `osrm-backend-{debug,release}`. Либо в `TARGET`, либо в `OSRM_TARGET`.\r\n* Для osrm-backend собирается отдельная копия omim. То есть да, omim собирается дважды. Хотя во второй раз — самый минимум. Зато под линуксом теперь основная сборка идёт со спеком `linux-clang-libc++`, а сборка библиотек для osrm-backend — с `linux-clang`.\r\n* Если написать `build_omim.sh -rdo`, то соберутся два omim (как раньше) и два osrm-backend (а вот это сюрприз). Другое дело, что теперь это писать незачем."},"comment":{"url":"https://api.github.com/repos/mapsme/omim/issues/comments/149330939","html_url":"https://github.com/mapsme/omim/pull/294#issuecomment-149330939","issue_url":"https://api.github.com/repos/mapsme/omim/issues/294","id":149330939,"user":{"login":"mapsmetest","id":13031967,"avatar_url":"https://avatars.githubusercontent.com/u/13031967?v=3","gravatar_id":"","url":"https://api.github.com/users/mapsmetest","html_url":"https://github.com/mapsmetest","followers_url":"https://api.github.com/users/mapsmetest/followers","following_url":"https://api.github.com/users/mapsmetest/following{/other_user}","gists_url":"https://api.github.com/users/mapsmetest/gists{/gist_id}","starred_url":"https://api.github.com/users/mapsmetest/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mapsmetest/subscriptions","organizations_url":"https://api.github.com/users/mapsmetest/orgs","repos_url":"https://api.github.com/users/mapsmetest/repos","events_url":"https://api.github.com/users/mapsmetest/events{/privacy}","received_events_url":"https://api.github.com/users/mapsmetest/received_events","type":"User","site_admin":false},"created_at":"2015-10-19T20:05:00Z","updated_at":"2015-10-19T20:05:00Z","body":"Jenkins asks: can anyone have a look at this request?\nIf you want Jenkins to compile and test this, write \"test this please\" or JT_TP (without the underscore in the middle)."}}