Переименовывание `module` в `module.bsl`. #56

dmpas · 2017-03-02T13:30:12Z

После распаковки толстых форм файлы module переименовываются в
module.bsl. В renames.txt добавляется соответствующая запись.

После распаковки толстых форм файлы `module` переименовываются в `module.bsl`. В `renames.txt` добавляется соответствующая запись.

khorevaa · 2017-03-02T13:32:05Z

Собственно задача на эту доработку #14

artbear

Замечаний нет.
Один вопрос - тесты прогнал?

dmpas · 2017-03-03T06:25:38Z

@artbear

ПРЕДУПРЕЖДЕНИЕ - Попытка удаления C:\Users\sbatanov\AppData\Local\Temp\m2lxm5rb.5f2.tmp закончилась неудачей, по причине {Модуль C:\opt\oscript\lib\tempfiles\ВременныеФайлы.os / Ошибка в строке: 74 / Внешнее исключение (System.UnauthorizedAccessException): Отказано в доступе по пути "1f1162cb2b66a0d3815576705becf33aa5e3fe".}
ПРЕДУПРЕЖДЕНИЕ - Попытка удаления C:\Users\sbatanov\AppData\Local\Temp\qdgqp1vy.m25.tmp закончилась неудачей, по причине {Модуль C:\opt\oscript\lib\tempfiles\ВременныеФайлы.os / Ошибка в строке: 74 / Внешнее исключение (System.UnauthorizedAccessException): Отказано в доступе по пути "d307c2ae1f936a415d760c745134eb4e3dd7d1".}
ПРЕДУПРЕЖДЕНИЕ - Попытка удаления C:\Users\sbatanov\AppData\Local\Temp\h2z4yzie.rww.tmp закончилась неудачей, по причине {Модуль C:\opt\oscript\lib\tempfiles\ВременныеФайлы.os / Ошибка в строке: 74 / Внешнее исключение (System.UnauthorizedAccessException): Отказано в доступе по пути "1f1162cb2b66a0d3815576705becf33aa5e3fe".}
ПРЕДУПРЕЖДЕНИЕ - Попытка удаления C:\Users\sbatanov\AppData\Local\Temp\fq3lcfiy.fiq.tmp закончилась неудачей, по причине {Модуль C:\opt\oscript\lib\tempfiles\ВременныеФайлы.os / Ошибка в строке: 74 / Внешнее исключение (System.UnauthorizedAccessException): Отказано в доступе по пути "1f1162cb2b66a0d3815576705becf33aa5e3fe".}
ПРЕДУПРЕЖДЕНИЕ - Попытка удаления C:\Users\sbatanov\AppData\Local\Temp\e4zjrkgv.it5.tmp закончилась неудачей, по причине {Модуль C:\opt\oscript\lib\tempfiles\ВременныеФайлы.os / Ошибка в строке: 74 / Внешнее исключение (System.UnauthorizedAccessException): Отказано в доступе по пути "8e1a25b1e2e1e3f47ea722dd6f4ef48f868ef3".}
ПРЕДУПРЕЖДЕНИЕ - Попытка удаления C:\Users\sbatanov\AppData\Local\Temp\bzh2z12l.4fy.tmp закончилась неудачей, по причине {Модуль C:\opt\oscript\lib\tempfiles\ВременныеФайлы.os / Ошибка в строке: 74 / Внешнее исключение (System.UnauthorizedAccessException): Отказано в доступе по пути "da76c725eab65fa543ac29deeb4562e0d2be86".}
ПРЕДУПРЕЖДЕНИЕ - Попытка удаления C:\Users\sbatanov\AppData\Local\Temp\lp4us1rc.n1k.tmp закончилась неудачей, по причине {Модуль C:\opt\oscript\lib\tempfiles\ВременныеФайлы.os / Ошибка в строке: 74 / Внешнее исключение (System.UnauthorizedAccessException): Отказано в доступе по пути "da76c725eab65fa543ac29deeb4562e0d2be86".}
ПРЕДУПРЕЖДЕНИЕ - Попытка удаления C:\Users\sbatanov\AppData\Local\Temp\izjx0w33.gqc.tmp закончилась неудачей, по причине {Модуль C:\opt\oscript\lib\tempfiles\ВременныеФайлы.os / Ошибка в строке: 74 / Внешнее исключение (System.UnauthorizedAccessException): Отказано в доступе по пути "915ba9890db61b7f4375b620023367f16afee1".}
ПРЕДУПРЕЖДЕНИЕ - Попытка удаления C:\Users\sbatanov\AppData\Local\Temp\1m005owj.ggi.tmp закончилась неудачей, по причине {Модуль C:\opt\oscript\lib\tempfiles\ВременныеФайлы.os / Ошибка в строке: 74 / Внешнее исключение (System.UnauthorizedAccessException): Отказано в доступе по пути "a163a84cae62735c4f534f88c95cf23c2f346a".}

