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

Fix MSVC linking issue with workaround #10865

Merged
merged 1 commit into from
Feb 27, 2024

Conversation

samansmink
Copy link
Contributor

@samansmink samansmink commented Feb 27, 2024

Was an easy workaround thanks to the investigation done by the people at TileDB who ran into the same issue: TileDB-Inc/TileDB#4759.

this PR reenables the azure and aws extension where this link issue would trigger

@Mytherin Mytherin merged commit 0b32ec2 into duckdb:main Feb 27, 2024
44 checks passed
@Mytherin
Copy link
Collaborator

Thanks! LGTM

github-actions bot pushed a commit to duckdb/duckdb-r that referenced this pull request Mar 15, 2024
Merge pull request duckdb/duckdb#10862 from Mytherin/batchinsertmemory
Merge pull request duckdb/duckdb#10865 from samansmink/fix-mvsc-link-problem
Merge pull request duckdb/duckdb#10860 from carlopi/build_fixes
carlopi added a commit to carlopi/duckdb that referenced this pull request May 16, 2024
This requires some thoughs since this is very brittle, connected to duckdb#10865
carlopi added a commit to carlopi/duckdb that referenced this pull request May 16, 2024
This requires some thoughts since specifiying version explicitly is very brittle\
Connected to duckdb#10865
carlopi added a commit to carlopi/duckdb that referenced this pull request Jun 7, 2024
It seems to not be necessessary anymore
carlopi added a commit to carlopi/duckdb that referenced this pull request Jun 7, 2024
It seems to not be necessary anymore
Mytherin added a commit that referenced this pull request Jun 8, 2024
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.

None yet

2 participants