Bug 499885

Summary:

emelfm2-debuginfo created from stripped binaries

Product:

[Fedora] Fedora

Reporter:

Ville Skyttä <ville.skytta>

Component:

emelfm2

Assignee:

Christoph Wickert <christoph.wickert>

Status:

CLOSED NEXTRELEASE

QA Contact:

Fedora Extras Quality Assurance <extras-qa>

Severity:

medium

Docs Contact:

Priority:

low

Version:

rawhide

CC:

christoph.wickert

Target Milestone:

---

Keywords:

Patch

Target Release:

---

Hardware:

All

OS:

Linux

Whiteboard:

Fixed In Version:

0.6.0-1.fc11

Doc Type:

Bug Fix

Doc Text:

Story Points:

---

Clone Of:

Environment:

Last Closed:

2009-05-09 11:35:14 UTC

Type:

---

Regression:

---

Mount Type:

---

Documentation:

---

CRM:

Verified Versions:

Category:

---

oVirt Team:

---

RHEL 7.3 requirements from Atomic Host:

Cloudforms Team:

---

Target Upstream Version:

Embargoed:

Bug Depends On:

Bug Blocks:

496968

Attachments:

Description	Flags
Fix debuginfo	none

Description Ville Skyttä 2009-05-08 18:30:22 UTC

Created attachment 343141 [details]
Fix debuginfo

Fix attached, see bug 496968 for more info.

Comment 1 Christoph Wickert 2009-05-08 18:50:33 UTC

You are absolutely right and I have to admit that I was aware of this problem. Nevertheless I don't like your patch, something like this should IMO go upstream and have an option in Makefile.config, say STRIP=0 or simply use DEBUG=1. DEBUG=1 does not strip and sets different cflags, but these get overwritten in the spec anyway, so IMO DEBUG is sane. Any objections?

Comment 2 Ville Skyttä 2009-05-08 19:18:56 UTC

I considered that, but:

$ grep -B 1 ^DEBUG Makefile.config
# setting DEBUG to 0/1 turns off/on debugging messages and in-file data for debuggers
DEBUG ?= 0

...I suppose using DEBUG != 0 will not only affect CFLAGS and stripping but will also result in debug message spewage somewhere (stdout?).

Comment 3 Christoph Wickert 2009-05-09 02:37:01 UTC

You are right, so took the patch until we have something better. Fixed in rawhide, you want me to fix this also in F-11 or a stable release? If not, I suggest to wait for the 0.6.0 update that is coming soon (when I fixed a segfault that appears upon updating the config files).

Comment 4 Christoph Wickert 2009-05-09 03:36:14 UTC

FYI: 0.6.0 built for rawhide and F-11.

Comment 5 Ville Skyttä 2009-05-09 04:23:14 UTC

I think that's fine.

Comment 6 Christoph Wickert 2009-05-09 11:35:14 UTC

OK then, closing.

Comment 7 Fedora Update System 2009-05-26 07:55:07 UTC

emelfm2-0.6.0-1.fc11 has been pushed to the Fedora 11 stable repository.  If problems still persist, please make note of it in this bug report.