Last Comment Bug 659997 - Remove typo'd ENVIRONMENT_MAC_OS_X_VERION_MIN_REQUIRED define
Product: Core
Classification: Components
Component: Build Config (show other bugs)
: Trunk
: All All
-- normal (vote)
: mozilla7
Assigned To: Ed Morley [:emorley]
: Gregory Szorc [:gps] (away until 2017-03-20)
Depends on:
Blocks: 705532
  Show dependency treegraph
Reported: 2011-05-26 10:00 PDT by Ed Morley [:emorley]
Modified: 2011-11-27 05:34 PST (History)
2 users (show)
emorley: in‑testsuite-
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Remove define (1.79 KB, patch)
2011-05-26 10:02 PDT, Ed Morley [:emorley]
ted: review+
Details | Diff | Splinter Review
Remove define (1.83 KB, patch)
2011-06-08 12:21 PDT, Ed Morley [:emorley]
emorley: review+
Details | Diff | Splinter Review

Description User image Ed Morley [:emorley] 2011-05-26 10:00:10 PDT
Was added typo'd in bug 403224, so presumably not required:

Comment 1 User image Ed Morley [:emorley] 2011-05-26 10:02:45 PDT
Created attachment 535379 [details] [diff] [review]
Remove define

Two line change.

Thanks :-)
Comment 2 User image Ed Morley [:emorley] 2011-05-26 13:39:49 PDT
Comment 3 User image Ted Mielczarek [:ted.mielczarek] 2011-06-08 10:11:26 PDT
Comment on attachment 535379 [details] [diff] [review]
Remove define

Review of attachment 535379 [details] [diff] [review]:
Comment 4 User image Ed Morley [:emorley] 2011-06-08 12:21:03 PDT
Created attachment 538095 [details] [diff] [review]
Remove define

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 :-)
Comment 5 User image Ed Morley [:emorley] 2011-06-09 12:14:41 PDT
Comment 6 User image Simona B [:simonab ] 2011-08-26 00:31:41 PDT
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:

Is this enough to mark this as VERIFIED FIXED?

Comment 7 User image Ed Morley [:emorley] 2011-08-26 00:43:13 PDT
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.
Comment 8 User image Simona B [:simonab ] 2011-08-26 01:57:40 PDT
Marking this as VERIFIED FIXED based on Comment 7.

Thanks ED!

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