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 added this to the release 1.0 milestone Mar 1, 2017
@lcpz lcpz force-pushed the master branch 2 times, most recently from cd4f448 to 794f5d3 Compare September 11, 2017 09:08
@lcpz lcpz force-pushed the master branch 2 times, most recently from b14a2c8 to 055e663 Compare October 5, 2017 09:32
@lcpz lcpz force-pushed the master branch 5 times, most recently from d2fd3d5 to 24ca585 Compare February 16, 2018 16:52
@lcpz lcpz force-pushed the master branch 7 times, most recently from de63ab6 to f589709 Compare October 30, 2018 08:08
@lcpz lcpz force-pushed the master branch 2 times, most recently from 2426659 to 8112df6 Compare November 24, 2018 15:03
@lcpz lcpz force-pushed the master branch 2 times, most recently from 1291b1c to 012c6b6 Compare April 9, 2019 11:46
@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