------------------------------------------------------------
                       Общая статистика
------------------------------------------------------------

Тестов пройдено: 23

Тестов не пройдено: 0

Тестов не реализовано \ пропущено: 0

ОК. Зеленая полоса

тысяча предупреждений, но провалов 0.

dmpas · 2017-03-03T06:38:14Z

Думаю, надо переделать на Module.bsl - с большой буквы, для единообразия.

EvilBeaver

А с самим таким поведением по .bsl все согласились? Там гит не убьет историю? Если возражений нет, то мержим.

dmpas · 2017-03-05T13:27:57Z

@EvilBeaver в гиттере @demidych и @khorevaa предлагали вынести это поведение в отдельную настройку.

EvilBeaver · 2017-03-05T13:32:51Z

@dmpas ну так может эту настройку впилить? А то похерит историю кому нибудь, а виноваты мы.

dmpas · 2017-03-05T13:41:30Z

да, стоит впилить. буду ковырять, как в гитсинке настройки устроены.

nixel2007 · 2017-03-05T14:35:00Z

Там гит не убьет историю?

гит - нет. гитхаб - да.

Добавлен булев параметр командной строки `-rename-modulebsl`. * on - переименовывать module в Module.bsl * off (по-умолчанию) - старое поведение

dmpas · 2017-03-09T12:28:37Z

@artbear @EvilBeaver впилил настройку.

nixel2007 · 2017-03-09T13:08:26Z

src/core/Классы/КомандаExport.os

@@ -26,6 +26,7 @@
 	Парсер.ДобавитьИменованныйПараметрКоманды(ОписаниеКоманды, "-maxversion", "<номер максимальной версии для выгрузки>");
 	Парсер.ДобавитьИменованныйПараметрКоманды(ОписаниеКоманды, "-limit", "<выгрузить неболее limit версий от текущей выгруженной>");
 	Парсер.ДобавитьИменованныйПараметрКоманды(ОписаниеКоманды, "-tempdir", "<Путь к каталогу временных файлов>");
+	Парсер.ДобавитьИменованныйПараметрКоманды(ОписаниеКоманды, "-rename-modulebsl", "<on|off>");


Может использовать Параметр-флаг, а не именованный параметр?
указан - переименовываем, не указан - не переименовываем.

я сделал как дебуг и вербосе. Единообразие вроде как жжёт.

Они устарели, параметр флаг рулит

обратную совместимость рушить или как?

Расскажу вам вот, что.

Если есть "параметр-флаг", то должен быть и "параметр-флаг-не". Как это работает в никсах:
есть утилита tool, принимает параметры --enable-feature и --disable-feature. Почему два? для вот такой развлекухи: tool --enable-feature --disable-feature - тут сработает последний (disable feature). Для чего используется? Для программного формирования строки запуска и для алиасов. То есть, если у меня прописано alias tool=tool --enable-feature, то запуская просто tool я запускаю его с включенной фичей, а tool --disable-feature - с выключенной.

Так что, "параметр-флаг" стоит немного доработать. Сделаю задачку, как думаете?

artbear · 2017-03-09T13:18:09Z

пока не рушим
ты новый флаг добавляй, старые не трожь.

artbear · 2017-03-09T13:20:27Z

src/core/Классы/КомандаExport.os

