FriendsofECCE / ECCE

The purpose of this repo is to allow continued development of the Extensible Computational Chemistry Environment (ECCE) which used to be maintained by the PNNL
Other
9 stars 5 forks source link

Build fails #35

Open yurivict opened 6 years ago

yurivict commented 6 years ago

clang-6 fails to build it on FreeBSD 11.2:

c++ -O2 -pipe -fno-omit-frame-pointer  -fstack-protector -isystem /usr/local/include -fno-strict-aliasing -I/usr/ports/science/ecce/work/ECCE-7.3.4-beta-8-gaf79f45/include -I/usr/include/freetype2/freetype -I/usr/local/lib/wx/include/gtk2-unicode-3.0 -I/usr/local/include/wx-3.0 -D_FILE_OFFSET_BITS=64 -DWXUSINGDLL -D__WXGTK__ -pthread -D_THREAD_SAFE -I/usr/ports/science/ecce/work/ECCE-7.3.4-beta-8-gaf79f45/3rdparty/xerces/include -c BookmarkEditor.C -o /usr/ports/science/ecce/work/ECCE-7.3.4-beta-8-gaf79f45/obj/organizer/BookmarkEditor.o
BookmarkEditor.C:85:28: error: no viable conversion from 'wxCStrData' to 'std::__1::string' (aka 'basic_string<char, char_traits<char>, allocator<char> >')
      setBookmarkName(pos, value.c_str());
                           ^~~~~~~~~~~~~
mattasplund commented 6 years ago

Sorry to be slow getting back to you. I have never tried to compile with clang, only with gcc. The error looks like it is in the wxWindows code, which is one of the older less well maintained parts of the code, so I am not certain what to say. wx has moved a major version past the one the Ecce uses, specifically to deal with unicode issues, and this looks like where the problem is.

On Aug 28, 2018, at 1:36 PM, yuri@FreeBSD notifications@github.com wrote:

clang-6 fails to build it on FreeBSD 11.2:

c++ -O2 -pipe -fno-omit-frame-pointer -fstack-protector -isystem /usr/local/include -fno-strict-aliasing -I/usr/ports/science/ecce/work/ECCE-7.3.4-beta-8-gaf79f45/include -I/usr/include/freetype2/freetype -I/usr/local/lib/wx/include/gtk2-unicode-3.0 -I/usr/local/include/wx-3.0 -D_FILE_OFFSET_BITS=64 -DWXUSINGDLL -DWXGTK -pthread -D_THREAD_SAFE -I/usr/ports/science/ecce/work/ECCE-7.3.4-beta-8-gaf79f45/3rdparty/xerces/include -c BookmarkEditor.C -o /usr/ports/science/ecce/work/ECCE-7.3.4-beta-8-gaf79f45/obj/organizer/BookmarkEditor.o BookmarkEditor.C:85:28: error: no viable conversion from 'wxCStrData' to 'std::__1::string' (aka 'basic_string<char, char_traits, allocator >') setBookmarkName(pos, value.c_str()); ^~~~~ — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/FriendsofECCE/ECCE/issues/35, or mute the thread https://github.com/notifications/unsubscribe-auth/AcDHgEjRBrIo7hxUBG1KV6yDXGBztE5rks5uVZvPgaJpZM4WQQyv.

Matthew Asplund matt@asplundfamily.org

yurivict commented 6 years ago

There is also this error:

KeyValueReader.C:68:17: error: comparison between pointer and integer ('char *' and 'int')
    while (cptr != '\0') {
           ~~~~ ^  ~~~~