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

ActiveQt does not install enough headers

    XMLWordPrintable

Details

    • Bug
    • Resolution: Done
    • P1: Critical
    • 5.0.0
    • 5.0.0 Beta 2
    • ActiveX Support
    • None
    • 2fe214ca90cfda68126bf6e83e876a3a00635521,7d4051d3bbc341574ca7a9bdf31d5f14306c1367

    Description

      The public header qaxselect.h includes ui_qaxselect.h. I don't know if other include problems exist. I noticed this one when testing if #include <ActiveQt> works. (Update: this part is now fixed)

      The Qt CI system likely did not catch this as the install of Qt is not autotested at all. Tests are run against the build-dir. (See also QTBUG-27315)

      Also, the fact that headers are in the 'ActiveQt' directory instead of a directory prefixed with 'Qt' should be considered a bug that can be fixed for Qt 5.0. The porting script from QtGui/QWidget to QtWidgets/QWidget can probably be extended to help there.

      qtactiveqt also does not follow other Qt conventions, as it does not use the qt_module.prf, and it represents multiple libraries instead of one. It should really be two modules - AxContainer and AxServer.

      Attachments

        Issue Links

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

          Activity

            People

              buddenha Oswald Buddenhagen
              stephen.kelly Stephen Kelly (Unused account) (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              13 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes