Inverse search command not executed in development build

[hpfmn]

Hi,

I did a build of the development branch on macOS. With the current release my setting for inverse search does work. But with this branch it seems it doesn't get execute (I have a little bash script that stores the parameters in a file - but nothing is stored)

Is there anything changed regarding inverse search in the current development branch?

Best regards,
Johannes

[ahrm]

Inverse search should still work. Which program are you using for inverse search and what is your settings for that?

[ismasou]

For me too it didn't work, I thought it might be just my Qt version. It seems that in Qt6 the QProcess::startDetached needs separate arguments for the command and its arguments.
I use neovim, and I was able to make it work by changing the AUR patch to include this change to the startDetached.

diff --git a/pdf_viewer/main_widget.cpp b/pdf_viewer/main_widget.cpp
index 20f61e3..b33480b 100644
--- a/pdf_viewer/main_widget.cpp
+++ b/pdf_viewer/main_widget.cpp
@@ -4651,7 +4651,14 @@ std::wstring MainWidget::synctex_under_pos(WindowPos position) {
                 QString command = QString::fromStdWString(inverse_search_command).arg(file_name, line_string.c_str(), column_string.c_str());
 #endif
                 res = QString("%1 %2 %3").arg(new_path, QString::number(line), QString::number(column)).toStdWString();
-                QProcess::startDetached(command);
+                QString arg = QString("-c VimtexInverseSearch %1 '%2'").arg(line_string.c_str(), file_name);
+                QStringList args = {"--headless", arg};
+
+                QProcess::startDetached("/usr/bin/nvim", args);
             }
             else {
                 show_error_message(L"inverse_search_command is not set in prefs_user.config");
diff --git a/pdf_viewer_build_config.pro b/pdf_viewer_build_config.pro
index 83d286d..8a027b0 100644
--- a/pdf_viewer_build_config.pro
+++ b/pdf_viewer_build_config.pro
@@ -158,7 +158,8 @@ unix:!mac:!android {
     QMAKE_CXXFLAGS += -std=c++17
 
     CONFIG(linux_app_image){
-        LIBS += -ldl -Lmupdf/build/release -lmupdf -lmupdf-third -lmupdf-threads -lharfbuzz -lz
+        DEFINES += LINUX_STANDARD_PATHS
+        LIBS += -ldl -lmupdf -lz
     } else {
         DEFINES += NON_PORTABLE
         DEFINES += LINUX_STANDARD_PATHS

@ahrm Thank you for the very good software. I couldn't debug why sending the command with its arguments to startDetached doesn't work anymore in Qt6.
Let me know if you want me to create a PR with a proper fix for user defined inverse search, I guess now one needs to separate the command from its arguments before calling startDetached.

[jinjiaodawang]

Can confirm the issue; and the issue happen might because the method QProcess::startDetached(const QString &command) is removed in Qt6.
Please refer to https://doc.qt.io/qt-5/qprocess-obsolete.html#startDetached-2

[ahrm]

Should be fixed in ed77c64.