Добавлена установка x64 дистрибутивов #10
Conversation
|
И тесты бы :) |
There was a problem hiding this comment.
И тесты бы :)
Тревис ставит какую убунту? х64? Судя по его логу - да.
Если нет, то не протестируем. Потому что по алгоритму х64 выбирается по битности системы. Принудительно не запустим.
PS: https://oscript.io/downloads/night-build/deb?bitness=x64 - о! и так можно было? )
|
|
||
| УстановщикOneScript = Новый УстановщикOneScript(); | ||
|
|
||
| Для Каждого ВерсияКУстановке Из МассивВерсийКУстановке Цикл | ||
| УстановщикOneScript.УстановитьOneScript(ВерсияКУстановке, АлиасВерсии, ОчищатьКаталогУстановки); | ||
| УстановщикOneScript.УстановитьOneScript(ВерсияКУстановке, АлиасВерсии, ОчищатьКаталогУстановки, ИспользоватьХ64); |
There was a problem hiding this comment.
Пробросил для каждой версии в цикле, а не установил параметр в класс установщика, потому что каждую версию нужно проверить на возможность установки х64
| Если ИспользоватьХ64 Тогда | ||
| ИспользоватьХ64 = ПроверитьВозможностьУстановкиX64(ВерсияКУстановке); | ||
| КонецЕсли; | ||
|
|
||
| Лог.Информация("Установка OneScript %1...", ВерсияКУстановке); | ||
|
|
||
| Если НЕ ЗначениеЗаполнено(АлиасВерсии) Тогда |
There was a problem hiding this comment.
Была идея для неустановленного алиаса добавлять суффикс х64.
Тогда версии х86 и х64 сразу устанавливались бы в разные папки просто при использовании опции х86 или без нее, а не перезатирали бы друг друга.
Но тогда нарушается логика сопоставления с версиями на сервере.
Не уверен, что автоматом так делать хорошо. Будут вопросы.
Лучше все же оставить на усмотрение пользователя такой выбор, путем установки пути-алиаса. По крайней мере чтобы он осмысленно указал 1.4.0-x64 и понимал, что сопоставление ls -r только по версии.
Возможность установить разные битности одной версии при этом есть.
можно, но это deb, а у нас zip :) хотя у зипа вроде тоже такой адрес есть. хочешь переделать? |
Запрос к zip с параметром тоже работает |
|
@240596448 @nixel2007 а что с этим зависшим ПР ? |
|
Технически все работает. Нерешенный вопрос только в предыдущем коменте.
Был 3. Я добавил 5. @nixel2007, нужны 1 и 2 ? Или 1 и 3 сделать для лучшей совместимости? |
|
@EvilBeaver @otymko подскажите, пожалуйста, какие алиасы корректно использовать для актуального сайта? может быть что-то объявлено устаревшим, а что-то наоборот новое и светящееся. |
|
@nixel2007 и те и те ссылки корректны. Пока их дерикейдить никто не собирается. Я рекомендую использовать: |
|
Ну тогда можно оставлять как есть... |
|
Это будет вливатся? В некоторых кейсах при использовании ovm выпадает OutOfMemory на х32 |
Проверил руками под виндой и wsl - все работает