debrouxl / tilibs

TILP (formerly GtkTiLink) can transfer data between Texas Instruments graphing calculators and a computer. It works with all link cables (parallel, serial, Black/Gray/Silver/Direct Link) and it supports the TI-Z80 series (73..86), the TI-eZ80 series (83PCE, 84+CE), the TI-68k series (89, 92, 92+, V200, 89T) and the Nspire series (Nspire Clickpad / Touchpad / CX, both CAS and non-CAS)
http://lpg.ticalc.org/prj_tilp
63 stars 23 forks source link

install_tilp.sh and other items outdated on the LPG site #51

Open scruss opened 3 years ago

scruss commented 3 years ago

On Ubuntu 20.04 / x86_64:

...
Making all in tests
make[2]: Entering directory '/root/lpg/tilp/tilibs/libticalcs/trunk/tests'
depbase=`echo test_ticalcs_2.o | sed 's|[^/]*$|.deps/&|;s|\.o$||'`;\
g++ -DHAVE_CONFIG_H -I. -I..  -I../src -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include -I/usr/local/include/tilp2 -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include -I/usr/include/tilp2 -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include -I/usr/include/tilp2   -g -O2 -D__LINUX__ -fvisibility=hidden -MT test_ticalcs_2.o -MD -MP -MF $depbase.Tpo -c -o test_ticalcs_2.o test_ticalcs_2.cc &&\
mv -f $depbase.Tpo $depbase.Po
test_ticalcs_2.cc:1091:8: error: ‘ticables_event_hook_type’ does not name a type; did you mean ‘ticalcs_event_hook_type’?
 1091 | static ticables_event_hook_type old_ticables_event_hook;
      |        ^~~~~~~~~~~~~~~~~~~~~~~~
      |        ticalcs_event_hook_type
test_ticalcs_2.cc:1094:80: error: ‘CableEventData’ does not name a type; did you mean ‘CalcEventData’?
 1094 | event_hook(CableHandle * handle, uint32_t event_count, const CableEventData * event, void * user_pointer)
      |                                                              ^~~~~~~~~~~~~~
      |                                                              CalcEventData
test_ticalcs_2.cc: In function ‘int cables_event_hook(CableHandle*, uint32_t, const int*, void*)’:
test_ticalcs_2.cc:1101:6: error: ‘old_ticables_event_hook’ was not declared in this scope; did you mean ‘old_ticalcs_event_hook’?
 1101 |  if (old_ticables_event_hook)
      |      ^~~~~~~~~~~~~~~~~~~~~~~
      |      old_ticalcs_event_hook
test_ticalcs_2.cc: In function ‘int main(int, char**)’:
test_ticalcs_2.cc:1270:2: error: ‘old_ticables_event_hook’ was not declared in this scope; did you mean ‘old_ticalcs_event_hook’?
 1270 |  old_ticables_event_hook = ticables_cable_set_event_hook(cable, cables_event_hook);
      |  ^~~~~~~~~~~~~~~~~~~~~~~
      |  old_ticalcs_event_hook
test_ticalcs_2.cc:1270:28: error: ‘ticables_cable_set_event_hook’ was not declared in this scope; did you mean ‘ticalcs_calc_set_event_hook’?
 1270 |  old_ticables_event_hook = ticables_cable_set_event_hook(cable, cables_event_hook);
      |                            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |                            ticalcs_calc_set_event_hook
make[2]: *** [Makefile:669: test_ticalcs_2.o] Error 1
make[2]: Leaving directory '/root/lpg/tilp/tilibs/libticalcs/trunk/tests'
make[1]: *** [Makefile:503: all-recursive] Error 1
make[1]: Leaving directory '/root/lpg/tilp/tilibs/libticalcs/trunk'
make: *** [Makefile:412: all] Error 2
debrouxl commented 3 years ago

I see both -I/usr/local/include/tilp2 and -I/usr/include/tilp2 in your log excerpt. That shouldn't occur on a clean build environment. Nowadays, tilpinst defaults to installing to /usr, unless you overrode that. Make sure to clean up the product of any previous manual compilation of libti* in /usr/local, or any -dev package installation for libticonv, libtifiles2, libticables2, libticalcs2, and tell me whether it works better that way :)

scruss commented 3 years ago

Got it, thanks. There was a lurking /usr/local/include/tilp2/ path from an installation I'd tried a couple of years ago.

Also, it didn't help that I used the obsolete script linked from http://lpg.ticalc.org/prj_tilp/linux.html - it's two years older (2016) than the one at https://ti-pla.net/tilpinst

debrouxl commented 3 years ago

Alright, thanks for reporting back. I'll reopen the bug and change the title to match the bug report about the script on the LPG website (and the whole LPG website, in fact...) being outdated :)