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
purple-discord: init at 2018-04-10 #38686
Conversation
@GrahamcOfBorg build pidgin-discord |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: pidgin-discord Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: pidgin-discord Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: pidgin-discord Partial log (click to expand)
|
"DESTDIR=$(out)" | ||
]; | ||
|
||
PKG_CONFIG_PURPLE_PLUGINDIR = "/lib/pidgin"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should probably be lib/purple-2
, see #38311.
pkgs/top-level/all-packages.nix
Outdated
@@ -17278,6 +17278,8 @@ with pkgs; | |||
plugins = []; | |||
}; | |||
|
|||
pidgin-discord = callPackage ../applications/networking/instant-messengers/pidgin-plugins/pidgin-discord { }; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are you naming it pidgin-discord
when the repo is called purple-discord
?
@jtojnar changes applied, sorry for the oversight. |
Motivation for this change
Adds new pidgin plugin that brings Discord support. In addition, add myself to the maintainers list.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)@jtojnar mind reviewing this again? Sorry about the new pull request, did not know GitHub would close it, when I forced a push to squash my commits. Also thank you for telling me about
PKG_CONFIG_${package}_${variable}
it is awesome.