"@foo@.in: WARNING: no preprocessor directives found" when running Preprocessor unit test

RESOLVED FIXED in mozilla17

Status

RESOLVED FIXED
7 years ago
a year ago

People

(Reporter: glandium, Assigned: glandium)

Tracking

unspecified
mozilla17

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Comment hidden (empty)
(Assignee)

Comment 1

7 years ago
Created attachment 644376 [details] [diff] [review]
Avoid warning when running unit-Preprocessor.py
Attachment #644376 - Flags: review?(ted.mielczarek)
(Assignee)

Updated

7 years ago
Assignee: nobody → mh+mozilla
Comment on attachment 644376 [details] [diff] [review]
Avoid warning when running unit-Preprocessor.py

Review of attachment 644376 [details] [diff] [review]:
-----------------------------------------------------------------

This is fine, but why do we warn in this case? We are doing a substitution, just not using preprocessor directives. That seems like a valid use of Preprocessor.py.
Attachment #644376 - Flags: review?(ted.mielczarek) → review+
(Assignee)

Comment 3

7 years ago
Created attachment 645375 [details] [diff] [review]
Avoid Preprocessor.py warning when no commands are processed, but filters are
Attachment #645375 - Flags: review?(ted.mielczarek)
(Assignee)

Updated

7 years ago
Attachment #644376 - Attachment is obsolete: true
Attachment #645375 - Flags: review?(ted.mielczarek) → review+
https://hg.mozilla.org/mozilla-central/rev/e0a8b539d562
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED

Updated

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