Comment out everything else and just echo the initial environment var… #817

shankari · 2022-04-03T23:38:53Z

…iables

So that at least it shows up

Not sure why it is hanging through

+ ensure that this only runs when the prerequisite script is changed

Since we are running it with `yes Y | ...` and it does not appear to exit automatically

We will try different options to pass in the prompt since the `yes` pipe doesn't seem to work https://www.baeldung.com/linux/bash-interactive-prompts

…o work

- Diff check is expected to fail - Install check should be the last item so that the workflow will fail correctly

Which makes it easier to find errors in these giant scripts

``` $ if [ ! -f $ANDROID_SDK_ROOT/emulator ]; then echo "not found" fi > -bash: syntax error: unexpected end of file $ if [ ! -f $ANDROID_SDK_ROOT/emulator ]; then echo "not found"; fi not found ```

Reverts "Run the test CI for all pushes and PRs" which will not be a separate commit once we squash and merge

shankari added 21 commits April 3, 2022 16:37

Comment out everything else and just echo the initial environment var…

3dd3262

…iables

Run the test CI for all pushes and PRs

0344926

So that at least it shows up

Change the automated install to be identical to the CI workflow

4b1ac52

Fix the file ending

c179cc1

Now introduce the actual changes we want to test

19c10b0

Fix the list_installed syntax

fafeaee

Enable all the other steps as well

1c19c6a

actually run the install script

19eaa04

Not sure why it is hanging through

Add some logging statements to check when the install is complete

564fe9e

+ ensure that this only runs when the prerequisite script is changed

Ensure that we explicitly exit from the script

be5a8a1

Since we are running it with `yes Y | ...` and it does not appear to exit automatically

Simplify the install script so we can see why it doesn't exit properly

49295bb

Remove all prompts and make sure that it works

5c876de

Remove the pipe to confirm that this the problem

fc8ae0b

Restore the prompts

48450da

We will try different options to pass in the prompt since the `yes` pipe doesn't seem to work https://www.baeldung.com/linux/bash-interactive-prompts

Reintroduce the actual install now that the printf workaround seems t…

d0f9a8b

…o work

Fix install check and diff check

f655c38

- Diff check is expected to fail - Install check should be the last item so that the workflow will fail correctly

Add -x so we print lines before executing them

cd17dc2

Which makes it easier to find errors in these giant scripts

Fix syntax of single line command by adding appropriate semicolons

5f388ac

``` $ if [ ! -f $ANDROID_SDK_ROOT/emulator ]; then echo "not found" fi > -bash: syntax error: unexpected end of file $ if [ ! -f $ANDROID_SDK_ROOT/emulator ]; then echo "not found"; fi not found ```

Fix the checks to look for directories and not files

a5fcbba

Split the checks into two so we detect if either check fails

fe81b1f

Restore the default CI to only run on the master branch

f97b514

Reverts "Run the test CI for all pushes and PRs" which will not be a separate commit once we squash and merge

shankari merged commit 2423030 into e-mission:test_sdk_install Apr 4, 2022

shankari deleted the minor_fit_and_finish branch April 4, 2022 22:13

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Comment out everything else and just echo the initial environment var… #817

Comment out everything else and just echo the initial environment var… #817

shankari commented Apr 3, 2022

Comment out everything else and just echo the initial environment var… #817

Comment out everything else and just echo the initial environment var… #817

Conversation

shankari commented Apr 3, 2022