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

    • Bug
    • Resolution: Duplicate
    • P2: Important
    • None
    • 2.5.0 (rev.09)
    • Build-time integration
    • None
    • 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

              mecfc Miguel Costa
              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