-
Notifications
You must be signed in to change notification settings - Fork 33
Semver #235
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Semver #235
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,5 @@ | ||
#Использовать logos | ||
#Использовать semver | ||
|
||
Перем Лог; | ||
|
||
|
@@ -33,7 +34,7 @@ | |
ТребуемаяВерсияМанифеста, | ||
ТекущаяВерсияМанифеста | ||
); | ||
Если РаботаСВерсиями.СравнитьВерсии(ТребуемаяВерсияМанифеста, ТекущаяВерсияМанифеста) > 0 Тогда | ||
Если Версии.СравнитьВерсии(ТребуемаяВерсияМанифеста, ТекущаяВерсияМанифеста) > 0 Тогда | ||
nixel2007 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
ТекстСообщения = СтрШаблон( | ||
"Ошибка установки пакета <%1>: Обнаружена устаревшая версия opm. | ||
|Требуемая версия: %2 | ||
|
@@ -61,7 +62,15 @@ | |
СистемнаяИнформация = Новый СистемнаяИнформация; | ||
ВерсияСреды = СистемнаяИнформация.Версия; | ||
Лог().Отладка("ПроверитьВерсиюСреды: Перед вызовом СравнитьВерсии(ЭтаВерсия = <%1>, БольшеЧемВерсия = <%2>)", ТребуемаяВерсияСреды, ВерсияСреды); | ||
Если РаботаСВерсиями.СравнитьВерсии(ТребуемаяВерсияСреды, ВерсияСреды) > 0 Тогда | ||
|
||
Попытка | ||
// До приведения версии движка к семверу | ||
РезультатСравнения = РаботаСВерсиями.СравнитьВерсии(ТребуемаяВерсияСреды, ВерсияСреды); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ночник приведен к семверу начиная с 2.0.0-rc.7+591, однако старые версии 1.x остаются четырехзначными There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Всё так, именно по этому тут старый вариант в попытке, а в исключении semver There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Может просто тогда нафиг не нужен семвер, я про это? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Как вариант, семвер делать в попытке, а в исключении старый вариант. И пометить его как устаревший There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Старый парсер на 4 компонента раcсчитан и падает на "2.0.0.rc", а semver не падает а корректно разбирает и сравнивает, кроме того с семвером я смогу в ванюните указать с какого RC он будет работать и это будет корректно проверятся например. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||
Исключение | ||
РезультатСравнения = Версии.СравнитьВерсии(ТребуемаяВерсияСреды, ВерсияСреды); | ||
КонецПопытки; | ||
|
||
Если РезультатСравнения > 0 Тогда | ||
ТекстСообщения = СтрШаблон( | ||
"Ошибка установки пакета <%1>: Обнаружена устаревшая версия движка OneScript. | ||
|Требуемая версия: %2 | ||
|
Uh oh!
There was an error while loading. Please reload this page.