Closed Bug 1413348 Opened 6 years ago Closed 6 years ago

remove MOZ_HORIZON checks from build system

Categories

(Firefox Build System :: General, enhancement)

Product:
Firefox Build System
Component:
General
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

(firefox58 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla58
Tracking Flags:
Tracking Status
firefox58 --- fixed

People

(Reporter: dholbert, Assigned: dholbert)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Bug 1413348: Remove build system variable for no-longer-used/supported MOZ_HORIZON configuration.
59 bytes, text/x-review-board-request
froydnj
: review+
Details 
I ran across the MOZ_HORIZON variable in old-configure.in and I noticed it seems to be dead code.

It was added here, with a few usages:
  https://hg.mozilla.org/mozilla-central/rev/de3b5f164ac5#l90.55
...but those usages were all in b2g, and they're now gone.  So, let's remove this dead MOZ_HORIZON code.
Assignee: nobody → dholbert
Status: NEW → ASSIGNED
Comment on attachment 8924008 [details]
Bug 1413348: Remove build system variable for no-longer-used/supported MOZ_HORIZON configuration.

Stealing review.
Attachment #8924008 - Flags: review?(mh+mozilla) → review+
Comment on attachment 8924008 [details]
Bug 1413348: Remove build system variable for no-longer-used/supported MOZ_HORIZON configuration.

https://reviewboard.mozilla.org/r/195210/#review200456
Pushed by nfroyd@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5bc78f996c0d
Remove build system variable for no-longer-used/supported MOZ_HORIZON configuration. r=froydnj
https://hg.mozilla.org/mozilla-central/rev/5bc78f996c0d
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox58: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
Comment on attachment 8924008 [details]
Bug 1413348: Remove build system variable for no-longer-used/supported MOZ_HORIZON configuration.

https://reviewboard.mozilla.org/r/195210/#review201530
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.