Skip to content

Conversation

@jungleraptor
Copy link
Contributor

@jungleraptor jungleraptor commented Sep 28, 2023

Adds an upload action so that the resulting sysroot is uploaded for testing.

@jungleraptor jungleraptor changed the base branch from master to peddie/blas-lapack September 28, 2023 05:18
@jungleraptor
Copy link
Contributor Author

@peddie First will probably help to actually upload the sysroot on PR's.

@jungleraptor jungleraptor requested a review from peddie September 28, 2023 23:18
Copy link
Contributor

@peddie peddie left a comment

Choose a reason for hiding this comment

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

I don't fully understand the details of this, but I fully approve the intent.

@jungleraptor
Copy link
Contributor Author

I don't fully understand the details of this, but I fully approve the intent.

Was not very helpful for testing purposes that the only way to get a copy of the sysroot was to push a tag - so I've made it so that the resulting artifact is uploaded on every PR.

You can find it here: https://github.com/swift-nav/swift-toolchains/actions/runs/6334833738?pr=22 (under Actions -> Summary).

@jungleraptor jungleraptor changed the title Isaac/test sysroot add testing infrastructure for sysroot builds Sep 29, 2023
jungleraptor added a commit to swift-nav/rules_swiftnav that referenced this pull request Sep 29, 2023
@jungleraptor jungleraptor merged commit d6baf0e into peddie/blas-lapack Sep 29, 2023
@jungleraptor jungleraptor deleted the isaac/test-sysroot branch September 29, 2023 04:33
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.

3 participants