Closed Bug 659997 Opened 9 years ago Closed 9 years ago

Remove typo'd ENVIRONMENT_MAC_OS_X_VERION_MIN_REQUIRED define

Categories

(Firefox Build System :: General, defect)

defect
Not set

Tracking

(Not tracked)

VERIFIED FIXED
mozilla7

People

(Reporter: emorley, Assigned: emorley)

References

Details

Attachments

(1 file, 1 obsolete file)

Was added typo'd in bug 403224, so presumably not required:

> AC_DEFINE_UNQUOTED(__ENVIRONMENT_MAC_OS_X_VERION_MIN_REQUIRED__,$_MACOSX_DEPLOYMENT_TARGET)

http://mxr.mozilla.org/mozilla-central/search?string=MAC_OS_X_VERION_MIN_REQUIRED
Attached patch Remove define (obsolete) — Splinter Review
Two line change.

Thanks :-)
Attachment #535379 - Flags: review?(ted.mielczarek)
Comment on attachment 535379 [details] [diff] [review]
Remove define

Review of attachment 535379 [details] [diff] [review]:
-----------------------------------------------------------------
Attachment #535379 - Flags: review?(ted.mielczarek) → review+
Attached patch Remove defineSplinter Review
Only change is adding the patch author field, which hg missed off despite my hgrc; carrying forwards r+.

Has passed try (comment 2); author & commit message filled out.
Thanks to whomever checks it in :-)
Attachment #535379 - Attachment is obsolete: true
Attachment #538095 - Flags: review+
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/972980426471
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla7
Mozilla/5.0 (Windows NT 5.1; rv:7.0) Gecko/20100101 Firefox/7.0

Is there any way to verify this?

I looked in the files in the repo:
http://hg.mozilla.org/mozilla-central/file/e87454393401

Is this enough to mark this as VERIFIED FIXED?

Thanks!
For cleanup changes the answer to that is normally going to be that there is no way to verify apart from visual inspection. This is particularly the case for bugs filed under Core::Build Config - and even more so where no lines were added, only removed.
Marking this as VERIFIED FIXED based on Comment 7.

Thanks ED!
Status: RESOLVED → VERIFIED
Blocks: 705532
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.