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

added net-im/skypeweb4pidgin #2

Merged
merged 1 commit into from May 12, 2015
Merged

added net-im/skypeweb4pidgin #2

merged 1 commit into from May 12, 2015

Conversation

nsavin
Copy link
Contributor

@nsavin nsavin commented Apr 25, 2015

No description provided.

@powerman
Copy link
Owner

На первый взгляд там будет конфликт по файлам (темы, иконки) с пакетом skype4pidgin.
Почему просто не добавить сборку ещё и этой либы в skype4pidgin-9999.ebuild?

@nsavin
Copy link
Contributor Author

nsavin commented Apr 26, 2015

На потестировать - вполне (ну можно добавить блокера).
Для продакшена:

  • это две абсолютно разных библиотеки, реализующих похожую функциональность
    по-разному. Так что в принципе два пакета - нормельно
  • редко будут ставить оба - мне например оно нужно, чтобы не собирать скайп
    и qt
  • я начинал делать в одном пакете - но там получается перегружено по
    use-флагам.
  • можно будет переименовать тему и иконки патчем для одного из пакетов,
    вынести тему и иконки в ресурсный пакет ну или таки сделать один пакет с
    флагами (skypeapi, skypeweb. По умолчанию наверное пока skypeapi. Ну и
    ругаться, если ниодного флага нет)

2015-04-26 0:53 GMT+01:00 Alex Efros notifications@github.com:

На первый взгляд там будет конфликт по файлам (темы, иконки) с пакетом
skype4pidgin.
Почему просто не добавить сборку ещё и этой либы в
skype4pidgin-9999.ebuild?


Reply to this email directly or view it on GitHub
#2 (comment)
.

Sincerely yours,
Nikita Savin

ICQ: 78815160
MSN: asdfgroup@hotmail.com
Yahoo!: asdfgroupnik
AOL: asdfgroup

Voice: +380633284200

@powerman
Copy link
Owner

Hi!

On Sun, Apr 26, 2015 at 01:40:04AM -0700, Nikita Savin wrote:

  • это две абсолютно разных библиотеки, реализующих похожую функциональность
    по-разному. Так что в принципе два пакета - нормельно

Их можно поставить одновременно? Если да, то достаточно одного USE-флага,
который нужен даже не столько для того, чтобы управлять компиляцией,
сколько для того, чтобы сделать условной зависимость от net-im/skype.

  • редко будут ставить оба - мне например оно нужно, чтобы не собирать скайп
    и qt
  • я начинал делать в одном пакете - но там получается перегружено по
    use-флагам.

С чего бы?

  • можно будет переименовать тему и иконки патчем для одного из пакетов,
    вынести тему и иконки в ресурсный пакет ну или таки сделать один пакет с
    флагами (skypeapi, skypeweb. По умолчанию наверное пока skypeapi. Ну и
    ругаться, если ниодного флага нет)

Три пакета из одного репо, конечно, сделать можно. Но нафига?

        WBR, Alex.

skypeweb is default protocol. To compile and install skypeapi
protocol too, provide USE-flag "skypeapi"
powerman added a commit that referenced this pull request May 12, 2015
added skypeweb support for net-im/skypeweb4pidgin
@powerman powerman merged commit 78ea9a5 into powerman:master May 12, 2015
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.

None yet

2 participants