]> git.lyx.org Git - lyx.git/blobdiff - src/support/filetools.C
* lyxfunctional.h: delete compare_memfun and helper classes
[lyx.git] / src / support / filetools.C
index 05109c76b3808ae06659f201ee048a9413b4b5c9..265583d074165f026cfe9d8669edabe114842f07 100644 (file)
 #include "support/systemcall.h"
 
 #include "filetools.h"
+#include "format.h"
 #include "lstrings.h"
 #include "FileInfo.h"
+#include "forkedcontr.h"
 #include "path.h"
 #include "path_defines.h"
 #include "gettext.h"
 #include "lyxlib.h"
 #include "os.h"
 
-#include "support/std_sstream.h"
-
 #include <boost/assert.hpp>
 #include <boost/regex.hpp>
 
+#include <fcntl.h>
+
 #include <cctype>
 #include <cstdlib>
 #include <cstdio>
-#include <fcntl.h>
 #include <cerrno>
 
 #include <utility>
 #include <fstream>
+#include <sstream>
 
 
 // Which part of this is still necessary? (JMarc).
@@ -315,12 +317,8 @@ i18nLibFileSearch(string const & dir, string const & name,
           variable. But we don't use the value if the currently
           selected locale is the C locale. This is a GNU extension. */
        /* [Otherwise] We have to proceed with the POSIX methods of
-          looking to `LC_ALL', `LC_xxx', and `LANG'. On some systems
-          this can be done by the `setlocale' function itself.  */
+          looking to `LC_ALL', `LC_xxx', and `LANG'. */
 
-#if defined HAVE_SETLOCALE && defined HAVE_LC_MESSAGES && defined HAVE_LOCALE_NULL
-       lang = setlocale(LC_MESSAGES, NULL);
-#else
        string lang = GetEnv("LC_ALL");
        if (lang.empty()) {
                lang = GetEnv("LC_MESSAGES");
@@ -330,24 +328,23 @@ i18nLibFileSearch(string const & dir, string const & name,
                                lang = "C";
                }
        }
-#endif
 
        string const language = GetEnv("LANGUAGE");
-       if (lang != "C" && !language.empty())
+       if (lang != "C" && lang != "POSIX" && !language.empty())
                lang = language;
 
-       lang = token(lang, '_', 0);
-
-       if (lang.empty() || lang == "C")
-               return LibFileSearch(dir, name, ext);
-       else {
-               string const tmp = LibFileSearch(dir, lang + '_' + name,
+       string l;
+       lang = split(lang, l, ':');
+       while (!l.empty() && l != "C" && l != "POSIX") {
+               string const tmp = LibFileSearch(dir,
+                                                token(l, '_', 0) + '_' + name,
                                                 ext);
                if (!tmp.empty())
                        return tmp;
-               else
-                       return LibFileSearch(dir, name, ext);
+               lang = split(lang, l, ':');
        }
+
+       return LibFileSearch(dir, name, ext);
 }
 
 
@@ -376,7 +373,7 @@ string const LibScriptSearch(string const & command_in)
        } else {
                // Replace "$$s/some_script" with "$LYX_SCRIPT_PATH/some_script"
                string::size_type const size_replace = size_script + 4;
-               command.replace(pos1, size_replace, script);
+               command.replace(pos1, size_replace, QuoteName(script));
        }
 
        return command;
@@ -441,9 +438,11 @@ int DeleteAllFilesInDir(string const & path)
 
                bool deleted = true;
                FileInfo fi(unlinkpath);
-               if (fi.isOK() && fi.isDir())
+               if (fi.isOK() && fi.isDir()) {
                        deleted = (DeleteAllFilesInDir(unlinkpath) == 0);
-               deleted &= (unlink(unlinkpath) == 0);
+                       deleted &= (rmdir(unlinkpath) == 0);
+               } else
+                       deleted &= (unlink(unlinkpath) == 0);
                if (!deleted)
                        return_value = -1;
        }
@@ -895,6 +894,24 @@ string const GetExtension(string const & name)
                return string();
 }
 
+
+namespace {
+
+class FormatExtensionsEqual : public std::unary_function<Format, bool> {
+public:
+       FormatExtensionsEqual(string const & extension)
+               : extension_(extension) {}
+       bool operator()(Format const & f) const
+       {
+               return f.extension() == extension_;
+       }
+private:
+       string extension_;
+};
+
+} // namespace anon
+
+
 // the different filetypes and what they contain in one of the first lines
 // (dots are any characters).          (Herbert 20020131)
 // AGR Grace...
@@ -925,7 +942,7 @@ string const GetExtension(string const & name)
 /// return the "extension" which belongs to the contents.
 /// for no knowing contents return the extension. Without
 /// an extension and unknown contents we return "user"
