search

cucumber-sp / yandex-music-linux

Linux client of Yandex Music
Other
345 stars 8 forks source link

Change naming in Nix #16

Closed ein-shved closed 8 months ago

ein-shved commented 8 months ago

How does the yandex?music.exe named in windows? We have two styles of naming here:

Let's agree here on correct naming.

cucumber-sp commented 8 months ago

yandexmusic предпочтительный в большинстве дистрибутивов, использую для deb и arch сейчас.

ein-shved commented 8 months ago

Это ваше решение, или разработчиков Я.Музыки?

cucumber-sp commented 8 months ago

Это ваше решение, или разработчиков Я.Музыки?

Моё решение) В винде вообще YandexMusic.exe

ein-shved commented 8 months ago

Ну

так что предлагаю синхронизировать нейминг с апстримом.

cucumber-sp commented 8 months ago

Ну

  • во-первых в винде нейминг выглядит лучше. Это конечно вусовщина, но подкрепленная тем, что имя yandexmusic не соответствует ни одному из широко распространенных coding-syles
  • во-вторых если вы используете другой нейминг, то вы автоматически на сможете обеспечить один нейминг во всех дистрибутивах, т.к. на Винду повлиять не сможете

так что предлагаю синхронизировать нейминг с апстримом.

вообще стандартная практика по крайней мере для пакетов в AUR. Но если хотите переименовать в yandex-music и займетесь этим то я не против, но нужно сделать это везде

ein-shved commented 8 months ago

Не, я бы не стал переименовывать в yandex-music я бы переименовал в YandexMusix, как это сделано в апстриме.

cucumber-sp commented 8 months ago

Не, я бы не стал переименовывать в yandex-music я бы переименовал в YandexMusix, как это сделано в апстриме.

ну это уже неприемлемый нейминг для AUR пакетов да и для deb наверное тоже. Не вижу причин не оставить yandexmusic

ein-shved commented 8 months ago

@cucumber-sp хозяин-барин, на том и порешим. На днях докину патч в nix

cucumber-sp commented 8 months ago

@cucumber-sp хозяин-барин, на том и порешим. На днях докину патч в nix

👌

ein-shved commented 8 months ago

Done with #19.