Commit 5368be30 by Edward Thomson

clar: support long paths on Windows

parent 525516b4
...@@ -9,6 +9,7 @@ SET(CLAR_FIXTURES "${CMAKE_CURRENT_SOURCE_DIR}/resources/") ...@@ -9,6 +9,7 @@ SET(CLAR_FIXTURES "${CMAKE_CURRENT_SOURCE_DIR}/resources/")
SET(CLAR_PATH "${CMAKE_CURRENT_SOURCE_DIR}") SET(CLAR_PATH "${CMAKE_CURRENT_SOURCE_DIR}")
ADD_DEFINITIONS(-DCLAR_FIXTURE_PATH=\"${CLAR_FIXTURES}\") ADD_DEFINITIONS(-DCLAR_FIXTURE_PATH=\"${CLAR_FIXTURES}\")
ADD_DEFINITIONS(-DCLAR_TMPDIR=\"libgit2_tests\") ADD_DEFINITIONS(-DCLAR_TMPDIR=\"libgit2_tests\")
ADD_DEFINITIONS(-DCLAR_WIN32_LONGPATHS)
ADD_DEFINITIONS(-D_FILE_OFFSET_BITS=64) ADD_DEFINITIONS(-D_FILE_OFFSET_BITS=64)
# Ensure that we do not use deprecated functions internally # Ensure that we do not use deprecated functions internally
......
...@@ -8,6 +8,12 @@ ...@@ -8,6 +8,12 @@
#ifdef _WIN32 #ifdef _WIN32
#ifdef CLAR_WIN32_LONGPATHS
# define CLAR_MAX_PATH 4096
#else
# define CLAR_MAX_PATH MAX_PATH
#endif
#define RM_RETRY_COUNT 5 #define RM_RETRY_COUNT 5
#define RM_RETRY_DELAY 10 #define RM_RETRY_DELAY 10
...@@ -48,21 +54,48 @@ fs_rmdir_rmdir(WCHAR *_wpath) ...@@ -48,21 +54,48 @@ fs_rmdir_rmdir(WCHAR *_wpath)
return 0; return 0;
} }
static void translate_path(WCHAR *path, size_t path_size)
{
size_t path_len, i;
if (wcsncmp(path, L"\\\\?\\", 4) == 0)
return;
path_len = wcslen(path);
cl_assert(path_size > path_len + 4);
for (i = path_len; i > 0; i--) {
WCHAR c = path[i - 1];
if (c == L'/')
path[i + 3] = L'\\';
else
path[i + 3] = path[i - 1];
}
path[0] = L'\\';
path[1] = L'\\';
path[2] = L'?';
path[3] = L'\\';
path[path_len + 4] = L'\0';
}
static void static void
fs_rmdir_helper(WCHAR *_wsource) fs_rmdir_helper(WCHAR *_wsource)
{ {
WCHAR buffer[MAX_PATH]; WCHAR buffer[CLAR_MAX_PATH];
HANDLE find_handle; HANDLE find_handle;
WIN32_FIND_DATAW find_data; WIN32_FIND_DATAW find_data;
size_t buffer_prefix_len; size_t buffer_prefix_len;
/* Set up the buffer and capture the length */ /* Set up the buffer and capture the length */
wcscpy_s(buffer, MAX_PATH, _wsource); wcscpy_s(buffer, CLAR_MAX_PATH, _wsource);
wcscat_s(buffer, MAX_PATH, L"\\"); translate_path(buffer, CLAR_MAX_PATH);
wcscat_s(buffer, CLAR_MAX_PATH, L"\\");
buffer_prefix_len = wcslen(buffer); buffer_prefix_len = wcslen(buffer);
/* FindFirstFile needs a wildcard to match multiple items */ /* FindFirstFile needs a wildcard to match multiple items */
wcscat_s(buffer, MAX_PATH, L"*"); wcscat_s(buffer, CLAR_MAX_PATH, L"*");
find_handle = FindFirstFileW(buffer, &find_data); find_handle = FindFirstFileW(buffer, &find_data);
cl_assert(INVALID_HANDLE_VALUE != find_handle); cl_assert(INVALID_HANDLE_VALUE != find_handle);
...@@ -72,7 +105,7 @@ fs_rmdir_helper(WCHAR *_wsource) ...@@ -72,7 +105,7 @@ fs_rmdir_helper(WCHAR *_wsource)
if (fs__dotordotdot(find_data.cFileName)) if (fs__dotordotdot(find_data.cFileName))
continue; continue;
wcscpy_s(buffer + buffer_prefix_len, MAX_PATH - buffer_prefix_len, find_data.cFileName); wcscpy_s(buffer + buffer_prefix_len, CLAR_MAX_PATH - buffer_prefix_len, find_data.cFileName);
if (FILE_ATTRIBUTE_DIRECTORY & find_data.dwFileAttributes) if (FILE_ATTRIBUTE_DIRECTORY & find_data.dwFileAttributes)
fs_rmdir_helper(buffer); fs_rmdir_helper(buffer);
...@@ -123,7 +156,7 @@ fs_rm_wait(WCHAR *_wpath) ...@@ -123,7 +156,7 @@ fs_rm_wait(WCHAR *_wpath)
static void static void
fs_rm(const char *_source) fs_rm(const char *_source)
{ {
WCHAR wsource[MAX_PATH]; WCHAR wsource[CLAR_MAX_PATH];
DWORD attrs; DWORD attrs;
/* The input path is UTF-8. Convert it to wide characters /* The input path is UTF-8. Convert it to wide characters
...@@ -133,7 +166,9 @@ fs_rm(const char *_source) ...@@ -133,7 +166,9 @@ fs_rm(const char *_source)
_source, _source,
-1, /* Indicates NULL termination */ -1, /* Indicates NULL termination */
wsource, wsource,
MAX_PATH)); CLAR_MAX_PATH));
translate_path(wsource, CLAR_MAX_PATH);
/* Does the item exist? If not, we have no work to do */ /* Does the item exist? If not, we have no work to do */
attrs = GetFileAttributesW(wsource); attrs = GetFileAttributesW(wsource);
...@@ -158,21 +193,23 @@ fs_rm(const char *_source) ...@@ -158,21 +193,23 @@ fs_rm(const char *_source)
static void static void
fs_copydir_helper(WCHAR *_wsource, WCHAR *_wdest) fs_copydir_helper(WCHAR *_wsource, WCHAR *_wdest)
{ {
WCHAR buf_source[MAX_PATH], buf_dest[MAX_PATH]; WCHAR buf_source[CLAR_MAX_PATH], buf_dest[CLAR_MAX_PATH];
HANDLE find_handle; HANDLE find_handle;
WIN32_FIND_DATAW find_data; WIN32_FIND_DATAW find_data;
size_t buf_source_prefix_len, buf_dest_prefix_len; size_t buf_source_prefix_len, buf_dest_prefix_len;
wcscpy_s(buf_source, MAX_PATH, _wsource); wcscpy_s(buf_source, CLAR_MAX_PATH, _wsource);
wcscat_s(buf_source, MAX_PATH, L"\\"); wcscat_s(buf_source, CLAR_MAX_PATH, L"\\");
translate_path(buf_source, CLAR_MAX_PATH);
buf_source_prefix_len = wcslen(buf_source); buf_source_prefix_len = wcslen(buf_source);
wcscpy_s(buf_dest, MAX_PATH, _wdest); wcscpy_s(buf_dest, CLAR_MAX_PATH, _wdest);
wcscat_s(buf_dest, MAX_PATH, L"\\"); wcscat_s(buf_dest, CLAR_MAX_PATH, L"\\");
translate_path(buf_dest, CLAR_MAX_PATH);
buf_dest_prefix_len = wcslen(buf_dest); buf_dest_prefix_len = wcslen(buf_dest);
/* Get an enumerator for the items in the source. */ /* Get an enumerator for the items in the source. */
wcscat_s(buf_source, MAX_PATH, L"*"); wcscat_s(buf_source, CLAR_MAX_PATH, L"*");
find_handle = FindFirstFileW(buf_source, &find_data); find_handle = FindFirstFileW(buf_source, &find_data);
cl_assert(INVALID_HANDLE_VALUE != find_handle); cl_assert(INVALID_HANDLE_VALUE != find_handle);
...@@ -185,8 +222,8 @@ fs_copydir_helper(WCHAR *_wsource, WCHAR *_wdest) ...@@ -185,8 +222,8 @@ fs_copydir_helper(WCHAR *_wsource, WCHAR *_wdest)
if (fs__dotordotdot(find_data.cFileName)) if (fs__dotordotdot(find_data.cFileName))
continue; continue;
wcscpy_s(buf_source + buf_source_prefix_len, MAX_PATH - buf_source_prefix_len, find_data.cFileName); wcscpy_s(buf_source + buf_source_prefix_len, CLAR_MAX_PATH - buf_source_prefix_len, find_data.cFileName);
wcscpy_s(buf_dest + buf_dest_prefix_len, MAX_PATH - buf_dest_prefix_len, find_data.cFileName); wcscpy_s(buf_dest + buf_dest_prefix_len, CLAR_MAX_PATH - buf_dest_prefix_len, find_data.cFileName);
if (FILE_ATTRIBUTE_DIRECTORY & find_data.dwFileAttributes) if (FILE_ATTRIBUTE_DIRECTORY & find_data.dwFileAttributes)
fs_copydir_helper(buf_source, buf_dest); fs_copydir_helper(buf_source, buf_dest);
...@@ -205,7 +242,7 @@ fs_copydir_helper(WCHAR *_wsource, WCHAR *_wdest) ...@@ -205,7 +242,7 @@ fs_copydir_helper(WCHAR *_wsource, WCHAR *_wdest)
static void static void
fs_copy(const char *_source, const char *_dest) fs_copy(const char *_source, const char *_dest)
{ {
WCHAR wsource[MAX_PATH], wdest[MAX_PATH]; WCHAR wsource[CLAR_MAX_PATH], wdest[CLAR_MAX_PATH];
DWORD source_attrs, dest_attrs; DWORD source_attrs, dest_attrs;
HANDLE find_handle; HANDLE find_handle;
WIN32_FIND_DATAW find_data; WIN32_FIND_DATAW find_data;
...@@ -217,14 +254,17 @@ fs_copy(const char *_source, const char *_dest) ...@@ -217,14 +254,17 @@ fs_copy(const char *_source, const char *_dest)
_source, _source,
-1, -1,
wsource, wsource,
MAX_PATH)); CLAR_MAX_PATH));
cl_assert(MultiByteToWideChar(CP_UTF8, cl_assert(MultiByteToWideChar(CP_UTF8,
MB_ERR_INVALID_CHARS, MB_ERR_INVALID_CHARS,
_dest, _dest,
-1, -1,
wdest, wdest,
MAX_PATH)); CLAR_MAX_PATH));
translate_path(wsource, CLAR_MAX_PATH);
translate_path(wdest, CLAR_MAX_PATH);
/* Check the source for existence */ /* Check the source for existence */
source_attrs = GetFileAttributesW(wsource); source_attrs = GetFileAttributesW(wsource);
...@@ -238,8 +278,8 @@ fs_copy(const char *_source, const char *_dest) ...@@ -238,8 +278,8 @@ fs_copy(const char *_source, const char *_dest)
* Use FindFirstFile to parse the path */ * Use FindFirstFile to parse the path */
find_handle = FindFirstFileW(wsource, &find_data); find_handle = FindFirstFileW(wsource, &find_data);
cl_assert(INVALID_HANDLE_VALUE != find_handle); cl_assert(INVALID_HANDLE_VALUE != find_handle);
wcscat_s(wdest, MAX_PATH, L"\\"); wcscat_s(wdest, CLAR_MAX_PATH, L"\\");
wcscat_s(wdest, MAX_PATH, find_data.cFileName); wcscat_s(wdest, CLAR_MAX_PATH, find_data.cFileName);
FindClose(find_handle); FindClose(find_handle);
/* Check the new target for existence */ /* Check the new target for existence */
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment