epasveer / seer

Seer - a gui frontend to gdb
GNU General Public License v3.0
2.07k stars 69 forks source link

Configuration of RR #233

Closed wkordalski closed 3 months ago

wkordalski commented 4 months ago

Hi,

I try to configure RR module of Seer with following settings:

I modify them in the configuration window, click "OK".

Bug 1: After opening the configuration window again, the values are the old (default) ones. Probably it is also only the case that the values are not loaded to configuration window. Especially that after changing the configuration, Seer successfully runs rr commands.

Bug 2: GDB does not understand the GDB arguments. Fragment of GDB output below:

Undefined command: "".  Try "help".
auto-load: No such file or directory.
[redacted: current-working-directory]/safe-path: No such file or directory.

Probably the algorithm that splits arguments-string into array of argument is too simple and does not take quotes into account.

Bug 3: Found it accidentaly during analyzing Seer code. The "RR arguments" option is not taken into account during starting rr command.

epasveer commented 4 months ago

Thanks for trying Seer!

Bug 2: GDB does not understand the GDB arguments. Fragment of GDB output below:

I have a fix for this. It's in the branch for this task. I'll work on the other 2 bugs.

epasveer commented 4 months ago

These 3 bugs should be fixed now (plus another bug not related to RR).

"main" has been updated with them. Let me know. Thanks.

Also thanks for digging into the code to help me debug the problem! :^)

wkordalski commented 3 months ago

I've tested Seer 2.4 and all the bugs seems to be fixed. Thank you very much. Now Seer is much more usable in my use case.

I think we can close this issue.

epasveer commented 3 months ago

Closing task.