On 5/22/13 3:23 PM, jvilk wrote:
>
> In Makefile:
>
> >  # SECONDEXPANSION allows us to use '%' and '$@' in our prerequisites. These
> >  # variables are not bound when the first expansion occurs. The directive
> >  # applies to all rules from this point on, so put it at the bottom of the file.
> >  .SECONDEXPANSION:
> >  build/release/compressed.js build/benchmark/compressed.js: build/%/compressed.js:\
> >  	build/% $$(%_BROWSER_SRCS)
> > +	mkdir -p $(dir $@)/browser/doppio-source
> >  	for src in $($*_BROWSER_SRCS); do \
>
> Now that these items are backed by tangible files, would it make sense 
> to have a new Makefile rule for building the sed'd files?
>
> Also, to make the Makefile simpler, we can add any needed empty 
> directories to the repository with a small |README.md|, e.g. 
> https://github.com/dropbox/dropbox-js/tree/master/lib
>
> I also think that it would be cleaner to deposit the sed'd files and 
> any other build cruft into a |tmp| directory, as they are not needed 
> by the deployment. Although if this adds significant complexity to the 
> |Makefile|, it might not be worth it.
>
true (at least they are unneeded for now, until someone gets sourcemaps 
mapping back to the sed'd coffeescript)
For now one could just rm -rf  $(dir $@)/browser/doppio-source
Maybe rename to doppio-source-tmp ?
FYI I also intended to have separate build areas for the two build 
targets, build/release/ and  build/benchmark



> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/int3/doppio/pull/217/files#r4348014>.
>