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-WSL.md #668

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update Windows-WSL.md #668

wants to merge 1 commit into from

Conversation

ThatOtherOtherBob
Copy link

Clarifying pip3 isn't the continuation of a long line wrapping and adding another line needed before my PC could install gcc.

Clarifying pip3 isn't the continuation of a long line wrapping and adding another line needed before my PC could install gcc.
@Gadgetoid
Copy link
Contributor

Might be worth splitting these into different code-fences so it's clearer at a glance. IE:

The following commands enable cross-compile to 32Blit via ARM GCC:
(These are similar to the Linux requirements, see those for more details)

sudo apt update
sudo apt install gcc gcc-arm-none-eabi unzip cmake make python3 python3-pip

And you should also install the 32Blit tools:

pip3 install 32blit

(Note: "construct" and "bitstring" are automatically installed as part of "32blit" now so we can simplify that line)

@Daft-Freak
Copy link
Collaborator

So the Linux docs have a blank line between the two here, also doesn't apt update before the install... and has a separate bit for the unneeded deps. Guess there's more tidying to be done...

@Gadgetoid
Copy link
Contributor

I think it's probably on us to grab the gist of this change and improve the documentation. Probably ties in with #691 and #692 but I pretty much ditched Windows so I'll have to provision a VM or something to kick the tyres on the docs and wrap these up.

@Daft-Freak
Copy link
Collaborator

Running a windows VM to test what is essentially a linux VM seems a bit silly. I have a windows setup handy... but I don't think I have WSL set up on it... Not sure how much this ties in with the other two, those are actually running on windows.

(These changes look reasonable, but we should maybe make the WSL docs as close to the linux ones as possible)

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.

None yet

3 participants