-
Notifications
You must be signed in to change notification settings - Fork 22
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
Improving clarity at https://github.com/fyne-io/docs.fyne.io/blob/main/started/index.md to avoid "could not find gcc or clang compilers" issues. #18
Comments
Problem 1: The step "Open “MSYS2 MinGW 64-bit” from the start menu" is critical - if you open the right terminal then it will all be installed and set up correctly. Problem 2: Can't we assume that people who choose a non-standard location will know that is the folder to specify when pasting in the commands? Problem 3: The previous step in the text literally initialise a go module and change into the right directory. So it will work if the steps are followed... |
I have removed the references to legacy / non-modules Go as that is no longer supported and could lead to confusion. |
Thank you. |
There is no unless here - modules are now mandatory with any Go app. |
If you can think of a way that this could be helped do please open a pull request on the fyne-io/docs.fyne.io repository. |
Description
Problem 1:
\bin on the \msys64\mingw64 folder is empty even after using the command "pacman -S mingw-w64-ucrt-x86_64-gcc", there are no instructions either on the "https://www.msys2.org/" main page or "https://docs.fyne.io/started/" to install the "msys64\mingw64" version if there's one, therefore "Fyne Setup.exe" will fail the c compiler check.
Bandaid:
https://discuss.codecademy.com/t/problem-with-msys2-cannot-add-path-to-environment-variables-no-mingw64-bin-folder-after-install/656816/7
Problem 2:
Instructions assume the default folder will be selected for the installation of msys2, but do not say that it is just the default folder, therefore people who do not choose “C:\msys64" as the install folder but find the bandaid will still fail the compiler check, the instructions do not prompt the user to check which folder they selected to install it either.
Problem 3: "go get fyne.io/fyne/v2@latest" does not work outside modules, "go install fyne.io/fyne/v2/cmd/fyne@latest" does but they are not separated in any way, as if both commands were required to be used in sequence, or both able to be used in any kind of situation, while each has their own use. Splitting "go install fyne.io/fyne/v2/cmd/fyne@latest" from the "modules" instruction section into it's own area would add clarity to the steps.
The text was updated successfully, but these errors were encountered: