-
Notifications
You must be signed in to change notification settings - Fork 331
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
The commands to activate the opam switch on windows are wrong #2707
Comments
Thanks, Jacques, this is very useful. I've opened PR #2709 on this. How can we address the @Sudha247, @jonahbeckford, @dra27: Any views on the antivirus question? |
That missing space is a typo - #2709 indeed deals with that. The actual code for the PowerShell snippet is correct in the sources: ocaml.org/data/tutorials/getting-started/1_00_install_OCaml.md Lines 105 to 109 in e464efa
and ocaml.org/src/ocamlorg_frontend/pages/install.eml Lines 168 to 170 in e464efa
so what happens with the rendering? From my browser I'm getting: ? The virus scanner part is tracked in ocaml/opam#5741. The principal aim to mitigate this is to stop extracting the massive tarballs and Git repositories (which will improve |
Chrome |
That page is also displayed fine on my side. What about this one? https://ocaml.org/install#windows This is what I've made a screenshot of. |
Oh, I was on the wrong page. Sorry, it’s suddenly obvious - the .eml is an ML file so the |
Can we add a few words basically saying: “Sorry, this is going to take an awful lot of time”? |
Yes, indeed - we altered the opam init process in 2.2 so that there is at least a status indicator (the “loading data” and “extracting” markers, which make opam look considerably less “dead”) |
Here is the texts I've pushed in #2709:
|
On the page "install ocaml on windows", the first steps are fine, but the two commands to activate the switch are wrong.
The one for cmd reads
for /f"tokens=*" %i in ('opam env') do @%i
, and it misses a space between/f
and"tokens=*"
.The one for PoweShell reads
(& opam env) -split '?' | ForEach-Object { Invoke-Expression $_ }
, but the?
should be replaced with the endline code (maybe hard to put on a web page). This makes the installation confusing.Another issue I had is that windows 10 appears to spend most of its running the antivirus. A comment about how to stop it would be welcom.
The text was updated successfully, but these errors were encountered: