remove uses_cxx11 call in build_grpc causing macos build failure

[taddes]

Resolution of MacOS build issue I outlined in #654 and related context to cc-rs and cmake. I'd suggested removing uses_cx11, which was confusingly appending to the system output of OS value and also removing the example from the documentation comment and example in cross_compile.md under All diff in fn build_grpc

[ti-chi-bot]

Welcome @taddes! It looks like this is your first PR to tikv/grpc-rs 🎉

[taddes]

Hey @BusyJay , just have an issue with the DCO signoff as the instructions link to a dead end in the ti-chi-bot. My commits are GPG signed against my Mozilla email. Would you let me know what I need to do to unblock on this? Unfortunately, there's no contributing doc I could reference in the repo. Thanks!

[BusyJay]

Detail signoff instructions can be found here: https://github.com/tikv/grpc-rs/pull/657/checks?check_run_id=40342515992

[taddes]

Thanks @BusyJay , lmk if you're comfortable moving this to your main workflow for review and approval.

[taddes]

Hey @BusyJay , I've got the fix in place and made a lot of clippy adjustments to satisfy, but there are a considerable number of warnings regarding error: creating a mutable reference to mutable static is discouraged. See the recent CI check outputs for most platforms. This kind of goes beyond the scope of this contribution, and since these code blocks are run within unsafe areas, I am not sure if my tinkering with this makes sense, or if it's best to simply ignore the check outputs since it was designed this way. Just let me know how you want to proceed as I do not have the historical context on this project to make an informed decision.

[BusyJay]

OK, I will take a look and make a fix for those warnings in a separate PR. Thanks for your work!

[taddes]

Thanks a bunch @BusyJay ! Just keep me posted and when we can land this change on top of that! Will look for future opportunities to contribute since I've got my feet wet here. Appreciate the warm welcome and appreciate your work!

[BusyJay]

Hi @taddes, now CI should work, feel free to merge master when you have time.

[taddes]

Thanks @BusyJay !!

[taddes]

Btw, in my forked repo and branch, I get a message that I don't have proper access rights to do a git fetch, since I want to integrate the upstream changes you made. Do I have to be added to an ACL or something?

[BusyJay]

No, this repo is public. Make sure the upstream url is using https protocol.

[taddes]

That did it, didn't know why it was originally working but then for pulling down from remote. Must have defaulted to HTTP. Checks are running and then should hopefully be good to go @BusyJay