Skip to content

Fix error message from cpack#803

Closed
lovettchris wants to merge 24 commits intoleanprover:masterfrom
lovettchris:clovett/cifix
Closed

Fix error message from cpack#803
lovettchris wants to merge 24 commits intoleanprover:masterfrom
lovettchris:clovett/cifix

Conversation

@lovettchris
Copy link
Contributor

No description provided.

…dditional compile flag "-ftls-model=local-exec". I think cmake can easily do this just using target_compile_options.
@github-actions
Copy link
Contributor

Thanks for your contribution! Please make sure to follow our Commit Convention.

@Kha
Copy link
Member

Kha commented Nov 17, 2021

That line is currently in use (see absence of message in our release CI jobs), but we should be able to remove it starting with #795: f1e1375

@lovettchris
Copy link
Contributor Author

If I go back to https://github.com/leanprover/lean4/actions/runs/1466228779 I see the error in the builds called "Linux", "Linux Debug", "Linux fsanitize", and "Windows", but for some reason not in "macOS" or the NIX based build named "Linux release"...

image

But I see your LLVM PR is disabling the Pack step?

@lovettchris
Copy link
Contributor Author

And on macOs I see this:
image

so not sure why cpack doesn't complain on macOS but there is nothing matching "." there.

which only leaves the NIX build which I don't have handy...

@Kha
Copy link
Member

Kha commented Nov 18, 2021

Resolved with #795

@Kha Kha closed this Nov 18, 2021
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