CI related improvements

[hrw]

When we do builds on CI we may want to be able to define where build directory is (instead of ../build-*".

Another thing is location of download/ and ccache/ directories - GitHub Actions dislike paths with ".." in them. Moving those to checked out sources (where it was linked before) allows to make caches easier.

[smoser]

please quote all variables.

And lets use CI_BUILD to set OUT.

OUT="$PWD/../build-$VER"
if [ "$CI_BUILD" = "true" ]; then
    OUT=$WHERE_YOU_WANT_IT_FOR_CI"
fi

Last, check CI_BUILD lie BOOTTEST is checked. accepted values are empty, false or true.

CI_BUILD="${CI_BUILD:-false}"
case "$CI_BUILD" in
    true|false) echo "bad value '$CI_BUILD' for CI_BUILD"; exit 1;;
esac

[NucciTheBoss]

Just to note, this might also just be specific to me, but I did not have any issues with using .. in my original action. The actions/checkout@v3 changes the current working directory in the pipeline to the CirrOS source code. However, syntactically/readability speaking, I think the way it is done here will be better for the pipeline.

That being said, I think this pull request should also update .gitignore to include ccache and build-*. That way if someone is building locally and committing upstream, no files from these directories will end up polluting the commit and require rebasing their branch/fork to remove. Let me know what you think.

[hrw]

That being said, I think this pull request should also update .gitignore to include ccache and build-*. That way if someone is building locally and committing upstream, no files from these directories will end up polluting the commit and require rebasing their branch/fork to remove. Let me know what you think.

#83 does that

[hrw]

build works, artifacts upload works, cache is reused