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

Tracking issues of cpp binding #2981

Open
6 of 8 tasks
Tracked by #1769
silver-ymz opened this issue Aug 31, 2023 · 6 comments
Open
6 of 8 tasks
Tracked by #1769

Tracking issues of cpp binding #2981

silver-ymz opened this issue Aug 31, 2023 · 6 comments
Assignees

Comments

@silver-ymz
Copy link
Member

silver-ymz commented Aug 31, 2023

Tasks

@Xuanwo
Copy link
Member

Xuanwo commented Aug 31, 2023

I am excited about the upcoming iostream and async support. I anticipate being able to utilize cpp binding in critical projects such as clickhouse, doris, and nebula (cc @wey-gu).

@silver-ymz
Copy link
Member Author

silver-ymz commented Sep 11, 2023

cpp binding now uses some unreleased API in v0.39, e.g. list and lister. Should we pin the version of opendal to 0.39 in cpp binding, make the first release and then revert back to current implementation?

@Xuanwo
Copy link
Member

Xuanwo commented Sep 11, 2023

Should we pin the version of opendal to 0.39 in cpp binding, make the first release and then revert back to current implementation?

No, we will release all projects under opendal at the same version.

@Xuanwo
Copy link
Member

Xuanwo commented Sep 11, 2023

By the way, when you mention releasing the cpp binding, what exactly do you mean? Are you referring to packaging our cpp binding and making it available in a registry?

@silver-ymz
Copy link
Member Author

silver-ymz commented Sep 11, 2023

By the way, when you mention releasing the cpp binding, what exactly do you mean?

Because CMakeLists.txt isn't in the root directory of project, we need to use cargo package to create a opendal-cpp.tar.gz individually in github release assets. This allows our users to import directly via cmake FetchContent without having to download and write additional cmake scripts.

Are you referring to packaging our cpp binding and making it available in a registry?

Cpp seems to have no central registers. About integrating with other popular package management like conan, vcpkg, we can complete it in the future.

@Xuanwo
Copy link
Member

Xuanwo commented Sep 11, 2023

This allows our users to import directly via cmake FetchContent without having to download and write additional cmake scripts.

Got it.

Cpp seems to have no central registers. About integrating with other popular package management like conan, vcpkg, we can complete it in the future.

LGTM.


Please be aware that in ASF, release is a crucial matter and all our releases must undergo the voting process. So we can't simply tag a v0.1 for cpp binding and put them in github assets.

To reduce the burden of our release mangaers, it's better to tag the same version for all our projects and release them after the voting.

Our next release windows is opendal v0.40 which expected to happen this week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants