Skip to content

fix ydb: implementation of using installed instance of ydb-cpp-sdk - #1291

Open
vasily-sviridov wants to merge 1 commit into
userver-framework:developfrom
vasily-sviridov:sviridov-patch-ydb-install
Open

fix ydb: implementation of using installed instance of ydb-cpp-sdk#1291
vasily-sviridov wants to merge 1 commit into
userver-framework:developfrom
vasily-sviridov:sviridov-patch-ydb-install

Conversation

@vasily-sviridov

@vasily-sviridov vasily-sviridov commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Note: by creating a PR or an issue you automatically agree to the CLA. See CONTRIBUTING.md. Feel free to remove this note, the agreement holds.

@vasily-sviridov vasily-sviridov changed the title fix(ydb): load installed package dependencies before targets fix(ydb): implementation of using installed instance of ydb Jul 8, 2026
@vasily-sviridov vasily-sviridov changed the title fix(ydb): implementation of using installed instance of ydb fix(ydb): implementation of using installed instance of ydb-cpp-sdk Jul 9, 2026
@apolukhin apolukhin changed the title fix(ydb): implementation of using installed instance of ydb-cpp-sdk fix ydb: implementation of using installed instance of ydb-cpp-sdk Aug 2, 2026
@apolukhin

Copy link
Copy Markdown
Member

LGTM

apolukhin
apolukhin previously approved these changes Aug 3, 2026
@apolukhin apolukhin closed this Aug 9, 2026
@apolukhin apolukhin reopened this Aug 9, 2026
@apolukhin
apolukhin requested a review from kpavlov00 as a code owner August 9, 2026 10:52
@apolukhin

Copy link
Copy Markdown
Member

@vasily-sviridov please rebase the PR. We're eager to merge it :)

@vasily-sviridov

Copy link
Copy Markdown
Contributor Author

@apolukhin Done! I rebased it and dropped the SetupYdbCppSDK.cmake bits since upstream handles that now.

@apolukhin

Copy link
Copy Markdown
Member

LGTM

@robot-magpie

robot-magpie Bot commented Aug 9, 2026

Copy link
Copy Markdown

Many thanks for the PR! @apolukhin is now importing your pull request into our internal upstream repository.

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 this pull request may close these issues.

2 participants