Nil absolute path displayed for LineBreakpoint in ERB files
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 35/100
Direzione di ricerca
Start with lib/debug/breakpoint.rb around lines 192 and 241, comparing how paths are assigned for breakpoints. Reproduce the ERB breakpoint example and inspect show_bps output. Done means ERB line breakpoints display their file path while continuing to stop at the specified line.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
https://github.com/ruby/debug/blob/bead0984d241a91235e3d3bacd247f0363d2d530/lib/debug/breakpoint.rb#L192
The iseq.absolute_path seems to always be nil for .erb files even though the breakpoint works correctly and stops the execution
at the specifiied line in the erb file.
Therefore the show_bps is broken for such breakpoints showing only the line number:
break app/views/earth/users/index.html.erb:50
break
0 BP - Line :50 (line)
Should the mentioned code be changed to:
@path = iseq.absolute_path || iseq.path
like it is already done at:
https://github.com/ruby/debug/blob/bead0984d241a91235e3d3bacd247f0363d2d530/lib/debug/breakpoint.rb#L241
or should this be fixed at the ruby lang level and make iseq.absolute_path properly return the erb file's absolute path?
- Lingua principale
- Ruby
- Stelle
- 1.3k
- Fork
- 146
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di ruby/debug
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
-
Support exceptionOptions in setExceptionBreakpoints to catch arbitrary exception classes via DAP Aperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 55/100
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 52/100
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
palladius/rails8-app-on-gcp#145 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
rubocop/rubocop-rspec#2236 ·
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
riscv/riscv-unified-db#2624 · 1 reazione ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100