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
Add Chatty and feedbackd #94820
Add Chatty and feedbackd #94820
Conversation
ce6bb38
to
7338bfc
Compare
Right now Chatty is built without any libpurple plugin and only works as a xmpp client. I'm not sure if we want to do something similar to pidgin-with-plugins. What do you think? |
When this PR is merged the package |
pkgs/applications/networking/instant-messengers/chatty/default.nix
Outdated
Show resolved
Hide resolved
pkgs/applications/networking/instant-messengers/chatty/default.nix
Outdated
Show resolved
Hide resolved
5d9f980
to
bde01ec
Compare
bde01ec
to
7c71800
Compare
@jtojnar Fixed :) |
Yeah, agree that we will need |
What do you mean? |
Since it is a daemon, it probably requires installing by setting |
7c71800
to
0fda71a
Compare
Added chatty-with-plugins. Successfully tested the following ones:
|
Looks like |
Working on a module + fixing gsettings! |
@@ -70,6 +70,7 @@ mapAliases ({ | |||
casperjs = throw "casperjs has been removed, it was abandoned by upstream and broken."; | |||
catfish = xfce.catfish; # added 2019-12-22 | |||
cgmanager = throw "cgmanager was deprecated by lxc and therefore removed from nixpkgs."; # added 2020-06-05 | |||
chatty-with-plugins = chatty; # added 2020-08-16 |
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 add an legacy alias when this never existed?
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.
I'm sorry, we don't want an alias for the plugins or this file isn't the right place?
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.
I do not see a reason for adding a longer name for chatty-with-plugins
when chatty
allows setting plugins
argument too. Also this file is oriented more towards lesser aliases and can be disabled using allowAliases = false
.
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.
Please remove this. Aliases are for package renames/moves/etc.
@@ -70,6 +70,7 @@ mapAliases ({ | |||
casperjs = throw "casperjs has been removed, it was abandoned by upstream and broken."; | |||
catfish = xfce.catfish; # added 2019-12-22 | |||
cgmanager = throw "cgmanager was deprecated by lxc and therefore removed from nixpkgs."; # added 2020-06-05 | |||
chatty-with-plugins = chatty; # added 2020-08-16 |
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.
Please remove this. Aliases are for package renames/moves/etc.
Result of 1 package failed to build:
1 package built:
chatty:
|
@masipcat Are you still interested in working on this? |
Feedbackd is now packaged. Jordi did not seem to respond on their other Librem related PR so feel free to revive chatty in a new PR. |
closing in favor of #122373 |
Motivation for this change
Use this amazing application from Librem
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)