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

Support for unix socket in MPD widget #316

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

martindube
Copy link

Hi,
This is a small suggestion to support Unix Socket in MPD.
Thank you,

@lcpz
Copy link
Owner

lcpz commented Feb 17, 2017

This introduces GNU netcat as dependency, which is not part of every distro core installation.

Please, investigate if is there a Pure Lua/Awesome API implementation.

Also, note that if you want to use sockets, this recipe might be of interest.

@Veratil
Copy link
Contributor

Veratil commented Feb 24, 2017

An alternative to netcat I've seen is piping into /dev/tcp/.... Something like this:
echo 'blah blah' > /dev/tcp/address/port

@unode
Copy link
Contributor

unode commented Feb 25, 2017

Maybe this? - references same code that @copycat-killer linked above.

@lcpz
Copy link
Owner

lcpz commented Feb 25, 2017

@unode Another thing I want to do for lain first release is to get rid of curl in mpd, imap and weather, in order to remove any external dependency.

Eventually, this PR might be overridden, or pushed, along with the curl-dependent code, as a recipe on the awesome site.

@unode
Copy link
Contributor

unode commented Feb 25, 2017

@copycat-killer by "get rid" I hope you mean replacing them with lua native api's such as luacurl.
If you remove the widgets people will miss them :)

@lcpz
Copy link
Owner

lcpz commented Feb 25, 2017

@unode By "get rid" I'm referring to curl.

I want to replace it with GLib via lua-lgi (which is an Awesome dependency).

@lcpz lcpz force-pushed the master branch 2 times, most recently from a5b407f to a955bbb Compare March 30, 2021 10:14
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.

4 participants