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

Update Windows caveats #24567

Merged
merged 1 commit into from
Apr 19, 2015
Merged

Update Windows caveats #24567

merged 1 commit into from
Apr 19, 2015

Conversation

tamird
Copy link
Contributor

@tamird tamird commented Apr 18, 2015

- rust-lang#1237 is closed
- `libgcc` is now statically linked:
  rust-lang#17471
@rust-highfive
Copy link
Contributor

r? @brson

(rust_highfive has picked a reviewer for you, use r? to override)

@steveklabnik
Copy link
Member

@bors: r+ rollup

@bors
Copy link
Collaborator

bors commented Apr 18, 2015

📌 Commit a8aa4ee has been approved by steveklabnik

@@ -42,10 +42,7 @@ Let the fact that this is an easily countable number be a warning.

## Does it run on Windows?

Yes. All development happens in lockstep on all 3 target platforms (using MinGW, not Cygwin). Note that the Windows implementation currently has some limitations; in particular, the 64-bit build is [not fully supported yet][win64], and all executables created by rustc [depend on libgcc DLL at runtime][libgcc].
Copy link
Contributor

Choose a reason for hiding this comment

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

libgcc is dynamically linked on 32-bit windows. (It's a known limitation of dwarf implementation of win32.)

@bors
Copy link
Collaborator

bors commented Apr 19, 2015

⌛ Testing commit a8aa4ee with merge be41e98...

bors added a commit that referenced this pull request Apr 19, 2015
- #1237 is closed
- `libgcc` is now statically linked:
  #17471
@bors bors merged commit a8aa4ee into rust-lang:master Apr 19, 2015
@tamird tamird deleted the windows-caviats branch April 19, 2015 16:09
@steveklabnik steveklabnik mentioned this pull request Apr 25, 2015
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.

6 participants