[vlc-devel] [PATCH] qt: use local-path for ui header wrapper

Jean-Baptiste Kempf jb at videolan.org
Sat Mar 7 12:44:13 CET 2020


LGTM

On Sat, Mar 7, 2020, at 12:35, Alexandre Janniaux wrote:
> Those files could generated in implementation-specified location by
> build systems and we don't need a full path. With meson, it will be
> handled automatically and having a full path makes the file not found.
> 
> Use local-path and specify all include directories to simulate this.
> ---
>  modules/gui/qt/Makefile.am                         | 14 +++++++++++++-
>  .../gui/qt/dialogs/extended/extended_panels.hpp    |  6 ++++--
>  .../qt/dialogs/fingerprint/fingerprintdialog.cpp   |  4 +++-
>  modules/gui/qt/dialogs/help/help.hpp               |  6 ++++--
>  modules/gui/qt/dialogs/messages/messages.hpp       |  5 ++++-
>  modules/gui/qt/dialogs/open/open.hpp               |  4 +++-
>  modules/gui/qt/dialogs/open/open_panels.hpp        |  9 +++++----
>  .../qt/dialogs/podcast/podcast_configuration.hpp   |  4 +++-
>  .../qt/dialogs/preferences/simple_preferences.hpp  | 11 ++++++-----
>  modules/gui/qt/dialogs/sout/profile_selector.hpp   |  4 +++-
>  modules/gui/qt/dialogs/sout/sout.hpp               |  4 +++-
>  modules/gui/qt/dialogs/vlm/vlm.hpp                 |  4 +++-
>  12 files changed, 54 insertions(+), 21 deletions(-)
> 
> diff --git a/modules/gui/qt/Makefile.am b/modules/gui/qt/Makefile.am
> index 627cc5c5ab..ac424fd5fa 100644
> --- a/modules/gui/qt/Makefile.am
> +++ b/modules/gui/qt/Makefile.am
> @@ -13,8 +13,20 @@
>  
>  SUFFIXES += .ui .h .hpp .moc.cpp .qml
>  
> +uidirs = \
> +	dialogs/extended \
> +	dialogs/fingerprint \
> +	dialogs/help \
> +	dialogs/messages \
> +	dialogs/open \
> +	dialogs/podcast \
> +	dialogs/preferences \
> +	dialogs/sout \
> +	dialogs/vlm
> +
>  libqt_plugin_la_CPPFLAGS = $(AM_CPPFLAGS) \
> -	-I$(srcdir)/gui/qt -I$(builddir)/gui/qt/ -I$(builddir)/gui/qt/ui
> +	-I$(srcdir)/gui/qt -I$(builddir)/gui/qt/ \
> +	$(addprefix -I$(builddir)/gui/qt/,$(uidirs))
>  libqt_plugin_la_CXXFLAGS = $(AM_CXXFLAGS) $(QT_CFLAGS) $(CXXFLAGS_qt)
>  libqt_plugin_la_LIBADD = $(QT_LIBS) $(LIBS_qt) $(LIBM)
>  libqt_plugin_la_LDFLAGS = $(AM_LDFLAGS) -rpath '$(guidir)'
> diff --git a/modules/gui/qt/dialogs/extended/extended_panels.hpp 
> b/modules/gui/qt/dialogs/extended/extended_panels.hpp
> index 98f0d84376..f0c775d6fb 100644
> --- a/modules/gui/qt/dialogs/extended/extended_panels.hpp
> +++ b/modules/gui/qt/dialogs/extended/extended_panels.hpp
> @@ -31,10 +31,12 @@
>  #include <vlc_common.h>
>  
>  #include "qt.hpp"
> -#include "dialogs/extended/ui_equalizer.h"
> -#include "dialogs/extended/ui_video_effects.h"
>  #include "util/variables.hpp"
>  
> +/* Auto-generated from .ui files */
> +#include "ui_equalizer.h"
> +#include "ui_video_effects.h"
> +
>  #include <QTabWidget>
>  
>  #define BANDS 10
> diff --git a/modules/gui/qt/dialogs/fingerprint/fingerprintdialog.cpp 
> b/modules/gui/qt/dialogs/fingerprint/fingerprintdialog.cpp
> index a2efeda3ab..468b69bd3d 100644
> --- a/modules/gui/qt/dialogs/fingerprint/fingerprintdialog.cpp
> +++ b/modules/gui/qt/dialogs/fingerprint/fingerprintdialog.cpp
> @@ -19,7 +19,9 @@
>   
> *****************************************************************************/
>  
>  #include "dialogs/fingerprint/fingerprintdialog.hpp"
> -#include "dialogs/fingerprint/ui_fingerprintdialog.h"
> +
> +/* Auto-generated from .ui files */
> +#include "ui_fingerprintdialog.h"
>  
>  #include "dialogs/fingerprint/chromaprint.hpp"
>  #include <vlc_url.h>
> diff --git a/modules/gui/qt/dialogs/help/help.hpp 
> b/modules/gui/qt/dialogs/help/help.hpp
> index 5754e06b9a..96c10981dd 100644
> --- a/modules/gui/qt/dialogs/help/help.hpp
> +++ b/modules/gui/qt/dialogs/help/help.hpp
> @@ -31,8 +31,10 @@
>  
>  #include "widgets/native/qvlcframe.hpp"
>  #include "util/singleton.hpp"
> -#include "dialogs/help/ui_about.h"
> -#include "dialogs/help/ui_update.h"
> +
> +/* Auto-generated from .ui files */
> +#include "ui_about.h"
> +#include "ui_update.h"
>  
>  class QEvent;
>  
> diff --git a/modules/gui/qt/dialogs/messages/messages.hpp 
> b/modules/gui/qt/dialogs/messages/messages.hpp
> index c2fd2850d1..00a81fbeb3 100644
> --- a/modules/gui/qt/dialogs/messages/messages.hpp
> +++ b/modules/gui/qt/dialogs/messages/messages.hpp
> @@ -25,7 +25,10 @@
>  
>  #include "widgets/native/qvlcframe.hpp"
>  #include "util/singleton.hpp"
> -#include "dialogs/messages/ui_messages_panel.h"
> +
> +/* Auto-generated from .ui files */
> +#include "ui_messages_panel.h"
> +
>  #include <stdarg.h>
>  #include <QMutex>
>  #include <QAtomicInt>
> diff --git a/modules/gui/qt/dialogs/open/open.hpp 
> b/modules/gui/qt/dialogs/open/open.hpp
> index 0ae45ecc95..e14442cb03 100644
> --- a/modules/gui/qt/dialogs/open/open.hpp
> +++ b/modules/gui/qt/dialogs/open/open.hpp
> @@ -31,9 +31,11 @@
>  #include <vlc_url.h>
>  
>  #include "widgets/native/qvlcframe.hpp"
> -#include "dialogs/open/ui_open.h"
>  #include "dialogs/open/open_panels.hpp"
>  
> +/* Auto-generated from .ui files */
> +#include "ui_open.h"
> +
>  enum {
>      OPEN_FILE_TAB,
>      OPEN_DISC_TAB,
> diff --git a/modules/gui/qt/dialogs/open/open_panels.hpp 
> b/modules/gui/qt/dialogs/open/open_panels.hpp
> index 57fffd53d5..b91e9343f5 100644
> --- a/modules/gui/qt/dialogs/open/open_panels.hpp
> +++ b/modules/gui/qt/dialogs/open/open_panels.hpp
> @@ -33,10 +33,11 @@
>  
>  #include "dialogs/preferences/preferences_widgets.hpp"
>  
> -#include "dialogs/open/ui_open_file.h"
> -#include "dialogs/open/ui_open_disk.h"
> -#include "dialogs/open/ui_open_net.h"
> -#include "dialogs/open/ui_open_capture.h"
> +/* Auto-generated from .ui files */
> +#include "ui_open_file.h"
> +#include "ui_open_disk.h"
> +#include "ui_open_net.h"
> +#include "ui_open_capture.h"
>  
>  #include <QFileDialog>
>  
> diff --git a/modules/gui/qt/dialogs/podcast/podcast_configuration.hpp 
> b/modules/gui/qt/dialogs/podcast/podcast_configuration.hpp
> index 65308b9440..1e99d67bf6 100644
> --- a/modules/gui/qt/dialogs/podcast/podcast_configuration.hpp
> +++ b/modules/gui/qt/dialogs/podcast/podcast_configuration.hpp
> @@ -24,9 +24,11 @@
>  #define QVLC_PODCAST_CONFIGURATION_DIALOG_H_ 1
>  
>  #include "widgets/native/qvlcframe.hpp"
> -#include "dialogs/podcast/ui_podcast_configuration.h"
>  #include "util/singleton.hpp"
>  
> +/* Auto-generated from .ui files */
> +#include "ui_podcast_configuration.h"
> +
>  class PodcastConfigDialog : public QVLCDialog, public 
> Singleton<PodcastConfigDialog>
>  {
>      Q_OBJECT
> diff --git a/modules/gui/qt/dialogs/preferences/simple_preferences.hpp 
> b/modules/gui/qt/dialogs/preferences/simple_preferences.hpp
> index be31f6ecde..32d7df9fc2 100644
> --- a/modules/gui/qt/dialogs/preferences/simple_preferences.hpp
> +++ b/modules/gui/qt/dialogs/preferences/simple_preferences.hpp
> @@ -30,11 +30,12 @@
>  #include <vlc_common.h>
>  #include <vlc_interface.h>
>  
> -#include "dialogs/preferences/ui_sprefs_input.h"
> -#include "dialogs/preferences/ui_sprefs_audio.h"
> -#include "dialogs/preferences/ui_sprefs_video.h"
> -#include "dialogs/preferences/ui_sprefs_subtitles.h"
> -#include "dialogs/preferences/ui_sprefs_interface.h"
> +/* Auto-generated from .ui files */
> +#include "ui_sprefs_input.h"
> +#include "ui_sprefs_audio.h"
> +#include "ui_sprefs_video.h"
> +#include "ui_sprefs_subtitles.h"
> +#include "ui_sprefs_interface.h"
>  
>  #include <vlc_media_library.h>
>  #include <QDialogButtonBox>
> diff --git a/modules/gui/qt/dialogs/sout/profile_selector.hpp 
> b/modules/gui/qt/dialogs/sout/profile_selector.hpp
> index c1c37a50f8..ad08a785e5 100644
> --- a/modules/gui/qt/dialogs/sout/profile_selector.hpp
> +++ b/modules/gui/qt/dialogs/sout/profile_selector.hpp
> @@ -31,7 +31,9 @@
>  
>  #include "widgets/native/qvlcframe.hpp"
>  #include "util/soutchain.hpp"
> -#include "dialogs/sout/ui_profiles.h"
> +
> +/* Auto-generated from .ui files */
> +#include "ui_profiles.h"
>  
>  class QComboBox;
>  
> diff --git a/modules/gui/qt/dialogs/sout/sout.hpp 
> b/modules/gui/qt/dialogs/sout/sout.hpp
> index ca8c5aa5b3..d42fe6bf43 100644
> --- a/modules/gui/qt/dialogs/sout/sout.hpp
> +++ b/modules/gui/qt/dialogs/sout/sout.hpp
> @@ -29,7 +29,9 @@
>  
>  #include <vlc_common.h> /* Gettext functions */
>  
> -#include "dialogs/sout/ui_sout.h"
> +/* Auto-generated from .ui files */
> +#include "ui_sout.h"
> +
>  #include "widgets/native/qvlcframe.hpp"
>  #include "util/soutchain.hpp"
>  
> diff --git a/modules/gui/qt/dialogs/vlm/vlm.hpp 
> b/modules/gui/qt/dialogs/vlm/vlm.hpp
> index 234c676898..7f60ae5c19 100644
> --- a/modules/gui/qt/dialogs/vlm/vlm.hpp
> +++ b/modules/gui/qt/dialogs/vlm/vlm.hpp
> @@ -30,7 +30,9 @@
>  
>  #include <vlc_vlm.h>
>  
> -#include "dialogs/vlm/ui_vlm.h"
> +/* Auto-generated from .ui files */
> +#include "ui_vlm.h"
> +
>  #include "widgets/native/qvlcframe.hpp"
>  #include "util/singleton.hpp"
>  #include <QDateTime>
> -- 
> 2.25.1
> 
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel

-- 
Jean-Baptiste Kempf -  President
+33 672 704 734


More information about the vlc-devel mailing list