-string const getExtFromContents(string const & filename)
+string const getFormatFromContents(string const & filename)
 {
        // paranoia check
        if (filename.empty() || !IsFileReadable(filename))
@@ -955,7 +972,7 @@ string const getExtFromContents(string const & filename)
        while ((count++ < max_count) && format.empty()) {
                if (ifs.eof()) {
                        lyxerr[Debug::GRAPHICS]
-                               << "filetools(getExtFromContents)\n"
+                               << "filetools(getFormatFromContents)\n"
                                << "\tFile type not recognised before EOF!"
                                << endl;
                        break;
@@ -1072,25 +1089,31 @@ string const getExtFromContents(string const & filename)
 
        string const ext(GetExtension(filename));
        lyxerr[Debug::GRAPHICS]
-               << "filetools(getExtFromContents)\n"
-               << "\tCouldn't find a known Type!\n";
+               << "filetools(getFormatFromContents)\n"
+               << "\tCouldn't find a known format!\n";
        if (!ext.empty()) {
-           lyxerr[Debug::GRAPHICS]
-               << "\twill take the file extension -> "
-               << ext << endl;
-               return ext;
-       } else {
-           lyxerr[Debug::GRAPHICS]
-               << "\twill use ext or a \"user\" defined format" << endl;
-           return "user";
+               // this is ambigous if two formats have the same extension,
+               // but better than nothing
+               Formats::const_iterator cit =
+                       find_if(formats.begin(), formats.end(),
+                               FormatExtensionsEqual(ext));
+               if (cit != formats.end()) {
+                       lyxerr[Debug::GRAPHICS]
+                               << "\twill guess format from file extension: "
+                               << ext << " -> " << cit->name() << endl;
+                       return cit->name();
+               }
        }
+       lyxerr[Debug::GRAPHICS]
+               << "\twill use a \"user\" defined format" << endl;
+       return "user";
 }
 
 
 /// check for zipped file
 bool zippedFile(string const & name)
 {
-       string const type = getExtFromContents(name);
+       string const type = getFormatFromContents(name);
        if (contains("gzip zip compress", type) && !type.empty())
                return true;
        return false;
@@ -1175,17 +1198,31 @@ bool LyXReadLink(string const & file, string & link, bool resolve)
 
 cmd_ret const RunCommand(string const & cmd)
 {
+       // FIXME: replace all calls to RunCommand with ForkedCall
+       // (if the output is not needed) or the code in ispell.C
+       // (if the output is needed).
+
        // One question is if we should use popen or
        // create our own popen based on fork, exec, pipe
        // of course the best would be to have a
        // pstream (process stream), with the
        // variants ipstream, opstream
 
+       sigset_t newMask, oldMask;
+       sigemptyset(&oldMask);
+       sigemptyset(&newMask);
+       sigaddset(&newMask, SIGCHLD);
+
+       // Block the SIGCHLD signal.
+       sigprocmask(SIG_BLOCK, &newMask, &oldMask);
+
        FILE * inf = ::popen(cmd.c_str(), os::popen_read_mode());
 
        // (Claus Hentschel) Check if popen was succesful ;-)
-       if (!inf)
+       if (!inf) {
                return make_pair(-1, string());
+               lyxerr << "RunCommand:: could not start child process" << endl;
+               }
 
        string ret;
        int c = fgetc(inf);
@@ -1194,6 +1231,12 @@ cmd_ret const RunCommand(string const & cmd)
                c = fgetc(inf);
        }
        int const pret = pclose(inf);
+       if (pret == -1)
+               perror("RunCommand:: could not terminate child process");
+
+       // Unblock the SIGCHLD signal and restore the old mask.
+       sigprocmask(SIG_SETMASK, &oldMask, 0);
+
        return make_pair(pret, ret);
 }
 
@@ -1265,7 +1308,9 @@ void readBB_lyxerrMessage(string const & file, bool & zipped,
 {
        lyxerr[Debug::GRAPHICS] << "[readBB_from_PSFile] "
                << message << std::endl;
+#ifdef WITH_WARNINGS
 #warning Why is this func deleting a file? (Lgb)
+#endif
        if (zipped)
                unlink(file);
 }
@@ -1283,7 +1328,7 @@ string const readBB_from_PSFile(string const & file)
        bool zipped = zippedFile(file);
        string const file_ = zipped ?
                string(unzipFile(file)) : string(file);
-       string const format = getExtFromContents(file_);
+       string const format = getFormatFromContents(file_);
 
        if (format != "eps" && format != "ps") {
                readBB_lyxerrMessage(file_, zipped,"no(e)ps-format");