Skip to content

Conversation

kryksyh
Copy link
Member

@kryksyh kryksyh commented Aug 25, 2025

Update Qt to 6.9

  • I signed CLA
  • The title of the pull request describes an issue it addresses
  • If changes are extensive, then there is a sequence of easily reviewable commits
  • Each commit's message describes its purpose and effects
  • There are no behavior changes unnecessary for the stated purpose of the PR

Recommended:

  • Each commit compiles and runs on my machine without known undesirable changes of behavior

@chinakov
Copy link
Contributor

Should fix #9251 and #9245

@kryksyh kryksyh force-pushed the qt_6.9 branch 2 times, most recently from 8cfccd5 to 33b0453 Compare August 25, 2025 12:50
- name: Install Qt
uses: jurplel/install-qt-action@v4
with:
version: '6.9.1'

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

6.9.2 has been released this week, so you might want to start with that version immediately

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will try :)

@@ -20,6 +20,8 @@ set(MODULE_TEST_LINK

muse::uicomponents
muse::ui

Qt::Quick

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it playback_tests that needs to link to Qt::Quick, or is it playback?

(Also, at MU, everything links to global, and global publicly links to Qt::Quick because of PCH. So at MU, something like this is not necessary.)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That was my first thought.
timecodemodel_tests.cpp includes timecode.h which includes QQuickItem, so it is a test.

Adding all generic modules to global makes sense. How does it affect compile time?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How does it affect compile time?

Hm, I haven't investigated that, sorry

@@ -70,9 +70,6 @@ Item {

model: toolBarModel

width: contentItem.childrenRect.width
height: contentItem.childrenRect.height

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you ever encounter similar issues elsewhere, where just removing these lines doesn't help, see also here: musescore/MuseScore#24326 (a community member figured it out for us)

@kryksyh kryksyh force-pushed the qt_6.9 branch 2 times, most recently from 9859e20 to 80de184 Compare September 2, 2025 19:42
Comment on lines +194 to +201
//! NOTE: For unknown reasons, Linux scaling for 1 is defined as 1.003 in fractional scaling.
//! Because of this, some elements are drawn with a shift on the score.
//! Let's make a Linux hack and round values above 0.75(see RoundPreferFloor)
#ifdef Q_OS_LINUX
QGuiApplication::setHighDpiScaleFactorRoundingPolicy(Qt::HighDpiScaleFactorRoundingPolicy::RoundPreferFloor);
#elif defined(Q_OS_WIN)
QGuiApplication::setHighDpiScaleFactorRoundingPolicy(Qt::HighDpiScaleFactorRoundingPolicy::PassThrough);
#endif

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This code was removed from MU in musescore/MuseScore@0728fc7 and musescore/MuseScore@dc5341c

host: 'mac'
target: 'desktop'
arch: 'clang_64'
modules: 'qt5compat qtnetworkauth qtscxml qtshadertools qtwebsockets'

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You might want to check if qtscxml is necessary for Audacity. In MU, it's only needed because one legacy widget uses Qt StateMachine, and therefore I doubt Audacity needs it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Audacity is slow to quit UI flickers when window is moved
5 participants