Release = optimize + pdb
От: para  
Дата: 26.02.10 10:42
Оценка:
В продолжение:
http://www.rsdn.ru/forum/nemerle/3716871.1.aspx
Автор: para
Дата: 25.02.10

исправил баг.

следующий код добавил в
http://code.google.com/p/nemerle/source/browse/nemerle/trunk/ncc/optimization/CFG.n#119

        match(expr)
            ...                
            | DebugInfo =>      //patch for combinig
                null            //-optimize and -debug+ options
                
            | _ =>
                assert(false)

поверил — компилируются обе фазы.
нужная функциональность появилась.

перед тем, как залить изменения хотел удостоверится, правильно ли я делаю...

комитить каталог boot надо после build.cmd?
Re: Release = optimize + pdb
От: para  
Дата: 26.02.10 10:45
Оценка: 3 (1) +1
Также сделал поддержку в интеграции:

http://files.rsdn.org/89317/screen.JPG

готов залить в свн
Re[2]: Release = optimize + pdb
От: SergASh  
Дата: 26.02.10 11:18
Оценка:
запятые из последней фразы лучше убрать
Re[3]: Release = optimize + pdb
От: hardcase Пират http://nemerle.org
Дата: 26.02.10 11:45
Оценка:
Здравствуйте, SergASh, Вы писали:

SAS>запятые из последней фразы лучше убрать


+1 это какая-то смесь английской орфографии и русской пунктуации.

p>комитить каталог boot надо после build.cmd?


Да, boot комиттить после build.cmd.
http://nemerle.org/Banners/?t=Developer!&g=dark /* иЗвиНите зА неРовнЫй поЧерК */
Re: Release = optimize + pdb
От: Иванков Дмитрий Россия  
Дата: 26.02.10 13:00
Оценка:
Здравствуйте, para, Вы писали:

P>комитить каталог boot надо после build.cmd?


А зачем коммитить boot?
Re[2]: Release = optimize + pdb
От: para  
Дата: 26.02.10 13:24
Оценка:
Здравствуйте, Иванков Дмитрий, Вы писали:

ИД>Здравствуйте, para, Вы писали:

P>>комитить каталог boot надо после build.cmd?
ИД>А зачем коммитить boot?

изменение компилятора
http://code.google.com/p/nemerle/source/browse/nemerle/trunk/ncc/optimization/CFG.n
Re[3]: Release = optimize + pdb
От: Иванков Дмитрий Россия  
Дата: 26.02.10 14:09
Оценка: +1
Здравствуйте, para, Вы писали:

P>Здравствуйте, Иванков Дмитрий, Вы писали:


ИД>>А зачем коммитить boot?


P>изменение компилятора

P>http://code.google.com/p/nemerle/source/browse/nemerle/trunk/ncc/optimization/CFG.n

Почему это изменение компилятора надо сопровождать изменением boot?

— в boot лежат бинарники, мне кажется, что без необходимости их не надо коммитить
— boot используется только на первом круге сборки компилятора, на втором используется результат первого, который успешно соберёт и сам компилятор по-новому

Неизбежно boot надо коммитить, если со старым boot не выйдет собрать готовящееся к коммиту изменение. Как правило, это использование новых макросов в самом ncc или изменение внутренних типов компилятора.
Re[4]: Release = optimize + pdb
От: para  
Дата: 26.02.10 14:18
Оценка:
Здравствуйте, Иванков Дмитрий, Вы писали:

ИД>Здравствуйте, para, Вы писали:


P>>Здравствуйте, Иванков Дмитрий, Вы писали:


ИД>>>А зачем коммитить boot?


P>>изменение компилятора

P>>http://code.google.com/p/nemerle/source/browse/nemerle/trunk/ncc/optimization/CFG.n

ИД>Почему это изменение компилятора надо сопровождать изменением boot?


ИД>- в boot лежат бинарники, мне кажется, что без необходимости их не надо коммитить

ИД>- boot используется только на первом круге сборки компилятора, на втором используется результат первого, который успешно соберёт и сам компилятор по-новому

ИД>Неизбежно boot надо коммитить, если со старым boot не выйдет собрать готовящееся к коммиту изменение. Как правило, это использование новых макросов в самом ncc или изменение внутренних типов компилятора.


согласен
Re[2]: Release = optimize + pdb
От: para  
Дата: 26.02.10 14:37
Оценка:
Какое сделать значение по умолчанию?
1. Как сейчас
     Debug   -> true
     Release -> false

2. Как в шарпе
     Debug   -> true
     Release -> true
Re: Release = optimize + pdb
От: VladD2 Российская Империя www.nemerle.org
Дата: 26.02.10 16:32
Оценка:
Здравствуйте, para, Вы писали:

P>следующий код добавил в

P>http://code.google.com/p/nemerle/source/browse/nemerle/trunk/ncc/optimization/CFG.n#119

P>
P>        match(expr)
P>            ...                
P>            | DebugInfo =>      //patch for combinig
P>                null            //-optimize and -debug+ options
                
P>            | _ =>
P>                assert(false)
P>

P>поверил — компилируются обе фазы.
P>нужная функциональность появилась.

P>перед тем, как залить изменения хотел удостоверится, правильно ли я делаю...


Вроде бы все ОК. Как я понимаю null означает использовать старое выражение.

P>комитить каталог boot надо после build.cmd?


Бут нужно комитить только если произошли изменения такие, что старым бутом нельзя собрать компилятор. Обычно это изменения в макросах или АПИ компилятора (имена и т.п.).
http://nemerle.org/Banners/?g=dark
Есть логика намерений и логика обстоятельств, последняя всегда сильнее.
Re[3]: Release = optimize + pdb
От: _nn_ www.nemerleweb.com
Дата: 01.03.10 07:31
Оценка:
Здравствуйте, para, Вы писали:

P>Какое сделать значение по умолчанию?

P>1. Как сейчас
P>
P>     Debug   -> true
P>     Release -> false
P>

P>2. Как в шарпе
P>
P>     Debug   -> true
P>     Release -> true
P>


Можно сделать как в шарпе, ничего плохо pdb в релизе не сулит.

P.S.
Как сейчас не хорошо, как минимум, потому что при сборке инсталлятора ищет pdb файлы
Ну это надо компиляцию инсталлятора исправлять или же сделать pdb по умолчанию.
http://rsdn.nemerleweb.com
http://nemerleweb.com
Re[4]: Release = optimize + pdb
От: para  
Дата: 01.03.10 09:12
Оценка:
P>>1. Как сейчас
P>>
P>>     Debug   -> true
P>>     Release -> false
P>>

так определено по умолчанию в MsBuild

P>>2. Как в шарпе

P>>
P>>     Debug   -> true
P>>     Release -> true
P>>

__>Можно сделать как в шарпе, ничего плохо pdb в релизе не сулит.

__>P.S.

__>Как сейчас не хорошо, как минимум, потому что при сборке инсталлятора ищет pdb файлы
__>Ну это надо компиляцию инсталлятора исправлять или же сделать pdb по умолчанию.

чтобы в релизе компилятора создавались ПДБ, надо либо вручную (теперь и через VS) исправить файлы проектов — добавить соответствующий элемент.
либо править настройки MsBuild, для изменения политики по умолчанию

P>>Какое сделать значение по умолчанию?

я имел в виду для новых проектов, при создании через VS.
пока оставил вар.1
для проектов, в в которых эта переменная не определена, работает политика по умолчанию MsBuild — тоже вар.1
 
Подождите ...
Wait...
Пока на собственное сообщение не было ответов, его можно удалить.