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

fix: const name from RbTrace to RBTrace #93

Merged
merged 1 commit into from Dec 15, 2023
Merged

fix: const name from RbTrace to RBTrace #93

merged 1 commit into from Dec 15, 2023

Conversation

k0i
Copy link
Contributor

@k0i k0i commented Dec 15, 2023

Background

I noticed rbtrace throws the following error:

*** run `sudo sysctl kernel.msgmnb=1048576` to prevent losing events (currently: 16384 bytes)
/home/koyam/.rbenv/versions/debug/lib/ruby/gems/3.3.0+0/gems/rbtrace-0.5.0/lib/rbtrace/rbtracer.rb:326:in `send_cmd': uninitialized constant RBTracer::RbTrace (NameError)

      if msg.bytesize >= RbTrace::BUF_SIZE
                         ^^^^^^^
        from /home/koyam/.rbenv/versions/debug/lib/ruby/gems/3.3.0+0/gems/rbtrace-0.5.0/lib/rbtrace/rbtracer.rb:210:in `attach'
        from /home/koyam/.rbenv/versions/debug/lib/ruby/gems/3.3.0+0/gems/rbtrace-0.5.0/lib/rbtrace/rbtracer.rb:91:in `initialize'
        from /home/koyam/.rbenv/versions/debug/lib/ruby/gems/3.3.0+0/gems/rbtrace-0.5.0/lib/rbtrace/cli.rb:428:in `new'
        from /home/koyam/.rbenv/versions/debug/lib/ruby/gems/3.3.0+0/gems/rbtrace-0.5.0/lib/rbtrace/cli.rb:428:in `run'
        from /home/koyam/.rbenv/versions/debug/lib/ruby/gems/3.3.0+0/gems/rbtrace-0.5.0/bin/rbtrace:5:in `<top (required)>'
        from /home/koyam/.rbenv/versions/debug/bin/rbtrace:25:in `load'
        from /home/koyam/.rbenv/versions/debug/bin/rbtrace:25:in `<main>'

So, I checked the latest commit.

I realized that the commit contains RBTrace and RbTrace, and the correct variant is RBTrace.

Снимок экрана_2023-12-15_03-17-59

Notes

After fixing that, it works.
Снимок экрана_2023-12-15_03-21-05

@tmm1
Copy link
Owner

tmm1 commented Dec 15, 2023

Thank you!

@tmm1 tmm1 merged commit 1c67488 into tmm1:master Dec 15, 2023
5 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.

None yet

2 participants