0002102: Обновить pascalscript

(0012498)
vdemidov (manager)
20-08-2013 06:43

Хорошая мысль

(0012506)
zed (manager)
20-08-2013 18:22

Я так понимаю, надо удалить существующий репо и создать новый клон с гитхаба?

(0012507)
vdemidov (manager)
20-08-2013 18:56

Ну если не получается затащить в существующий то да, но не забудь проверить что бы Assigned работало.

(0012508)
zed (manager)
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 (manager)
20-08-2013 20:34

А оно точно в существующий нормально не затянется?

(0012510)
zed (manager)
21-08-2013 06:25

Пробуй. У меня ничего дельного не получилось.

Можешь и патч им по поводу Assigned заслать. Судя по тому, что там последний коммит был неделю назад, кто-то живой там точно есть, так что может и примут.

(0012511)
vdemidov (manager)
21-08-2013 06:48

Попробую вечером. Это дело вроде бы не срочное, так что можно и попробовать.

А патч действительно нужно попробовать заслать.

(0012518)
vdemidov (manager)
21-08-2013 07:33

Заслал патч. Благо он простой и делфы не требовал для проверки. Так его уже даже принять успели :)

(0012521)
zed (manager)
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 (manager)
21-08-2013 07:39

Нее, это уж ты как-то сам.

(0012524)
zed (manager)
21-08-2013 07:40

Мне там регистрироваться впадлу.

(0012525)
vdemidov (manager)
21-08-2013 07:41

Ну посмотрим. Такие изменения я пока не готов без пробной компиляции коммитить :)

(0012536)
vdemidov (manager)
21-08-2013 19:52

Попробовал. Таки нужно у нас грохать репозиторий и создавать заново с гитхаба.
Теперь вот думаю что проще. Грохнуть старый и залить новый вместо него или сделать отдельную копию и пусть пока живут параллельно?

(0012537)
vasketsov (manager)
21-08-2013 23:06

Например, переименовать старый, сделать новый под старым именем. Чтобы работать с новым, но если вдруг факап какой (например, по результатам будущего релиза), то вернуться назад, пока патч не будет готов.
зы. Это конечно если переименование меняет URL, иначе смысла нет.

(0012545)
zed (manager)
22-08-2013 08:44

>или сделать отдельную копию и пусть пока живут параллельно
Зачем? Грохай его с концами. Главное исправь потом линки в репо с зависимостями.

(0012547)
vdemidov (manager)
22-08-2013 08:50

>Зачем? Грохай его с концами. Главное исправь потом линки в репо с зависимостями.
Проблема в том, что меркуриал не расчехлится, что заменили репозиторий, если его положить в ту же папку, и всем это придется делать вручную. И возможность откатиться на любую старую ревизию sas.requires мы потеряем.
Самый надежный способ это положить новый в новую папку с отдельной ссылкой, а потом уже только грохать старый, и то не сразу.

(0012548)
zed (manager)
22-08-2013 08:53

Тогда:
>переименовать старый, сделать новый под старым именем

(0012549)
vdemidov (manager)
22-08-2013 09:03

>переименовать старый, сделать новый под старым именем
Ага, только если апдейтиться будешь минуя ревизию с переименованием, то будут те же глюки. Проверял в свое время на Imagine.

(0013279)
vdemidov (manager)
17-11-2013 16:49

Обновил репозиторий. Пока сделал копию с другим именем, но в ту же подпапку в sas.requires.
Zed проверь билд, скорее всего сломается.

(0013280)
zed (manager)
17-11-2013 18:36

У себя обновил вот так:

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

Users who viewed this issue
User List	Anonymous (2894x)
Total Views	2894
Last View	22-11-2024 03:27

Notes
(0012498) vdemidov (manager) 20-08-2013 06:43	Хорошая мысль

(0012506) zed (manager) 20-08-2013 18:22	Я так понимаю, надо удалить существующий репо и создать новый клон с гитхаба?

(0012507) vdemidov (manager) 20-08-2013 18:56	Ну если не получается затащить в существующий то да, но не забудь проверить что бы Assigned работало.

(0012508) zed (manager) 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 (manager) 20-08-2013 20:34	А оно точно в существующий нормально не затянется?

(0012510) zed (manager) 21-08-2013 06:25	Пробуй. У меня ничего дельного не получилось. Можешь и патч им по поводу Assigned заслать. Судя по тому, что там последний коммит был неделю назад, кто-то живой там точно есть, так что может и примут.

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

(0012518) vdemidov (manager) 21-08-2013 07:33	Заслал патч. Благо он простой и делфы не требовал для проверки. Так его уже даже принять успели :)

(0012521) zed (manager) 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 (manager) 21-08-2013 07:39	Нее, это уж ты как-то сам.

