-
-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
Reformat of README.windows.md #21285
Conversation
We should remove the msys2 instructions. They don't currently work. Can be put back later, less recommended than cygwin cross compile, if someone gets it working again. |
Reformatted to make it render correctly on Github. Only minor editorial changes (marked link as dead, corrected pacman update command). (and whitespace error hopefully corrected...)
599cac9
to
cc0cf4a
Compare
I would rather not make that change myself since I'm basically exclusively on Linux nowadays. |
README.windows.md
Outdated
|
||
# Binary distribution | ||
Instructions for adding fonts to the terminal are available at | ||
[Microsoft KB-247815 (dead link)]( |
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.
We should probably avoid dead links. Maybe https://superuser.com/a/5079 would be a good choice.
README.windows.md
Outdated
https://code.google.com/p/mintty/) (note that running Julia on Mintty needs a | ||
copy of `stty.exe` in your `%PATH%` to work properly). Alternatively, you may | ||
prefer the features of a more full-function IDE, such as [Juno](http:https://junolab.org), | ||
[LightTable](https://github.com/one-more-minute/Jupiter-LT), |
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.
Jupiter-LT appears no longer to exist.
README.windows.md
Outdated
- Windows 7: supported (32 and 64 bits) | ||
- Windows Vista: not officially supported (but probably works anyways) | ||
- Windows XP: not supported (but may work anyways, if you can get around the | ||
lack of junction points) |
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.
I wouldn't mention XP or Vista
As an aside, you can add [ci skip] to commit messages for commits that only touch READMEs. That avoids rerunning the entire hour-long build and test. |
|
||
**On Both:** | ||
Unfortunately, the version of gcc installed by Ubuntu targets pthreads. On Mac, | ||
the situation is similar: the version in MacPorts is very old and Homebrew does |
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.
These statements (and the rest of the section) are false now. Macports and Homebrew have mingw-w64
now, and our build now requires the pthreads thread model instead of win32.
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.
Also who uses MacPorts anymore 😛
Correctness of instructions is orthogonal to reformatting. Does anyone have any issues with the reformatting itself? |
README.windows.md
Outdated
2. From the *Net* category: `curl` | ||
3. From *Interpreters* (or *Python*) category: `m4`, `python` | ||
4. From the *Archive* category: `p7zip` | ||
5. For 32 bit Julia, only, and also from the *Devel* category: |
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.
drop the "only" here
[ci skip]
Done, and this time with |
Same as the README.md before, the README.windows.md also renders incorrectly on Github.
Thus, reformatted it for the various lists to count and indent correctly, and made only very minor editorial changes (marked a link as dead, corrected pacman update command).