Note: There are a few cases of duplicates in user autocompletion which are being worked on.

mobile/android/base/ $(shell cat) deps

RESOLVED FIXED in mozilla15



Build Config
5 years ago
5 years ago


(Reporter: joey, Assigned: jhford)



Firefox Tracking Flags

(Not tracked)



(2 attachments, 1 obsolete attachment)



5 years ago

Data loading 'shell cat' commands in this makefile can be optimized.  Another patch has wrapped these calls in a conditional so they will only impose overhead once when a lib* target is being processed.


  o If the *.mn files are only used for makefile dependencies contents can be re-ordered to one file per line removing the need for calling the 'tr' command.

  o Replace shell overhead with a makefile include.  Add make targets to derive *.mk from *.mn data, possibly defining SYNC variables as needed.  Add deps to regenerate individual makefiles when *.mn changes.  A simple include of the data makefile should be enough to force regeneration.

SYNC_JAVA_FILES=$(shell cat $(topsrcdir)/mobile/android/sync/ | tr '\n' ' ';)
SYNC_PP_JAVA_FILES=$(shell cat $(topsrcdir)/mobile/android/sync/ | tr '\n' ' ';)
SYNC_THIRDPARTY_JAVA_FILES=$(shell cat $(topsrcdir)/mobile/android/sync/ | tr '\n' ' ';)
SYNC_RES_DRAWABLE=$(shell cat $(topsrcdir)/mobile/android/sync/ | tr '\n' ' ';)
SYNC_RES_DRAWABLE_LDPI=$(shell cat $(topsrcdir)/mobile/android/sync/ | tr '\n' ' ';)
SYNC_RES_DRAWABLE_MDPI=$(shell cat $(topsrcdir)/mobile/android/sync/ | tr '\n' ' ';)
SYNC_RES_DRAWABLE_HDPI=$(shell cat $(topsrcdir)/mobile/android/sync/ | tr '\n' ' ';)
SYNC_RES_LAYOUT=$(shell cat $(topsrcdir)/mobile/android/sync/ | tr '\n' ' ';)
SYNC_RES_VALUES=$(shell cat $(topsrcdir)/mobile/android/sync/ | tr '\n' ' ';)
I'm not really sure if these files are used for anything else, CCing rnewman who is apparently the author of the script that creates them:

If they only exist for the sake of the Fennec build, then we should be able to just generate Makefiles instead. Perhaps something like:

Then we could include that directly.
We actually want to switch to using the file input to javac, rather than filenames-as-arguments, so we will still want some kind of manifest file for some of these. (That's Bug 736463.)

At that point the source files will be file-per-line.

I'd be very happy to take patches for these post-release; Fennec's and Sync's build system are a little bit rushed, to say the least.

I'm not wedded to a particular approach; I just suck at Make, and needed a solution quickly that wouldn't involve editing Fennec's Makefile every time we add a Sync source file.

These manifests are only used for Fennec's build.
I've created a pull requeset, which creates the manifests with one file per line
Created attachment 622419 [details] [diff] [review]
remove the tr call

This patch corresponds to my pull request in github.  This removes the call to tr from the mozilla build system.  I'll put together a patch that generates a makefile as well as the manifests.
Attachment #622419 - Flags: review?(khuey)
Attachment #622419 - Flags: feedback?(rnewman)
Comment on attachment 622419 [details] [diff] [review]
remove the tr call

Review of attachment 622419 [details] [diff] [review]:

If everything still works, r=me.
Attachment #622419 - Flags: review?(khuey) → review+
Created attachment 622424 [details] [diff] [review]
alternate approach where the android-sync build system generates the makfile and we include it

This doesn't have the sample makefile generated, but I'll include that.
Created attachment 622425 [details]
sample makefile generated by android-sync changes

This is a sample makefile generated by my android-sync changes that I'm about to add to my pull request on github
Comment on attachment 622424 [details] [diff] [review]
alternate approach where the android-sync build system generates the makfile and we include it

this works, and

This does depend on to work.  I did my own sync code drop for the try run.
Attachment #622424 - Flags: review?(khuey)
Attachment #622424 - Flags: review?(khuey) → review+
Merged with fixes:

Landed on inbound:
Assignee: nobody → jhford
Target Milestone: --- → mozilla15
Attachment #622419 - Attachment is obsolete: true
Attachment #622419 - Flags: feedback?(rnewman)
And on mozilla-central:
Blocks: 748452
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.