]> git.lyx.org Git - lyx.git/blobdiff - src/support/filetools.C
various changes
[lyx.git] / src / support / filetools.C
index bc2f9925e390e0e716dd46028b5c376bf6b198ec..10335730aac05a05de296288debc686638ebdd97 100644 (file)
@@ -32,9 +32,9 @@
 #include "lyx_gui_misc.h"
 #include "FileInfo.h"
 #include "support/path.h"        // I know it's OS/2 specific (SMiyata)
-#include "support/syscall.h"
 #include "gettext.h"
 #include "lyxlib.h"
+#include "os.h"
 
 // Which part of this is still necessary? (JMarc).
 #if HAVE_DIRENT_H
@@ -60,13 +60,6 @@ using std::endl;
 using std::ifstream;
 using std::vector;
 
-#if 0
-using std::getenv;
-using std::isalpha;
-using std::isalnum;
-using std::popen;
-#endif
-
 extern string system_lyxdir;
 extern string build_lyxdir;
 extern string user_lyxdir;
@@ -75,13 +68,13 @@ extern string system_tempdir;
 
 bool IsLyXFilename(string const & filename)
 {
-       return contains(filename, ".lyx");
+       return suffixIs(filename, ".lyx");
 }
 
 
 bool IsSGMLFilename(string const & filename)
 {
-       return contains(filename, ".sgml");
+       return suffixIs(filename, ".sgml");
 }
 
 
@@ -111,12 +104,9 @@ string const MakeLatexName(string const & file)
 // Substitutes spaces with underscores in filename (and path)
 string const QuoteName(string const & name)
 {
-       // CHECK Add proper emx support here!
-#ifndef __EMX__
-       return "\'" + name + "\'";
-#else
-       return name; 
-#endif
+       return (os::shell() == os::UNIX) ?
+               "\'" + name + "\'":
+               "\"" + name + "\"";
 }
 
 
