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

[DO-000] use specific artifact name to download it #370

Merged
merged 8 commits into from
Feb 10, 2025

Conversation

duje-begonja-rdx
Copy link
Contributor

No description provided.

Copy link

codecov bot commented Feb 4, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.68%. Comparing base (a18623a) to head (b5fb0fe).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #370   +/-   ##
=======================================
  Coverage   92.68%   92.68%           
=======================================
  Files        1184     1184           
  Lines       26689    26689           
  Branches       85       85           
=======================================
  Hits        24737    24737           
  Misses       1941     1941           
  Partials       11       11           
Flag Coverage Δ
kotlin 98.86% <ø> (ø)
rust 92.27% <ø> (ø)
swift 92.87% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


- uses: RDXWorks-actions/download-artifact-v4@main
with:
name: x86_64-unknown-linux-gnu
Copy link
Contributor

Choose a reason for hiding this comment

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

should this really be x86_64-unknown-linux-gnu but the one above is aarch64-apple-darwin ? looks wrong with the assymetry?

duje-begonja-rdx and others added 8 commits February 10, 2025 18:39
@micbakos-rdx micbakos-rdx force-pushed the DO-000-fix-desktop-binaries-workflow branch from 39333dd to b5fb0fe Compare February 10, 2025 16:40
@micbakos-rdx micbakos-rdx marked this pull request as ready for review February 10, 2025 16:40
@micbakos-rdx
Copy link
Contributor

micbakos-rdx commented Feb 10, 2025

In summary:

The upload-artifact job was uploading the artifacts and was renaming them to the target triple's name:
Upload from:

path: target/x86_64-unknown-linux-gnu/release/libsargon_uniffi.so
name: x86_64-unknown-linux-gnu

AND

path: target/aarch64-apple-darwin/release/libsargon_uniffi.dylib
name: aarch64-apple-darwin

This still works as it used to, but, inn the past the download-artifact job we used, was downloading each artifact and when unwrapping it was also naming it with the name of the directory it was saved into. So

// It used to be like this
artifacts/x86_64-unknown-linux-gnu/libsargon_uniffi.so
artifacts/aarch64-apple-darwin/libsargon_uniffi.dylib
// Now changed to
artifacts/libsargon_uniffi.dylib
artifacts/libsargon_uniffi.so

So gradle's scripts could not pick up the correct files. It seems that the latest main of the download job is unwrapping in place without extracting the binaries in their respective directory

@micbakos-rdx micbakos-rdx merged commit 059e84b into main Feb 10, 2025
13 checks passed
@micbakos-rdx micbakos-rdx deleted the DO-000-fix-desktop-binaries-workflow branch February 10, 2025 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants