Closed
Bug 1461964
Opened 7 years ago
Closed 7 years ago
remove old osx 10.6 references in manifest files
Categories
(Core :: DMD, enhancement)
Tracking
()
RESOLVED
FIXED
mozilla62
| Tracking | Status | |
|---|---|---|
| firefox62 | --- | fixed |
People
(Reporter: jmaher, Assigned: apavel)
References
Details
(Whiteboard: [manifest_cleanup])
Attachments
(1 file, 1 obsolete file)
|
622 bytes,
patch
|
n.nethercote
:
review+
|
Details | Diff | Splinter Review |
we run our automation on osx 10.10, not 10.6 anymore. We can remove references to 10.6.
there is one reference to osx 10.6 in here:
https://searchfox.org/mozilla-central/source/memory/replace/dmd/test/xpcshell.ini#35
| Assignee | ||
Comment 1•7 years ago
|
||
Assignee: nobody → apavel
Attachment #8978822 -
Flags: review?(gbrown)
Comment 2•7 years ago
|
||
Comment on attachment 8978822 [details] [diff] [review]
Removed osx reference
Review of attachment 8978822 [details] [diff] [review]:
-----------------------------------------------------------------
::: memory/replace/dmd/test/xpcshell.ini
@@ +32,1 @@
> # Bug 1077230 explains why this test is disabled on Mac 10.6.
Please remove this comment.
@@ +32,4 @@
> # Bug 1077230 explains why this test is disabled on Mac 10.6.
> [test_dmd.js]
> dmd = true
> +skip-if = !(os=='linux' || os=='win')
This isn't right. I believe you want this:
> skip-if = !(os=='linux' || os=='win' || os=='mac')
because `os_version!='10.6'` is now equivalent to `true`.
::: toolkit/components/telemetry/tests/unit/xpcshell.ini
@@ -1,2 @@
> -[DEFAULT]
> -head = head.js
I don't know anything about telemetry tests, but deleting this entire file doesn't seem right.
| Assignee | ||
Comment 3•7 years ago
|
||
:njn thank you. done the modifications you mentioned, however, i'm not sure about
::: toolkit/components/telemetry/tests/unit/xpcshell.ini
@@ -1,2 @@
> -[DEFAULT]
> -head = head.js
i did not touch that file.
Attachment #8978822 -
Attachment is obsolete: true
Attachment #8978822 -
Flags: review?(gbrown)
Attachment #8979157 -
Flags: review?(n.nethercote)
Updated•7 years ago
|
Attachment #8979157 -
Flags: review?(n.nethercote) → review+
| Assignee | ||
Updated•7 years ago
|
Keywords: checkin-needed
Pushed by apavel@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/1a3b9492f30d
removed mac 10.6 reference r=njn
Keywords: checkin-needed
Comment 5•7 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox62:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in
before you can comment on or make changes to this bug.
Description
•