Bug 7399 - %if construct doesn't parse macro names correctly
%if construct doesn't parse macro names correctly
Product: Red Hat Linux
Classification: Retired
Component: rpm (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Jeff Johnson
Depends On:
  Show dependency treegraph
Reported: 1999-11-28 16:02 EST by yates
Modified: 2008-05-01 11:37 EDT (History)
0 users

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 1999-11-29 09:17:51 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description yates 1999-11-28 16:02:09 EST
The %if construct doesn't recognize "_" as a legas first character in a
macro name.  For example
	%define _foo bar
	%if _foo == "bar"
gets an error, wheras
	%define foo bar
	%if foo == "bar"
does not.

Suggest changing line 265 in expression.c
	    } else if (isalpha(*p)) {
	    } else if (isalpha(*p) || *p == '_') {
Comment 1 Jeff Johnson 1999-11-29 09:17:59 EST
This had been fixed in (at least) rpm-3.0.4. Thanks for the patch.

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