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

Adopt swift-toolchain-sqlite to reduce system dependencies #127

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

MaxDesiatov
Copy link
Contributor

This means we no longer require users to provide their own system SQLite installation on Linux or Windows.

This means we no longer require users to provide their own system SQLite installation on Linux or Windows.
@MaxDesiatov MaxDesiatov requested a review from bnbarham August 27, 2024 18:14
@MaxDesiatov
Copy link
Contributor Author

@swift-ci test

MaxDesiatov added a commit that referenced this pull request Aug 27, 2024
Verified manually that `swift-sdk-generator` itself can be cross-compiled when using #127.
@MaxDesiatov MaxDesiatov added the dependencies Changes to dependencies label Aug 27, 2024
@bnbarham
Copy link
Contributor

Suppose this is a question for @jakepetroules - should swift-toolchain-sqlite use the system dependency for Apple platforms, or is it intentional that all platforms are just using the amalgamation?

MaxDesiatov added a commit that referenced this pull request Aug 28, 2024
Verified manually that `swift-sdk-generator` itself can be cross-compiled when using #127.
@MaxDesiatov
Copy link
Contributor Author

@swift-ci test

@MaxDesiatov
Copy link
Contributor Author

FWIW without this change due to SwiftPM's bug in systemLibrary support we get this warning on every macOS build

warning: failed to retrieve search paths with pkg-config; maybe pkg-config is not installed
warning: couldn't find pc file for sqlite3

@MaxDesiatov
Copy link
Contributor Author

@swift-ci test

…/swift-toolchain-sqlite

# Conflicts:
#	Package.swift
@MaxDesiatov
Copy link
Contributor Author

@swift-ci test

@xtremekforever
Copy link
Contributor

Are you going to merge this? Just curious, as it seems like it could be a good improvement.

@MaxDesiatov
Copy link
Contributor Author

@swift-ci test

@jakepetroules
Copy link

jakepetroules commented Mar 13, 2025

Suppose this is a question for @jakepetroules - should swift-toolchain-sqlite use the system dependency for Apple platforms, or is it intentional that all platforms are just using the amalgamation?

Sorry I'm only seeing this now -- it's the intention that only platforms which need it are using the amalgamation, while platforms that have an accessible system dependency use that one. So amalgamation would currently be Windows and WebAssembly IIRC.

See llbuild for example of usage.

@bnbarham
Copy link
Contributor

See llbuild for example of usage.

I was more wondering whether we think swift-toolchain-sqlite should do this itself (ie. have an empty target that depends on the system sqlite in the non-windows/wasm case), rather than all clients in the toolchain.

@MaxDesiatov
Copy link
Contributor Author

Swift SDK Generator is not a part of the installable toolchain, although it's a part of some toolchain CI jobs. Swift SDK Generator users build it locally on their machine vast majority of the time, and minimizing the amount of prerequisites for them is crucial. That's why I'd prefer to use this new package on Linux too, especially SQLite dev headers installation instructions can be so different for every Linux distribution.

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

Successfully merging this pull request may close these issues.

4 participants