Skip to content

Conversation

@kachick
Copy link
Member

@kachick kachick commented Mar 28, 2021

Sorry, I don't know the details of the difference of this.
But...

So this change is okay...?

@kachick
Copy link
Member Author

kachick commented Mar 28, 2021

Oh, it fails on head too 😮 https://github.com/ruby/strscan/runs/2211057093

Run gem install pkg/*.gem
ERROR:  Error installing pkg/strscan-3.0.0.gem:
	ERROR: Failed to build gem native extension.

    No such file or directory @ dir_s_mkdir - /home/runner/.rubies/ruby-head/lib/ruby/gems/3.1.0/gems/strscan-3.0.0/ext/strscan/.gem.20210328-1691-cvspni

Gem files will remain installed in /home/runner/.rubies/ruby-head/lib/ruby/gems/3.1.0/gems/strscan-3.0.0 for inspection.
Results logged to /home/runner/.rubies/ruby-head/lib/ruby/gems/3.1.0/extensions/x86_64-linux/3.1.0/strscan-3.0.0/gem_make.out
Error: Process completed with exit code 1.

@kachick kachick marked this pull request as draft March 28, 2021 04:29
@kachick kachick changed the title Use head instead of debug in CI CI is fault in ruby-{head/debug}, I don't know why... 😢 Mar 28, 2021
@hsbt
Copy link
Member

hsbt commented Jan 4, 2022

@kachick It's caused by the same gem version of ruby-head. Maybe, It's caused by the current implementation of RubyGems.

@eregon
Copy link
Member

eregon commented Apr 22, 2022

The workaround is

# If strscan in Ruby's master has the same version of this strscan,
# "gem install pkg/*.gem" fails.
- run: bundle exec rake version:bump
if: >-
matrix.ruby == 'debug' ||
matrix.ruby == 'mingw' ||
matrix.ruby == 'mswin'

I think we close this. debug is head + extra assertions.

@eregon eregon closed this Apr 22, 2022
@kachick
Copy link
Member Author

kachick commented May 20, 2022

Thank you! 🙇‍♂️

@kachick kachick deleted the use-head-instead-of-debug branch May 20, 2022 05:16
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