<div dir="ltr">Yes, you are right, config.h inside os_win32\vc16 has those defines commented out.<div>Despite that, the build fails. It appears VS 2022 is referencing the config.h from the root folder instead of the one inside vc16.</div><div>In the Solution Explorer, the config.h file there is the vc16 version, so I'm not sure why it's using the other one in the root.</div><div>When I replace the root config.h with the vc16 copy, it fixes the problem right away.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Apr 27, 2022 at 12:49 PM Christian Franke <<a href="mailto:Christian.Franke@t-online.de" target="_blank">Christian.Franke@t-online.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Eaton Zveare wrote:<br>
> Hi Christian, the line endings/format are correct in the files I have. <br>
> I did not need to make changes.<br>
> I made the int128 change and it actually made no difference. Digging <br>
> deeper, I found that I also had to remove these defines:<br>
><br>
>   * HAVE_UNISTD_H<br>
>   * HAVE_ATTR_PACKED<br>
><br>
> Only then did VS 2022 finally compile. You didn't have to remove those <br>
> defines to get it to work?<br>
<br>
Correct. The sed script in the Makefile should remove these during copy from<br>
   .../smartmontools/vctmp/config.h<br>
to<br>
   .../smartmontools/os_win32/vc16/config.h<br>
<br>
Try to diff both files.<br>
<br>
Because newer 'sed' versions no longer implicitly convert input files to <br>
LF, this currently only works if the original config.h has already LF <br>
line endings.<br>
<br>
I only tested with Cygwin, so this might also be a MSYS specific problem.<br>
<br>
</blockquote></div>