Fix #3912: Added local property protobuf condition for M1 Mac

[FareesHussain]

Fixes #3912

protoc-3.8.0-osx-aarch_64.exe is not available in 3.8.0 release and gives the following error on build

Here to fix this we the following property to local.properties (Only for M1 Mac) which uses osx release instead for the given condition when built on gradle.

protobuf_platform=osx-x86_64

PS: This issue doesn't appear for bazel build

[BenHenning]

Thanks @FareesHussain. Nice fix. Some questions:

1. Is there a tracking issue on protoc's GitHub repo tracking this issue that we can point to?
2. If (1) doesn't exist, could you file one? It seems like a nice thing to do for a fellow open source project.
3. Could you file an issue on our repository to add references to anywhere that helped solve this, to capture the context above, and to link to the GitHub issue for protoc?
4. Could you update the comment to point to the issue from (2) as a reference?
5. Could you update the PR to mark the new issue as fixed?

I think this'll better help centralize the investigation to an issue (which is the default place where investigation contexts should live for each consulting in the future).

[oppiabot]

Assigning @BenHenning for code owner reviews. Thanks!

[FareesHussain]

Thanks @FareesHussain. Nice fix. Some questions:

1. Is there a tracking issue on protoc's GitHub repo tracking this issue that we can point to?

Yes, there is (grpc/grpc-java#7690). this is where i found the solution.

2. If (1) doesn't exist, could you file one? It seems like a nice thing to do for a fellow open source project.
3. Could you file an issue on our repository to add references to anywhere that helped solve this, to capture the context above, and to link to the GitHub issue for protoc?

Done, #3912

4. Could you update the comment to point to the issue from (2) as a reference?

Since the issue is already filed I didn't have to create these issues, protocolbuffers/protobuf#8557 grpc/grpc-java#7690

5. Could you update the PR to mark the new issue as fixed?

Done

I think this'll better help centralize the investigation to an issue (which is the default place where investigation contexts should live for each consulting in the future).

[BenHenning]

Thanks @FareesHussain. Just one follow-up.

[BenHenning]

Thanks @FareesHussain. I think I wasn't very clear in my earlier comment, but I think the latest is a bit clearer. Please resolve & send back. PR otherwise LGTM.

[BenHenning]

Suggested change

	// To build protoc in M1 mac. For Context, see:
	// https://github.com/grpc/grpc-java/issues/7690.
	// To build protoc in M1 mac. For context, see: #3912.

Prefer linking to our issue for context.

[FareesHussain]

Done

[BenHenning]

Thanks again for the fast fix @FareesHussain! LGTM.