precommit1c 2.0 #72

nixel2007 · 2016-03-21T14:31:05Z

Summary:

Обновлен v8reader. Тексты модулей теперь раскладываются в bsl
Полный переход на oscript-версию
- удалена python-версия скрипта
- удалена информация по функциональности файла precommit.ini
Обновлено README
Мажорная версия 2.0 + вывод информации о версии

Fix #9.

В новой версии v8reader добавлена функциональность по раскладыванию обработок в расширение bsl.
Проверено, что git корректно распознает смену расширения как переименование.

@artbear @pumbaEO если возражений нет, смержите, пожалуйста.

pumbaEO · 2016-03-21T15:18:20Z

Сборка не сломается? Там же теперь в толстые форма добавляется расширение bsl , а раньше не было, т.е. при сборке формы module.bsl надо переименовать на module и только потом паковать.

nixel2007 · 2016-03-21T15:34:17Z

@pumbaEO да, сломается.
Варианта 2.

при сборке анализируется каталог, если в нем есть файл module.bsl, то он переименовывается в module и в таком виде подсовывается v8unpack
на стороне v8reader в файл renames.txt добавляется запись о переименовании. Тогда при сборке файл переименуется сам. В этом случае надо гарантировать, что в renames.txt запись о каталоге будет строго до записи о файле.

Хотел с тобой посоветоваться, как сделать лучше.

nixel2007 · 2016-03-21T15:35:52Z

А нет, в 2 нифига он не переименуется... В каталоге просто окажутся два файла module и module.bsl

artbear · 2016-03-21T16:41:11Z

Я против подобного исправления.
Текущая доработка будет неожиданностью для пользователей продукта.
Кто-то сделает небольшую доработку обработки и у него будет несколько исправленных файлов вместо одного.
ИМХО лучше добавить доп.ключ командной строки

artbear · 2016-03-21T16:42:41Z

@bambr1975 Что скажешь насчет ключа командной строки в v8reader?

pumbaEO · 2016-03-21T17:04:59Z

Артур, git поймет переименование и у нас все очищается и потом новое пишеться, я не вижу проблемы в добавлении расширения bsl. Да и я не знаю, где-бы к расширению txt привязывались.

pumbaEO · 2016-03-21T17:06:16Z

Варианта 2.

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

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

nixel2007 · 2016-03-21T17:18:55Z

@artbear

Задача по смене расширения висит уже больше года ;)
Де-факто стандартное расширение для модулей 1С - bsl. Поэтому это прекоммит сейчас ведет себя нестандартно, и в идеале мы должны вернуться к ожидаемому поведению
При даже маленьком изменении прекоммит выгрузит около десятка файлов, ибо скобочки и прочие известные "приветы" от 1С. И переименование гит обработает корректно - пример - последняя выгрузка в исходники самой v8reader.

Имхо, еще один ключ в v8reader - излишнее усложнение.

nixel2007 · 2016-03-21T17:20:30Z

@pumbaEO могу взять на себя реализацию для v8files-extractor

bambr1975 · 2016-03-21T18:00:42Z

@artbear xDrivenDevelopment/v8Reader@c2436e7 вот пример такого коммита. По-моему, четко понятно, что изменено, а что переименовано. Разговор о ключе был в свете заблуждения, что git не сможет показать такое сравнение. Если может, то по-моему, ключ не нужен. Скажи конкретную причину, зачем тебе нужен этот ключ?

artbear · 2016-03-21T18:34:12Z

Само собой, я знаю про переименование файлов в гит.

Мне не очень нравится, что в случае очередной правки обработок в репозитарии в истории будет куча измененных файлов вместо наличия только доработанных файлов.
Конечно, это не слишком критичная проблема, все-таки количество переименованных файлов не будет так велико, как в случае разбора конфигурации на файлы.

Давайте хотя бы мажорный номер релиза поменяем, чтобы показать расхождение в совместимости.

nixel2007 · 2016-03-21T18:42:45Z

@artbear про мажорную версию сам думал.
единственная проблема, что precommit1c сейчас вроде вообще без версий :)
2.0?

artbear · 2016-03-21T18:46:49Z

Да, 2.0

pumbaEO · 2016-03-21T18:53:34Z

https://github.com/xDrivenDevelopment/precommit1c/blob/develop/pyv8unpack.py#L13

nixel2007 · 2016-03-21T18:55:15Z

https://github.com/xDrivenDevelopment/precommit1c/blob/develop/pyv8unpack.py#L13

