SASGIS - SAS.Планета |
View Issue Details |
|
ID | Project | Category | View Status | Date Submitted | Last Update |
0002534 | SAS.Планета | Рефакторинг | public | 05-11-2014 22:13 | 08-11-2014 18:27 |
|
Reporter | vdemidov | |
Assigned To | vdemidov | |
Priority | normal | Severity | minor | Reproducibility | have not tried |
Status | resolved | Resolution | fixed | |
Platform | | OS | | OS Version | |
Product Version | 140303 | |
Target Version | 141111 | Fixed in Version | 141111 | |
|
Summary | 0002534: Переместить в репозитории все исходники в отдельную папку |
Description | Сейчас папки с исходниками перемешаны с другими папками. ИМХО было бы неплохо переместить все исходники в папку Src, что бы в корне репозитория остались только такие папки .bin, .dcu, Benchmark, Includes, Resources, Test, Tools |
Steps To Reproduce | |
Additional Information | |
Tags | No tags attached. |
Relationships | |
Attached Files | SASUnitsRenamer_141108.zip (815) 08-11-2014 16:18 https://bugtracker.sasgis.org/file_download.php?file_id=1785&type=bug |
|
Issue History |
Date Modified | Username | Field | Change |
05-11-2014 22:13 | vdemidov | New Issue | |
05-11-2014 22:24 | vdemidov | Note Added: 0014859 | |
06-11-2014 08:16 | zed | Note Added: 0014862 | |
06-11-2014 08:22 | vdemidov | Note Added: 0014863 | |
06-11-2014 08:26 | zed | Note Added: 0014865 | |
06-11-2014 08:39 | vdemidov | Note Added: 0014866 | |
06-11-2014 08:44 | zed | File Added: SASUnitsRenamer.zip | |
06-11-2014 08:47 | zed | Note Added: 0014868 | |
06-11-2014 08:54 | vdemidov | Note Added: 0014869 | |
06-11-2014 08:59 | zed | Note Added: 0014870 | |
06-11-2014 09:21 | vdemidov | Note Added: 0014871 | |
06-11-2014 10:58 | vdemidov | Assigned To | => vdemidov |
06-11-2014 10:58 | vdemidov | Status | new => confirmed |
08-11-2014 16:16 | zed | File Added: 8331.patch.zip | |
08-11-2014 16:17 | zed | File Deleted: SASUnitsRenamer.zip | |
08-11-2014 16:18 | zed | File Added: SASUnitsRenamer_141108.zip | |
08-11-2014 16:19 | zed | Note Added: 0014875 | |
08-11-2014 16:42 | vdemidov | Note Added: 0014876 | |
08-11-2014 16:50 | zed | Note Added: 0014877 | |
08-11-2014 18:27 | vdemidov | Status | confirmed => resolved |
08-11-2014 18:27 | vdemidov | Fixed in Version | => 141111 |
08-11-2014 18:27 | vdemidov | Resolution | open => fixed |
08-11-2014 18:56 | zed | File Deleted: 8331.patch.zip | |
Notes |
|
|
У кого-нибудь есть принципиальные возражения? |
|
|
(0014862)
|
zed
|
06-11-2014 08:16
|
|
Не возражаю. У самого такие же мысли были.
Только, наверное, помимо папок, нужно ещё оставить файлы проектов и лицензию на английском языке. А лицензию на русском языке, имхо, надо удалить. |
|
|
|
Лицензию оставить это без вопросов, а вот файлы проектов, я думал тоже переместить. Просто если переместить файлы проектов со всеми исходниками, то их почти не придется изменять, а если оставить на месте, то придется править положение каждого юнита в трех файлах. |
|
|
(0014865)
|
zed
|
06-11-2014 08:26
|
|
Придётся. Но надеюсь ты это не руками делаешь? |
|
|
|
Руками, чем же еще. В интерфейсе делфи. А потом в меркуриале долго и нудно указываю перемещения файлов, что бы непрерывность истории сохранялась. |
|
|
(0014868)
|
zed
|
06-11-2014 08:47
|
|
Я когда-то писал себе скрипт, который правит dpr и dproj, исходя из текущего расположения юнитов. Скрипт на python27.
И в меркуриале, на сколько я помню, можно выделить сразу все файлы и указать что они переименованы. По-одному тыркать не нужно. |
|
|
|
> И в меркуриале, на сколько я помню, можно выделить сразу все файлы и указать что они переименованы. По-одному тыркать не нужно.
Не знал. Нужно будет посмотреть.
Но все равно, мне кажется что место файлов проектов рядом с исходниками. |
|
|
(0014870)
|
zed
|
06-11-2014 08:59
|
|
|
|
|
Должен признать, что вариант с оставить файлы проекта в корне есть плюс, что не придется трогать билд-систему. |
|
|
(0014875)
|
zed
|
08-11-2014 16:19
|
|
Обновил скрипт и приложил патч с перенесёнными юнитами в папку Src. |
|
|
|
А чего просто не закоммитил? |
|
|
(0014877)
|
zed
|
08-11-2014 16:50
|
|
Предоставляю это право тебе. |
|