search

oscript-library / opm

Пакетный менеджер OneScript
Apache License 2.0
66 stars 31 forks source link

[WIP] Исправил ошибку, которая приводила к удалению установленного пакета при его неудачном обновлении #185

Closed ovcharenko-di closed 1 year ago

ovcharenko-di commented 4 years ago

close #176

khorevaa commented 4 years ago

@ovcharenko-di @nixel2007 @nixel2007 Коллеги - мне видится, что мы делаем заплатку. На мой взгляд правильное поведение - это установка нового пакета в промежуточный каталог ( вместе с тем зависимостями, что добавляются или обновляются), а только потом уже перенос в рабочий "либ".

Хотя может я просто слишком замороченный :)

ovcharenko-di commented 4 years ago

@ovcharenko-di @nixel2007 @nixel2007 Коллеги - мне видится, что мы делаем заплатку. На мой взгляд правильное поведение - это установка нового пакета в промежуточный каталог ( вместе с тем зависимостями, что добавляются или обновляются), а только потом уже перенос в рабочий "либ".

Хотя может я просто слишком замороченный :)

Я именно так и планировал дорабатывать этот PR! Заодно посмотрю как в других менеджерах пакетов решается подобная проблема.

EvilBeaver commented 4 years ago

Заодно посмотрю как в других менеджерах пакетов решается подобная проблема.

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

EvilBeaver commented 4 years ago

@ovcharenko-di мы же ждем доработки, да? Можешь в таком случае WIP поставить в заголовке?

EvilBeaver commented 2 years ago

@ovcharenko-di подниму?

ovcharenko-di commented 2 years ago

@EvilBeaver спасибо за пинг, я уже и забыл об этом PR! Я постараюсь еще поработать над ним.