Bug 528668 - [RFE] extend "set debug-file-directory DIR" to support a list of directories
Summary: [RFE] extend "set debug-file-directory DIR" to support a list of directories
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: gdb
Version: rawhide
Hardware: All
OS: Linux
low
medium
Target Milestone: ---
Assignee: Jan Kratochvil
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2009-10-13 09:31 UTC by Denys Vlasenko
Modified: 2009-10-23 14:53 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2009-10-23 14:53:21 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
Proposed implementation (1.49 KB, patch)
2009-10-13 10:47 UTC, Denys Vlasenko
no flags Details | Diff
Patch fixing GNU coding style, printing path if the search fails, cross-platform pathnames, error exceptions safety, application on all the usages of debug-file-directory. (5.69 KB, patch)
2009-10-14 14:34 UTC, Jan Kratochvil
no flags Details | Diff

Description Denys Vlasenko 2009-10-13 09:31:59 UTC
There can be more than one directory with debuginfos on the machine.

Please extend "set debug-file-directory DIR" command so that it accepts
this syntax:

set debug-file-directory DIR1[:DIR2]...

or any equivalent method of achieving the same functionality.

Comment 1 Denys Vlasenko 2009-10-13 10:47:41 UTC
Created attachment 364571 [details]
Proposed implementation

Diffed against current cvs. Run tested.

Comment 2 Jan Kratochvil 2009-10-14 14:34:48 UTC
Created attachment 364755 [details]
Patch fixing GNU coding style, printing path if the search fails, cross-platform pathnames, error exceptions safety, application on all the usages of debug-file-directory.

Scratch-built gdb-7.0-1bz528668v0.fc12:
http://koji.fedoraproject.org/scratch/jkratoch/task_1746015/

Please request if this feature is really required for F-12 ABRT as normally such extension would go only for F-13.

Comment 3 Jan Kratochvil 2009-10-14 14:35:29 UTC
And patch is against Fedora which is a completely different codebase in this regards against FSF GDB due to build-id locating patch.

Comment 4 Denys Vlasenko 2009-10-14 14:52:29 UTC
Personally I'm fine with F13. Thanks a lot for quick response.

Comment 6 Denys Vlasenko 2009-10-19 14:15:53 UTC
I implemented much better debuginfo install in abrt.

Without the gdb feature from this bug I can only copy debuginfos directly to
/usr/lib/debug/build-id.

This works, but Jiri thinks that this won't be acceptable for many users, and
he is probably right. We should use a separate directory for them.

In other words, we need this gdb feature sooner, i.e. in F12.

Can you do this for us, Jan?

Comment 7 Jan Kratochvil 2009-10-23 06:16:53 UTC
gdb-7.0-3.fc12:
http://koji.fedoraproject.org/koji/buildinfo?buildID=137900

Please ping if it works to push it for dist-f12, it is only built now.
A review is welcome as it is very late in the F-12 release process now:
http://sourceware.org/ml/gdb-patches/2009-10/msg00509.html
http://sourceware.org/ml/gdb-patches/2009-10/msg00508.html

Comment 8 Denys Vlasenko 2009-10-23 14:01:21 UTC
Tested with modified abrt which runs gdb like this:

gdb -batch -ex set debug-file-directory /usr/lib/debug:/var/cache/abrt-di/usr/lib/debug ...

Seems to work fine.

Comment 9 Denys Vlasenko 2009-10-23 14:02:35 UTC
I meant:

-ex 'set debug-file-directory /usr/lib/debug:/var/cache/abrt-di/usr/lib/debug'

Comment 10 Jan Kratochvil 2009-10-23 14:53:21 UTC
Thanks for the test.
dist-f12 request: https://fedorahosted.org/rel-eng/ticket/2674


Note You need to log in before you can comment on or make changes to this bug.