View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0002102 | SAS.Планета | Рефакторинг / Refactoring | public | 19-08-2013 20:23 | 17-11-2013 18:37 |
| Reporter | vasketsov | Assigned To | vdemidov | ||
| Priority | normal | Severity | minor | Reproducibility | N/A |
| Status | resolved | Resolution | fixed | ||
| Platform | Windows | OS | Vista | OS Version | Ultimate |
| Product Version | 121010 | ||||
| Target Version | 140303 | Fixed in Version | 140303 | ||
| Summary | 0002102: Обновить pascalscript | ||||
| Description | Не пора ли обновиться? https://github.com/remobjects/pascalscript там уже поддерживается вплоть до XE4. И ещё замените пожалуйста ссылку в репо PascalScript-а: http://www.remobjects.com/ps старая сдохла совсем | ||||
| Tags | No tags attached. | ||||
|
|
Хорошая мысль |
|
|
Я так понимаю, надо удалить существующий репо и создать новый клон с гитхаба? |
|
|
Ну если не получается затащить в существующий то да, но не забудь проверить что бы Assigned работало. |
|
|
Да, там нужно опять вносить правку по поводу 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 |
|
|
А оно точно в существующий нормально не затянется? |
|
|
Пробуй. У меня ничего дельного не получилось. Можешь и патч им по поводу 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 Там просто закомментировать/удалить "Result:=true;" в самом начале функций нужно. |
|
|
Нее, это уж ты как-то сам. |
|
|
Мне там регистрироваться впадлу. |
|
|
Ну посмотрим. Такие изменения я пока не готов без пробной компиляции коммитить :) |
|
|
Попробовал. Таки нужно у нас грохать репозиторий и создавать заново с гитхаба. Теперь вот думаю что проще. Грохнуть старый и залить новый вместо него или сделать отдельную копию и пусть пока живут параллельно? |
|
|
Например, переименовать старый, сделать новый под старым именем. Чтобы работать с новым, но если вдруг факап какой (например, по результатам будущего релиза), то вернуться назад, пока патч не будет готов. зы. Это конечно если переименование меняет URL, иначе смысла нет. |
|
|
>или сделать отдельную копию и пусть пока живут параллельно Зачем? Грохай его с концами. Главное исправь потом линки в репо с зависимостями. |
|
|
>Зачем? Грохай его с концами. Главное исправь потом линки в репо с зависимостями. Проблема в том, что меркуриал не расчехлится, что заменили репозиторий, если его положить в ту же папку, и всем это придется делать вручную. И возможность откатиться на любую старую ревизию sas.requires мы потеряем. Самый надежный способ это положить новый в новую папку с отдельной ссылкой, а потом уже только грохать старый, и то не сразу. |
|
|
Тогда: >переименовать старый, сделать новый под старым именем |
|
|
>переименовать старый, сделать новый под старым именем Ага, только если апдейтиться будешь минуя ревизию с переименованием, то будут те же глюки. Проверял в свое время на Imagine. |
|
|
Обновил репозиторий. Пока сделал копию с другим именем, но в ту же подпапку в sas.requires. Zed проверь билд, скорее всего сломается. |
|
|
У себя обновил вот так: > hg pull > hg update -r 75 > %удалить каталог pascalscript% > hg update -C |
| 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 | |
| 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 => 41xxxx |
| 04-11-2013 14:22 | vdemidov | Target Version | 41xxxx => 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 |
| 08-08-2025 13:25 | zed | Category | Рефакторинг => Рефакторинг / Refactoring |