-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
Redox Cross Compilation #38401
Redox Cross Compilation #38401
Changes from 1 commit
341d2d1
c7aa284
a621d12
d2707aa
ece703a
f86e014
3e7543a
773a0a2
07e313d
6d7c2ec
57bc1a9
86f85c1
01157e6
e55596f
65eecf8
fd4bc88
7697c72
2ca1f0b
bf50acb
e909e43
92c8e0f
7d3ae87
e7b006d
1eb6c44
2ddd117
474eb62
c59bb49
4dcb867
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -37,6 +37,7 @@ pub fn opts() -> TargetOptions { | |
function_sections: false, | ||
dynamic_linking: true, | ||
executables: true, | ||
target_family: Some("unix".to_string()), | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Everything that isn't windows, is set to unix. That includes Linux, OS X, iOS, DragonflyBSD, NetBSD, OpenBSD, FreeBSD, Bitrig, Solaris, Fuchsia, and Haiku |
||
is_like_osx: true, | ||
has_rpath: true, | ||
dll_prefix: "lib".to_string(), | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -38,7 +38,7 @@ pub fn opts() -> TargetOptions { | |
relocation_model: "static".to_string(), | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There is no support for relocation in Redox. |
||
disable_redzone: true, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Red zones are not currently supported by Redox. This may be enabled in the future after testing. |
||
eliminate_frame_pointer: false, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We do not eliminate the frame pointer so that stack frames are easily traversable. |
||
target_family: Some("redox".to_string()), | ||
target_family: None, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Redox no longer has a family :-( There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 😢 |
||
linker_is_gnu: true, | ||
no_default_libraries: true, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. No default libraries must be linked, as they would come from the host. |
||
lib_allocation_crate: "alloc_system".to_string(), | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -24,6 +24,7 @@ pub fn opts() -> TargetOptions { | |
staticlib_prefix: "".to_string(), | ||
staticlib_suffix: ".lib".to_string(), | ||
no_default_libraries: true, | ||
target_family: Some("windows".to_string()), | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Both the windows bases are covered |
||
is_like_windows: true, | ||
allows_weak_linkage: false, | ||
pre_link_args: vec![ | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since the windows platforms now correctly define target_family, this is no longer necessary.