Re: Recent newlib/cygwin change breaking cris targets

Jeff Law <[email protected]>
Newsgroups gmane.comp.lib.newlib
Message-ID <[email protected]>
On 8/19/19 4:23 PM, Jeff Law wrote:
> On 8/19/19 4:05 PM, Jeff Johnston wrote:
>> I have posted a fix that should solve the problem.  I don't have a
>> cris-elf toolchain to test so if there are any issues,
>> please let me know.
> THanks.  My tester will pick it up ~24hrs after the commit across all
> the targets it knows about.
cris32-elf and crisv32-elf are now working.

visium has similar breakage which appears to be fixable via the attached
patch.  I'm not sure if I've got commit privs in newlib, so if it looks
good to you, I'd appreciate it if you could commit on my behalf.


That's the last fallout I see from those changes.  Everything else in my
tester is building OK.

jeff
P (text/plain, 1.2 KB)
diff --git a/libgloss/visium/io-gdb.c b/libgloss/visium/io-gdb.c
index 4f16b2f7b..e662226e3 100644
--- a/libgloss/visium/io-gdb.c
+++ b/libgloss/visium/io-gdb.c
@@ -94,9 +94,9 @@ __hosted_from_gdb_stat (const struct gdb_stat *gs,
   s->st_size = gs->st_size;
   s->st_blksize = gs->st_blksize;
   s->st_blocks = gs->st_blocks;
-  s->st_atime = gs->st_atime;
-  s->st_mtime = gs->st_mtime;
-  s->st_ctime = gs->st_ctime;
+  s->st_atime = gs->st_atim;
+  s->st_mtime = gs->st_mtim;
+  s->st_ctime = gs->st_ctim;
 }
 
 void
diff --git a/libgloss/visium/io.h b/libgloss/visium/io.h
index a1bb879fb..9cb76ea30 100644
--- a/libgloss/visium/io.h
+++ b/libgloss/visium/io.h
@@ -57,9 +57,9 @@ struct gdb_stat {
   uint64_t    st_size;    /* total size, in bytes */
   uint64_t    st_blksize; /* blocksize for filesystem I/O */
   uint64_t    st_blocks;  /* number of blocks allocated */
-  gdb_time_t  st_atime;   /* time of last access */
-  gdb_time_t  st_mtime;   /* time of last modification */
-  gdb_time_t  st_ctime;   /* time of last change */
+  gdb_time_t  st_atim;   /* time of last access */
+  gdb_time_t  st_mtim;   /* time of last modification */
+  gdb_time_t  st_ctim;   /* time of last change */
 };
 
 struct gdb_timeval {
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.