Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[bug] Symlinks are preserved by default #590

Open
daskol opened this issue Aug 2, 2024 · 6 comments
Open

[bug] Symlinks are preserved by default #590

daskol opened this issue Aug 2, 2024 · 6 comments
Labels
bug Something isn't working

Comments

@daskol
Copy link

daskol commented Aug 2, 2024

What happened?

Transition from v4.3.4 to v4.3.5 broke regular building pipelines because actions/upload-artifact does not derefernce symlink anymore by default and upload symlinks as is but not target files.

See "Upload StarPU libraries" job for details.

What did you expect to happen?

  1. Pipelines are expected to work as before.
  2. Symlinks are dereferenced by default.
  3. This action providesd special option/flag for (de)referencing symlinks (kind of -L/-l in common POSIX utils).

How can we reproduce it?

You can fork repo or create pull request and trigger pipeline on push event.

Anything else we need to know?

No response

What version of the action are you using?

v4.3.5 (no issue with the previous v4.3.4)

What are your runner environments?

linux

Are you on GitHub Enterprise Server? If so, what version?

No response

@daskol daskol added the bug Something isn't working label Aug 2, 2024
daskol added a commit to nntile/nntile that referenced this issue Aug 3, 2024
Action `action/upload-artifact` with new release no longer dereferences
symlinks by default.

See actions/upload-artifact#590 for details.
daskol added a commit to nntile/nntile that referenced this issue Aug 3, 2024
Action `action/upload-artifact` with new release no longer dereferences
symlinks by default.

See actions/upload-artifact#590 for details.
@erick-xanadu
Copy link

This also affected us. Thanks @daskol for reporting this issue.

@cbaeberle
Copy link

#589 Seems the same issue.

@cipolleschi
Copy link

Hi there! Riccardo from Meta, working in the React team and taking care of the React Native build pipeline on GH.
This affected us as well.

@daskol
Copy link
Author

daskol commented Aug 6, 2024

@robherley Could we revert the changes in the latest release 4.3.5 and create a new release 4.3.6? After that, we can continue working on this issue.

facebook-github-bot pushed a commit to facebook/react-native that referenced this issue Aug 6, 2024
Summary:
We had CI on main failing consistently the past couple of days.
The problem is that the hermes pipeline is failing to create the iOS XCFramework with the error:
> unable to create a Mach-O from the binary at '/Users/runner/work/react-native/react-native/packages/react-native/sdks/hermes/destroot/Library/Frameworks/catalyst/hermes.framework/hermes'

The main cause is this upgrade of [upload-artifacts](actions/upload-artifact#590) which breaks symlinks.

The solution is to bump the caches and downgrade the `upload-artifact` actions.
## Changelog:
[Internal] - Try to fix CI for Hermes

Pull Request resolved: #45908

Test Plan: GHA must be green

Reviewed By: cortinico

Differential Revision: D60828616

Pulled By: cipolleschi

fbshipit-source-id: 6976b86dd67e2fd9d806ebaa62f47e39dc44b30d
@robherley
Copy link
Contributor

Hey folks, we reverted & released a new version for v4.3.6 to address the regression.

@sanjacob
Copy link

sanjacob commented Aug 15, 2024

It would be really nice if this behaviour could be opt-in instead #93 #508.

Edit: after testing it seems that not all symlinks are preserved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

6 participants