Castro-Fidel / PortWINE

MIT License
462 stars 33 forks source link

Check PW_NO_AUTOCREATE_SHORTCUT var #314

Closed zorn-v closed 2 weeks ago

zorn-v commented 2 weeks ago

Например VKPlayLoader каждый раз при запуске создает ярлык на рабочем столе и постоянно после закрытия спрашивает про создание ярлыка.

Htylol commented 2 weeks ago

@zorn-v Вроде ведь в devel пофиксили это уже, там не смотрел? Коммит у тебя в master, а не в devel. Да и сам коммит, автошорткаты наоборот сделаны, чтобы после установки приложения по лнк предлагалось создать ярлык, чтобы самому не лазить по директориям, искать и т.д., своим коммитом, ты просто отключаешь весь этот функционал, да и всё. Точнее этой переменной PW_NO_AUTOCREATE_SHORTCUT вообще нет нигде, смысл от неё

zorn-v commented 2 weeks ago

Вроде ведь в devel пофиксили это уже

Ага, посмотрел. https://github.com/Castro-Fidel/PortWINE/blob/4b27dd3670d22e26f575459c898a5fd5cb108515/data_from_portwine/scripts/functions_helper#L4874

ты просто отключаешь весь этот функционал, да и всё

Да, просто отключаю если установлена переменная (которую я собирался установить в user.conf) :smile: Для VKGameLoader можно в portwine_db файле ее установить например, а не перебирать все варианты прямо в скрипте.

Еще как вариант add_in_stop_portwine запускать перед pw_auto_create_shortcut, а не после и в ppdb файле удалять ярлык в этой функции, но я пошел более простым путем. https://github.com/Castro-Fidel/PortWINE/blob/4b27dd3670d22e26f575459c898a5fd5cb108515/data_from_portwine/scripts/functions_helper#L1094-L1095

zorn-v commented 2 weeks ago

Для VKGameLoader можно в portwine_db файле ее установить например, а не перебирать все варианты прямо в скрипте.

Хотя не поможет. При запуске другой игры (без этой переменной) опять вылезет предложение создать ярлык. Надо проверять уже после удаления lnk

zorn-v commented 2 weeks ago

При запуске другой игры (без этой переменной) опять вылезет предложение создать ярлык

И кстати с тем фиксом что в devel ветке, будет тоже самое

Htylol commented 2 weeks ago

@zorn-v ну дак это специально сделано, когда при установке игры или программы, выбираешь создать ярлык на рабочем столе, он создаёт lnk, и по завершению portproton предлагает создать ярлык. И при каждом новом запуске portproton, удаляет все lnk которые могли бы остаться

Htylol commented 2 weeks ago

@zorn-v просто некоторые лаучнеры сами по себе без галок тянут lnk, в остальном функционал создан чтобы по галочке создавались ярлыки тоже

zorn-v commented 2 weeks ago

Да это все понятно, но фикс в devel не рабочий - там выход из функции до того как lnk удаляется. А это значит, что при завершении любого другого ехе, который не подходит под регулярку, опять предложит создать ярлык для VKGameLoader - lnk то остался на рабочем столе.

Да и странно проверять жестко прописанной регуляркой на имя файла.

zorn-v commented 2 weeks ago

Создал другой PR, чтобы была понятна мысля https://github.com/Castro-Fidel/PortWINE/pull/315

zorn-v commented 2 weeks ago

И при каждом новом запуске portproton, удаляет все lnk которые могли бы остаться

Хм, а вот это вообще не очевидно. Как то пропустил.