R devel - Mar 2021 - [Solved] Possible x11 window manager window aggregation under one icon?

>>>>> Dirk Eddelbuettel 
>>>>>     on Tue, 23 Mar 2021 12:36:47 -0500 writes:
    > It all works now, thanks mostly to some very detailed reading of the
specs by
    > Ivan.  In short, I made the following changes:

    > - add the missing WM hint to the .desktop file we install
    > - add the svg logo as 'scalable'
    > - create a new (square) 48x48 default png logo from the new one
    > - deactivate yesterday's patch

    > and it is all good now.  Duncan's rgl windows aggregate under the
item, as do
    > the standard R x11 devices.  I will try to attach a small screenshot,
we'll
    > see how mailman likes it.  Martin should still be able to get the old
(and to
    > me, buggy) behaviour back by removing the one key line from the
.desktop
    > file, if his Fedora environment ever updates as I plan.

    > For anyone on Ubuntu 20.10, updated binaries are in my PPA, see
    >
https://launchpad.net/~edd/+archive/ubuntu/misc/+packages?field.name_filter=r-base&field.status_filter=published&field.series_filter=groovy

    > Changelog entries below, these have not been committed to Debian's
git yet
    > but I think I will activate this for R 4.0.5 next week (and test it til
    > then). Screenshot attached below too.

    > ------ changelog for these three test builds follows 
-----------------------

    > r-base (4.0.4-1.2010.3) groovy; urgency=medium

    > * debian/r-base-core.dirs: Also create the directory
    > usr/share/icons/hicolor/scalable/apps for the svg logo

    > -- Dirk Eddelbuettel <edd at debian.org>  Tue, 23 Mar 2021
11:05:17 -0500

    > r-base (4.0.4-1.2010.2) groovy; urgency=medium

    > * icon-class-patch/R.desktop: Add 'StartupWMClass=R_x11'
    > * icon-class-patch/rlogo_icon.OLD.png.mpack: Renamed old icon
    > * icon-class-patch/rlogo_icon.png.mpack: New 48x48 png from svg
    > * icon-class-patch/Rlogo.svg: Copy of official logo
    > * debian/rules: Also install Rlogo.svg in 'scalable' icons dir

    > * debian/patches/series: Deactivate unneeded grouping patch 

    > -- Dirk Eddelbuettel <edd at debian.org>  Tue, 23 Mar 2021
10:27:43 -0500

    > r-base (4.0.4-1.2010.1) groovy; urgency=medium

    > * PPA build on Ubuntu 20.10 "groovy"
    > * src/modules/X11/devX11.c: Apply patch by Ivan Krylov (posted to
r-devel
    >   on 2021-03-22) enabling grouping of x11 plot device windows

    > -- Dirk Eddelbuettel <edd at debian.org>  Mon, 22 Mar 2021
21:33:09 -0500

Thank you, Dirk for raising the issue, providing the nice summary,
and again to Ivan for his patches and Duncan for testing and comments

I've checked the first of Ivan's patches -- IIRC the one Dirk
now also is going to patch the Debian/Ubuntu/... R-base packages with.
The code is nice, short (but not too short), partly self explainable,
and it also works fine under the Fedora_32--current version of
Gnome.

For this reason I've committed to R (the trunk, i.e., R-devel,
for R 4.1.0 in a month or so)  in svn rev 80110.

However we definitely want a stable R 4.0.5 with basically only
a bug fix of the character/UTF-8/.. problem so R 4.0.x itself
will surely not get such a patch.

Best regards,
Martin

On 24 March 2021 at 10:30, Martin Maechler wrote:
| For this reason I've committed to R (the trunk, i.e., R-devel,
| for R 4.1.0 in a month or so)  in svn rev 80110.

I just saw that via the (still extremely helpful) RSS feed of SVN changes and
then pulled.

You may have missed that Ivan concluded, and I followed, that the _patch is
not needed_.  All one needs is to adjust the .desktop file. I posted my full
changelog from the Debian package (of which I currently run a test build on
two Ubuntu machines using the binaries from Launchpad I pointed to).

So in that sense I think r80110 may wants to be reverted.

Dirk

-- 
https://dirk.eddelbuettel.com | @eddelbuettel | edd at debian.org