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

Building issues #4

Open
biagio-trimarchi opened this issue Jan 8, 2024 · 2 comments
Open

Building issues #4

biagio-trimarchi opened this issue Jan 8, 2024 · 2 comments

Comments

@biagio-trimarchi
Copy link

I have found a series of issues in the building of the package.

First of all, there is missing include statements in the file Settings.h, in particular the file misses the #include<string_view> to use the string_view data type.

Moreover, the compiler raise the following error after fixing the missing include statement:

$ make
Consolidate compiler generated dependencies of target ScsEigen
[  7%] Building CXX object src/ScsEigen/CMakeFiles/ScsEigen.dir/src/Settings.cpp.o
/home/docker-dev/ws/scs_eigen_test/scs-eigen/src/ScsEigen/src/Settings.cpp: In member function ‘bool ScsEigen::Settings::eps(double)’:
/home/docker-dev/ws/scs_eigen_test/scs-eigen/src/ScsEigen/src/Settings.cpp:87:23: error: ‘struct ScsSettings’ has no member named ‘eps’
   87 |     m_pimpl->settings.eps = eps;
      |                       ^~~
/home/docker-dev/ws/scs_eigen_test/scs-eigen/src/ScsEigen/src/Settings.cpp: In member function ‘double ScsEigen::Settings::eps() const’:
/home/docker-dev/ws/scs_eigen_test/scs-eigen/src/ScsEigen/src/Settings.cpp:93:30: error: ‘struct ScsSettings’ has no member named ‘eps’
   93 |     return m_pimpl->settings.eps;
      |                              ^~~
make[2]: *** [src/ScsEigen/CMakeFiles/ScsEigen.dir/build.make:90: src/ScsEigen/CMakeFiles/ScsEigen.dir/src/Settings.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:158: src/ScsEigen/CMakeFiles/ScsEigen.dir/all] Error 2
make: *** [Makefile:136: all] Error 2```
@S-Dafarra
Copy link
Collaborator

cc @GiulioRomualdi

@GiulioRomualdi
Copy link
Owner

Hi @biagio-trimarchi, Thank you for reporting 😊 Let me check it better!

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

3 participants