Skip to content

build: always copy the modulemaps #412

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

Merged
merged 1 commit into from
Oct 29, 2018
Merged

Conversation

compnerd
Copy link
Member

Change from a symlink to a copy. This is more portable and fixes the
distribution aspect. When the installation occurs, the symbolic link is
not followed and a symbolic link is installed. Rather if we copy, we
can get the contents. The files are small so the cost is relatively
low.

Change from a symlink to a copy.  This is more portable and fixes the
distribution aspect.  When the installation occurs, the symbolic link is
not followed and a symbolic link is installed.  Rather if we copy, we
can get the contents.  The files are small so the cost is relatively
low.
@compnerd
Copy link
Member Author

@swift-ci please test

@compnerd
Copy link
Member Author

@swift-ci please build toolchain

@aciidgh
Copy link
Contributor

aciidgh commented Oct 27, 2018

CI failed because build-script invocation is missing this argument for some reason: '--llvm-install-components=llvm-cov;llvm-profdata'

/cc @shahmishal @gottesmm

@gottesmm
Copy link
Contributor

Ok. We should get this in ASAP to unbreak the package build. Let me look at this real quick.

@gottesmm
Copy link
Contributor

Dispatch isn't using a build-script preset...

@gottesmm
Copy link
Contributor

This preset will work: swiftlang/swift#20109. I asked @shahmishal to update the bots once it lands.

@compnerd
Copy link
Member Author

preset=buildbot_linux_1604,notest
@swift-ci Please test with preset

@gottesmm
Copy link
Contributor

@compnerd until @shahmishal changes the PR job to invoke the preset can't work

@shahmishal
Copy link
Member

Updated the PR job with '--llvm-install-components=llvm-cov;llvm-profdata' for now, I will update the bot to start using preset.

@shahmishal
Copy link
Member

@swift-ci test

@shahmishal shahmishal merged commit 9acbab3 into swiftlang:master Oct 29, 2018
@shahmishal
Copy link
Member

@ktopley-apple,

FYI - I merged this PR to unblock swift.org toolchains.
https://bugs.swift.org/browse/SR-9080

@compnerd compnerd deleted the copy branch February 15, 2020 19:39
rokhinip pushed a commit that referenced this pull request Nov 5, 2021
build: always copy the modulemaps
Signed-off-by: Kim Topley <ktopley@apple.com>
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.

5 participants