put a useful gdbinit in the tree so it's found automagically

RESOLVED FIXED in mozilla15

Status

()

Core
Build Config
RESOLVED FIXED
15 years ago
5 years ago

People

(Reporter: dmose, Assigned: Ehsan)

Tracking

(Blocks: 1 bug)

Trunk
mozilla15
All
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 3 obsolete attachments)

(Reporter)

Description

15 years ago
So that even people who don't read the FAQ get a gdb that's minimially usable.
(Reporter)

Comment 1

15 years ago
Created attachment 108560 [details] [diff] [review]
gdbinit patch, v1

This is taken from the FAQ, but has modifications to the way set auto-solib-add
is used so that attach works better.
(Reporter)

Updated

15 years ago
Status: NEW → ASSIGNED
Did you know that with really recent snapshots of gdb that you don't need the
auto-solib-add hacks wrt performance?  It still uses lots of memory, though.

Comment 3

15 years ago
disabling loading the libraries also makes one more step in telling newbies how
to get a stack, which is basically going to be 

% mozilla -g -d gdb
(gdb) set auto-solib-add 1
(gdb) run
[crash]
(gdb) bt

loading libs individually is too complicated for newbies.

the real advantage is to users who do a lot of debugging, but (for me at least)
it's a lot faster to get mozilla to the point where it will crash (or I want to
debug) and then attach gdb.  it still uses lots of memory (230MB) and isn't
useful for catching stuff during startup, but it's more than 2x faster.
Product: Browser → Seamonkey
(Reporter)

Comment 4

10 years ago
Assigning bugs that I'm not actively working on back to nobody; use
SearchForThis as a search term if you want to delete all related bugmail at
once.
Assignee: dmose → nobody
Status: ASSIGNED → NEW

Updated

8 years ago
Component: Build Config → Build Config
Product: SeaMonkey → Core
QA Contact: granrosebugs → build-config
Is there interest in this bug any more?  I'm willing to create a new patch if there is.
I don't know, what would you put in there? I would be willing to bet there are some nice Archer-gdb scripts around these days...
http://www.mozilla.org/unix/.gdbinit
and any other useful things that folks have.

I don't know much about Archer-gdb.
(Reporter)

Comment 8

7 years ago
I think it would help drive down the barrier to entry for developmen, so I'd say it would be great to get a patch here and land it...
Created attachment 437095 [details] [diff] [review]
Patch (v1)

Here's an updated patch.
Assignee: nobody → ehsan
Status: NEW → ASSIGNED
Attachment #437095 - Flags: review?(ted.mielczarek)
Created attachment 442948 [details] [diff] [review]
Patch (v2)

Add cjone's ptarray:

http://blog.mozilla.com/cjones/2010/04/30/print-nstarrays-in-gdb/
Attachment #108560 - Attachment is obsolete: true
Attachment #437095 - Attachment is obsolete: true
Attachment #442948 - Flags: review?(ted.mielczarek)
Attachment #437095 - Flags: review?(ted.mielczarek)
Comment on attachment 442948 [details] [diff] [review]
Patch (v2)

I defer to jimb.
Attachment #442948 - Flags: review?(ted.mielczarek) → review?(jim)
Whiteboard: [has patch][needs review jimb]

Updated

6 years ago
Blocks: 652719

Comment 12

5 years ago
Ehsan, if you could check that the .gdbinit here is up to date, I'll r+ it.
Created attachment 620513 [details] [diff] [review]
Patch (v3)

I've added a few other macros since I attached the patch here!
Attachment #442948 - Attachment is obsolete: true
Attachment #620513 - Flags: review?(jimb)
Attachment #442948 - Flags: review?(jimb)
I also recommend this:

>def jsinfo
> call JS_PCToLineNumber(cx, script, regs.pc)
> printf "%s:%d\n", script.ptr->filename, $
>end
(In reply to Josh Matthews [:jdm] (travelling until June 25th, not reading non-CCed bugmail) from comment #14)
> I also recommend this:
> 
> >def jsinfo
> > call JS_PCToLineNumber(cx, script, regs.pc)
> > printf "%s:%d\n", script.ptr->filename, $
> >end

Let's land this first, I don't wanna stall this more than necessary.  Once this is in the tree, everyone can add all sorts of macros.  :-)

Updated

5 years ago
Attachment #620513 - Flags: review?(jimb) → review+
https://hg.mozilla.org/mozilla-central/rev/5aa794f7d5cd
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Whiteboard: [has patch][needs review jimb]
Target Milestone: --- → mozilla15
You need to log in before you can comment on or make changes to this bug.