Re: Is svn release 1758 missing fixes?

[Dr . Jürgen Sauermann]

    Hi Paul,
    
    thanks, fixed in SVN 1759 as proposed. I was initially
    thinking of 
    
    return st.st_mtime;
    
    myself but didn't do it because is is a macro and I wasn't sure if
    it is #define'd
    on all platform.
    
    Best Regards,
    Jürgen

On 3/4/24 02:49, Paul Rockwell wrote:

Mike,

I noticed the same issues in SVN 1757 and 1758 as well. I have made similar fixes in the source that I have.

Our fixes differ in that the fix I made to Command.cc is to change line 2030 from 

if (sort == SORT_TIME) return st.st_mtim.tv_sec;

2031

to

if (sort == SORT_TIME) return st.st_mtime;

2031

Both Linux and macOS #define st_mtime to be st.st_mtim.tv_sec (Linux) or st.st_mtimespec.tv_sec (macOS) to provide backward compatibility. That way we don't need the 

test for __APPLE__.

- Paul Rockwell