Last Comment Bug 753939 - Add logic so can autoload library makefiles on demand.
: Add logic so can autoload library makefiles on demand.
Product: Core
Classification: Components
Component: Build Config (show other bugs)
: unspecified
: All All
: -- normal (vote)
: mozilla15
Assigned To: Joey Armstrong [:joey]
: Gregory Szorc [:gps]
Depends on:
Blocks: 747540 748452 748470 751167
  Show dependency treegraph
Reported: 2012-05-10 11:53 PDT by Joey Armstrong [:joey]
Modified: 2012-05-12 09:00 PDT (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Attachments .mk loading logic with unit tests (18.18 KB, patch)
2012-05-10 13:10 PDT, Joey Armstrong [:joey]
ted: review+
Details | Diff | Splinter Review

Description Joey Armstrong [:joey] 2012-05-10 11:53:02 PDT
+++ This bug was initially created as a clone of Bug #748470 +++

Split the dependency patch for 748470 into smaller pieces.

Separate out edits setup to load and unit test for the edits into a separate patch.
Comment 1 Joey Armstrong [:joey] 2012-05-10 13:10:19 PDT
Created attachment 622874 [details] [diff] [review] .mk loading logic with unit tests
Comment 2 Joey Armstrong [:joey] 2012-05-10 13:18:04 PDT
Comment on attachment 622874 [details] [diff] [review] .mk loading logic with unit tests logic and unit tests from bug 748470 modified to act as a front end loader for other library makefiles.

# Include to define mkdir-deps() early:
include $(topsrcdir)/config/makefiles/

Unit tests setup to verify loading.
Also compile time unit test code moved out of and into sub-makefiles so the main makefile will be easier to read.  Timestamp dependencies added so check tests will only run when a makefile or unit test is modified.

mkdir_deps: filter the pattern '//' from strings.  Very easy to create using:
  $(call mkdir_deps,$(dir $(path)))

Also fixed a GARBAGE/clean rule.  GARBAGE_DIRS should be assigned GENERATED_DIRS not the timestamp files contained within them.
Comment 3 Joey Armstrong [:joey] 2012-05-10 13:18:34 PDT
code was tested before but another try job is in the pipeline for these isolated edits.
Comment 4 Joey Armstrong [:joey] 2012-05-11 05:47:14 PDT
Try job passed last night:
Comment 5 John Ford [:jhford] CET/CEST Berlin Time 2012-05-11 11:48:13 PDT
Comment on attachment 622874 [details] [diff] [review] .mk loading logic with unit tests

Landed on inbound
Comment 6 Matt Brubeck (:mbrubeck) 2012-05-12 09:00:28 PDT

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