Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-65015

Visual Studio 2017 15.5 breaks VCRedist search in WinDeployQt

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done
    • Icon: P3: Somewhat important P3: Somewhat important
    • 5.9.4
    • 5.10.0 RC
    • None
    • Windows, MSVC 2017
    • f8bb798ade81f152bc8ff2028e4974150171fa08 (qttools/5.9, 8.12.2017, 5.9.4)

      Dear Maintainers,

      the release of Visual Studio 15.5 changed the name of the vc_redist files.

      Before: vcredist_$ARCH.exe (eg C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\VC\Redist\MSVC\14.11.25325\vcredist_x64.exe)

      Now: vc_redist.$ARCH.exe (eg ... 14.12.25810\vc_redist.x64.exe)

      They added a "." between redist and $ARCH, and a "_" in front.

       

      My proposal is adding a second check to qttools/src/windeployqt/main.cpp, Line 1122: (First part unchanged)

      const QFileInfo fi(releaseRedistDir + QLatin1Char('/')
       + QStringLiteral("vcredist_") + wordSizeString
       + QStringLiteral(".exe"));
       if (fi.isFile())
       redistFiles.append(fi.absoluteFilePath());
      
      // Second check for VS >= 14.12.25810
      
      const QFileInfo fiNewFormat(releaseRedistDir + QLatin1Char('/')
       + QStringLiteral("vc_redist.") + wordSizeString
       + QStringLiteral(".exe"));
       if (fiNewFormat.isFile())
       redistFiles.append(fiNewFormat.absoluteFilePath());
      

       

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

            kleint Friedemann Kleint
            phiber Philipp Berger
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved:

                There are no open Gerrit changes