blocktronics / moebius

Modern ANSI & ASCII Art Editor
https://blocktronics.github.io/moebius/
Apache License 2.0
731 stars 40 forks source link

debian package requires libappindicator3-1 #210

Open rfb opened 2 years ago

rfb commented 2 years ago

libappindicator3-1 has been removed from debian bullseye. I guessing what's really needed is an upgrade of electron?

andyherbert commented 2 years ago

Updating to the latest version of electron is problematic as the code relies heavily on the remote object provided by the api in the renderer thread. At some point this was deprecated as a security concern, which is probably a wise move, but represents a major effort to update this code base. I’m personally motivated to update this repo because bringing electron up to date would also provide native support for Apple silicon based macs, doing a quick investigation brings up an electron remote package on npm which might provide an easy workaround. I’m writing this comment in case anyone reads and implements a fix before I get around to starting it.

andyherbert commented 2 years ago

Related stack overflow thread.

https://stackoverflow.com/questions/37884130/electron-remote-is-undefined

andyherbert commented 2 years ago

I spent a couple of hours on this not expecting to get anywhere and updated to the latest version of electron with fairly minimal changes, sadly building on Apple Silicon mandates digital signing via Apple's Developer Program. On the up side potentially this issue is resolved.