Meta.Rectangle is deprecated, use Mtk.Rectangle instead

Bug #2034664 reported by Daniel van Vugt
34
This bug affects 4 people
Affects Status Importance Assigned to Milestone
gnome-shell-extension-tiling-assistant (Ubuntu)
Status tracked in Mantic
Mantic
Fix Released
Undecided
Jeremy Bícha
gnome-shell-extension-ubuntu-dock (Ubuntu)
Status tracked in Mantic
Mantic
Fix Committed
Undecided
Marco Trevisan (Treviño)

Bug Description

[ Impact ]

The gnome-shell 45.rc log fills with:

Meta.Rectangle is deprecated, use Mtk.Rectangle instead

[ Test case ]

Run ubuntu with ubuntu extensions enabled (only).
journalctl /usr/bin/gnome-shell should not contain any reference to Meta.Rectangle deprecation

[ Regression potential ]

Introspected types are not defined and so extensions may try to use undefined code.

Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in gnome-shell-extension-tiling-assistant (Ubuntu):
status: New → Confirmed
Changed in gnome-shell-extension-ubuntu-dock (Ubuntu):
status: New → Confirmed
Revision history for this message
Daniel van Vugt (vanvugt) wrote :
Changed in gnome-shell-extension-ubuntu-dock (Ubuntu):
assignee: nobody → Marco Trevisan (Treviño) (3v1n0)
status: Confirmed → Fix Committed
Revision history for this message
Daniel van Vugt (vanvugt) wrote :
Changed in gnome-shell-extension-tiling-assistant (Ubuntu):
assignee: nobody → Marco Trevisan (Treviño) (3v1n0)
status: Confirmed → In Progress
Revision history for this message
Ubuntu QA Website (ubuntuqa) wrote :

This bug has been reported on the Ubuntu ISO testing tracker.

A list of all reports related to this bug can be found here:
https://iso.qa.ubuntu.com/qatracker/reports/bugs/2034664

tags: added: iso-testing
Changed in gnome-shell-extension-tiling-assistant (Ubuntu):
status: In Progress → Fix Committed
Jeremy Bícha (jbicha)
Changed in gnome-shell-extension-tiling-assistant (Ubuntu):
assignee: Marco Trevisan (Treviño) (3v1n0) → Jeremy Bícha (jbicha)
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package gnome-shell-extension-tiling-assistant - 44-1ubuntu1

---------------
gnome-shell-extension-tiling-assistant (44-1ubuntu1) mantic; urgency=medium

  * Merge with debian, remaining changes:
    - Rename extension to gnome-shell-extension-ubuntu-tiling-assistant
    - debian/control: Suggests gnome-shell-extension-prefs
    - d/p: tilingWindowManager: Override window constraints style