вот оно что...)
Тогда можно и 1.0 :D

nixel2007 · 2016-03-21T19:39:26Z

Добавил переименование.
@pumbaEO добавишь в питоноверсию?

pumbaEO · 2016-03-21T19:41:42Z

Та я вообще хочу питоновскую убрать уже из релиза, перейти так сказать
полностью на oscript ...

2016-03-21 22:39 GMT+03:00 Nikita Gryzlov notifications@github.com:

Добавил переименование.
@pumbaEO https://github.com/pumbaEO добавишь в питоноверсию?

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#72 (comment)

З повагою Сосна Євген,
mailto:shenja@sosna.zp.ua
skype:shenjasosna
+380933897103

nixel2007 · 2016-03-21T19:42:44Z

@pumbaEO ну, кстати, неплохая идея. у него и функциональности меньше. плюс oscript-версия покрыта тестами.

@artbear ?

nixel2007 · 2016-03-21T20:01:16Z

Я это к чему - надо либо релизить новую версию и в питоне и в односкрипте, либо объявлять о питоноустаревании и в 2.0 включить только os.

ghost · 2016-03-21T20:53:07Z

для vanessa-behavior сделано специально, что подмодуль смотрит в основной @pumbaEO репозиторий. Поэтому питоноустаревание поддерживаю.

artbear · 2016-03-22T07:58:41Z

ИМХО нужно выпускать версию 2.0
Фактически у нас была версия 1 - питон
сейчас мы активно юзаем версию с оскрипт. Предлагаю и обозначить ее версией 2.0

artbear · 2016-03-22T08:00:10Z

для vanessa-behavior сделано специально, что подмодуль смотрит в основной @pumbaEO репозиторий. Поэтому питоноустаревание поддерживаю.

@allustin Не очень понятно, зачем так сделано.
Основной ведь проект ведется именно в текущем репозитарии, а у Жени был/есть инициирующий проект.

nixel2007 · 2016-03-22T08:33:44Z

@artbear постараюсь сегодня сделать второй pr с ридми и убранным питоном.

Этот pr принимаем?

artbear · 2016-03-22T09:38:55Z

В новом pr поменяешь номер релиза на 2.0 ?

artbear · 2016-03-22T09:44:14Z

Предлагаю все-таки именно в этот pr добавить метод Версия для 2.0 и показ версии при запуске

artbear · 2016-03-22T09:44:33Z

Потом я замержу pr

artbear · 2016-03-22T14:47:40Z

@nixel2007 Вечером посмотрю.

artbear · 2016-03-22T18:35:17Z

