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

broken with ActiveSupport error #121

Closed
apotheon opened this issue Feb 22, 2022 · 7 comments
Closed

broken with ActiveSupport error #121

apotheon opened this issue Feb 22, 2022 · 7 comments

Comments

@apotheon
Copy link

.rbenv/versions/3.0.0/lib/ruby/gems/3.0.0/gems/activesupport-7.0.2.2/lib/active_support/xml_mini.rb:184:in `current_thread_backend': uninitialized constant ActiveSupport::XmlMini::IsolatedExecutionState (NameError)
@siaw23-retired
Copy link
Collaborator

siaw23-retired commented Feb 22, 2022

Thank you for this issue. Looks like we depend on a gem that's broken because of updates in AS. There's an umerged PR carmen-ruby/carmen#298 for this. I'm thinking of looking for alternatives or forking that gem. A PR will be very welcome. 😊

@apotheon
Copy link
Author

I commented on the pull request for carmen. Hopefully it spurs some action. That simple require change does indeed make Kovid work locally.

I haven't looked at how carmen is used in Kovid yet, though.

@siaw23
Copy link
Owner

siaw23 commented Mar 13, 2022

@Jberczel maybe you can help with this? Is there a chance you could help us find an alternative to Carmen or maybe fork and publish the fix mentioned in carmen-ruby/carmen#298?

@Jberczel
Copy link
Contributor

@siaw23 - It looks like carmen is only used to lookup state codes for USA. I think we can just remove it and use a ruby hash to look those up? I think when I originally added the carmen dependency, I expected us to be able to use it for other country and province lookups.

Let me know if you're okay with that approach, and I can submit a PR.

@siaw23
Copy link
Owner

siaw23 commented Mar 16, 2022

I'm OK with that approach. I think we only look up states for USA so we might not need Carmen after all. Please submit a PR if you can. @Jberczel

@siaw23
Copy link
Owner

siaw23 commented Mar 21, 2022

We have a fix for this thanks to @Jberczel. @apotheon you can upgrade to see if this works as should. 😀

@apotheon
Copy link
Author

The code looks functional and the fix works well so far. Thanks!

@siaw23 siaw23 closed this as completed Mar 22, 2022
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

No branches or pull requests

4 participants