search

ClickHouse / clickhouse-odbc

ODBC driver for ClickHouse
https://clickhouse.tech
Apache License 2.0
239 stars 84 forks source link

Please provide builds for the usual Linux distributions (Ubuntu, EL7, EL8, Debian) #420

Open aadant opened 1 year ago

aadant commented 1 year ago

This would solve the libicu issues that have been reported in a couple of places :

https://github.com/ClickHouse/clickhouse-odbc/issues/384 https://github.com/ClickHouse/clickhouse-odbc/issues/358

dima-altinity commented 1 year ago

there are multiple opened issues here with the same root cause " the pre-built Linux package is built on CentOS 7, and it depends on the default system version of ICU, which is ICU 50. " ICU 50 doesnt come in all Linux distributions and cause a pain for users to install clickhouse-odbc package. It makes sense to change build process to fix it. Possible solutions:

  1. Make builds for every major Linux distributions (Ubuntu 18,20,22..,EL7, EL8, Debian)
  2. Build it static
alexey-milovidov commented 1 year ago

@dima-altinity

Dependency on packaged libraries is nonsense; it should build all dependencies from the sources and link them statically. See how ClickHouse itself does it. The build with OS dependencies should not exist at all.

aholster commented 10 months ago

Fell down this rabbit hole for a few hours as well, please fix this issue, it was kinda of hard to figure out