Closed Bug 184013 Opened 22 years ago Closed 12 years ago

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

Categories

(Firefox Build System :: General, defect)

All
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla15

People

(Reporter: dmosedale, Assigned: ehsan.akhgari)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 3 obsolete files)

So that even people who don't read the FAQ get a gdb that's minimially usable.
Attached patch gdbinit patch, v1 (obsolete) — Splinter Review
This is taken from the FAQ, but has modifications to the way set auto-solib-add
is used so that attach works better.
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.
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
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
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.
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...
Attached patch Patch (v1) (obsolete) — Splinter Review
Here's an updated patch.
Assignee: nobody → ehsan
Status: NEW → ASSIGNED
Attachment #437095 - Flags: review?(ted.mielczarek)
Attached patch Patch (v2) (obsolete) — Splinter Review
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]
Ehsan, if you could check that the .gdbinit here is up to date, I'll r+ it.
Attached patch Patch (v3)Splinter Review
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.  :-)
Attachment #620513 - Flags: review?(jimb) → review+
https://hg.mozilla.org/mozilla-central/rev/5aa794f7d5cd
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [has patch][needs review jimb]
Target Milestone: --- → mozilla15
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.