RPM

rpm: split mono debuginfo

Bug #633750 reported by Jeff Johnson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
RPM
Triaged
Low
Unassigned
openSUSE
Won't Fix
High

Bug Description

Tracker

Revision history for this message
In , Mmeeks (mmeeks) wrote :

Created an attachment (id=311117)
patch

Revision history for this message
In , Meissner-novell (meissner-novell) wrote :

you can just submit this change as submitreq if you want (there is no clear maintainer of find-debuginfo.sh)

i'll assign to mls though just in general ;)

Revision history for this message
In , Mmeeks (mmeeks) wrote :

Oh - I'd really prefer someone who knows what they are doing to review my patch - manifestly I have no idea here; if they could submit it too - that'd be really helpful.

I assume the prospect of more space on the various Live-CDs is generally interesting to people too.

Revision history for this message
In , Coolo (coolo) wrote :

Mon Jan 28 16:35:46 CET 2008 - <email address hidden>
- finddebuginfo.diff: disable debuginfo for mono at the moment

No idea why.

Revision history for this message
In , Rguenther (rguenther) wrote :

I don't see where finddebuginfo.diff changes anything for mono.

But ...

+# Find mono debugging data
+if [ -x "/usr/bin/mono" ] ; then
+ echo "find in $RPM_BUILD_ROOT/usr/lib"
+ find $RPM_BUILD_ROOT/usr/lib -type f \( -name "*.dll.mdb" -or -name "*.exe.mdb" \) | sed "s|$RPM_BUILD_ROOT||" >> "$LISTFILE"
+fi
+

so you are special-casing packages that have /usr/bin/mono. Why not simply
manually add a

Package: debuginfo
...

%files debuginfo
...

which is possible now (well, not mono-debuginfo as that exists already and
you can't add to it - I didn't see that would be necessary yet). Likely
what you want is something like mono-runtime-debuginfo anyway (or wherever
that pieces are).

The proposed solution to finddebuginfo.sh is a hack. If more packages
would provide debuginformation in a mono format then if a mono
build environment is present there needs to be a way to identify a
*.dll.mdb or *.exe.mdb file as proper debuginformation and the packaging
should be done for all packages not just the one containing /usr/bin/mono.

Revision history for this message
In , Mmeeks (mmeeks) wrote :

Hi Richard !

> so you are special-casing packages that have /usr/bin/mono.
> Why not simply manually add a [ .spec file change]

oh - well, this is not very simple when you have to add it to a lot of packages (60 or so) - and in general, some pleasant way to drop (known to be) debuginfo information automatically into -debuginfo files is surely more pleasant ? This presumably is the thinking underlying a central find-debuginfo.sh script.

The check for /usr/bin/mono - is not intended to make this specific to a single package - but to make it specific to those packages that have a BuildRequires of mono - ie. to reduce any chance of false positives with people calling their files "foo.dll.mdb" but them not being either a DLL or a debuginfo file ;-) I tend to think that is pretty unlikely, and in these cases they would want those files in a -debuginfo file anyway - so we could easily remove that.

Revision history for this message
In , Rguenther (rguenther) wrote :

Ah, indeed - I misread that part of the patch. Is there a way to have
mono say "yes, this is proper debuginfo for me!"? Like

mono --verify-this-is-debuginfo /path/to/the/file

?

Does the mono debugger find the debuginformation if it is copied to
the /usr/lib/debug/ shadow tree? I see the debuginfo files do not
end in .debug, so at the moment they wouldn't be picked up by rpm,
but that would be easy to fix.

Revision history for this message
In , Mmeeks (mmeeks) wrote :

Martin - any idea about this ? it would be nice to have some debug sniffing tool that we could use to reliably detect real debuginfo files; and Richard wonders - could we look in the shadow path for these files ? [ moving them across to there would be a little cleaner, and more consistent obviously ].

Revision history for this message
Jeff Johnson (n3npq) wrote :

Tracker

tags: added: opensuse
Jeff Johnson (n3npq)
tags: added: rpmbuild
Changed in rpm:
importance: Undecided → Low
Changed in opensuse:
importance: Unknown → High
status: Unknown → Incomplete
Jeff Johnson (n3npq)
tags: added: debuginfo
Revision history for this message
In , Jeff Johnson (n3npq) wrote :
Changed in rpm:
status: New → Triaged
Changed in opensuse:
status: Incomplete → Confirmed
Revision history for this message
In , Mls-i (mls-i) wrote :

Closing. Please reopen if you still want this.

Changed in opensuse:
status: Confirmed → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Related blueprints

Remote bug watches

Bug watches keep track of this bug in other bug trackers.