(0012524) zed (manager) 21-08-2013 07:40	Мне там регистрироваться впадлу.

(0012525) vdemidov (manager) 21-08-2013 07:41	Ну посмотрим. Такие изменения я пока не готов без пробной компиляции коммитить :)

(0012536) vdemidov (manager) 21-08-2013 19:52	Попробовал. Таки нужно у нас грохать репозиторий и создавать заново с гитхаба. Теперь вот думаю что проще. Грохнуть старый и залить новый вместо него или сделать отдельную копию и пусть пока живут параллельно?

(0012537) vasketsov (manager) 21-08-2013 23:06	Например, переименовать старый, сделать новый под старым именем. Чтобы работать с новым, но если вдруг факап какой (например, по результатам будущего релиза), то вернуться назад, пока патч не будет готов. зы. Это конечно если переименование меняет URL, иначе смысла нет.

(0012545) zed (manager) 22-08-2013 08:44	>или сделать отдельную копию и пусть пока живут параллельно Зачем? Грохай его с концами. Главное исправь потом линки в репо с зависимостями.

(0012547) vdemidov (manager) 22-08-2013 08:50	>Зачем? Грохай его с концами. Главное исправь потом линки в репо с зависимостями. Проблема в том, что меркуриал не расчехлится, что заменили репозиторий, если его положить в ту же папку, и всем это придется делать вручную. И возможность откатиться на любую старую ревизию sas.requires мы потеряем. Самый надежный способ это положить новый в новую папку с отдельной ссылкой, а потом уже только грохать старый, и то не сразу.

(0012548) zed (manager) 22-08-2013 08:53	Тогда: >переименовать старый, сделать новый под старым именем

(0012549) vdemidov (manager) 22-08-2013 09:03	>переименовать старый, сделать новый под старым именем Ага, только если апдейтиться будешь минуя ревизию с переименованием, то будут те же глюки. Проверял в свое время на Imagine.

(0013279) vdemidov (manager) 17-11-2013 16:49	Обновил репозиторий. Пока сделал копию с другим именем, но в ту же подпапку в sas.requires. Zed проверь билд, скорее всего сломается.

(0013280) zed (manager) 17-11-2013 18:36	У себя обновил вот так: > hg pull > hg update -r 75 > %удалить каталог pascalscript% > hg update -C

Issue History
Date Modified	Username	Field	Change
19-08-2013 20:23	vasketsov	New Issue
20-08-2013 06:43	vdemidov	Note Added: 0012498
20-08-2013 06:43	vdemidov	Status	new => confirmed
20-08-2013 06:43	vdemidov	Product Version	=> 121010
20-08-2013 06:43	vdemidov	Target Version	=> 131111
20-08-2013 18:22	zed	Note Added: 0012506
20-08-2013 18:56	vdemidov	Note Added: 0012507
20-08-2013 19:46	zed	Note Added: 0012508
20-08-2013 19:54	zed	Note Edited: 0012508	View Revisions
20-08-2013 20:34	vdemidov	Note Added: 0012509
21-08-2013 06:25	zed	Note Added: 0012510
21-08-2013 06:48	vdemidov	Note Added: 0012511
21-08-2013 07:33	vdemidov	Note Added: 0012518
21-08-2013 07:36	zed	Note Added: 0012521
21-08-2013 07:39	vdemidov	Note Added: 0012523
21-08-2013 07:40	zed	Note Added: 0012524
21-08-2013 07:41	vdemidov	Note Added: 0012525
21-08-2013 19:52	vdemidov	Note Added: 0012536
21-08-2013 23:06	vasketsov	Note Added: 0012537
22-08-2013 08:44	zed	Note Added: 0012545
22-08-2013 08:50	vdemidov	Note Added: 0012547
22-08-2013 08:53	zed	Note Added: 0012548
22-08-2013 09:03	vdemidov	Note Added: 0012549
04-11-2013 13:41	vdemidov	Target Version	131111 => 24xxxx
04-11-2013 14:22	vdemidov	Target Version	24xxxx => 140303
17-11-2013 16:49	vdemidov	Note Added: 0013279
17-11-2013 16:49	vdemidov	Assigned To	=> zed
17-11-2013 16:49	vdemidov	Status	confirmed => assigned
17-11-2013 18:36	zed	Note Added: 0013280
17-11-2013 18:37	zed	Assigned To	zed => vdemidov
17-11-2013 18:37	zed	Status	assigned => resolved
17-11-2013 18:37	zed	Resolution	open => fixed
17-11-2013 18:37	zed	Fixed in Version	=> 140303
17-11-2013 21:32	vdemidov	Relationship added	related to 0002252

SASGIS