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

Fix missing escape in Makefile comments #39022

Closed
wants to merge 1 commit into from
Closed

Conversation

musm
Copy link
Contributor

@musm musm commented Dec 29, 2020

This was causing a shell escape error when attempting to cross-compile. Fix is from @vtjnash, opening this so he doesn't forget again 😅

...Hmm this doesn't want to work on Cygwin

@musm musm added the domain:building Build system, or building Julia or its dependencies label Dec 29, 2020
Keno added a commit that referenced this pull request Apr 10, 2021
@Keno Keno mentioned this pull request Apr 10, 2021
Keno added a commit that referenced this pull request Apr 10, 2021
@Keno Keno closed this in #40425 Apr 12, 2021
Keno added a commit that referenced this pull request Apr 12, 2021
* Fix wine bootstrap

Fixes #39124
Closes #39022

* fix cross-build, and other build issues

* this should probably be part of spawn, if it is needed

* Revert "this should probably be part of spawn, if it is needed"

This reverts commit eb88a9c.

Seems to be needed

* fixup! fix cross-build, and other build issues

Co-authored-by: Jameson Nash <[email protected]>
@musm musm deleted the makef branch April 13, 2021 02:30
ElOceanografo pushed a commit to ElOceanografo/julia that referenced this pull request May 4, 2021
* Fix wine bootstrap

Fixes JuliaLang#39124
Closes JuliaLang#39022

* fix cross-build, and other build issues

* this should probably be part of spawn, if it is needed

* Revert "this should probably be part of spawn, if it is needed"

This reverts commit eb88a9c.

Seems to be needed

* fixup! fix cross-build, and other build issues

Co-authored-by: Jameson Nash <[email protected]>
antoine-levitt pushed a commit to antoine-levitt/julia that referenced this pull request May 9, 2021
* Fix wine bootstrap

Fixes JuliaLang#39124
Closes JuliaLang#39022

* fix cross-build, and other build issues

* this should probably be part of spawn, if it is needed

* Revert "this should probably be part of spawn, if it is needed"

This reverts commit eb88a9c.

Seems to be needed

* fixup! fix cross-build, and other build issues

Co-authored-by: Jameson Nash <[email protected]>
johanmon pushed a commit to johanmon/julia that referenced this pull request Jul 5, 2021
* Fix wine bootstrap

Fixes JuliaLang#39124
Closes JuliaLang#39022

* fix cross-build, and other build issues

* this should probably be part of spawn, if it is needed

* Revert "this should probably be part of spawn, if it is needed"

This reverts commit eb88a9c.

Seems to be needed

* fixup! fix cross-build, and other build issues

Co-authored-by: Jameson Nash <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain:building Build system, or building Julia or its dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants