separate binary finding logic from automation-build.mk

RESOLVED FIXED

Status

()

RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: k0scist, Assigned: k0scist)

Tracking

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

8 years ago
Currently, the only way to find the binary location in the build system is via automation-build.mk:

http://mxr.mozilla.org/mozilla-central/source/build/automation-build.mk

However, this does not suffice for test harnesses to reference the binary internal to the makefiles as the path to the binary is bizarrely quoted (e.g. dist/bin/firefox-bin is a valid path, but "dist/bin/firefox-bin" is not).  This should be remedied.  

In addition, the binary-finding logic is separate from the further intention of automation-build.mk.  If automation-build.mk is going to be included as cross-platform binary-finding logic (see bug 613804), then this functionality should be separated out to its own file and included into automation-build.mk
(Assignee)

Updated

8 years ago
Blocks: 613804
(Assignee)

Updated

8 years ago
Blocks: 568642
(Assignee)

Comment 1

8 years ago
Created attachment 492828 [details] [diff] [review]
separate binary finding logic from automation-build.mk

first round of the patch; try on try atm
Attachment #492828 - Flags: review?(ted.mielczarek)
Assignee: nobody → jhammel
Attachment #492828 - Flags: review?(ted.mielczarek) → review+

Comment 2

8 years ago
Landed after getting a+ from ted on irc: http://hg.mozilla.org/mozilla-central/rev/4182651505fb
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
I would have expected to see license boilerplate; any reason for its absence?
I landed a follow-up to this to fix automation-build.mk usage from comm-central applications.

Basically the include for binary-location.mk should have used $(MOZILLA_DIR) rather than $(topsrcdir) - the way comm-central is structured means that $(topsrcdir) there is pointing to the comm-central repo, not the mozilla-central one. This was the actual patch:

http://hg.mozilla.org/mozilla-central/rev/1b0158a0a67e
(Assignee)

Comment 5

8 years ago
Created attachment 497297 [details] [diff] [review]
add license to build-location.mk
Attachment #497297 - Flags: review?(ted.mielczarek)
(Assignee)

Comment 6

8 years ago
(In reply to comment #3)
> I would have expected to see license boilerplate; any reason for its absence?

Sorry, forgot about it; see attachment 497297 [details] [diff] [review]
Attachment #497297 - Flags: review?(ted.mielczarek)
Attachment #497297 - Flags: review+
Attachment #497297 - Flags: approval2.0+

Comment 7

8 years ago
(In reply to comment #6)
> (In reply to comment #3)
> > I would have expected to see license boilerplate; any reason for its absence?
> 
> Sorry, forgot about it; see attachment 497297 [details] [diff] [review]

Pushed: http://hg.mozilla.org/mozilla-central/rev/e7d4ff8c03ad
You need to log in before you can comment on or make changes to this bug.