Skip to content
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

Новое событие ПередУстановкой внутри packagedef #134

Closed
1 of 2 tasks
artbear opened this issue Aug 21, 2018 · 7 comments
Closed
1 of 2 tasks
Milestone

Comments

@artbear
Copy link
Member

artbear commented Aug 21, 2018

Существующих событий перестало хватать
Опишу свой кейс:
пакет vanessa.add, все о нем знают.
этот пакет внутри себя имеет набор каталогов разной вложенности с набором epf-файлов (внешних обработок)
и выясняется проблема - при обновлении новой версии add поверх существующей, старой версиии через opm из архива поставки происходит установка всех файлов из пакета.

Но старые файлы не удаляются :(
в итоге файлы двух версий смешиваются и получается баг обновления.
Вот ишуз vanessa-opensource/add#291

Существующего события ПриУстановке для решения проблемы недостаточно, т.к. оно срабатывает уже после извлечения всех файлов из поставки и установки их в каталог установки, т.е. когда все файлы перемешались.

и в этом событии нет доступа к архиву пакета, так что даже сверить файлы с источником нельзя :(

Предлагаю:

  • добавить новое событие ПередУстановкой(Знач КаталогУстановкиПакета, Знач АрхивПакета, СтандартнаяОбработка) Экспорт
    которое поможет выполнять действие перед установкой

  • Также дополнительно будет полезен спец.метод Описание.ОчиститьПередУстановкой(МаскаФайлов, ВключитьПодкаталоги)
    для простого решения указанной задачи без написания доп.обработки

@khorevaa @EvilBeaver @nixel2007 Могу сделать, если нет возражений.

@artbear
Copy link
Member Author

artbear commented Aug 30, 2018

Выполнено

@artbear artbear closed this as completed Aug 30, 2018
@artbear
Copy link
Member Author

artbear commented Aug 30, 2018

@khorevaa @EvilBeaver @nixel2007 Выпустим релиз 1.0.0 с переводом opm на cli и моими последними правками ?

давно пора версию 1.0.0 выпускать

@nixel2007
Copy link
Member

nixel2007 commented Aug 30, 2018 via email

@khorevaa
Copy link
Member

@artbear ДУмаю можно выпустить! Хотя я бы сменить формат packagedef перед версией 1.0.0 иначе потом совместимость и прочие! А тут cli и так фактически вводит другую строку вызова

@EvilBeaver
Copy link
Member

Ох, доберусь я до вас!

В смысле, никак не доберусь подключиться к opm, хотя и ключевой продукт

@artbear
Copy link
Member Author

artbear commented Sep 6, 2018

@khorevaa @EvilBeaver @nixel2007 Релиз будем выпускать??
0.15 ??
событие ПередУстановкой очень нужно.

@khorevaa
Copy link
Member

khorevaa commented Sep 6, 2018

@artbear Я не против!

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

No branches or pull requests

4 participants