Fix PBD::copy_files so that it uses O_BINARY on windows and doesn't add line endings...
authorTim Mayberry <mojofunk@gmail.com>
Tue, 24 Jun 2014 23:05:43 +0000 (09:05 +1000)
committerPaul Davis <paul@linuxaudiosystems.com>
Wed, 25 Jun 2014 16:40:11 +0000 (12:40 -0400)
libs/pbd/file_utils.cc

index f4efea00f3a3785dac03e0a33f795ec1fd3e7a0c..fd98cb733f96292a37ec77df8a7b73533514008f 100644 (file)
@@ -276,6 +276,14 @@ find_files_matching_filter (vector<string>& result,
        run_functor_for_paths (result, paths, filter, arg, true, pass_fullpath, return_fullpath, recurse);
 }
 
+#ifdef PLATFORM_WINDOWS
+#define WRITE_FLAGS O_RDWR | O_CREAT | O_BINARY
+#define READ_FLAGS O_RDONLY | O_BINARY
+#else
+#define WRITE_FLAGS O_RDWR | O_CREAT
+#define READ_FLAGS O_RDONLY
+#endif
+
 bool
 copy_file(const std::string & from_path, const std::string & to_path)
 {
@@ -286,12 +294,12 @@ copy_file(const std::string & from_path, const std::string & to_path)
        char buf[4096]; // BUFSIZ  ??
        ssize_t nread;
 
-       fd_from = ::open(from_path.c_str(), O_RDONLY);
+       fd_from = ::open(from_path.c_str(), READ_FLAGS);
        if (fd_from < 0) {
                goto copy_error;
        }
 
-       fd_to = ::open(to_path.c_str(), O_WRONLY | O_CREAT, 0666);
+       fd_to = ::open(to_path.c_str(), WRITE_FLAGS, 0666);
        if (fd_to < 0) {
                goto copy_error;
        }