diff options
author | Gerald Combs <gerald@wireshark.org> | 2018-04-05 18:00:16 -0700 |
---|---|---|
committer | Anders Broman <a.broman58@gmail.com> | 2018-04-06 03:11:52 +0000 |
commit | 0cb3176ab770c6a5f69ffc5a5cd8889a3d5b23a0 (patch) | |
tree | ae4b2e16b94120dedf7170fb2dd7cd4e8a69ed7a | |
parent | 5fd4659164536a494480b4717e6f6b5ae530c119 (diff) |
Remove some references to Qt4.
Change-Id: I50f5abafb8adf18a5407427f7feb260301df45cc
Reviewed-on: https://code.wireshark.org/review/26771
Reviewed-by: Gerald Combs <gerald@wireshark.org>
Petri-Dish: Gerald Combs <gerald@wireshark.org>
Tested-by: Petri Dish Buildbot
Reviewed-by: Anders Broman <a.broman58@gmail.com>
-rw-r--r-- | README.bsd | 12 | ||||
-rw-r--r-- | doc/README.developer | 13 | ||||
-rw-r--r-- | docbook/wsdg_src/WSDG_chapter_libraries.asciidoc | 3 | ||||
-rw-r--r-- | packaging/nsis/windeployqt-to-nsis.ps1 | 10 | ||||
-rw-r--r-- | packaging/wix/windeployqt-to-wix.ps1 | 22 | ||||
-rw-r--r-- | ui/qt/CMakeLists.txt | 23 | ||||
-rw-r--r-- | ui/qt/Makefile.am | 5 | ||||
-rw-r--r-- | ui/qt/main_window_slots.cpp | 3 | ||||
-rw-r--r-- | ui/qt/rpc_service_response_time_dialog.cpp | 2 |
9 files changed, 16 insertions, 77 deletions
diff --git a/README.bsd b/README.bsd index 09f636afe3..e2a747548b 100644 --- a/README.bsd +++ b/README.bsd @@ -2,7 +2,7 @@ Installing Wireshark on FreeBSD/OpenBSD/NetBSD/DragonFly BSD ======================================================================== 1. Extra packages required - 2. Compiling Wireshark + 2. Compiling Wireshark 3. Berkeley Packet Filter (BPF) requirement 4. Running Wireshark as a non-root user @@ -16,23 +16,19 @@ The easiest way to install these is by using your operating system's ports or packages system. If you prefer to build from source, the programs can be found at the following sites: - glib 2.16 or later: + glib 2.32 or later: ftp.gnome.org:/pub/gnome/sources/glib/ http://ftp.gnome.org/pub/gnome/sources/glib/ pkgconfig: http://pkgconfig.freedesktop.org/releases/ - python 2.5 or later: + python 2.7 or later, including Python 3: https://www.python.org/downloads/source/ If you want to use the Wireshark GUI, install one or both of these toolkits: - gtk+ 2.12 or later: - ftp.gnome.org:/pub/gnome/sources/gtk+/ - http://ftp.gnome.org/pub/gnome/sources/gtk+/ - - Qt 4.7 or later: + Qt 5.2 or later: http://download.qt-project.org/official_releases/qt/ diff --git a/doc/README.developer b/doc/README.developer index 6d2872a5cd..42f2d0e31a 100644 --- a/doc/README.developer +++ b/doc/README.developer @@ -409,14 +409,11 @@ own mkstemp implementation for use on platforms that lack mkstemp. Note: mkstemp does not accept NULL as a parameter. Wireshark requires mininum versions of each of the libraries it uses, in -particular GLib 2.22.0, Qt 4.8.0, and GTK+ 2.12.0 or newer. If you -require a mechanism that is available only in a newer version of a -library then use its version detection macros, e.g. "#if -GLIB_CHECK_VERSION(...)", "#if QT_VERSION_CHECK(...)", and "#if -GTK_CHECK_VERSION(...)" to conditionally compile code using that -mechanism. If you're using GTK_CHECK_VERSION be aware that GTK+ will be -removed in a future release. Adding new features to the GTK+ UI is -discouraged. +particular GLib 2.32.0 and Qt 5.2.0 or newer. If you require a mechanism +that is available only in a newer version of a library then use its +version detection macros, e.g. "#if GLIB_CHECK_VERSION(...)" and "#if +QT_VERSION_CHECK(...)" to conditionally compile code using that +mechanism. When different code must be used on UN*X and Win32, use a #if or #ifdef that tests _WIN32, not WIN32. Try to write code portably whenever diff --git a/docbook/wsdg_src/WSDG_chapter_libraries.asciidoc b/docbook/wsdg_src/WSDG_chapter_libraries.asciidoc index c184f867cf..0e8a12dc65 100644 --- a/docbook/wsdg_src/WSDG_chapter_libraries.asciidoc +++ b/docbook/wsdg_src/WSDG_chapter_libraries.asciidoc @@ -86,8 +86,7 @@ must be set appropriately. === Qt The Qt library is used to build the UI for Wireshark and is used to provide a -platform independent UI. Wireshark can be built with Qt 4.7.0 or later, but we -recommend using Qt5. +platform independent UI. Wireshark can be built with Qt 5.2 or later. For more information on the Qt libraries, see <<ChUIQt>>. diff --git a/packaging/nsis/windeployqt-to-nsis.ps1 b/packaging/nsis/windeployqt-to-nsis.ps1 index 8fed6baff0..a28139ba48 100644 --- a/packaging/nsis/windeployqt-to-nsis.ps1 +++ b/packaging/nsis/windeployqt-to-nsis.ps1 @@ -100,14 +100,6 @@ File "..\..\wireshark-qt-release\Qt5PrintSupport.dll" File /r "..\..\wireshark-qt-release\platforms" "@ - } else { - # Assume Qt 4 - - $nsisCommands += @" -File "..\..\wireshark-qt-release\QtCore4.dll" -File "..\..\wireshark-qt-release\QtGui4.dll" -"@ - } } @@ -126,4 +118,4 @@ Set-Content $FilePath @" # "@ -Add-Content $FilePath $nsisCommands
\ No newline at end of file +Add-Content $FilePath $nsisCommands diff --git a/packaging/wix/windeployqt-to-wix.ps1 b/packaging/wix/windeployqt-to-wix.ps1 index b621e55e63..70623a2b3d 100644 --- a/packaging/wix/windeployqt-to-wix.ps1 +++ b/packaging/wix/windeployqt-to-wix.ps1 @@ -179,28 +179,6 @@ try { </Fragment> "@ - } else { - # Assume Qt 4 - - $wixComponents += @" - <Fragment> - <DirectoryRef Id=`"INSTALLFOLDER`"> - <Component Id=`"cmpQt4Core_dll`" Guid=`"*`"> - <File Id=`"filQt4Core_dll`" KeyPath=`"yes`" Source=`"`$(var.WiresharkQt.Dir)\QtCore4.dll`" /> - </Component> - <Component Id=`"cmpQt4Gui_dll`" Guid=`"*`"> - <File Id=`"filQt4Gui_dll`" KeyPath=`"yes`" Source=`"`$(var.WiresharkQt.Dir)\QtGui4.dll`" /> - </Component> - </DirectoryRef> - </Fragment> - <Fragment> - <ComponentGroup Id=`"CG.QtDependencies`"> - <ComponentRef Id=`"cmpQt4Core_dll`" /> - <ComponentRef Id=`"cmpQt4Gui_dll`" /> - </ComponentGroup> - </Fragment> -"@ - } $wixComponents += @" diff --git a/ui/qt/CMakeLists.txt b/ui/qt/CMakeLists.txt index 901b8c5308..a052a7dfa9 100644 --- a/ui/qt/CMakeLists.txt +++ b/ui/qt/CMakeLists.txt @@ -589,24 +589,13 @@ include_directories( ${CMAKE_CURRENT_SOURCE_DIR} ) -# Define compatibility macros for Qt 4.8 -if(QT_VERSION EQUAL 4) - add_definitions(-DQ_NULLPTR=NULL) -endif() - SET(CMAKE_AUTOMOC TRUE) if(CMAKE_VERSION VERSION_LESS "3.0.2") # AUTOUIC and AUTORCC are not supported in CMAKE 2.8.x - if (QT_VERSION EQUAL 5) - QT5_ADD_TRANSLATION(WIRESHARK_QT_QM ${WIRESHARK_QT_TS}) - QT5_ADD_RESOURCES(WIRESHARK_QT_QRC_SRC ${WIRESHARK_QT_QRC}) - QT5_WRAP_UI(WIRESHARK_QT_UI_SRC ${WIRESHARK_QT_UI}) - else() - QT4_ADD_TRANSLATION(WIRESHARK_QT_QM ${WIRESHARK_QT_TS}) - QT4_ADD_RESOURCES(WIRESHARK_QT_QRC_SRC ${WIRESHARK_QT_QRC}) - QT4_WRAP_UI(WIRESHARK_QT_UI_SRC ${WIRESHARK_QT_UI}) - endif() + QT5_ADD_TRANSLATION(WIRESHARK_QT_QM ${WIRESHARK_QT_TS}) + QT5_ADD_RESOURCES(WIRESHARK_QT_QRC_SRC ${WIRESHARK_QT_QRC}) + QT5_WRAP_UI(WIRESHARK_QT_UI_SRC ${WIRESHARK_QT_UI}) else() SET(CMAKE_AUTOUIC TRUE) SET(CMAKE_AUTORCC TRUE) @@ -616,11 +605,7 @@ else() # Add .qrc files to library dependencies for AUTORCC. set(WIRESHARK_QT_QRC_SRC ${WIRESHARK_QT_QRC}) - if (QT_VERSION EQUAL 5) - QT5_ADD_TRANSLATION(WIRESHARK_QT_QM ${WIRESHARK_QT_TS}) - else() - QT4_ADD_TRANSLATION(WIRESHARK_QT_QM ${WIRESHARK_QT_TS}) - endif() + QT5_ADD_TRANSLATION(WIRESHARK_QT_QM ${WIRESHARK_QT_TS}) endif() add_custom_target( diff --git a/ui/qt/Makefile.am b/ui/qt/Makefile.am index 7426dd68d4..cf84267c76 100644 --- a/ui/qt/Makefile.am +++ b/ui/qt/Makefile.am @@ -27,11 +27,6 @@ if HAVE_SPEEXDSP AM_CPPFLAGS += $(SPEEXDSP_CFLAGS) endif -# Define compatibility macros for Qt 4.8 -if HAVE_QT_VERSION_4 -AM_CPPFLAGS += -DQ_NULLPTR=NULL -endif - noinst_LIBRARIES = libqtui.a include Makefile_custom.common diff --git a/ui/qt/main_window_slots.cpp b/ui/qt/main_window_slots.cpp index 4193a62c7e..b23d22c408 100644 --- a/ui/qt/main_window_slots.cpp +++ b/ui/qt/main_window_slots.cpp @@ -1836,9 +1836,6 @@ void MainWindow::on_actionNewDisplayFilterExpression_triggered() main_ui_->filterExpressionFrame->addExpression(df_combo_box_->lineEdit()->text()); } -// On Qt4 + macOS with unifiedTitleAndToolBarOnMac set it's possible to make -// the main window obnoxiously wide. - void MainWindow::displayFilterButtonClicked() { QAction *dfb_action = qobject_cast<QAction*>(sender()); diff --git a/ui/qt/rpc_service_response_time_dialog.cpp b/ui/qt/rpc_service_response_time_dialog.cpp index 30db4c5e06..360f7b1789 100644 --- a/ui/qt/rpc_service_response_time_dialog.cpp +++ b/ui/qt/rpc_service_response_time_dialog.cpp @@ -150,7 +150,7 @@ TapParameterDialog *RpcServiceResponseTimeDialog::createDceRpcSrtDialog(QWidget // dcerpc,srt,<uuid>,<major version>.<minor version>[,<filter>] QStringList args_l = QString(opt_arg).split(','); if (args_l.length() > 1) { - // Alas, QUuid requires Qt 4.8. + // XXX Switch to QUuid. unsigned d1, d2, d3, d4_0, d4_1, d4_2, d4_3, d4_4, d4_5, d4_6, d4_7; if(sscanf(args_l[0].toUtf8().constData(), "%08x-%04x-%04x-%02x%02x-%02x%02x%02x%02x%02x%02x", |