b2d865f180 introduced a call to exportVariable() to export the MSYS env
var, which configures the symlink strategy for MSYS2/cygwin binaries it calls.
By setting the env var globally, this also changes the behaviour of other MSYS2
using tools in a CI job, and also overrides MSYS configuration set by the user,
which I think was not intended.
To avoid this leakage set the MSYS env var only for the commands which
@actions/cache calls.
Fixes#1312
* bsd + zstd fallback implementation
* bsd + zstd fallback implementation
* Fix tar operations
* Add -v option for testing
* Fix order of args for tar
* Add GNUtar as default on windows
* Fix test
* Fix tar tests
* Fix lint issues
* Fix windows gnutar test case
* Temporarily remove thhe condition that prevents zstd usage on windows unless with GNUtar
* Address some comments and correct compression commands
* Add windows bsdtar test
* Fix windows test
* Fix test
* Separate args
* Fix old tests
* Add new tests
* Fix tests
* Fix lint test
* Refactor code
* Address review comments
* Fix test
* Fix tar test
* Add await to async function calls
* Fix test
* Update for beta release
* Fix audit issues
* Add fallback to gzip compression if cache not found
* Fix test
* Add test
* Address review comments
* Revert Address review comments
* Release 3.1.0-beta.2 cache package
* Fix issues
* Reconfigure catch block
* Add debug logging for gzip fall back
* Fix test
* Add end to end test for cache using bsd on windows
and address review comments
* Fix test
* Fix test
* Fix tests
* Add better comments
* Update packages/cache/src/internal/cacheHttpClient.ts
Co-authored-by: Bishal Prasad <bishal-pdmsft@github.com>
* Address review comments
* Update for new beta cache package release
* Address bugbash issues
* Fix tests
* Release new actions/cache minor version
Co-authored-by: Lovepreet Singh <pdotl@github.com>
Co-authored-by: Bishal Prasad <bishal-pdmsft@github.com>
* Initial changes
* added info error as well
* Format
* Unused package
* adding message field
* removed line
* Review comments
* review comment to add validation as errors handling
We moved `@actions/http-client` to be part of the toolkit in https://github.com/actions/toolkit/pull/1062. We also made some breaking changes to exported types and released v2.
The biggest change in terms of lines of code affected was to get rid of the `I-` prefix for interfaces since TypeScript doesn't follow this convention.
I bumped the patch version of all packages except for `tool-cache`, where I bumped the major version. The rationale is explained in the release notes for that package.
* Added cacheSize in ReserveCache API request
* minor
* minor
* minor
* Cleanup
* package-lock revert
* Modified tests
* New Response Type
* cleanup
* Linting
* Lint fix
* Resolved comments
* Added tests
* package-lock
* Resolved package-lock mismatch
* Liniting
* Update packages/cache/src/cache.ts
Co-authored-by: Bishal Prasad <bishal-pdmsft@github.com>
* Linting issue
* Resolved few comments
* version upgrade
* Savecache tests
* RequestUtil test
* test
* test
* test
* test
* test
* test
* test
* test
* test
* test
Co-authored-by: Apple <apple@Apples-MacBook-Pro.local>
Co-authored-by: Bishal Prasad <bishal-pdmsft@github.com>
* Added support to check if Artifact cache service is enabled or not.
* enablingForGHES
* added ACTIONS_CACHE_URL in fixtures
* Fix CI
* CI fix
* changed function name
* Function rename
* Updated release
* added test case
* Update RELEASES.md
* Lint errors
* lint
* linting
* lint
* update name to actions service
* Update packages/cache/src/internal/cacheUtils.ts
Co-authored-by: Brian Cristante <33549821+brcrista@users.noreply.github.com>
* review comments
* linting
* linting
* push to start CI
* Update RELEASES.md
* remove extra spaces
* reverting version update
* Revert "reverting version update"
This reverts commit af84eba61e.
* Update RELEASES.md
Co-authored-by: Brian Cristante <33549821+brcrista@users.noreply.github.com>
Christoph Reiter