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

The driver is obviously wrong #428

Closed
alexey-milovidov opened this issue Sep 24, 2023 · 2 comments
Closed

The driver is obviously wrong #428

alexey-milovidov opened this issue Sep 24, 2023 · 2 comments

Comments

@alexey-milovidov
Copy link
Member

This is a wrong practice and not going to work:

milovidov@milovidov-desktop:~/work/clickhouse-odbc/build$ ldd driver/libclickhouseodbc.so 
        linux-vdso.so.1 (0x00007fffab910000)
        libodbcinst.so.2 => /lib/x86_64-linux-gnu/libodbcinst.so.2 (0x00007fd6f8cc3000)
        libicuuc.so.70 => /lib/x86_64-linux-gnu/libicuuc.so.70 (0x00007fd6f8ac8000)
        libstdc++.so.6 => /lib/x86_64-linux-gnu/libstdc++.so.6 (0x00007fd6f889c000)
        libm.so.6 => /lib/x86_64-linux-gnu/libm.so.6 (0x00007fd6f87b5000)
        libgcc_s.so.1 => /lib/x86_64-linux-gnu/libgcc_s.so.1 (0x00007fd6f8795000)
        libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007fd6f856b000)
        /lib64/ld-linux-x86-64.so.2 (0x00007fd6f9191000)
        libltdl.so.7 => /lib/x86_64-linux-gnu/libltdl.so.7 (0x00007fd6f8560000)
        libicudata.so.70 => /lib/x86_64-linux-gnu/libicudata.so.70 (0x00007fd6f6942000)

The right way to build a self-contained shared library is to use --whole-archive and to remove all irrelevant symbols with a linker script. I did it initially: ff642ef

@mshustov
Copy link
Member

related #245

@alesapin
Copy link
Member

alesapin commented Oct 2, 2024

Fixed by #456

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

Successfully merging a pull request may close this issue.

3 participants