Uploaded image for project: 'Qt for Python'
  1. Qt for Python
  2. PYSIDE-506

shiboken (5.9 branch) Wrong include option in ApiExtractor/clangparser/compilersupport.cpp

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: P2: Important
    • Resolution: Done
    • Affects Version/s: 5.9, dev
    • Fix Version/s: 5.9
    • Component/s: Shiboken
    • Labels:
    • Commits:
      6c816e0b6302f436143680359b787ab7a81a390e

      Description

      includeOption(const HeaderPath &p) should return -isystem instead of -I for system header path otherwise build is broken.

      Look at the difference:

      $ clang++ -E -x c++ - -v < /dev/null 
      #include <...> search starts here:
       /usr/include/c++/v1
       /usr/bin/../lib/clang/4.0.0/include
       /usr/include
      End of search list.
      
      $ clang++ -nostdinc -isystem /usr/include/c++/v1 -isystem /usr/bin/../lib/clang/4.0.0/include -isystem /usr/include -E -x c++ - -v < /dev/null
      #include <...> search starts here:
       /usr/include/c++/v1
       /usr/bin/../lib/clang/4.0.0/include
       /usr/include
      End of search list.
      
      $ clang++ -nostdinc -I/usr/include/c++/v1 -I/usr/bin/../lib/clang/4.0.0/include  -I/usr/include -E -x c++ - -v < /dev/null
      #include <...> search starts here:
       /usr/bin/../lib/clang/4.0.0/include
       /usr/include
       /usr/include/c++/v1
      End of search list.
      

        Attachments

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

          Activity

            People

            Assignee:
            alexandru.croitor Alexandru Croitor
            Reporter:
            sylvaingarrigues Sylvain Garrigues
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Gerrit Reviews

                There are no open Gerrit changes