README.md


 4. Путь к платформе находится автоматически в случае стандартной установки 1С. Если необходимо явно указать путь к платформе, то нужно: указать переменную окружения PATH1C c путём к каталогу, в который установлена 1С
 ```
 set PATH1C = d:\program\


ИМХО этот код в оскрипт-версии не используется

artbear · 2016-03-22T18:43:53Z

@nixel2007 Добавишь в коммит #47 (показ версии) ?

artbear · 2016-03-22T18:53:24Z

@nixel2007 @pumbaEO Чем вам отладка не понравилась?
ИМХО вывод подробностей не создает слишком большой шум и позволяет сразу анализировать возникающие ошибки

pumbaEO · 2016-03-22T18:55:50Z

Ошибок не должно быть, успех или не успех. Но захломлять вывод неправильно.
А если не успех, тогда уже запускаем с режимом отладки и смотрим.

22 марта 2016 г., 21:53 пользователь Artur Ayukhanov <
notifications@github.com> написал:

@nixel2007 https://github.com/nixel2007 @pumbaEO
https://github.com/pumbaEO Чем вам отладка не понравилась?
ИМХО вывод подробностей не создает слишком большой шум и позволяет сразу
анализировать возникающие ошибки

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#72 (comment)

З повагою Сосна Євген,
mailto:shenja@sosna.zp.ua
skype:shenjasosna
+380933897103

nixel2007 · 2016-03-22T21:31:20Z

Убрал информацию по precommit.ini

nixel2007 · 2016-03-22T21:39:17Z

Ошибок не должно быть, успех или не успех. Но захломлять вывод неправильно.
А если не успех, тогда уже запускаем с режимом отладки и смотрим.

+1.
Я еще думал убрать debug-вывод cmdline, но без него вообще сложно понять, почему не работают те или иные ключи, если вдруг кому-то захочется поменять сам файл pre-commit

Необходимая функциональность уже включена в мастер

artbear · 2016-03-23T13:11:57Z

v8files-extractor.os

 		Возврат Ложь;
 	КонецЕсли;

+	Лог.Информация("precommit1c " + ПолучитьВерсию() + Символы.ПС);


Есть метод Версия(), а вызывается метод ПолучитьВерсию() :)
А как же тесты?

Тьфу! Сейчас поправлю

artbear · 2016-03-23T13:14:56Z

Лады, по отладке оставляем, как реализовано в pr
А я в свободное время посмотрю, возможно, какие-то полезные сообщения переведу из режима отладки в режиме информации.

nixel2007 · 2016-03-23T13:19:40Z

Версию поправил.

тесты падают из-за долгого доступа к файлу логов. =/

artbear · 2016-03-23T13:25:32Z

@nixel2007

Я еще думал убрать debug-вывод cmdline, но без него вообще сложно понять, почему не работают те или иные ключи, если вдруг кому-то захочется поменять сам файл pre-commit

Ага, убирать не нужно, полезные сообщения.

@pumbaEO

Ошибок не должно быть, успех или не успех. Но захломлять вывод неправильно.
А если не успех, тогда уже запускаем с режимом отладки и смотрим.

Тут проблема в том, что мы запускает команды git, затем автоматом стартует хук гита.
Этот хук всегда работает в одном режиме :(
Если возникают проблемы/ошибки, то повторный запуск команды git также будет выполнен в том же режиме, что и при первом запуске.

Я говорю о работе пользователей продукта.
Запуск из командной строки со спец.командой отладки не очень хотелось рекомендовать, т.к. тут нужно сделать несколько лишних, "неудобных" движений. Например, при работе с графическим клиентом Гит не очень хочется/не очень удобно переходить в терминал.
Эти движения не очень нужны обычным пользователям продукта.

Решаю эту проблему, я в свое время реализовал так - пусть будет больше полезной инфы. Все равно лог намного чаще изучается в случае проблем, чем в случае успеха.

pumbaEO · 2016-03-23T13:36:55Z

Захломляет просто вывод, имхо правильно:
Файл Путь разобрали в Путь
и все. А в случаи ошибок тогда выводить весь массив накопленных сообщений.

23 марта 2016 г., 16:25 пользователь Artur Ayukhanov <
notifications@github.com> написал:

@nixel2007 https://github.com/nixel2007

Я еще думал убрать debug-вывод cmdline, но без него вообще сложно понять,
почему не работают те или иные ключи, если вдруг кому-то захочется поменять
сам файл pre-commit

Ага, убирать не нужно, полезные сообщения.

@pumbaEO https://github.com/pumbaEO

Ошибок не должно быть, успех или не успех. Но захломлять вывод неправильно.
А если не успех, тогда уже запускаем с режимом отладки и смотрим.

Тут проблема в том, что мы запускает команды git, затем автоматом стартует
хук гита.
Этот хук всегда работает в одном режиме :(
Если возникают проблема, повторный запуск команды git также будет выполнен
в том же режиме, что и при первом запуске.

Я говорю о работе пользователей продукта.
Запуск из командной строки со спец.командой отладки не очень хотелось
рекомендовать, т.к. тут нужно сделать несколько лишних, "неудобных"
движений. Например, при работе с графическим клиентом Гит не очень
хочется/не очень удобно переходить в терминал.
Эти движения не очень нужны обычным пользователям продукта.

Решаю эту проблему, я в свое время реализовал так - пусть будет больше
полезной инфы. Все равно лог намного чаще изучается в случае проблем, чем в
случае успеха.

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#72 (comment)

З повагою Сосна Євген,
mailto:shenja@sosna.zp.ua
skype:shenjasosna
+380933897103

nixel2007 · 2016-03-23T13:41:53Z

@pumbaEO разбор в попытке?

pumbaEO · 2016-03-23T13:46:35Z

Нет, пишем все сообщения в массив сообщений, а в случаи необходимости
аварийного выхода, тогда все что накопили в этот массив, то и выводим.
Т.е. в log при уровне info не пропускаем вывод при вызове log.debug() а
запоминаем где либо и при необходимости вызываем log.ВывестиБуфер() или же
ОчиститьБуфер().

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

2016-03-23 16:41 GMT+03:00 Nikita Gryzlov notifications@github.com:

@pumbaEO https://github.com/pumbaEO разбор в попытке?

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#72 (comment)

З повагою Сосна Євген,
mailto:shenja@sosna.zp.ua
skype:shenjasosna
+380933897103

nixel2007 · 2016-03-23T13:50:11Z

@pumbaEO как ты обработаешь аварийный выход без попытки?
Я бы понял, если бы logos был встроен напрямую в oscript, и это был бы "платформенный" метод. Но у нас-то, если падает скрипт, падает все.

nixel2007 · 2016-03-23T13:50:55Z

@artbear вроде по замечаниям все. Мержим?

Обновление v8reader. xDrivenDevelopment#9

1776ef0

nixel2007 force-pushed the feature/bsl branch from d833e43 to d635978 Compare March 21, 2016 19:40

artbear self-assigned this Mar 22, 2016

artbear added the oscript-port label Mar 22, 2016

ПолучитьВерсию -> Версия

3a118a8

artbear reviewed Mar 22, 2016
View reviewed changes

artbear added this to the 2.0.0-oscript-port milestone Mar 22, 2016

nixel2007 added 2 commits March 23, 2016 00:26

Убрал старую информацию о precommit.ini. Передвинул пункты

3a4b819

Убран сам precommit.ini

faab9b0

nixel2007 mentioned this pull request Mar 22, 2016

Показ версии продукта #47

Closed

nixel2007 changed the title ~~Обновление v8reader~~ precommit1c 2.0 Mar 22, 2016

Для oscript-library больше не нужна develop-версия

cb9c2e6

Необходимая функциональность уже включена в мастер

artbear reviewed Mar 23, 2016
View reviewed changes

Версия - ПолучитьВерсию

92205e0

artbear mentioned this pull request Mar 23, 2016

Утилита не самодостаточна, нужна связь с v8unpack #46

Closed

artbear merged commit 1df5738 into xDrivenDevelopment:develop Mar 23, 2016

nixel2007 deleted the feature/bsl branch March 24, 2016 07:12

artbear mentioned this pull request Mar 24, 2016

Добавить логирование в файл #59

Closed

precommit1c 2.0 #72

precommit1c 2.0 #72

Uh oh!

Conversation

nixel2007 commented Mar 21, 2016

Uh oh!

pumbaEO commented Mar 21, 2016

Uh oh!

nixel2007 commented Mar 21, 2016

Uh oh!

nixel2007 commented Mar 21, 2016

Uh oh!

artbear commented Mar 21, 2016

Uh oh!

artbear commented Mar 21, 2016

Uh oh!

pumbaEO commented Mar 21, 2016

Uh oh!

pumbaEO commented Mar 21, 2016

Uh oh!

nixel2007 commented Mar 21, 2016

Uh oh!

nixel2007 commented Mar 21, 2016

Uh oh!

bambr1975 commented Mar 21, 2016

Uh oh!

artbear commented Mar 21, 2016

Uh oh!

nixel2007 commented Mar 21, 2016

Uh oh!

artbear commented Mar 21, 2016

Uh oh!

pumbaEO commented Mar 21, 2016

Uh oh!

nixel2007 commented Mar 21, 2016

Uh oh!

nixel2007 commented Mar 21, 2016

Uh oh!

pumbaEO commented Mar 21, 2016

Uh oh!

nixel2007 commented Mar 21, 2016

Uh oh!

nixel2007 commented Mar 21, 2016

Uh oh!

ghost commented Mar 21, 2016

Uh oh!

artbear commented Mar 22, 2016

Uh oh!

artbear commented Mar 22, 2016

Uh oh!

nixel2007 commented Mar 22, 2016

Uh oh!

artbear commented Mar 22, 2016

Uh oh!

artbear commented Mar 22, 2016

Uh oh!

artbear commented Mar 22, 2016

Uh oh!

artbear commented Mar 22, 2016

Uh oh!

artbear Mar 22, 2016

Choose a reason for hiding this comment

Uh oh!

artbear commented Mar 22, 2016

Uh oh!

artbear commented Mar 22, 2016

Uh oh!

pumbaEO commented Mar 22, 2016

Uh oh!

nixel2007 commented Mar 22, 2016

Uh oh!

nixel2007 commented Mar 22, 2016

Uh oh!

artbear Mar 23, 2016

Choose a reason for hiding this comment

Uh oh!

nixel2007 Mar 23, 2016

Choose a reason for hiding this comment

Uh oh!

artbear commented Mar 23, 2016