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

Resolve test failure from bundler 2.2.24+ #544

Merged
merged 3 commits into from
Nov 10, 2023

Conversation

iterateNZ
Copy link
Contributor

Bundler v2.2.24 changed the behaviour of Bundler::Definition#specs_for when the requested groups are empty from returning nothing to everything.

@enebo
Copy link
Member

enebo commented Nov 7, 2023

Just poking in to note that jruby-9.2 is pretty ancient so it failing with the new .lock file is not so surprising. This work should probably update to 9.4 (9.3 is still supported but not for too much longer). Not sure if that will start to pull the sweater apart or be relatively error-free.

@iterateNZ iterateNZ force-pushed the bundler-2.4-support-fix branch 2 times, most recently from c9c167a to a82ce21 Compare November 7, 2023 21:51
@iterateNZ
Copy link
Contributor Author

Trying a slightly different tack and including two bundler versions in the build matrix (sneaking in jruby 9.3, 9.4 and java 11 as well). I haven't been able to get clean consistent builds of these environments locally so relying on the CI setup here to verify the changes.

The bundler version running the rspec should be what is then used for the bundle install within the examples upgrading the lock file as it goes 🤞

@iterateNZ iterateNZ force-pushed the bundler-2.4-support-fix branch 2 times, most recently from 0eb2f9a to 015888b Compare November 8, 2023 00:25
@@ -8,7 +8,7 @@ def self.reset_local_repository

describe Warbler::WebServer::Artifact do

@@_env = ENV.dup
@@_env = ENV.to_h
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Should resolve test failures on jruby v9.4+

iterateNZ and others added 3 commits November 8, 2023 15:06
Bundler v2.2.24 changed the behaviour of `Bundler::Definition#specs_for` when the requested groups are empty from returning nothing to everything.
* TypeError: Cannot dup ENV, use ENV.to_h to get a copy of ENV as a hash
* LoadError: no such file to load -- rexml/document
@iterateNZ
Copy link
Contributor Author

Looks like this is passing on jruby-head now as well, that's a nice bonus.

@iterateNZ iterateNZ marked this pull request as ready for review November 8, 2023 20:15
@olleolleolle
Copy link
Member

Yes! So, this looks amazing, and I think that we are ready for it!

♥️ 💛 💚 💙 💜

@olleolleolle olleolleolle merged commit 2868c4f into jruby:master Nov 10, 2023
65 of 66 checks passed
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