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

Incorrect invocation of moc when include directories are quoted

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: P2: Important
    • Resolution: Fixed
    • Affects Version/s: 2.4.3, 2.5.0, 2.5.0 (rev.07), 2.5.0 (rev.09), 2.5.1 (rev.08)
    • Fix Version/s: 2.5.1 (rev.18), 2.5.2
    • Component/s: Build-time integration
    • Labels:
      None
    • Environment:
      VS2017
    • Platform/s:
      Windows
    • Commits:
      e93d8f4c2df8cb05bc4ca6edbd09b396c2cbb25b

      Description

      Reporting against 2.4.3 but probably applies to all releases.

      When trying to build my project after adding QWinWidget, I got a nastygram out of moc which was the result of directory names in the include path that contained spaces:

      10>moc qwinwidget.h
      10> moc: Too many input files specified: 'D:\Users\amonra\Documents\GitHub\DSS\DeepSkyStacker\qwinwidget.h' 'Files' '(x86)\Visual' 'Leak' 'Detector\include'

      My VS2017 C++ Additional Include directories was set to:

      .\GeneratedFiles$(ConfigurationName);.\GeneratedFiles;"C:\Program Files (x86)\Visual Leak Detector\include";.;../Libraw;..\ZClass;..\tools;..\LibTIFF;..\CFitsIO;..\Zlib;%(AdditionalIncludeDirectories)

      and the moc settings in VS project properties are set like this:

      It worked OK when I removed the quotes around

      "C:\Program Files (x86)\Visual Leak Detector\include"

      So that does rather confirm the problem.

      PS Yes, I do know VLD is less useful with Qt than it could be because of all the global objects Qt creates but ...

        Attachments

          Issue Links

          For Gerrit Dashboard: QTVSADDINBUG-777
          # Subject Branch Project Status CR V

            Activity

              People

              Assignee:
              mecfc Miguel Costa
              Reporter:
              perdrix David Partridge
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Gerrit Reviews

                  There are no open Gerrit changes