I have gone through path '0001-Support-building-with-visual-studio-2019.patch' 
only, but I am sure some comments will also apply to back branches.

1. The VisualStudioVersion value looks odd:

+       $self->{VisualStudioVersion}        = '16.0.32.32432';

Are you using a pre-release version [1]?

2. There is a typo: s/stuido/studio/:

+       # The major visual stuido that is suppored has nmake version >= 14.20 
and < 15.

There is something in the current code that I think should be also updated. The 
code for _GetVisualStudioVersion contains:

  if ($major > 14)
        {
        carp
         "The determined version of Visual Studio is newer than the latest 
supported version. Returning the latest supported version instead.";
        return '14.00';
        }

Shouldn't the returned value be '14.20' for Visual Studio 2019?

Regards,

Juan José Santamaría Flecha

[1] 
https://docs.microsoft.com/en-us/visualstudio/releases/2019/history#release-dates-and-build-numbers

Reply via email to