Hello Bastian et al, On Saturday 14 Aug 2010 15:46:33 Kurt Roeckx wrote: > On Sat, Aug 14, 2010 at 09:58:24AM +0200, Bastian Blank wrote: > > On Sat, Aug 14, 2010 at 04:17:27AM +0100, Ben Hutchings wrote: > > > [I also wonder why we are copying the unstripped modules into > > > /usr/lib/debug and not just debug info? > > > > Because the tools can't cope with them. > > > > > And shouldn't this really be > > > > > > part of Kbuild? Here's how we could do it in Kbuild: > > > quiet_cmd_modules_install = INSTALL $@ > > > > > > +ifdef INSTALL_MOD_DEBUGLINK > > > + cmd_modules_install = mkdir -p $(2); mkdir -p $(3); $(OBJCOPY) > > > --only-keep-debug $@ $(3)/$(@F) && $(OBJCOPY) --strip-debug > > > --add-gnu-debuglink=$(3)/$(@F) $@ $(2)/$(@F) > > > > The link looks wrong, it includes the absolute path during install time. > > That's no problem. Only the filename itself is stored in the > file. But objcopy wants to read that file to see if it has > debug info. > So what has been finalized ? Like Kurt already mentioned, objcopy does need to read the file. I can put the conditional if DEBUG in the initial patch that I sent. Ritesh -- Ritesh Raj Sarraf | http://people.debian.org/~rrs Debian - The Universal Operating System
Attachment:
signature.asc
Description: This is a digitally signed message part.