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

4.2.0.43 compile failure with gcc 8.2 #408

Closed
glensc opened this issue Dec 10, 2018 · 2 comments
Closed

4.2.0.43 compile failure with gcc 8.2 #408

glensc opened this issue Dec 10, 2018 · 2 comments

Comments

@glensc
Copy link
Contributor

glensc commented Dec 10, 2018

prepare:

git checkout v4.2.0
git submodule update
cmake .
make
[~/rpm/packages/digidoc4-client/DigiDoc4-Client] ➔ make
[  2%] Built target translations_1
[  2%] Automatic MOC for target qtsingleapplication
[  2%] Built target qtsingleapplication_autogen
[  5%] Built target qtsingleapplication
[  6%] Automatic MOC for target qdigidoccommon
[  6%] Built target qdigidoccommon_autogen
[ 27%] Built target qdigidoccommon
[ 27%] Automatic MOC for target qdigidoc4
[ 27%] Built target qdigidoc4_autogen
[ 28%] Automatic MOC for target TSLDownload
[ 28%] Built target TSLDownload_autogen
[ 29%] Built target TSLDownload
[ 30%] Building CXX object client/CMakeFiles/qdigidoc4.dir/Application.cpp.o
/home/users/glen/rpm/packages/digidoc4-client/DigiDoc4-Client/client/Application.cpp:170:7: error: ‘void DigidocConf::setTSUrl(const string&)’ marked ‘override’, but does not override
  void setTSUrl(const std::string &url) override
       ^~~~~~~~
/home/users/glen/rpm/packages/digidoc4-client/DigiDoc4-Client/client/Application.cpp: In destructor ‘virtual Application::~Application()’:
/home/users/glen/rpm/packages/digidoc4-client/DigiDoc4-Client/client/Application.cpp:413:12: warning: possible problem detected in invocation of delete operator: [-Wdelete-incomplete]
  delete d->bar;
            ^~~
/home/users/glen/rpm/packages/digidoc4-client/DigiDoc4-Client/client/Application.cpp:413:12: warning: invalid use of incomplete type ‘class MacMenuBar’
  delete d->bar;
         ~~~^~~
/home/users/glen/rpm/packages/digidoc4-client/DigiDoc4-Client/client/Application.cpp:67:7: note: forward declaration of ‘class MacMenuBar’
 class MacMenuBar;
       ^~~~~~~~~~
/home/users/glen/rpm/packages/digidoc4-client/DigiDoc4-Client/client/Application.cpp:413:12: note: neither the destructor nor the class-specific operator delete will be called, even if they are declared when the class is defined
  delete d->bar;
            ^~~
/home/users/glen/rpm/packages/digidoc4-client/DigiDoc4-Client/client/Application.cpp: In static member function ‘static void Application::setConfValue(Application::ConfParameter, const QVariant&)’:
/home/users/glen/rpm/packages/digidoc4-client/DigiDoc4-Client/client/Application.cpp:851:19: error: ‘digidoc::XmlConfCurrent’ {aka ‘class digidoc::XmlConfV2’} has no member named ‘setTSUrl’; did you mean ‘TSUrl’?
   case TSAUrl: i->setTSUrl(v.isEmpty()? std::string() : v.constData()); break;
                   ^~~~~~~~
                   TSUrl
make[2]: *** [client/CMakeFiles/qdigidoc4.dir/build.make:391: client/CMakeFiles/qdigidoc4.dir/Application.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:383: client/CMakeFiles/qdigidoc4.dir/all] Error 2
make: *** [Makefile:130: all] Error 2
[~/rpm/packages/digidoc4-client/DigiDoc4-Client] ➔
➔ g++ --version
g++ (PLD-Linux) 8.2.0 20181009 (release)
Copyright (C) 2018 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
@metsma
Copy link
Contributor

metsma commented Dec 10, 2018

/home/users/glen/rpm/packages/digidoc4-client/DigiDoc4-Client/client/Application.cpp:170:7: error: > ‘void DigidocConf::setTSUrl(const string&)’ marked ‘override’, but does not override
void setTSUrl(const std::string &url) override

You need latest (3.13.8) libdigidocpp

@metsma metsma closed this as completed Dec 10, 2018
@glensc
Copy link
Contributor Author

glensc commented Dec 10, 2018

the versioned dependency should be then defined in cmake rules. please re-open.

@metsma metsma reopened this Dec 10, 2018
glensc added a commit to pld-linux/digidoc4-client that referenced this issue Dec 10, 2018
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

2 participants