gnome-shell-extension-tiling-assistant (44-1) experimental; urgency=medium

  * New upstream release
    - Officially support GNOME Shell 45 (Closes: #1052109) (LP: #2034664)
  * debian/control: Bump minimum GNOME Shell to 45 to match metadata.json
  * Drop all patches except locale patch

gnome-shell-extension-tiling-assistant (41-3) unstable; urgency=medium

  * debian/gbp.conf: Use debian/unstable branch
  * debian/control: Do not support gnome-shell 45

gnome-shell-extension-tiling-assistant (41-2) unstable; urgency=medium

  * Upload to unstable. (Closes: #1041580)
  * debian/control.in:
      - Bumped Standards-Version to 4.6.2.
      - Adjusted gnome-shell versions in Depends field in package stanza to
        support GNOME 43 to 45.
  * debian/copyright: updated packaging data.
  * debian/patches/series: commented out GNOME 45 support patches for now.

 -- Jeremy Bícha <email address hidden> Mon, 25 Sep 2023 11:46:07 -0400

Changed in gnome-shell-extension-tiling-assistant (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
corrado venturini (corradoventu) wrote (last edit ):

I still have the problem
corrado@corrado-n17-mm-1004:~$ apt policy gnome-shell-extension-tiling-assistant
gnome-shell-extension-tiling-assistant:
  Installed: (none)
  Candidate: (none)
  Version table:
corrado@corrado-n17-mm-1004:~$ apt policy gnome-shell-extension-ubuntu-dock
gnome-shell-extension-ubuntu-dock:
  Installed: 87ubuntu1
  Candidate: 87ubuntu1
  Version table:
 *** 87ubuntu1 500
        500 http://archive.ubuntu.com/ubuntu mantic/main amd64 Packages
        500 http://archive.ubuntu.com/ubuntu mantic/main i386 Packages
        100 /var/lib/dpkg/status
corrado@corrado-n17-mm-1004:~$
corrado@corrado-n17-mm-1004:~$ apt policy gnome-shell-extension-ubuntu-tiling-assistant
gnome-shell-extension-ubuntu-tiling-assistant:
  Installed: 44-1ubuntu1
  Candidate: 44-1ubuntu1
  Version table:
 *** 44-1ubuntu1 500
        500 http://archive.ubuntu.com/ubuntu mantic/main amd64 Packages
        500 http://archive.ubuntu.com/ubuntu mantic/main i386 Packages
        100 /var/lib/dpkg/status
corrado@corrado-n17-mm-1004:~$

Revision history for this message
Daniel van Vugt (vanvugt) wrote :

I know...

$ grep -r Meta.Rectangle /usr/share/gnome-shell/extensions/
/<email address hidden>/appIcons.js: const rect = new Meta.Rectangle();

Changed in gnome-shell-extension-ubuntu-dock (Ubuntu):
status: Fix Committed → Triaged
status: Triaged → Fix Committed
description: updated
Revision history for this message
Utkarsh Gupta (utkarsh) wrote :

Hey, gnome-shell-extension-ubuntu-dock is not something that we (release team) are planning to accept ahead of Mantic release, so we're going to let it be in the unapproved queue and consider this for a 0-day SRU.

Revision history for this message
corrado venturini (corradoventu) wrote :

gnome-shell-extension-ubuntu-dock is installed in Ubuntu Full install in the repository mantic/main so you (relese team) should explain...

https://cdimages.ubuntu.com/daily-live/pending/mantic-desktop-amd64.manifest

corrado@corrado-n4-mm-1005:~/Videos$ apt policy gnome-shell-extension-ubuntu-dock
gnome-shell-extension-ubuntu-dock:
  Installed: 87ubuntu1
  Candidate: 87ubuntu1
  Version table:
 *** 87ubuntu1 500
        500 http://archive.ubuntu.com/ubuntu mantic/main amd64 Packages
        500 http://archive.ubuntu.com/ubuntu mantic/main i386 Packages
        100 /var/lib/dpkg/status
corrado@corrado-n4-mm-1005:~/Videos$ apt show gnome-shell-extension-ubuntu-dock
Package: gnome-shell-extension-ubuntu-dock
Version: 87ubuntu1
Priority: optional
Section: gnome
Origin: Ubuntu
Maintainer: Marco Trevisan <email address hidden>
Bugs: https://bugs.launchpad.net/ubuntu/+filebug
Installed-Size: 913 kB
Depends: gnome-shell (<< 46~), gnome-shell (>= 45~), dconf-gsettings-backend | gsettings-backend
Replaces: gnome-shell-extension-dashtodock
Homepage: https://github.com/micheleg/dash-to-dock/blob/ubuntu-dock/README.md
Task: ubuntu-desktop-minimal, ubuntu-desktop, edubuntu-desktop-gnome
Download-Size: 117 kB
APT-Manual-Installed: no
APT-Sources: http://archive.ubuntu.com/ubuntu mantic/main amd64 Packages
Description: Ubuntu Dock for GNOME Shell
 A dock for the Gnome Shell, default Ubuntu experience.
 .
 This extension is a modified version of Dash To Dock with different defaults.
 Dash To Dock can be installed to replace it and give more (but unsupported)
 configuration options.

corrado@corrado-n4-mm-1005:~/Videos$

Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

Corrado, I think this is not a big deal. The deprecation warning is just a warning, so everything should still work even when using Meta instead of Mtk.

This is an update that we can also get at later point, so I think Utkarsh is fine to wait (and I also was expecting this to happen, thus I adapted the bug report to the SRU template).

description: updated
Revision history for this message
corrado venturini (corradoventu) wrote :

Yes The deprecation warning is just a warning, but I have the log full of warning messages and i want debug a problem my journalctl is almost unusable

Revision history for this message
Daniel van Vugt (vanvugt) wrote (last edit ):

Only if a message happens frequently will it be a big performance problem. Because then it's triggering a filesystem journal flush every 5 seconds, which blocks in the log write, which blocks the shell, which is seen as screen stutter.

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

This bug is ready for verification. Due to the status mantic was in when the package was accepted, the tooling failed to add the template comment to this bug, and add the tags, and I did all of that manually now.

tags: added: verification-needed verification-needed-mantic
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.