Notes |
|
|
|
|
(0012506)
|
zed
|
20-08-2013 18:22
|
|
Я так понимаю, надо удалить существующий репо и создать новый клон с гитхаба? |
|
|
|
Ну если не получается затащить в существующий то да, но не забудь проверить что бы Assigned работало. |
|
|
(0012508)
|
zed
|
20-08-2013 19:46
(edited on: 20-08-2013 19:54) |
|
Да, там нужно опять вносить правку по поводу Assigned.
Плюс ещё пару мелких хинтов нужно фиксить:
[DCC Warning] uPSRuntime.pas(9255): H2077 Value assigned to 'Include_' never used
[DCC Warning] uPSRuntime.pas(9271): H2077 Value assigned to 'Exclude_' never used
А XE2 ещё и варнинг нашла:
[DCC Warning] uPSRuntime.pas(2757): W1036 Variable 'Curr' might not have been initialized
|
|
|
|
А оно точно в существующий нормально не затянется? |
|
|
(0012510)
|
zed
|
21-08-2013 06:25
|
|
Пробуй. У меня ничего дельного не получилось.
Можешь и патч им по поводу Assigned заслать. Судя по тому, что там последний коммит был неделю назад, кто-то живой там точно есть, так что может и примут. |
|
|
|
Попробую вечером. Это дело вроде бы не срочное, так что можно и попробовать.
А патч действительно нужно попробовать заслать. |
|
|
|
Заслал патч. Благо он простой и делфы не требовал для проверки. Так его уже даже принять успели :) |
|
|
(0012521)
|
zed
|
21-08-2013 07:36
|
|
Зашли тогда и варнинги:
[DCC Warning] uPSRuntime.pas(9255): H2077 Value assigned to 'Include_' never used
[DCC Warning] uPSRuntime.pas(9271): H2077 Value assigned to 'Exclude_' never used
Там просто закомментировать/удалить "Result:=true;" в самом начале функций нужно. |
|
|
|
Нее, это уж ты как-то сам. |
|
|
(0012524)
|
zed
|
21-08-2013 07:40
|
|
Мне там регистрироваться впадлу. |
|
|
|
Ну посмотрим. Такие изменения я пока не готов без пробной компиляции коммитить :) |
|
|
|
Попробовал. Таки нужно у нас грохать репозиторий и создавать заново с гитхаба.
Теперь вот думаю что проще. Грохнуть старый и залить новый вместо него или сделать отдельную копию и пусть пока живут параллельно? |
|
|
|
Например, переименовать старый, сделать новый под старым именем. Чтобы работать с новым, но если вдруг факап какой (например, по результатам будущего релиза), то вернуться назад, пока патч не будет готов.
зы. Это конечно если переименование меняет URL, иначе смысла нет. |
|
|
(0012545)
|
zed
|
22-08-2013 08:44
|
|
>или сделать отдельную копию и пусть пока живут параллельно
Зачем? Грохай его с концами. Главное исправь потом линки в репо с зависимостями. |
|
|
|
>Зачем? Грохай его с концами. Главное исправь потом линки в репо с зависимостями.
Проблема в том, что меркуриал не расчехлится, что заменили репозиторий, если его положить в ту же папку, и всем это придется делать вручную. И возможность откатиться на любую старую ревизию sas.requires мы потеряем.
Самый надежный способ это положить новый в новую папку с отдельной ссылкой, а потом уже только грохать старый, и то не сразу. |
|
|
(0012548)
|
zed
|
22-08-2013 08:53
|
|
Тогда:
>переименовать старый, сделать новый под старым именем |
|
|
|
>переименовать старый, сделать новый под старым именем
Ага, только если апдейтиться будешь минуя ревизию с переименованием, то будут те же глюки. Проверял в свое время на Imagine. |
|
|
|
Обновил репозиторий. Пока сделал копию с другим именем, но в ту же подпапку в sas.requires.
Zed проверь билд, скорее всего сломается. |
|
|
(0013280)
|
zed
|
17-11-2013 18:36
|
|
У себя обновил вот так:
> hg pull
> hg update -r 75
> %удалить каталог pascalscript%
> hg update -C |
|