Uploaded image for project: 'Qt Visual Studio Tools'
  1. Qt Visual Studio Tools
  2. QTVSADDINBUG-776

Incorrect quoting of paths used in calls to Qt tools

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: P2: Important
    • Resolution: Duplicate
    • Affects Version/s: 2.5.0 (rev.09)
    • Fix Version/s: None
    • Component/s: Qt/MSBuild
    • Labels:
      None
    • Platform/s:
      Windows

      Description

      When using paths with surrounding quotes in C++ properties, a second set of surrounding quotes is added when passing command-line arguments to Qt tools like moc.

      For example, setting the Additional Include Directories property to "C:\foo bar";%(AdditionalIncludeDirectories) will result in quotes added to all include paths, even those already containing surrounding quotes:

      "-I"C:\foo bar"" "-ID:\lib\Qt\5.13.2\msvc2017_64\include" "-ID:\lib\Qt\5.13.2\msvc2017_64\include\QtWidgets" "-ID:\lib\Qt\5.13.2\msvc2017_64\include\QtGui" "-ID:\lib\Qt\5.13.2\msvc2017_64\include\QtANGLE" "-ID:\lib\Qt\5.13.2\msvc2017_64\include\QtCore" "-I/include" "-ID:\lib\Qt\5.13.2\msvc2017_64\mkspecs\win32-msvc" "-Ix64\Debug\moc" "-Ix64\Debug\uic"

      The C++ compiler options do not have this issue:

      /Ix64\Debug\uic /Ix64\Debug\moc /I"C:\foo bar" /ID:\lib\Qt\5.13.2\msvc2017_64\include /ID:\lib\Qt\5.13.2\msvc2017_64\include\QtWidgets /ID:\lib\Qt\5.13.2\msvc2017_64\include\QtGui /ID:\lib\Qt\5.13.2\msvc2017_64\include\QtANGLE /ID:\lib\Qt\5.13.2\msvc2017_64\include\QtCore /I/include /I"D:\lib\Qt\5.13.2\msvc2017_64\mkspecs\win32-msvc"

       
       
      (reported originally in: https://forum.qt.io/topic/114726/bug-with-moc-parsing-of-include-directories)

        Attachments

          Issue Links

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

            Activity

              People

              Assignee:
              mecfc Miguel Costa
              Reporter:
              mecfc Miguel Costa
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Gerrit Reviews

                  There are no open Gerrit changes