SASGIS - SAS.Планета
View Issue Details
0002102SAS.ПланетаРефакторингpublic19-08-2013 20:2317-11-2013 18:37
vasketsov 
vdemidov 
normalminorN/A
resolvedfixed 
WindowsVistaUltimate
121010 
140303140303 
0002102: Обновить pascalscript
Не пора ли обновиться?
https://github.com/remobjects/pascalscript
там уже поддерживается вплоть до XE4.

И ещё замените пожалуйста ссылку в репо PascalScript-а:
http://www.remobjects.com/ps
старая сдохла совсем
No tags attached.
related to 0002252resolved vdemidov Обновить репозиторий Alcinoe 
Issue History
19-08-2013 20:23vasketsovNew Issue
20-08-2013 06:43vdemidovNote Added: 0012498
20-08-2013 06:43vdemidovStatusnew => confirmed
20-08-2013 06:43vdemidovProduct Version => 121010
20-08-2013 06:43vdemidovTarget Version => 131111
20-08-2013 18:22zedNote Added: 0012506
20-08-2013 18:56vdemidovNote Added: 0012507
20-08-2013 19:46zedNote Added: 0012508
20-08-2013 19:54zedNote Edited: 0012508bug_revision_view_page.php?bugnote_id=12508#r5635
20-08-2013 20:34vdemidovNote Added: 0012509
21-08-2013 06:25zedNote Added: 0012510
21-08-2013 06:48vdemidovNote Added: 0012511
21-08-2013 07:33vdemidovNote Added: 0012518
21-08-2013 07:36zedNote Added: 0012521
21-08-2013 07:39vdemidovNote Added: 0012523
21-08-2013 07:40zedNote Added: 0012524
21-08-2013 07:41vdemidovNote Added: 0012525
21-08-2013 19:52vdemidovNote Added: 0012536
21-08-2013 23:06vasketsovNote Added: 0012537
22-08-2013 08:44zedNote Added: 0012545
22-08-2013 08:50vdemidovNote Added: 0012547
22-08-2013 08:53zedNote Added: 0012548
22-08-2013 09:03vdemidovNote Added: 0012549
04-11-2013 13:41vdemidovTarget Version131111 => 24xxxx
04-11-2013 14:22vdemidovTarget Version24xxxx => 140303
17-11-2013 16:49vdemidovNote Added: 0013279
17-11-2013 16:49vdemidovAssigned To => zed
17-11-2013 16:49vdemidovStatusconfirmed => assigned
17-11-2013 18:36zedNote Added: 0013280
17-11-2013 18:37zedAssigned Tozed => vdemidov
17-11-2013 18:37zedStatusassigned => resolved
17-11-2013 18:37zedResolutionopen => fixed
17-11-2013 18:37zedFixed in Version => 140303
17-11-2013 21:32vdemidovRelationship addedrelated to 0002252

Notes
(0012498)
vdemidov   
20-08-2013 06:43   
Хорошая мысль
(0012506)
zed   
20-08-2013 18:22   
Я так понимаю, надо удалить существующий репо и создать новый клон с гитхаба?
(0012507)
vdemidov   
20-08-2013 18:56   
Ну если не получается затащить в существующий то да, но не забудь проверить что бы 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

(0012509)
vdemidov   
20-08-2013 20:34   
А оно точно в существующий нормально не затянется?
(0012510)
zed   
21-08-2013 06:25   
Пробуй. У меня ничего дельного не получилось.

Можешь и патч им по поводу Assigned заслать. Судя по тому, что там последний коммит был неделю назад, кто-то живой там точно есть, так что может и примут.
(0012511)
vdemidov   
21-08-2013 06:48   
Попробую вечером. Это дело вроде бы не срочное, так что можно и попробовать.

А патч действительно нужно попробовать заслать.
(0012518)
vdemidov   
21-08-2013 07:33   
Заслал патч. Благо он простой и делфы не требовал для проверки. Так его уже даже принять успели :)
(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;" в самом начале функций нужно.
(0012523)
vdemidov   
21-08-2013 07:39   
Нее, это уж ты как-то сам.
(0012524)
zed   
21-08-2013 07:40   
Мне там регистрироваться впадлу.
(0012525)
vdemidov   
21-08-2013 07:41   
Ну посмотрим. Такие изменения я пока не готов без пробной компиляции коммитить :)
(0012536)
vdemidov   
21-08-2013 19:52   
Попробовал. Таки нужно у нас грохать репозиторий и создавать заново с гитхаба.
Теперь вот думаю что проще. Грохнуть старый и залить новый вместо него или сделать отдельную копию и пусть пока живут параллельно?
(0012537)
vasketsov   
21-08-2013 23:06   
Например, переименовать старый, сделать новый под старым именем. Чтобы работать с новым, но если вдруг факап какой (например, по результатам будущего релиза), то вернуться назад, пока патч не будет готов.
зы. Это конечно если переименование меняет URL, иначе смысла нет.
(0012545)
zed   
22-08-2013 08:44   
>или сделать отдельную копию и пусть пока живут параллельно
Зачем? Грохай его с концами. Главное исправь потом линки в репо с зависимостями.
(0012547)
vdemidov   
22-08-2013 08:50   
>Зачем? Грохай его с концами. Главное исправь потом линки в репо с зависимостями.
Проблема в том, что меркуриал не расчехлится, что заменили репозиторий, если его положить в ту же папку, и всем это придется делать вручную. И возможность откатиться на любую старую ревизию sas.requires мы потеряем.
Самый надежный способ это положить новый в новую папку с отдельной ссылкой, а потом уже только грохать старый, и то не сразу.
(0012548)
zed   
22-08-2013 08:53   
Тогда:
>переименовать старый, сделать новый под старым именем
(0012549)
vdemidov   
22-08-2013 09:03   
>переименовать старый, сделать новый под старым именем
Ага, только если апдейтиться будешь минуя ревизию с переименованием, то будут те же глюки. Проверял в свое время на Imagine.
(0013279)
vdemidov   
17-11-2013 16:49   
Обновил репозиторий. Пока сделал копию с другим именем, но в ту же подпапку в sas.requires.
Zed проверь билд, скорее всего сломается.
(0013280)
zed   
17-11-2013 18:36   
У себя обновил вот так:

> hg pull
> hg update -r 75
> %удалить каталог pascalscript%
> hg update -C