[Android Wiki] Fix Part of #2746 : Added Bazel Installations instructions for different Operating System

[MohitGupta121]

Explanation

Fix Part of #2746 : Added Bazel Installations instructions for different Operating System

Essential Checklist

• The PR title and explanation each start with "Fix #bugnum: " (If this PR fixes part of an issue, prefix the title with "Fix part of #bugnum: ...".)
• Any changes to scripts/assets files have their rationale included in the PR explanation.
• The PR follows the style guide.
• The PR does not contain any unnecessary code changes from Android Studio (reference).
• The PR is made from a branch that's not called "develop" and is up-to-date with "develop".
• The PR is assigned to the appropriate reviewers (reference).

[MohitGupta121]

@BenHenning PTAL, I added Bazel installation instructions mac m1/m2, make instrcutions more clear for other OS.

[MohitGupta121]

@adhiamboperes Can you please review on the Mac setup doc.

@kkmurerwa Can you please have a look on Ubuntu/Fedora setup doc from your setup experinece feel free to give any suggestions.

[kkmurerwa]

Hey @MohitGupta121. The Fedora part looks good.

[MohitGupta121]

@kkmurerwa Thanks, you have any suggestions for Ubuntu?

[kkmurerwa]

I haven't used Ubuntu that much so I can't really say a lot about it. But I believe the Wiki is sufficient for most Linux distros, especially the two that you focused on. You could add how to make Python 2 and jdk8 the default if that is required for Bazel to run.

[BenHenning]

I haven't used Ubuntu that much so I can't really say a lot about it. But I believe the Wiki is sufficient for most Linux distros, especially the two that you focused on. You could add how to make Python 2 and jdk8 the default if that is required for Bazel to run.

With our pending upgrade to Bazel 6.x, these requirements should change. Python 3 should be fine to use, and our dependencies are actually pushing us toward JDK 11 (and possibly later).

@MohitGupta121 I can look especially at the Ubuntu part since that's my regular development environment (Linux Mint). Note that part of this may need to wait until #4886 is merged since that's a significant blocker in being able to do regular Bazel development.

[MohitGupta121]

@BenHenning Okay Thanks, I will wait for the Bazel updation PR got merged first.

[seanlip]

Thanks @MohitGupta121. Took a grammar/comprehensibility pass, PTAL.

[MohitGupta121]

Thanks @seanlip, PTAL, I make changes as per your review.

[adhiamboperes]

The M1 instructions look good as per our prior tests.

[oppiabot]

Unassigning @adhiamboperes since they have already approved the PR.

[seanlip]

@MohitGupta121 Took another pass, thanks. Please read previous comments carefully, some issues are still present in the updated PR.

[MohitGupta121]

Okay Thanks @BenHenning

[oppiabot]

Hi @MohitGupta121, I'm going to mark this PR as stale because it hasn't had any updates for 7 days. If no further activity occurs within 7 days, it will be automatically closed so that others can take up the issue.
If you are still working on this PR, please make a follow-up commit within 3 days (and submit it for review, if applicable). Please also let us know if you are stuck so we can help you!

[oppiabot]

Hi @MohitGupta121, I'm going to mark this PR as stale because it hasn't had any updates for 7 days. If no further activity occurs within 7 days, it will be automatically closed so that others can take up the issue.
If you are still working on this PR, please make a follow-up commit within 3 days (and submit it for review, if applicable). Please also let us know if you are stuck so we can help you!

[MohitGupta121]

@adhiamboperes PTAL, final review. Thanks!

[adhiamboperes]

@MohitGupta121, minor comments, looks good otherwise

[oppiabot]

Unassigning @adhiamboperes since the review is done.

[oppiabot]

Hi @MohitGupta121, it looks like some changes were requested on this pull request by @adhiamboperes. PTAL. Thanks!

[MohitGupta121]

@adhiamboperes PTAL, maked requested changes. Thanks!

[adhiamboperes]

Hi @MohitGupta121, this LGTM, thanks!