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

clickhouse-cpp abseil link error #33914

Open
Chrys4lisfag opened this issue Sep 21, 2023 · 6 comments
Open

clickhouse-cpp abseil link error #33914

Chrys4lisfag opened this issue Sep 21, 2023 · 6 comments
Assignees
Labels
requires:repro The issue is not currently repro-able Stale

Comments

@Chrys4lisfag
Copy link

Chrys4lisfag commented Sep 21, 2023

Describe the bug
Can't compile the app with clickhouse-cpp package

clickhouse-cpp-lib-static.lib(decimal.cpp.obj) : error LNK2001: unresolved external symbol "class absl::int128 __cdecl absl::operator/(class absl::int128,class absl::int128)" (??Kabsl@@ya?AVint128@0@V10@0@Z)

Environment

  • OS: windows 10
  • Compiler: MVSC 22 x64 c++20

To Reproduce
Steps to reproduce the behavior:

  1. ./vcpkg install clickhouse-cpp:x64-windows-static

`clickhouse::ClientOptions options;
options.SetHost( "censored" );
options.SetPort( 8123 );
options.SetDefaultDatabase( "censored" );
options.SetUser( "censored" );
options.SetPassword( "censored" );
clickhouse::Client client( options );

client.Select( "SELECT max(censored) as max FROM censored.censored;",
			   []( const clickhouse::Block& block )
{


	auto censored = block[ 0 ]->As<clickhouse::ColumnUInt64>()->At( 0 );
}
);`
@LilyWangLL LilyWangLL added the requires:more-information This Issue requires more information to solve label Sep 22, 2023
@LilyWangLL
Copy link
Contributor

Thanks for posting this issue. Could you please provide the sample repro project?

@Chrys4lisfag
Copy link
Author

Chrys4lisfag commented Sep 22, 2023

Thanks for posting this issue. Could you please provide the sample repro project?

https://github.com/Chrys4lisfag/vcpkg-clickhouse-issue-repo
x64-release is configured

I've also seen mention of this issue here: #27250 (review)

@Chrys4lisfag
Copy link
Author

Thanks for posting this issue. Could you please provide the sample repro project?

Hi, has the issue been confirmed? Do you need anything more for me to check?

@LilyWangLL LilyWangLL added requires:repro The issue is not currently repro-able and removed requires:more-information This Issue requires more information to solve labels Sep 26, 2023
@LilyWangLL
Copy link
Contributor

I will test this issue locally soon.

@Chrys4lisfag
Copy link
Author

Chrys4lisfag commented Oct 21, 2023

I will test this issue locally soon.

Hi, I've seen that package was updated and the issue is fixed now.

Everything works except for the compression option, I get crc mismatch when enable it.

As I understand this can be caused by wrong cityhash package bundled
ClickHouse/clickhouse-cpp#325

Thank you for fixing compiling issue, I hope that compression will be fixed as well!

Copy link

This is an automated message. Per our repo policy, stale issues get closed if there has been no activity in the past 28 days. The issue will be automatically closed in 14 days. If you wish to keep this issue open, please add a new comment.

@github-actions github-actions bot added the Stale label Nov 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
requires:repro The issue is not currently repro-able Stale
Projects
None yet
Development

No branches or pull requests

2 participants