Bug 225108 – BibTool: optflags not honored

Bug 225108 - BibTool: optflags not honored

Summary:	BibTool: optflags not honored

Status:	CLOSED RAWHIDE

Aliases:	None

Product:	Fedora
Classification:	Fedora
Component:	BibTool (Show other bugs)
Sub Component:
Version:	rawhide
Hardware:	All Linux

Priority	medium Severity medium
Target Milestone:	---
Target Release:	---
Assigned To:	Patrice Dumas
QA Contact:	Fedora Extras Quality Assurance
Docs Contact:

URL:
Whiteboard:
Keywords:	Patch

Depends On:
Blocks:
	Show dependency tree / graph

Reported:	2007-01-28 18:47 EST by Ville Skyttä
Modified:	2007-11-30 17:11 EST (History)
CC List:	0 users

See Also:
Fixed In Version:	2.48-6
Doc Type:	Bug Fix
Doc Text:
Story Points:	---
Clone Of:
Environment:
Last Closed:	2007-01-29 13:00:22 EST
Type:	---
Regression:	---
Mount Type:	---
Documentation:	---
CRM:
Verified Versions:
Category:	---
oVirt Team:	---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---

Attachments	(Terms of Use)
Honor optflags (438 bytes, patch) 2007-01-28 18:47 EST, Ville Skyttä	no flags	Details \| Diff
Add an attachment (proposed patch, testcase, etc.)

Groups: None (edit)

Description Ville Skyttä 2007-01-28 18:47:36 EST

BibTool does not honor $RPM_OPT_FLAGS, resulting in no-no such as no compiler
generated security features and more or less useless debuginfo.  The included
configure script fails to affect the optflags used during compilation.

Patch attached, however this seems to cause build failures in
regex-0.12/regex.c, suggesting that there's some more work to be done.

glibc contains a regex library which looks similar to this (see
/usr/include/regex.h), perhaps it should be used instead of the bundled copy?

Comment 1 Ville Skyttä 2007-01-28 18:47:36 EST

Created attachment 146800 [details]
Honor optflags

Comment 2 Patrice Dumas 2007-01-29 13:00:22 EST

It should be fixed in 2.48-6, with system regex used.
There are tons of warnings, I am not willing to fix them.
I didn't used the system regex because of a comment in 
makefile, but I hope it will be right anyway. regex-0.12
is very outdated.

Because it may break with the system regex, I only update
in devel.

Thanks for the report.

Note You need to log in before you can comment on or make changes to this bug.