Fix makefile style and fallback to general cc commands & temp windows builds fix

[sagudev]

• use cc instead of gcc and c++ instead of g++ for fallback on linux
• format makefile
• pin window ci to rust 1.66 due to Builds on Windows with the Beta branch are broken #334
• update nightly version used for UWP

[jdm]

@bors-servo r+

[bors-servo]

📌 Commit d23ff38 has been approved by jdm

[bors-servo]

⌛ Testing commit d23ff38 with merge 405b1a4...

[bors-servo]

💔 Test failed - checks-github

[CYBAI]

@bors-servo retry

[bors-servo]

⌛ Testing commit d23ff38 with merge f84c114...

[bors-servo]

☀️ Test successful - checks-github
Approved by: jdm
Pushing f84c114 to master...

[sagudev]

Hm, something is still weird with UWP build.
there is rustup component add rust-src --toolchain nightly-2023-02-02-x86_64-pc-windows-msvc
so I do not know how can this error happened:

error: "C:\\Users\\runneradmin\\.rustup\\toolchains\\nightly-2023-02-02-x86_64-pc-windows-msvc\\lib\\rustlib\\src\\rust\\Cargo.lock" does not exist, unable to build with the standard library, try:
        rustup component add rust-src --toolchain nightly-2023-02-02-x86_64-pc-windows-msvc

and whats mor interesting is that again rustup default nightly-2023-02-02 does nothing as rustc --version --verbose still prints stable:

rustc 1.66.1 (90743e729 2023-01-10)
binary: rustc
commit-hash: 90743e7298aca107ddaa0c202a4d3604e29bfeb6
commit-date: 2023-01-10
host: x86_64-pc-windows-msvc
release: 1.66.1
LLVM version: 15.0.2

The reasons may be failing rustup updates. Anyway it looks like rustup bug to me.

[sagudev]

I checked working builds and rustup default nightly-2023-02-02 still does nothing, but we get different errors when trying to update:

info: checking for self-updates
info: downloading self-update
error: could not remove 'setup' file: 'C:\Users\runneradmin\.cargo\bin/rustup-init.exe': Access is denied. (os error 5)
error: could not remove 'rustup-bin' file: 'C:\Users\runneradmin\.cargo\bin\rustup.exe': Access is denied. (os error 5)

while in failing build:

info: checking for self-updates
info: downloading self-update
error: could not remove 'setup' file: 'C:\Users\runneradmin\.cargo\bin/rustup-init.exe': Access is denied. (os error 5)
warning: tool `rustfmt` is already installed, remove it from `C:\Users\runneradmin\.cargo\bin`, then run `rustup update` to have rustup manage this tool.
warning: tool `cargo-fmt` is already installed, remove it from `C:\Users\runneradmin\.cargo\bin`, then run `rustup update` to have rustup manage this tool.

so either there are some differences in GH runners or race condition in rustup.