Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Fix zip step in build-and-release.yml#19

Merged
albertodebortoli merged 1 commit intomasterStellarTools/Stellar:masterfrom
feature/fix-release-zip-contentStellarTools/Stellar:feature/fix-release-zip-contentCopy head branch name to clipboard
Jan 23, 2023
Merged

Fix zip step in build-and-release.yml#19
albertodebortoli merged 1 commit intomasterStellarTools/Stellar:masterfrom
feature/fix-release-zip-contentStellarTools/Stellar:feature/fix-release-zip-contentCopy head branch name to clipboard

Conversation

@albertodebortoli
Copy link
Member

The release zip was created with missing files deeper than 2 levels, only reproducible on GitHub Actions.
It seriously looks to me like a bug in zip, as the ** glob is very much Linux standard, and zip is the same version locally and remotely (v3.0.0).

I'm now using the -r parameter to recursively zip all files.

Go figure!

@albertodebortoli albertodebortoli added the bug Something isn't working label Jan 22, 2023
@albertodebortoli albertodebortoli added this to the 0.1.0 milestone Jan 22, 2023
@malcommac malcommac self-requested a review January 23, 2023 22:00
@albertodebortoli albertodebortoli merged commit b9dcf51 into master Jan 23, 2023
@albertodebortoli albertodebortoli deleted the feature/fix-release-zip-content branch January 23, 2023 23:03
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

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.