Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: Incorrect use of desktopFileName #7581

Closed
4 of 8 tasks
kleifgch opened this issue Nov 27, 2024 · 1 comment · Fixed by #7699
Closed
4 of 8 tasks

[Bug]: Incorrect use of desktopFileName #7581

kleifgch opened this issue Nov 27, 2024 · 1 comment · Fixed by #7699

Comments

@kleifgch
Copy link
Contributor

⚠️ Before submitting, please verify the following: ⚠️

Bug description

On my system, the Nextcloud client emits the following warning shortly after being started:

QGuiApplication::setDesktopFileName: the specified desktop file name ends with .desktop. For compatibility reasons, the .desktop suffix will be removed. Please specify a desktop file name without .desktop suffix

This seems to be caused by this snippet:

QString desktopFileName = QString(QLatin1String(LINUX_APPLICATION_ID)
+ QLatin1String(".desktop"));
setDesktopFileName(desktopFileName);

The documentation for this function has been clarified in qt/qtbase@0c5135a and the detection of the suffix has been introduced in qt/qtbase@fb86c5a

Steps to reproduce

  1. Start the Nextcloud client
  2. Observe the output

Expected behavior

This warning should not be issued.

Which files are affected by this bug

https://github.com/nextcloud/desktop/blob/ae363c90e907efb8d9610f1f8b84ed4d3484dd84/src/gui/application.cpp

Operating system

Linux

Which version of the operating system you are running.

NixOS unstable

Package

Distro package manager

Nextcloud Server version

28.0.5

Nextcloud Desktop Client version

3.14.3

Is this bug present after an update or on a fresh install?

Fresh desktop client install

Are you using the Nextcloud Server Encryption module?

Encryption is Enabled

Are you using an external user-backend?

  • Default internal user-backend
  • LDAP/ Active Directory
  • SSO - SAML
  • Other

Nextcloud Server logs

Additional info

No response

@Dominique-M
Copy link

Idem under Debian testing with clientVersion=3.15.0 (build 27049) (Appimage) and serverVersion=29.0.6.1

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

Successfully merging a pull request may close this issue.

2 participants