GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Revert compression changes related to windows but keep version logging by Phantsure · Pull Request #1049 · actions/cache

Conversation

@Phantsure

@Phantsure Phantsure commented Dec 27, 2022 •
edited by lvpx
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

due to symlink issues

Description

Revert compression changes related to windows but keep version logging

Motivation and Context

Some issues have been reported due to linux/mac cache not being recovered on windows because of symlink issue. We would be testing possible solutions for them till then reverting this change.

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation (add or update README or docs)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@Phantsure
Revert compression changes related to windows
c9cbdaf
 due to symlink issues
@Phantsure
Phantsure requested a review from a team as a code owner December 27, 2022 10:43
@github-actions
github-actions Bot requested a review from tanuj077 December 27, 2022 10:43
@github-actions github-actions Bot assigned tanuj077 Dec 27, 2022
@Phantsure
Phantsure requested a review from lvpx December 27, 2022 10:44
@Phantsure
Added tips and workarounds for cross os
90d7628
lvpx
lvpx approved these changes Dec 27, 2022
View reviewed changes

@lvpx lvpx left a comment

Copy link
Copy Markdown
Contributor

There was a problem hiding this comment.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@Phantsure
Phantsure merged commit 4723a57 into main Dec 27, 2022
@mikelane mikelane mentioned this pull request Dec 28, 2022
Bump actions/cache from 3.2.1 to 3.2.2 mikelane/reddit-get#119
Merged
@Teddy-van-Jerry Teddy-van-Jerry mentioned this pull request Jan 3, 2023
Bump actions/cache from 3.2.1 to 3.2.2 mmcesim/mmcesim#16
Merged
@Phantsure Phantsure mentioned this pull request Jan 4, 2023
Add support to opt-in enable cross-os caching on windows #1056
Merged
1 of 10 tasks
@mmuetzel mmuetzel mentioned this pull request Jan 17, 2023
Cache save fails with Cygwin in PATH #1073
Closed
Dakshjain1 pushed a commit to SvavaCapital/cache that referenced this pull request Apr 19, 2024
@Phantsure @Dakshjain1
Revert compression changes related to windows but keep version logging (
1551d2b
actions#1049)

* Revert compression changes related to windows
 due to symlink issues

* Added tips and workarounds for cross os
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tanuj077 tanuj077 Awaiting requested review from tanuj077
1 more reviewer
@lvpx lvpx lvpx approved these changes
Reviewers whose approvals may not affect merge requirements

Uh oh!

There was an error while loading. Please reload this page.

Assignees

@tanuj077 tanuj077

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@Phantsure @lvpx @tanuj077
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402520 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.