RE: [PATCH] Fix for GCC '-MF' option cannot deal with long paths in Windows

[<sundeep.kokkonda@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 975 bytes --]

Hello Andrew,

I updated the patch. Please see attached patch files and let me know your feedback.



Thanks,
Sundeep K.

-----Original Message-----
From: Andrew Pinski <pinskia@gmail.com> 
Sent: Tuesday, January 18, 2022 9:22 AM
To: sundeep.kokkonda@gmail.com
Cc: GCC Patches <gcc-patches@gcc.gnu.org>
Subject: Re: [PATCH] Fix for GCC '-MF' option cannot deal with long paths in Windows

On Mon, Jan 17, 2022 at 8:35 AM Sundeep KOKKONDA via Gcc-patches <gcc-patches@gcc.gnu.org> wrote:
>
> Hello,
>
>
>
> The '-MF' option cannot deal with path size >256 characters in 
> Windows. This patch is to fix this long path issue with -MF option.
>
> Let me know is it ok to commit?

I don't think this is the right place to put this.
Maybe a better way to implement this is to have a wrapper around fopen for windows and do it that way.
Does open have the same issue?

Thanks,
Andrew Pinski

>
>
>
>
>
> --
>
> Thanks,
>
> Sundeep K.
>

[-- Attachment #2: lrealpath.patch --]
[-- Type: application/octet-stream, Size: 1219 bytes --]

diff --git a/libiberty/lrealpath.c b/libiberty/lrealpath.c
index 3c7053b0b70..593b51c2d96 100644
--- a/libiberty/lrealpath.c
+++ b/libiberty/lrealpath.c
@@ -36,6 +36,8 @@ components will be simplified.  The returned value will be allocated using
 #include "config.h"
 #include "ansidecl.h"
 #include "libiberty.h"
+#include <malloc.h>
+#include "filenames.h"
 
 #ifdef HAVE_LIMITS_H
 #include <limits.h>
@@ -138,6 +140,32 @@ lrealpath (const char *filename)
   {
     char buf[MAX_PATH];
     char* basename;
+
+    size_t filelen;
+    
+    filelen = strlen (filename) + 1;
+
+    if (filelen > MAX_PATH - 1)
+    {
+	    char * fullpath;
+	    fullpath = (char *) malloc (filelen+8);
+
+	    filelen = strlen (filename)+1;
+	    strcpy(fullpath,"\\\\?\\");
+	    strcat(fullpath,filename);
+
+	    for (int i = 0; fullpath[i]; i++)
+	    {
+		    if (IS_UNIX_DIR_SEPARATOR (fullpath[i]))
+			    fullpath[i] = '\\';
+	    }
+
+	    strcpy(filename,fullpath);
+	    free(fullpath);
+
+	    return (strdup(filename));
+    }
+
     DWORD len = GetFullPathName (filename, MAX_PATH, buf, &basename);
     if (len == 0 || len > MAX_PATH - 1)
       return strdup (filename);

[-- Attachment #3: c-opts.patch --]
[-- Type: application/octet-stream, Size: 492 bytes --]

diff --git a/gcc/c-family/c-opts.c b/gcc/c-family/c-opts.c
index aa6801fa811..0f019fb1caa 100644
--- a/gcc/c-family/c-opts.c
+++ b/gcc/c-family/c-opts.c
@@ -1291,7 +1291,7 @@ c_common_finish (void)
 	deps_stream = stdout;
       else
 	{
-	  deps_stream = fopen (deps_file, deps_append ? "a": "w");
+	  deps_stream = fopen (lrealpath(deps_file), deps_append ? "a": "w");
 	  if (!deps_stream)
 	    fatal_error (input_location, "opening dependency file %s: %m",
 			 deps_file);