search

imbamike666 / psi-dev

Automatically exported from code.google.com/p/psi-dev
0 stars 0 forks source link

Реализация сборки посредством cmake #504

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
Hi,

Here's the patch that solves compilation issue triggered when config.h is not 
(yet) generated.

Thanks.

Original issue reported on code.google.com by fer...@gmail.com on 5 Apr 2012 at 11:15

Attachments:

GoogleCodeExporter commented 8 years ago 
Ты можешь смело писать по-русски, вся 
команда рускоговорящая :)
по патчу - не соблюден кодстайл. Мы не 
используем отступы пробеллами. Только 
табы. Перед дерективами препроцессора 
отступов быть не должно.
Далее, вопрос - в каких таких случаях 
конфиг.h может быть еще не сгененрирован?

Original comment by wadea...@gmail.com on 5 Apr 2012 at 11:51

GoogleCodeExporter commented 8 years ago 
Просто привык писать багрепорты на англ. :)
Кодстайл исправил, пробелы заменил табами.

Патч нужен для той системы сборки (cmake), над 
которой я работаю. Его можно не применять 
прямо сейчас, хотя логически он исправляет 
некорректное поведение при 
отсутствии/наличии config.h. То есть, если уж 
введена директива HAVE_CONFIG, то она как 
минимум должна работать в случае, если не 
установлена.

Скорее всего для адекватной работы cmake 
будут еще патчи, поэтому потом их можно 
будет объединить по какому-нибудь 
признаку. Это просто первый.

Original comment by fer...@gmail.com on 5 Apr 2012 at 12:17

Attachments:

GoogleCodeExporter commented 8 years ago 
Понятно. Давай тогда пока я не буду его 
перемещать в репу, пусть висит сдесь. Когда 
все будет готово - сделаем 1 патч, 
позволяющий осуществлять сборку в cmake. В 
дальнейшем, можешь новые патчи цеплять к 
этой задаче

Original comment by wadea...@gmail.com on 5 Apr 2012 at 12:24

GoogleCodeExporter commented 8 years ago 
на самом деле подразумевается что будет 
использован скрипт конфигурации перед 
сборкой.
который и пропишет PSI_LIBDIR

Original comment by rion4ik on 5 Apr 2012 at 12:27

GoogleCodeExporter commented 8 years ago 
интересно, автор планирует довести до ума 
свою работу, или проблему можно закрывать?

Original comment by wadea...@gmail.com on 3 Mar 2013 at 12:23

GoogleCodeExporter commented 8 years ago 
Автор забросил эту затею, доведя сборку до 
этапа компоновки. Причина - ужасное 
состояние на тот момент (лето 2012) дерева 
исходников, с кучей закомметированного, 
дублирующего друг друга кода, странных 
зависимостей от qt3 (выключенных, но не 
удаленных), экспериментальных нерабочих 
целых кусков дерева и тп
после 15 разных исправлений и практически 
околонулевого понимания, как это все 
работает, решено не тратить время, пока 
текущая сборка не примет надлежащий вид.

Original comment by fer...@gmail.com on 22 Mar 2013 at 11:59

GoogleCodeExporter commented 8 years ago

Original comment by wadea...@gmail.com on 22 Mar 2013 at 1:54