Skip to content

Commit

Permalink
Remove support for multiple Linux shell extensions
Browse files Browse the repository at this point in the history
  • Loading branch information
TheOneRing authored and erikjv committed Jul 17, 2024
1 parent 0c6005b commit d47f789
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 6 deletions.
6 changes: 6 additions & 0 deletions changelog/unreleased/11743
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
Change: Remove support for branded shell extensions on Linux

From now on user will have to install dolphin-owncloud nautilus-owncloud
instead of a braned version.

https://github.com/owncloud/client/issues/11743
2 changes: 1 addition & 1 deletion src/gui/guiutility_unix.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ void Utility::startShellIntegration()

QString Utility::socketApiSocketPath()
{
return QStringLiteral("%1/%2/socket").arg(QStandardPaths::writableLocation(QStandardPaths::RuntimeLocation), Theme::instance()->appName());
return QStringLiteral("%1/ownCloud/socket").arg(QStandardPaths::writableLocation(QStandardPaths::RuntimeLocation));
}

} // namespace OCC
6 changes: 1 addition & 5 deletions src/gui/guiutility_win.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -25,11 +25,7 @@ void Utility::startShellIntegration()

QString Utility::socketApiSocketPath()
{
return QLatin1String("\\\\.\\pipe\\") + QLatin1String("ownCloud-") + qEnvironmentVariable("USERNAME");
// TODO: once the windows extension supports multiple
// client connections, switch back to the theme name
// See issue #2388
// + Theme::instance()->appName();
return QStringLiteral(R"(\\.\pipe\ownCloud-%1)").arg(qEnvironmentVariable("USERNAME"));
}

} // namespace OCC

0 comments on commit d47f789

Please sign in to comment.