Skip to content

Conversation

@pull
Copy link

@pull pull bot commented Sep 17, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

nobu and others added 10 commits September 17, 2025 11:23
No one uses `sync_default_gems_with` on other platforms; it is used by
the dedicated workflow and a few developers only.
When we're extracting a gem, it should be extracted to an empty
directory.  Trying to remove every file before extracting the file
greatly slows the tar extraction process.

This change increases tar extraction speed by about 10%:

Master branch:

```
aaron@tc ~/g/tartest (main)> ruby -I../rubygems/lib:../vernier/lib test.rb
{MEAN: 0.ruby/rubygems@670195550005}
{stddev: 0.ruby/rubygems@029875687861}
{"mib/s": 20.ruby/rubygems@889425481677}
```

This commit:

```
aaron@tc ~/g/tartest (main)> ruby -I../rubygems/lib:../vernier/lib test.rb
{MEAN: 0.ruby/rubygems@600223749969}
{stddev: 0.ruby/rubygems@037224226667}
{"mib/s": 23.ruby/rubygems@324635189326}
```

ruby/rubygems@e40a153560
@pull pull bot locked and limited conversation to collaborators Sep 17, 2025
@pull pull bot added the ⤵️ pull label Sep 17, 2025
@pull pull bot merged commit 6094cb5 into turkdevops:master Sep 17, 2025
1 of 2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants