public inbox for [email protected]  
help / color / mirror / Atom feed
From: Dave Page <[email protected]>
To: Neel Patel <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Cc: Akshay Joshi <[email protected]>
Cc: Ashesh Vashi <[email protected]>
Cc: Joao Pedro De Almeida Pereira <[email protected]>
Cc: Robert Eckhardt <[email protected]>
Subject: Re: Showstopper desktop runtime issue
Date: Thu, 22 Mar 2018 10:22:05 +0000
Message-ID: <CA+OCxowUBKcCAt8hFginMP=_S0V033gM7oBUPniO4EsCsGx97w@mail.gmail.com> (raw)
In-Reply-To: <CACCA4P1eCtXGJZic61dz+YqiXqVAO=c6hgU57_qfnKVKECN8sQ@mail.gmail.com>
References: <CA+OCxoyjbMa6EK=Wu+d56S7teJ7YRVP2sY73BbYFwiRZAwUetA@mail.gmail.com>
	<CACCA4P1eCtXGJZic61dz+YqiXqVAO=c6hgU57_qfnKVKECN8sQ@mail.gmail.com>

On Thu, Mar 22, 2018 at 10:19 AM, Neel Patel <[email protected]>
wrote:

> Hi,
>
> On Thu, Mar 22, 2018 at 3:19 PM, Dave Page <[email protected]> wrote:
>
>> All,
>>
>> As you know, the 3.0 release is currently on hold as we discovered late
>> yesterday that the re-vamped desktop runtime will not run on Gnome 3.26 and
>> later. This is because the GTK project, and later Gnome, have removed
>> support for the System Tray on which the new runtime relies.
>>
>> They have replaced it with a notification mechanism, however this doesn't
>> really meet our needs as what we want is a place (the tray icon) to attach
>> a menu to control the pgAdmin server; we don't really use notifications as
>> such.
>>
>> I see a number of possible ways around this:
>>
>> 1) Return to the previous runtime. I think this is at best a short-term
>> solution, as the re-visited Annulen version of the QtWebKit seems to be
>> getting little attention at the moment, and this would re-introduce many
>> known bugs caused by WebKit.
>>
>> 2) Re-work the current runtime code to remove the tray icon, and utilise
>> desktop/start menu items to signal the running instance to show the logs,
>> configure the server, exit etc. This should work, but will be kinda klunky.
>>
>> 3) Put effort into polishing Joao's Electron based runtime. This might be
>> a good long term solution as it would remove the need to have any C++ code
>> of our own, and might allow us to use Electron's update mechanism to do
>> software updates. The downsides are that we would lose support for dockable
>> tabs (new windows only), and it wouldn't work on CentOS/RHEL 6 which we
>> currently support.
>>
>> Thoughts and comments please folks? How do we want to proceed? I'm
>> currently leaning towards 2 for v3, and possibly moving to 3 in the long
>> term.
>>
>
> As per our previous efforts on Qt's web rendering framework, I am not in
> favor of adding it again as it add lots of dependency and complexity.
> I would suggest to go for option 2 - To re-work the current runtime code
> and find out how to get already running instance of pgadmin4.
>

We have a shared memory interlock as well as temp/lock files that encode
the executable path into their name that handles that already, though it
doesn't seem to be working quite as expected when you run from a Mac
Appbundle (which Akshay is working on). The issue would be how we signal
the running instance and tell it to do something, e.g. exit.

-- 
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


view thread (16+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected], [email protected], [email protected]
  Subject: Re: Showstopper desktop runtime issue
  In-Reply-To: <CA+OCxowUBKcCAt8hFginMP=_S0V033gM7oBUPniO4EsCsGx97w@mail.gmail.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox