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

pip3 install => build.cpp:68:1: error: expected '}' at end of input #99

Closed
tseemann opened this issue Apr 3, 2018 · 4 comments
Closed
Assignees

Comments

@tseemann
Copy link

tseemann commented Apr 3, 2018

Any ideas?

   Scanning dependencies of target gramtools
    [ 47%] Building CXX object libgramtools/CMakeFiles/gramtools.dir/src/common/utils.cpp.o
    [ 48%] Building CXX object libgramtools/CMakeFiles/gramtools.dir/src/common/timer_report.cpp.o
    [ 49%] Building CXX object libgramtools/CMakeFiles/gramtools.dir/src/search/search.cpp.o
    /tmp/linuxbrew/pip-hx3exc5r-build/libgramtools/src/search/search.cpp: In function 'std::__cxx11::string serialize_search_state(const SearchState&)':
    /tmp/linuxbrew/pip-hx3exc5r-build/libgramtools/src/search/search.cpp:675:41: warning: switch condition has type bool [-Wswitch-bool]
         switch (search_state.cache_populated) {
                                             ^
    [ 50%] Building CXX object libgramtools/CMakeFiles/gramtools.dir/src/build/build.cpp.o
    In file included from /tmp/linuxbrew/pip-hx3exc5r-build/libgramtools/src/build/build.cpp:10:0:
    /tmp/linuxbrew/pip-hx3exc5r-build/libgramtools/include/build/build.hpp:4:19: error: expected '{' before '::' token
     namespace commands::build {
                       ^
    /tmp/linuxbrew/pip-hx3exc5r-build/libgramtools/include/build/build.hpp:4:21: error: 'build' in namespace '::' does not name a type
     namespace commands::build {
                         ^
    /tmp/linuxbrew/pip-hx3exc5r-build/libgramtools/src/build/build.cpp:13:16: error: 'commands::build' has not been declared
     void commands::build::run(const Parameters &parameters) {
                    ^
    /tmp/linuxbrew/pip-hx3exc5r-build/libgramtools/src/build/build.cpp:68:1: error: expected '}' at end of input
     }
     ^
    make[2]: *** [libgramtools/CMakeFiles/gramtools.dir/build.make:102: libgramtools/CMakeFiles/gramtools.dir/src/build/build.cpp.o] Error 1
    make[1]: *** [CMakeFiles/Makefile2:103: libgramtools/CMakeFiles/gramtools.dir/all] Error 2
    make: *** [Makefile:84: all] Error 2
    error: [Errno 2] No such file or directory: '/tmp/linuxbrew/pip-hx3exc5r-build/cmake-build-debug/libgramtools/tests/test_main': '/tmp/linuxbrew/pip-hx3exc5r-build/cmake-build-debug/libgramtools/tests/test_main'

    ----------------------------------------
Command "/home/linuxbrew/.linuxbrew/opt/python/bin/python3.6 -u -c "import setuptools, tokenize;__file__='/tmp/linuxbrew/pip-hx3exc5r-build/setup.py';f=getattr(tokenize, 'open', open)(__file__);code=f.read().replace('\r\n', '\n');f.close();exec(compile(code, __file__, 'exec'))" install --record /tmp/linuxbrew/pip-3pi1_8nx-record/install-record.txt --single-version-externally-managed --compile" failed with error code 1 in /tmp/linuxbrew/pip-hx3exc5r-build/

@iqbal-lab
Copy link
Collaborator

The upshot of our pip experience is we are building stand alone executable binaries.
In the middle of that process @tseemann

@ffranr ffranr self-assigned this Apr 4, 2018
@ffranr
Copy link
Contributor

ffranr commented Apr 4, 2018

@tseemann Thank you for taking the time to point out this problem!

I think this occurs because the C++ compiler used for building the backend does not support C++17.

The solution to this problem has two parts:

  1. CMake ensures compiler handles C++17 (CMake ensures compiler handles C++17 #100)
  2. Release includes pre built backend binaries (Release includes pre built backend binaries #101)

@tseemann
Copy link
Author

tseemann commented Apr 7, 2018

Ok, yeah I don't have a C++17 capable compiler by default. If i get time i'll try brew install [email protected] and see how I go. But I suspect i'll get problems integrating into my python setup.

@ffranr
Copy link
Contributor

ffranr commented Jun 7, 2018

Solving #101 (Release includes pre built backend binaries) will take a bit more time than expected. #100 (CMake ensures compiler handles C++17) has been implemented and I think that it should be enough to provide an informative error. With that, I'll close this issue and leave #101 open.

@ffranr ffranr closed this as completed Jun 7, 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

3 participants