@@ -26,6 +26,7 @@
 	Парсер.ДобавитьИменованныйПараметрКоманды(ОписаниеКоманды, "-maxversion", "<номер максимальной версии для выгрузки>");
 	Парсер.ДобавитьИменованныйПараметрКоманды(ОписаниеКоманды, "-limit", "<выгрузить неболее limit версий от текущей выгруженной>");
 	Парсер.ДобавитьИменованныйПараметрКоманды(ОписаниеКоманды, "-tempdir", "<Путь к каталогу временных файлов>");
+	Парсер.ДобавитьИменованныйПараметрКоманды(ОписаниеКоманды, "-rename-modulebsl", "<on|off>");


Может быть, сделать имя параметра -rename2modulebsl
Иначе сейчас читается странно.

вот двойка точно мозг вынесет и глаз режет :) давай придумаем понятное название.

давайте только без двоек)))

-rename-to-bsl rename-into-bsl

-make-module-bsl сделать модуль БСЛьным

Переделано на параметр-флаг `-process-fatform-modules`.

dmpas · 2017-03-13T10:54:59Z

@artbear @nixel2007 сделал параметр-флаг с добротным именем.

theshadowco · 2017-03-15T08:56:48Z

@dmpas ты добавил только для команды export, для sync тоже нада

dmpas · 2017-03-15T09:00:51Z

@theshadowco я синком не пользуюсь, потому мне сложно будет проверять

theshadowco · 2017-03-15T09:03:18Z

@dmpas патч аналогичен export, т.к. используются те же модули.

dmpas · 2017-03-15T09:05:02Z

@theshadowco хорошо, сделаю. будет возможность проверить?

theshadowco · 2017-03-15T09:19:51Z

@dmpas да

Флаг `-process-fatforms-modules` добавлен также в команду sync.

dmpas · 2017-03-19T14:18:42Z

@theshadowco добавил.

Переименовывание module в module.bsl.

18e0c95

После распаковки толстых форм файлы `module` переименовываются в `module.bsl`. В `renames.txt` добавляется соответствующая запись.

dmpas requested review from pumbaEO, nixel2007, EvilBeaver and artbear March 2, 2017 13:30

nixel2007 approved these changes Mar 2, 2017

View reviewed changes

artbear approved these changes Mar 3, 2017

View reviewed changes

EvilBeaver approved these changes Mar 5, 2017

View reviewed changes

dmpas added 3 commits March 9, 2017 14:37

Merge remote-tracking branch 'evil/develop' into feature/fat-forms

6ee42da

Переделано на Module.bsl

8ce2a24

nixel2007 reviewed Mar 9, 2017

View reviewed changes

artbear reviewed Mar 9, 2017

View reviewed changes

Изменён параметр обработки модуля толстых форм.

5d97e52

Переделано на параметр-флаг `-process-fatform-modules`.

Распаковка толстых форм в команде sync.

685349e

Флаг `-process-fatforms-modules` добавлен также в команду sync.

EvilBeaver merged commit 7d5279e into oscript-library:develop Mar 21, 2017

Переименовывание module в module.bsl. #56

Переименовывание module в module.bsl. #56

Uh oh!

Conversation

dmpas commented Mar 2, 2017

Uh oh!

khorevaa commented Mar 2, 2017

Uh oh!

artbear left a comment

Choose a reason for hiding this comment

Uh oh!

dmpas commented Mar 3, 2017

Uh oh!

dmpas commented Mar 3, 2017

Uh oh!

EvilBeaver left a comment

Choose a reason for hiding this comment

Uh oh!

dmpas commented Mar 5, 2017

Uh oh!

EvilBeaver commented Mar 5, 2017

Uh oh!

dmpas commented Mar 5, 2017

Uh oh!

nixel2007 commented Mar 5, 2017

Uh oh!

dmpas commented Mar 9, 2017

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

artbear commented Mar 9, 2017

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

dmpas commented Mar 13, 2017

Uh oh!

theshadowco commented Mar 15, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

dmpas commented Mar 15, 2017

Uh oh!

theshadowco commented Mar 15, 2017

Uh oh!

dmpas commented Mar 15, 2017

Uh oh!

theshadowco commented Mar 15, 2017

Uh oh!

dmpas commented Mar 19, 2017

Uh oh!

Uh oh!

Переименовывание `module` в `module.bsl`. #56

Переименовывание `module` в `module.bsl`. #56

theshadowco commented Mar 15, 2017 •

edited

Loading