Mate command applet problem #163

Open
fennectech opened this Issue Nov 2, 2015 · 15 comments

Projects

None yet

4 participants

@fennectech
fennectech commented Nov 2, 2015 edited

I use the mate command applet to fetch my ip from ifconfig.me however it fowls up the applet and locks it up sometimes bringing the panel down with it.
setps to reproduce
1 set command to curl ifconfig.me

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/27904355-mate-command-applet-problem?utm_campaign=plugin&utm_content=tracker%2F517519&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F517519&utm_medium=issues&utm_source=github).
@fennectech

https://screencloud.net/v/bT9l at screenshot by a friend who reproduced it on their machine

@flexiondotorg
Member

@stefano-k could you look at this one please?

@monsta
Member
monsta commented Dec 8, 2015

The command is executed synchronously, so it's no wonder such things happen. However, there's no appropriate g_spawn_* API for running a process and getting its output.

I think we can use async functions of GSubprocess instead - but then we'll need to decide on setting the minimum required GLib version to 2.40.

@monsta monsta closed this Dec 8, 2015
@monsta monsta reopened this Dec 8, 2015
@monsta
Member
monsta commented Dec 8, 2015

Oops

@stefano-k
Member

Or keep g_spawn_* for old glib versions

@monsta
Member
monsta commented Dec 9, 2015

We can't keep g_spawn_command_line_sync as it blocks, and I couldn't find any async function that would invoke a callback on read completion...

@monsta monsta added the confirmed label Mar 30, 2016
@fennectech

has this been patched yet?

@monsta
Member
monsta commented Apr 18, 2016

No, I didn't have enough time to finish it for 1.14. Will do for 1.16.

@fennectech

okay thanks there is also an issue on one of my machines that i think
may be related i have another website that i poll and the command
applet lags and appears frozen for seconds to minutes usually when the
network is down this occors. is it possable to make an applet that just
has one purpous aquiring and displaying the public ip's?

On Mon, Apr 18, 2016 at 6:26 AM, monsta notifications@github.com wrote:

No, I didn't have enough time to finish it for 1.14. Will do for 1.16.


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#163 (comment)

Fennec

@monsta
Member
monsta commented Apr 19, 2016

We have enough work here already, but any 3rd party developers are free to write any applets.

@ghost
ghost commented Aug 24, 2016

@monsta Will you have time to work on that or have already started? If not I would like to implement that. I would just use g_shell_parse_argv () and g_spawn_async_with_pipes ().

@ghost
ghost commented Aug 24, 2016

Ah, wait, the problem is to get to know when it finishes, right? So I would actually use GSubprocess then. But I won't start until I got to know whether nobody is working on that.

@monsta
Member
monsta commented Aug 24, 2016

Well, I need to find some time to get back to it and finish it. Hope to get it done before 1.16 release.
You can check the current (unfinished) work at 892025e.

@ghost
ghost commented Aug 25, 2016

@monsta Ok, so you already do it, I just wanted to ask.

@monsta
Member
monsta commented Sep 22, 2016

Sorry, didn't make it in time 😞
Hoping for 1.18.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment