apache / apache/cordova-android

More information in requirements error message

Open
#641 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
3.8k
Forks
1.6k
Avg merge
16h 18m
Merged PRs (30d)
11

Description

Followup to PR #624:

- For some reason we determined in issue #623 & PR #624 that logging the value of JAVA_HOME was misleading. Part of the explanation may be that check_reqs.js overwrites the setting which I think is not the best solution. I would favor that we look for a better solution, then log the actual JAVA_HOME setting in case there was an issue with Java.
- Looking at the changes in PR #624, I think it would be ideal if the code would make it clear if there is an issue with both Java and Android SDK.

Contributor guide

Open the contributing guide

Research direction

Start with check_reqs.js and review the context in issue #623 and PR #624. Trace how JAVA_HOME and the Java and Android SDK checks are handled, then verify that the requirements error message reports the actual relevant setting and distinguishes Java problems from Android SDK problems.

Written by the indexing model from the issue text.

Assessment

Tech stack
android, java, javascript, nodejs
Domain
mobile, tooling
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.