edeforas / Astree

Astree is a free open source optical ray tracing and design software
GNU Lesser General Public License v3.0
76 stars 9 forks source link

Compilation error #22

Closed probonopd closed 5 years ago

probonopd commented 5 years ago

Tried to build like this: https://github.com/probonopd/Astree/blob/96ae25cb7afb01758337a345933b7c16078ed1f4/.travis.yml

GlassCatalogIo.cpp:43:26: error: invalid user-defined conversion from ‘std::ifstream {aka std::basic_ifstream<char>}’ to ‘char**’ [-fpermissive]

         ::getline(f,sLine);

                          ^

In file included from /usr/include/c++/4.8/ios:44:0,

                 from /usr/include/c++/4.8/istream:38,

                 from /usr/include/c++/4.8/sstream:38,

                 from GlassCatalogIo.cpp:5:

/usr/include/c++/4.8/bits/basic_ios.h:115:7: note: candidate is: std::basic_ios<_CharT, _Traits>::operator void*() const [with _CharT = char; _Traits = std::char_traits<char>] <near match>

       operator void*() const

       ^

/usr/include/c++/4.8/bits/basic_ios.h:115:7: note:   no known conversion for implicit ‘this’ parameter from ‘void*’ to ‘char**’

GlassCatalogIo.cpp:43:26: error: cannot convert ‘std::string {aka std::basic_string<char>}’ to ‘size_t* {aka long unsigned int*}’ for argument ‘2’ to ‘__ssize_t getline(char**, size_t*, FILE*)’

         ::getline(f,sLine);

                          ^

make: *** [GlassCatalogIo.o] Error 1

make: *** Waiting for unfinished jobs....

The command "make -j$(nproc)" exited with 2.

Full build log: https://travis-ci.com/probonopd/Astree/builds/123544056

ghost commented 5 years ago

As first portability test run successfully,...

... I try build AppImage again:

But has something wrong with coping astree.desktop file into /usr/share/applications/astree.desktop

How should I solve this issue?

probonopd commented 5 years ago

I am now getting

The command "qmake CONFIG+=release PREFIX=/usr" exited with 1.. https://travis-ci.com/probonopd/Astree/builds/123605406

Here is how I an trying to compile it: https://github.com/probonopd/Astree/blob/96ae25cb7afb01758337a345933b7c16078ed1f4/.travis.yml

@edeforas how can I compile this using qmake Qt 5?

edeforas commented 5 years ago

Hello,

Thanks, i appreciate the idea to create portable apps under linux! Will do my best for Astree By the way , how do we store the data files ( i.e. glass catalog?) , near the executable?

Le dim. 18 août 2019 à 16:53, probonopd notifications@github.com a écrit :

I am now getting

The command "qmake CONFIG+=release PREFIX=/usr" exited with 1.. https://travis-ci.com/probonopd/Astree/builds/123605406

Here is how I an trying to compile it:

https://github.com/probonopd/Astree/blob/96ae25cb7afb01758337a345933b7c16078ed1f4/.travis.yml

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/edeforas/Astree/issues/22?email_source=notifications&email_token=AEBQLGYETGQEKYMQ56PLVL3QFFPAJA5CNFSM4IMLNRKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4RBUXY#issuecomment-522328671, or mute the thread https://github.com/notifications/unsubscribe-auth/AEBQLG5ILHXVYFH7RGMLTULQFFPAJANCNFSM4IMLNRKA .

ghost commented 5 years ago

Astree should check next paths on Linux:

AppImage should export distributed data (glass and samples) to ~/.local/share/astree/{glass|samples}/ and will use it.

ghost commented 5 years ago

As I see compilation still broken.

So, @edeforas try additinally include next PPA in .travis.yml (in before_install section) for get GCC 9 for Ubuntu 14.04

probonopd commented 5 years ago

I want to avoid that because it possibly introduces new runtime dependencies not present in Ubuntu 14.04 out of the box. Ubutnu 16.04 would also be an acceptable target system at this point, since 14.04 is no longer supported by Canonical.

probonopd commented 4 years ago

@azubieta now has a solution where you don't need Ubuntu 16.04: https://github.com/AppImageCrafters/appimage-builder

edeforas commented 4 years ago

Thanks!

Will try. Got also a Linux user so will have more tests.

Le ven. 8 mai 2020 à 08:48, probonopd notifications@github.com a écrit :

@azubieta https://github.com/azubieta now has a solution where you don't need Ubuntu 16.04: https://github.com/AppImageCrafters/appimage-builder

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/edeforas/Astree/issues/22#issuecomment-625664930, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEBQLGZ4GA6W36SSZSOLUR3RQOTMDANCNFSM4IMLNRKA .