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

Create new engine release workflow #1950

Merged
merged 4 commits into from
Feb 17, 2025

Conversation

p2004a
Copy link
Collaborator

@p2004a p2004a commented Feb 15, 2025

A new release workflow that uses v2 engine build to create new releases.

There is a change in how release tags and artifacts are named. Best see example output https://github.com/p2004a/spring/releases/tag/recoil%7Bmaster%7D2025.01.2-93-g44d2480.

- Uses the new v2 build to create releases
- Builds both regular and tracy builds as part of the same release
- Is able to create and push a new annoted release tag
@p2004a p2004a requested review from Beherith, lhog and sprunk February 15, 2025 20:17
@lhog
Copy link
Collaborator

lhog commented Feb 16, 2025

One quick question. Why zst for debug symbols?

@p2004a
Copy link
Collaborator Author

p2004a commented Feb 16, 2025

One quick question. Why zst for debug symbols?

  1. Way quicker to compress in comparison to standard gzip, still quicker to compress even in comparison to pigz -1
  2. Smaller output size in all cases
  3. Quicker to decompress
  4. ZSTD is nowadays quite uniformly well supported
$ time tar -cf - dbgsym | zstd -T0 > dbgsym.tar.zst
tar -cf - dbgsym  0.05s user 1.58s system 73% cpu 2.207 total
zstd -T0 > dbgsym.tar.zst  15.56s user 1.92s system 776% cpu 2.250 total
$ time tar -czf dbgsym.tar.gz dbgsym
tar -czf dbgsym.tar.gz dbgsym  47.88s user 2.04s system 103% cpu 48.244 total
$ time tar -cf - dbgsym | pigz -1 > dbgsym.tar.gz
tar -cf - dbgsym  0.08s user 1.87s system 63% cpu 3.051 total
pigz -1 > dbgsym.tar.gz  32.94s user 3.91s system 1019% cpu 3.615 total
$ time tar -cf - dbgsym | pigz > dbgsym.tar.gz
tar -cf - dbgsym  0.15s user 1.53s system 26% cpu 6.364 total
pigz > dbgsym.tar.gz  87.47s user 2.98s system 1316% cpu 6.872 total
$ ls -l -h dbgsym.tar.*
-rw-rw-r-- 1 p2004a p2004a 546M Feb 16 18:57 dbgsym.tar.gz
-rw-rw-r-- 1 p2004a p2004a 507M Feb 16 18:57 dbgsym.tar.zst
$ time tar -xf dbgsym.tar.zst -C dbgsym
tar -xf dbgsym.tar.zst -C dbgsym  2.60s user 2.95s system 235% cpu 2.355 total
$ time tar -xf dbgsym.tar.gz -C dbgsym
tar -xf dbgsym.tar.gz -C dbgsym  8.50s user 2.74s system 125% cpu 8.920 total

There are less and less reasons not to use zstd nowadays.

@lhog lhog merged commit 8e9cd08 into beyond-all-reason:master Feb 17, 2025
2 checks passed
github-actions bot pushed a commit that referenced this pull request Feb 18, 2025
- Uses the new v2 build to create releases
- Builds both regular and tracy builds as part of the same release
- Is able to create and push a new annoted release tag

* Change release auto tag and archive naming scheme
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