Allows Customizing Project Location on Gradle Wrapper

[andreban]

@JudahGabriel drafted this to enable customizing the Gradle Wrapper.

[JudahGabriel]

Running this code locally, I get a different exception:

FAILURE: Build failed with an exception. 
* What went wrong:
Task 'assembleRelease' not found in root project 'pwabuilder-llamapack'

In other words, even though we've specified the path to gradlew.bat, it's still looking in the process' current directory for the assembleRelease task.

I'll follow up here when I find out the additional changes needed. Perhaps I can submit a PR for it.

[JudahGabriel]

OK, I think the issue is we should instead be setting the current directory of the node child process.

I think the final code would look like this:

class GradleWrapper {
  constructor(process, androidSdkTools, gradleDirectory) {
    this.process = process;
    this.gradleDirectory = gradleDirectory;
    this.androidSdkTools = androidSdkTools;
    if (process.platform === 'win32') {
      this.gradleCmd = 'gradlew.bat';
    } else {
      this.gradleCmd = './gradlew';
    }
  }

  async assembleRelease() {
    const env = this.androidSdkTools.getEnv();
    await exec(`${this.gradleCmd} assembleRelease --stacktrace`, {
      env: env,
      cwd: this.gradleDirectory
    });
  }

I'll test this first and follow up shortly with the results.

[JudahGabriel]

Instead of appending the directory to the gradle file, we only need to set the working directory option on the child process:

await exec(`${this.gradleCmd} assembleRelease --stacktrace`, {
   env: androidSdkTools.getEnv(),
   cwd: gradleDirectory // set the current directory for the child process
});

Once we do that, everything works. 😎

[andreban]

Ha! Sounds like I was overcomplicating things. Thanks for checking this out. I'll update the PR.

[andreban]

@JudahGabriel does this sound good?

[JudahGabriel]

Looks great! 😎