configure.in uses and misuses '[' and ']'

RESOLVED FIXED in mozilla1.9alpha1

Status

RESOLVED FIXED
14 years ago
10 months ago

People

(Reporter: daa, Unassigned)

Tracking

Trunk
mozilla1.9alpha1

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

14 years ago
configure.in uses [[ and ]] to escape [ and ] in some places 
uses changequote(,) and changequote([,]) in others and
does not quote at all in still others

patch converts all to use changequote(,) and changequote([,])
(Reporter)

Comment 1

14 years ago
Created attachment 193016 [details] [diff] [review]
configure.in patch
timeless@ did a related checkin a few days ago, maybe he can review...
Summary: configure.in uses and misuses '[' and ']' → configure.in uses and misuses '[' and ']'

Comment 3

14 years ago
Could you please provide this patch as -u12? It is much easier to review that way.
(Reporter)

Comment 4

14 years ago
Created attachment 193078 [details] [diff] [review]
-u12 vserion of patch
Attachment #193016 - Attachment is obsolete: true

Updated

14 years ago
Attachment #193078 - Flags: review?(cls)

Updated

13 years ago
Attachment #193078 - Flags: review?(cls) → review+

Updated

13 years ago
Status: NEW → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9alpha

Updated

10 months ago
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.