@@ -151,8 +141,10 @@ int IsFileWriteable (string const & path)
 //      -1: error- couldn't find out
 int IsDirWriteable (string const & path)
 {
-        string const tmpfl(lyx::tempName(path)); //TmpFileName(path));
-
+        string const tmpfl(lyx::tempName(path, "lyxwritetest"));
+       // We must unlink the tmpfl.
+       lyx::unlink(tmpfl);
+       
        if (tmpfl.empty()) {
                WriteFSAlert(_("LyX Internal Error!"), 
                             _("Could not test if directory is writeable"));
@@ -178,7 +170,7 @@ string const FileOpenSearch (string const & path, string const & name,
        string tmppath = split(path, path_element, ';');
        
        while (notfound && !path_element.empty()) {
-               path_element = CleanupPath(path_element);
+               path_element = os::slashify_path(path_element);
                if (!suffixIs(path_element, '/'))
                        path_element+= '/';
                path_element = subst(path_element, "$$LyX", system_lyxdir);
@@ -209,14 +201,14 @@ vector<string> const DirList(string const & dir, string const & ext)
 {
        // This is a non-error checking C/system implementation
        string extension(ext);
-       if (extension[0] != '.')
+       if (!extension.empty() && extension[0] != '.')
                extension.insert(0, ".");
        vector<string> dirlist;
        DIR * dirp = ::opendir(dir.c_str());
        dirent * dire;
        while ((dire = ::readdir(dirp))) {
                string const fil = dire->d_name;
-               if (prefixIs(fil, extension)) {
+               if (suffixIs(fil, extension)) {
                        dirlist.push_back(fil);
                }
        }
@@ -255,7 +247,6 @@ string const FileSearch(string const & path, string const & name,
        // Expand Environmentvariables in 'name'
        string const tmpname = ReplaceEnvironmentPath(name);
        string fullname = MakeAbsPath(tmpname, path);
-       
        // search first without extension, then with it.
        if (IsFileReadable(fullname))
                return fullname;
@@ -327,7 +318,7 @@ string const GetEnvPath(string const & name)
 #ifndef __EMX__
         string const pathlist = subst(GetEnv(name), ':', ';');
 #else
-        string const pathlist = subst(GetEnv(name), '\\', '/');
+        string const pathlist = os::slashify_path(GetEnv(name));
 #endif
         return strip(pathlist, ';');
 }
@@ -380,7 +371,8 @@ bool PutEnvPath(string const & envstr)
 }
 
 
-static
+namespace {
+
 int DeleteAllFilesInDir (string const & path)
 {
        // I have decided that we will be using parts from the boost
@@ -433,7 +425,6 @@ int DeleteAllFilesInDir (string const & path)
 }
 
 
-static
 string const CreateTmpDir(string const & tempdir, string const & mask)
 {
        lyxerr[Debug::FILES]
@@ -447,7 +438,7 @@ string const CreateTmpDir(string const & tempdir, string const & mask)
        // safe because of the gap between unlink and mkdir. (Lgb)
        lyx::unlink(tmpfl.c_str());
        
-       if (tmpfl.empty() || lyx::mkdir(tmpfl, 0777)) {
+       if (tmpfl.empty() || lyx::mkdir(tmpfl, 0700)) {
                WriteFSAlert(_("Error! Couldn't create temporary directory:"),
                             tempdir);
                return string();
@@ -456,7 +447,6 @@ string const CreateTmpDir(string const & tempdir, string const & mask)
 }
 
 
-static
 int DestroyTmpDir(string const & tmpdir, bool Allfiles)
 {
 #ifdef __EMX__
@@ -469,12 +459,25 @@ int DestroyTmpDir(string const & tmpdir, bool Allfiles)
                return -1;
        }
        return 0; 
-} 
+}
+
+} // namespace anon
 
 
 string const CreateBufferTmpDir(string const & pathfor)
 {
-       return CreateTmpDir(pathfor, "lyx_tmpbuf");
+       static int count;
+       static string const tmpdir(pathfor.empty() ? os::getTmpDir() : pathfor);
+       // We are in our own directory.  Why bother to mangle name?
+       // In fact I wrote this code to circumvent a problematic behaviour (bug?)
+       // of EMX mkstemp().
+       string const tmpfl = tmpdir + "/lyx_tmpbuf" + tostr(count++);
+       if (lyx::mkdir(tmpfl, 0777)) {
+               WriteFSAlert(_("Error! Couldn't create temporary directory:"),
+                            tmpdir);
+               return string();
+       }
+       return tmpfl;
 }
 
 
@@ -514,7 +517,7 @@ int DestroyLyXTmpDir(string const & tmpdir)
 // Creates directory. Returns true if succesfull
 bool createDirectory(string const & path, int permission)
 {
-       string temp(strip(CleanupPath(path), '/'));
+       string temp(strip(os::slashify_path(path), '/'));
 
        if (temp.empty()) {
                WriteAlert(_("Internal error!"),
@@ -551,32 +554,20 @@ string const MakeAbsPath(string const & RelPath, string const & BasePath)
 {
        // checks for already absolute path
        if (AbsolutePath(RelPath))
-#ifdef __EMX__
-               if (RelPath[0]!= '/' && RelPath[0]!= '\\')
-#endif
                return RelPath;
 
        // Copies given paths
-       string TempRel(CleanupPath(RelPath));
+       string TempRel(os::slashify_path(RelPath));
+       // Since TempRel is NOT absolute, we can safely replace "//" with "/"
+       TempRel = subst(TempRel, "//", "/");
 
        string TempBase;
 
-       if (!BasePath.empty()) {
-#ifndef __EMX__
+       if (AbsolutePath(BasePath))
                TempBase = BasePath;
-#else
-               char * with_drive = new char[_MAX_PATH];
-               _abspath(with_drive, BasePath.c_str(), _MAX_PATH);
-               TempBase = with_drive;
-               delete[] with_drive;
-#endif
-       } else
-               TempBase = lyx::getcwd(); //GetCWD();
-#ifdef __EMX__
-       if (AbsolutePath(TempRel))
-               return TempBase.substr(0, 2) + TempRel;
-#endif
-
+       else
+               TempBase = AddPath(lyx::getcwd(), BasePath);
+       
        // Handle /./ at the end of the path
        while(suffixIs(TempBase, "/./"))
                TempBase.erase(TempBase.length() - 2);
@@ -605,6 +596,9 @@ string const MakeAbsPath(string const & RelPath, string const & BasePath)
                                TempBase.erase(i, string::npos);
                        else
                                TempBase += '/';
+               } else if (Temp.empty() && !RTemp.empty()) {
+                               TempBase = os::current_root() + RTemp;
+                               RTemp.erase();
                } else {
                        // Add this piece to TempBase
                        if (!suffixIs(TempBase, '/'))
@@ -614,7 +608,7 @@ string const MakeAbsPath(string const & RelPath, string const & BasePath)
        }
 
        // returns absolute path
-       return TempBase;
+       return os::slashify_path(TempBase);
 }
 
 
@@ -629,7 +623,7 @@ string const AddName(string const & path, string const & fname)
        string buf;
 
        if (path != "." && path != "./" && !path.empty()) {
-               buf = CleanupPath(path);
+               buf = os::slashify_path(path);
                if (!suffixIs(path, '/'))
                        buf += '/';
        }
@@ -653,28 +647,13 @@ string const OnlyFilename(string const & fname)
 }
 
 
-// Strips filename from path
-string const BasePath(string const & fname)
-{
-       if (fname.empty())
-               return fname;
-
-       string::size_type j = fname.rfind('/');
-       if (j == string::npos) // no '/' in fname
-               return string();
-
-       // Strip to basename
-       return fname.substr(0,j + 1);
-}
-
-
 // Is a filename/path absolute?
 bool AbsolutePath(string const & path)
 {
 #ifndef __EMX__
        return (!path.empty() && path[0] == '/');
 #else
-       return (!path.empty() && (path[0] == '/' || (isalpha(static_cast<unsigned char>(path[0])) && path.length()>1 && path[1] == ':')));
+       return (!path.empty() && isalpha(static_cast<unsigned char>(path[0])) && path.length() > 1 && path[1] == ':');
 #endif
 }
 
@@ -746,18 +725,6 @@ string const NormalizePath(string const & path)
 }
 
 
-string const CleanupPath(string const & path) 
-{
-#ifdef __EMX__   /* SMiyata: This should fix searchpath bug. */
-       string temppath = subst(path, '\\', '/');
-       temppath = subst(temppath, "//", "/");
-       return lowercase(temppath);
-#else // On unix, nothing to do
-       return path;
-#endif
-}
-
-
 string const GetFileContents(string const & fname)
 {
        FileInfo finfo(fname);
@@ -869,33 +836,16 @@ string const ReplaceEnvironmentPath(string const & path)
 
 
 // Make relative path out of two absolute paths
-string const MakeRelPath(string const & abspath0, string const & basepath0)
+string const MakeRelPath(string const & abspath, string const & basepath)
 // Makes relative path out of absolute path. If it is deeper than basepath,
 // it's easy. If basepath and abspath share something (they are all deeper
 // than some directory), it'll be rendered using ..'s. If they are completely
 // different, then the absolute path will be used as relative path.
 {
-       // This is a hack. It should probaly be done in another way. Lgb.
-       string const abspath = CleanupPath(abspath0);
-       string const basepath = CleanupPath(basepath0);
-       if (abspath.empty())
-               return "<unknown_path>";
-
        string::size_type const abslen = abspath.length();
        string::size_type const baselen = basepath.length();
-       
-       // Find first different character
-       string::size_type i = 0;
-       while (i < abslen && i < baselen && abspath[i] == basepath[i]) ++i;
-
-       // Go back to last /
-       if (i < abslen && i < baselen
-           || (i < abslen && abspath[i] != '/' && i == baselen)
-           || (i < baselen && basepath[i] != '/' && i == abslen))
-       {
-               if (i) --i;     // here was the last match
-               while (i && abspath[i] != '/') --i;
-       }
+
+       string::size_type i = os::common_path(abspath, basepath);
 
        if (i == 0) {
                // actually no match - cannot make it relative
@@ -932,10 +882,10 @@ string const MakeRelPath(string const & abspath0, string const & basepath0)
 string const AddPath(string const & path, string const & path_2)
 {
        string buf;
-       string const path2 = CleanupPath(path_2);
+       string const path2 = os::slashify_path(path_2);
 
        if (!path.empty() && path != "." && path != "./") {
-               buf = CleanupPath(path);
+               buf = os::slashify_path(path);
                if (path[path.length() - 1] != '/')
                        buf += '/';
        }
@@ -970,7 +920,7 @@ ChangeExtension(string const & oldname, string const & extension)
        else
                ext = extension;
 
-       return CleanupPath(oldname.substr(0, last_dot) + ext);
+       return os::slashify_path(oldname.substr(0, last_dot) + ext);
 }
 
 
@@ -1053,8 +1003,10 @@ bool LyXReadLink(string const & File, string & Link)
 }
 
 
+namespace {
+
 typedef pair<int, string> cmdret;
-static
+
 cmdret const do_popen(string const & cmd)
 {
        // One question is if we should use popen or
@@ -1073,6 +1025,8 @@ cmdret const do_popen(string const & cmd)
        return make_pair(pret, ret);
 }
 
+} // namespace anon
+
 
 string const
 findtexfile(string const & fil, string const & /*format*/)
@@ -1135,151 +1089,3 @@ void removeAutosaveFile(string const & filename)
                }
        }
 }
-
-
-bool WriteableDir(string const & name, string & errorMessage)
-{
-       errorMessage.erase();
-
-       if (!AbsolutePath(name)) {
-               errorMessage = N_("The absolute path is required.");
-               return false;
-       }
-
-       FileInfo const tp(name);
-       if (!tp.isDir()) {
-               errorMessage = N_("Directory does not exist.");
-               return false;
-       }
-
-       if (!tp.writable()) {
-               errorMessage = N_("Cannot write to this directory.");
-               return false;
-       }
-
-       return true;
-}
-
-
-bool ReadableDir(string const & name, string & errorMessage)
-{
-       errorMessage.erase();
-
-       if (!AbsolutePath(name)) {
-               errorMessage = N_("The absolute path is required.");
-               return false;
-       }
-
-       FileInfo const tp(name);
-       if (!tp.isDir()) {
-               errorMessage = N_("Directory does not exist.");
-               return false;
-       }
-
-       if (!tp.readable()) {
-               errorMessage = N_("Cannot read this directory.");
-               return false;
-       }
-
-       return true;
-}
-
-
-bool WriteableFile(string const & name, string & errorMessage)
-{
-       // A writeable file is either:
-       // * An existing file to which we have write access, or
-       // * A file that doesn't yet exist but that would exist in a writeable
-       //   directory.
-
-       errorMessage.erase();
-
-       if (name.empty()) {
-               errorMessage = N_("No file input.");
-               return false;
-       }
-
-       string const dir = OnlyPath(name);
-       if (!AbsolutePath(dir)) {
-               errorMessage = N_("The absolute path is required.");
-               return false;
-       }
-
-       FileInfo d(name);
-       if (!d.isDir()) {
-               d.newFile(dir);
-       }
-
-       if (!d.isDir()) {
-               errorMessage = N_("Directory does not exist.");
-               return false;
-       }
-       
-       if (!d.writable()) {
-               errorMessage = N_("Cannot write to this directory.");
-               return false;
-       }
-
-       FileInfo f(name);
-       if (dir == name || f.isDir()) {
-               errorMessage = N_("A file is required, not a directory.");
-               return false;
-       }
-
-       if (f.exist() && !f.writable()) {
-               errorMessage = N_("Cannot write to this file.");
-               return false;
-       }
-       
-       return true;
-}
-
-
-bool ReadableFile(string const & name, string & errorMessage)
-{
-       errorMessage.erase();
-
-       if (name.empty()) {
-               errorMessage = N_("No file input.");
-               return false;
-       }
-
-       string const dir = OnlyPath(name);
-       if (!AbsolutePath(dir)) {
-               errorMessage = N_("The absolute path is required.");
-               return false;
-       }
-
-       FileInfo d(name);
-       if (!d.isDir()) {
-               d.newFile(dir);
-       }
-
-       if (!d.isDir()) {
-               errorMessage = N_("Directory does not exist.");
-               return false;
-       }
-       
-       if (!d.readable()) {
-               errorMessage = N_("Cannot read from this directory.");
-               return false;
-       }
-
-       FileInfo f(name);
-       if (dir == name || f.isDir()) {
-               errorMessage = N_("A file is required, not a directory.");
-               return false;
-       }
-
-       if (!f.exist()) {
-               errorMessage = N_("File does not exist.");
-               return false;
-       }
-       
-       if (!f.readable()) {
-               errorMessage = N_("Cannot read from this file.");
-               return false;
-       }
-
-       return true;
-}