Skip to content

add 'swift' - #4840

Merged
mxcl merged 6 commits into
pkgxdev:mainfrom
krzyzanowskim:new/swift.org
Jan 17, 2024
Merged

mxcl merged 6 commits into
pkgxdev:mainfrom
krzyzanowskim:new/swift.org

Conversation

@krzyzanowskim

@krzyzanowskim krzyzanowskim commented Jan 14, 2024 •

Copy link
Copy Markdown
Contributor

Vendored Swift packages for Darwin and Linux; however, the Linux platform is disabled for now due to platform requirement: ubuntu 22.04. Unless somebody can make it run tests on Ubuntu

@krzyzanowskim

This comment was marked as resolved.

@krzyzanowskim
krzyzanowskim marked this pull request as ready for review January 14, 2024 20:22
@jhheider
jhheider requested a review from mxcl January 15, 2024 02:34

@mxcl mxcl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great job

@mxcl

mxcl commented Jan 17, 2024

Copy link
Copy Markdown
Contributor

Ideally we’d avoid resigning this since Apple signed it.

Also: I assume we have license to redistribute, but one day we may find out we didn’t.

@mxcl
mxcl merged commit 231e832 into pkgxdev:main Jan 17, 2024
@krzyzanowskim
krzyzanowskim deleted the new/swift.org branch January 17, 2024 11:03
@krzyzanowskim

Copy link
Copy Markdown
Contributor Author

I assume we have license to redistribute

since it is downloaded by the user from the https://swift.org it's not really distributed by pkgx, or is it?

@mxcl

mxcl commented Jan 17, 2024

Copy link
Copy Markdown
Contributor

It is distributed by us. The build script runs on our infra to convert everything into bottles that pkgx then downloads from our hosting.

@krzyzanowskim

Copy link
Copy Markdown
Contributor Author

right, I assume it all is under Apache 2.0 as it states on https://www.swift.org/download/ let's assume that ;)

@krzyzanowskim

Copy link
Copy Markdown
Contributor Author

@mxcl I'd like you to weigh in on Linux support. I lack the skills how to make it "per distribution" only for a vendored package

@mxcl

mxcl commented Jan 17, 2024 •

Copy link
Copy Markdown
Contributor

@krzyzanowskim distributions for a specific hardware platform are (almost always) just variations on underlying deps. So if Apple provide a swift for ubuntu where the ubuntu glibc matches our requirements (v23 I think) then it'll be fine. We provide the other dependencies and it should just work at runtime.

Failing that we have to build from source.

Though once we finish packaging glibc (#147) that issue could also be alleviated.

Important

I say all this but distributions can do bad things with underlying dependencies that cause binaries built for them to
become extremely specific. Off hand I don't recall ever seeing this, but I'm sure it has happened.

@mxcl mxcl mentioned this pull request Jan 18, 2024
tannevaled pushed a commit to tannevaled/pantry that referenced this pull request Aug 27, 2026
* add 'swift'
* Conditional for platforms
* Update provides per platform
* linux dep on gcc
* Add deps
* Disable linux platform
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