Skip to content

Conversation

@nixel2007
Copy link
Member

No description provided.

nixel2007 and others added 16 commits October 12, 2017 21:57
@nixel2007
Copy link
Member Author

Пересекается с #72. Это PR - первая реализация независимой app. Дальше опыт можно распространить на gitsync/vanessa-runner/etc

@nixel2007
Copy link
Member Author

Предвижу потенциальные проблемы со сборкой/установкой не-self-contained пакетов. Надо проверить.

@nixel2007 nixel2007 changed the title Feature/self contained opm WIP: Feature/self contained opm Dec 8, 2017
@nixel2007
Copy link
Member Author

Да, есть конфликт в установке зависимостей.
Имеем: opm, имеющий зависимость от strings.
gitsync, имеющий зависимость от strings.

При установке gitsync пакет strings не скачивается из хаба, так как КэшУстановленныхПакетов находит его в opm/oscript_modules.

Если добавить ограничение каталога, по которому строится КэшУстановленныхПакетов (процедура ПолучитьУстановленныеПакеты), то гитсинк поставится нормально, а вот self-contained opm начнет затягивать свои зависимости из интернета, т.к. при установщик перестает смотреть в lib.additional, в котором прописал oscript_modules

@nixel2007
Copy link
Member Author

Конфликт режимов установки разрешен.

@nixel2007 nixel2007 requested a review from EvilBeaver December 9, 2017 13:15
@artbear
Copy link
Member

artbear commented Dec 10, 2017

@nixel2007 Поясни сценарии использования.
Как и что тестировать-то?

Процесс.Запустить();
Процесс.ОжидатьЗавершения();

Если Процесс.КодВозврата <> 0 Тогда
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

А выхлоп будет спрятан или к тому же sdout подцепится?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@EvilBeaver в тот же stdout. Это важно?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Важно чтобы совсем не пропадал, а то непонятно будет. Все ок

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Затрудняюсь сказать, что делает эта логика. Наверное она нужна

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Если обнаружены метаданные пакета (opm-metadata.xml), то выводим ещё и версию пакета

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

О, конструкторчики! )

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

А то! 👍

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Предупреждение или фэйл?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

По идее это систкмный загрузчик копируется в новый каталог. Если системного загрузчика нет, то здесь как раз будет предупреждение. Но в целом, если системного загрузчика нет, то это вообще странно и маловероятно (сам опм не запустится?)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вот этот метод не осознал. Перебираем создаваемые батники и в их каталогах создаем локальные конфиги?

Copy link
Member Author

@nixel2007 nixel2007 Dec 10, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Именно так. Каждый исполняемый файл - это энтрипоинт. А рядом с энтрипоинтом должен лежать конфиг, ведущий на оскрипт_модулес. Но ты спросил, и сейчас я подумал, что надо добавить условие, что устанавливаемое приложение имеет эти самые оскрипт_модулес, иначе эта процедура все сломает.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

проверил, файлы создаются только при установке через -l

@nixel2007
Copy link
Member Author

@artbear при чем тут тестирование?

@nixel2007
Copy link
Member Author

@artbear а, понял о чем ты. Собрать, протестировать установку и работу других пакетов/приложений

@nixel2007 nixel2007 merged commit 89e185d into develop Dec 11, 2017
@nixel2007 nixel2007 deleted the feature/self-contained-opm branch December 11, 2017 11:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants