]> git.lyx.org Git - lyx.git/commitdiff
Use real path names for buffer lookup by name of temporary external files.
authorStephan Witt <switt@lyx.org>
Sat, 22 Feb 2020 11:11:42 +0000 (12:11 +0100)
committerRichard Kimberly Heck <rikiheck@lyx.org>
Thu, 7 Jan 2021 17:08:36 +0000 (12:08 -0500)
In case of path names for external files containing symbolic links the real path
and the logical path name may be different for the same file or directory.
LyX is using QDir::tempPath() to create the path name of the temporary directory.
The Qt implementation is free to return the logical or the real path name here and
it happens to be different for various platforms and versions.
The most stable and clean solution is to use the real path name consistently.

(cherry picked from commit f2f861f017bd598c9e5b72f64e10587cbe1e3405)

development/MacOSX/lyxeditor
src/BufferList.cpp
src/BufferList.h
src/frontends/qt4/GuiView.cpp

index 7431b3b99165ee529b34e609b74f56d7f4b1d661..ad8c407e018161cf2d8863e7594ca0c3d129102a 100755 (executable)
@@ -82,9 +82,7 @@ test -z "${LYXPIPE}" && {
 }
 
 if [ -n "$LYXPIPE" -a -p "$LYXPIPE".in ]; then
-       file=$(echo "$1" | sed 's|^/private||')
-
-       MAC_LYXPIPE_CONTENTS="LYXCMD:macdvix:server-goto-file-row:$file $2"
+       MAC_LYXPIPE_CONTENTS="LYXCMD:macdvix:server-goto-file-row:$1 $2"
        # echo "$MAC_LYXPIPE_CONTENTS"
        echo "$MAC_LYXPIPE_CONTENTS" > "${LYXPIPE}".in || { echo "Cannot write to lyxpipe." ; exit 2 ; }
        while read line ; do
index 5e83c7b63ebe967dcad3d87f933a005ab07f9313..f5b106d329c5d9894213331eae28cc37598a110b 100644 (file)
@@ -331,11 +331,12 @@ Buffer * BufferList::getBuffer(support::FileName const & fname, bool internal) c
 }
 
 
-Buffer * BufferList::getBufferFromTmp(string const & s)
+Buffer * BufferList::getBufferFromTmp(string const & s, bool realpath)
 {
        BufferStorage::iterator it = bstore.begin();
        BufferStorage::iterator end = bstore.end();
        for (; it < end; ++it) {
+               string const temppath = realpath ? FileName((*it)->temppath()).realPath() : (*it)->temppath();
                if (prefixIs(s, (*it)->temppath())) {
                        // check whether the filename matches the master
                        string const master_name = (*it)->latexName();
index ca55abe8c9fdd63ce9be84fb947f53a3641fa394..8ccebf0ded0385f549e1ce2b6c75180d3fd08cd5 100644 (file)
@@ -100,8 +100,9 @@ public:
        /// \return a pointer to the buffer with the given number
        Buffer * getBuffer(unsigned int);
 
-       /// \return a pointer to the buffer whose temppath matches the given path
-       Buffer * getBufferFromTmp(std::string const & path);
+       /// \return a pointer to the buffer whose temppath matches the given \p path
+       ///  If optional \p realpath is \c true the lookup is done with real path names
+       Buffer * getBufferFromTmp(std::string const & path, bool realpath = false);
 
        /** returns a pointer to the buffer that follows argument in
         * buffer list. The buffer following the last in list is the
index 8d73c027b06d250ed4d834d7aebe14e3734cead5..9ad0834b6a32a06db6297adea3becd6c4ff96aeb 100644 (file)
@@ -3493,7 +3493,7 @@ bool GuiView::goToFileRow(string const & argument)
        int row;
        size_t i = argument.find_last_of(' ');
        if (i != string::npos) {
-               file_name = os::internal_path(trim(argument.substr(0, i)));
+               file_name = os::internal_path(FileName(trim(argument.substr(0, i))).realPath());
                istringstream is(argument.substr(i + 1));
                is >> row;
                if (is.fail())
@@ -3504,20 +3504,14 @@ bool GuiView::goToFileRow(string const & argument)
                return false;
        }
        Buffer * buf = 0;
-       string const abstmp = package().temp_dir().absFileName();
        string const realtmp = package().temp_dir().realPath();
        // We have to use os::path_prefix_is() here, instead of
        // simply prefixIs(), because the file name comes from
        // an external application and may need case adjustment.
-       if (os::path_prefix_is(file_name, abstmp, os::CASE_ADJUSTED)
-               || os::path_prefix_is(file_name, realtmp, os::CASE_ADJUSTED)) {
-               // Needed by inverse dvi search. If it is a file
-               // in tmpdir, call the apropriated function.
-               // If tmpdir is a symlink, we may have the real
-               // path passed back, so we correct for that.
-               if (!prefixIs(file_name, abstmp))
-                       file_name = subst(file_name, realtmp, abstmp);
-               buf = theBufferList().getBufferFromTmp(file_name);
+       if (os::path_prefix_is(file_name, realtmp, os::CASE_ADJUSTED)) {
+               buf = theBufferList().getBufferFromTmp(file_name, true);
+               LYXERR(Debug::FILES, "goToFileRow: buffer lookup for " << file_name
+                          << (buf ? " success" : " failed"));
        } else {
                // Must replace extension of the file to be .lyx
                // and get full path