Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[vogleditor] Build failure with Qt 5.5 #209

Open
imirkin opened this issue Jul 13, 2015 · 11 comments
Open

[vogleditor] Build failure with Qt 5.5 #209

imirkin opened this issue Jul 13, 2015 · 11 comments

Comments

@imirkin
Copy link

imirkin commented Jul 13, 2015

I'm seeing the following build failures. I have Qt 5.5.0 installed:

In file included from .../src/vogleditor/vogleditor_qsettings.cpp:1:0:
.../src/vogleditor/vogleditor_qsettings.h:50:1: error: expected class-name before '{' token
 {
 ^
.../src/vogleditor/vogleditor_qsettings.h:51:5: error: 'Q_OBJECT' does not name a type
     Q_OBJECT
     ^
.../src/vogleditor/vogleditor_qsettings.h:65:1: error: 'signals' does not name a type
 signals:
 ^
.../src/vogleditor/vogleditor_qsettings.h: In member function 'void vogleditor_qsettings::update_group_active_lists()':
.../src/vogleditor/vogleditor_qsettings.h:259:9: error: 'emit' was not declared in this scope
         emit treeDisplayChanged();
         ^
.../src/vogleditor/vogleditor_qsettings.h:259:14: error: expected ';' before 'treeDisplayChanged'
         emit treeDisplayChanged();
              ^

.../src/vogleditor/vogleditor_qsettings.cpp: At global scope:
.../src/vogleditor/vogleditor_qsettings.cpp:13:44: error: definition of implicitly-declared 'vogleditor_qsettings::vogleditor_qsettings()'
 vogleditor_qsettings::vogleditor_qsettings()
                                            ^
.../src/vogleditor/vogleditor_qsettings.cpp:1:0:
.../src/vogleditor/vogleditor_qsettings.h: In function 'void __static_initialization_and_destruction_0(int, int)':
.../src/vogleditor/vogleditor_qsettings.h:55:13: error: 'virtual vogleditor_qsettings::~vogleditor_qsettings()' is private
     virtual ~vogleditor_qsettings()
             ^
@karolherbst
Copy link

seems to be like this issue: QupZilla/qupzilla#970

@imirkin
Copy link
Author

imirkin commented Jul 13, 2015

Looks like the following patch fixes the problem.

diff --git a/src/vogleditor/vogleditor_qsettings.h b/src/vogleditor/vogleditor_qsettings.h
index 215d495..db8c171 100644
--- a/src/vogleditor/vogleditor_qsettings.h
+++ b/src/vogleditor/vogleditor_qsettings.h
@@ -3,6 +3,7 @@

 #include "vogl_dynamic_string.h"
 #include "vogl_json.h"
+#include <QObject>
 #include <QStringList>
 #include <QVector>

Courtesy of @karolherbst

@kankaristo
Copy link

Thanks @imirkin, fixed the build for me. 👍

kingtaurus added a commit to kingtaurus/vogl that referenced this issue Dec 6, 2015
Issue: [vogleditor] Build failure with Qt 5.5

(1) Requires an additional header file QObject to be included;

Build still works with Qt 5.5 and 5.4 (Ubuntu 15.10);
kingtaurus added a commit to kingtaurus/vogl that referenced this issue Dec 6, 2015
Issue: [vogleditor] Build failure with Qt 5.5

(1) Requires an additional header file QObject to be included;

Build still works with Qt 5.5 and 5.4 (Ubuntu 15.10);
@ghost
Copy link

ghost commented Jan 4, 2016

Same issue here, seems to not be fixed

@ghost
Copy link

ghost commented Jan 4, 2016

The missing include fixed it, thanks @kingtaurus

@AzP
Copy link

AzP commented Mar 20, 2016

Any chance someone will merge the pull request that fixes this? It's a shame that development seems to have stopped completely since several people still has use of this tool.

@perkele1989
Copy link

^ volvo plz

Plagman added a commit that referenced this issue May 13, 2016
@Plagman
Copy link
Member

Plagman commented May 13, 2016

I merged the pull request, but we don't have a maintainer for vogl anymore and it can be considered deprecated unless someone wants to take ownership of it.

@karolherbst
Copy link

karolherbst commented May 13, 2016

@Plagman and is there anything new to use or just good ol' apitrace?

@AzP
Copy link

AzP commented May 13, 2016

@karolherbst My impression from discussions at GDC is that RenderDoc has more or less replaced VoGL since it supports both OpenGL (only forward-compatible contexts) and Vulkan. But I'm also interested in a more concrete answer from @Plagman and Valve's view on it.

@Plagman
Copy link
Member

Plagman commented May 13, 2016

RenderDoc is what we'll be focusing on; mainly from a Vulkan perspective, but it should also work for OpenGL. Not clear when the new Qt UI will be fully up to speed to let it be hosted on Linux, but my understanding is that it's being worked on.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants