rscons 1.4.1 → 1.4.2

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    Yzg0NDA4ZjBmYzA5MGQ5MmFiM2QxZDIyNmI1Yzg5YWM4Y2Q2ZDViOQ==
+    NTA0YjY5YjRkMGY4Y2QxOGFjNzhlOTJjMmRjMzA5MjZiZjI2MTUzOQ==
   data.tar.gz: !binary |-
-    Y2ZmMGVkZTEyYjY1NWY2YzNhYWI4ZmUyYmNhZGM1ZDVhMzdmNzRiZA==
+    MmJjM2ZmZjRkMGUwMTdjNjE3ZmFmNTI5NTVlYjFhNTg0YzNhMmNmNg==
 SHA512:
   metadata.gz: !binary |-
-    Y2EzODEzMDZkYTUzMzkzNzIwZTcyOTMzNjk4NmVhMjM2MTc2MGZhYTZkNWIx
-    ZjU2YzA3NWMzYzJiM2Q3OWRhMDYyMmEzZjRhMTkxYzM3YzAyZjUyY2Y5NGNk
-    NjIzOGJkODlhOTVhZDVhZjkxOWI2OGU1ODk2NTVhY2VhNGI2Njg=
+    N2M3YjM1ZGI4ODEyNjI4M2E5YzhkNTU5NTI2MDFiMjE2MDY0ODRhNDI2YzM1
+    MTg1YmFlMTRkNWYyMjdjMzBiZTU2M2QzY2NjMDA2NzZlMDEwYmVhMGIxZjYy
+    NWY2YzAxNDQxMmFiZWNhNzk0MjZkODA4ZWEzMzMyZGNhMjFkMWU=
   data.tar.gz: !binary |-
-    MDE5YzhkY2FjNmU3NGEzZWVkZDNjZDBiOGFiYTVmMDI0MmU1MjhhOTRhYTdm
-    N2JhNGJlNzkxMjIzMzc4MDQwYTkzMDk3MzU1MjlmYzBhZTQyOTM3ZTlkMDAw
-    Njg0MTU0YTUxOGFlNzJjZWNjMTIyZGJhZWY4MDcwZTA2ODQ3OWQ=
+    MmEwNDAzZTc1ZDgyMjEzODJhOGQ4Njk2ZDlkY2ZkZDc2ZGU1ZDk0ZGU3MDgw
+    NjdmZjNkMmZmNmQzZjI5NmFlN2NmMDg0ZWU1ZjczYzQyNjZmYWMzN2ZhMTcz
+    NWRlYjcxZDc3NWY5ODYyYmFjMmFmMGMwNTE2MDNmNmNiNjMxYTA=

data/README.md

@@ -329,6 +329,15 @@ http://rubydoc.info/github/holtrop/rscons/frames.
 
 ## Release Notes
 
+### v1.4.2
+
+- add Environment#expand_path
+- expand construction variable references in builder targets and sources before invoking builder
+
+### v1.4.1
+
+- fix invoking a builder with no sources while a build root defined
+
 ### v1.4.0
 
 - add CFile builder

data/lib/rscons/environment.rb

@@ -172,7 +172,7 @@ module Rscons
     # called after the block returns.
     def process
       unless @targets.empty?
-        clean_target_paths!
+        expand_paths!
         cache = Cache.instance
         cache.clear_checksum_cache!
         targets_processed = {}
@@ -337,24 +337,35 @@ module Rscons
       builder.run(target, sources, cache, self, vars)
     end
 
-    private
+    # Expand a path to be relative to the Environment's build root.
+    #
+    # Paths beginning with "^/" are expanded by replacing "^" with the
+    # Environment's build root.
+    #
+    # @param path [String] The path to expand.
+    #
+    # @return [String] The expanded path.
+    def expand_path(path)
+      path.sub(%r{^\^(?=[\\/])}, @build_root)
+    end
 
-    # Expand all target paths that begin with ^/ to be relative to the
-    # Environment's build root, if present
-    def clean_target_paths!
-      if @build_root
-        expand = lambda do |path|
-          path.sub(%r{^\^(?=[\\/])}, @build_root)
-        end
+    private
 
-        new_targets = {}
-        @targets.each_pair do |target, target_params|
-          target_params[:sources].map! do |source|
-            expand[source]
-          end
-          new_targets[expand[target]] = target_params
+    # Expand target and source paths before invoking builders.
+    #
+    # This method expand construction variable references in the target and
+    # source file names before passing them to the builder. It also expands
+    # "^/" prefixes to the Environment's build root if a build root is defined.
+    def expand_paths!
+      @targets = @targets.reduce({}) do |result, (target, target_params)|
+        sources = target_params[:sources].map do |source|
+          source = expand_path(source) if @build_root
+          expand_varref(source)
         end
-        @targets = new_targets
+        target = expand_path(target) if @build_root
+        target = expand_varref(target)
+        result[target] = target_params.merge(sources: sources)
+        result
       end
     end
 

data/lib/rscons/version.rb

@@ -1,4 +1,4 @@
 module Rscons
   # gem version
-  VERSION = "1.4.1"
+  VERSION = "1.4.2"
 end

data/spec/build_tests_spec.rb

@@ -498,4 +498,30 @@ EOF
       env.TestBuilder("file")
     end
   end
+
+  it "expands construction variables in builder target and sources before invoking the builder" do
+    test_dir('custom_builder')
+    class MySource < Rscons::Builder
+      def run(target, sources, cache, env, vars)
+        File.open(target, 'w') do |fh|
+          fh.puts <<EOF
+    #define THE_VALUE 678
+EOF
+        end
+        target
+      end
+    end
+
+    Rscons::Environment.new do |env|
+      env["hdr"] = "inc.h"
+      env["src"] = "program.c"
+      env.add_builder(MySource.new)
+      env.MySource('${hdr}')
+      env.Program('program', "${src}")
+    end
+
+    lines.should == ['CC program.o', 'LD program']
+    File.exists?('inc.h').should be_true
+    `./program`.should == "The value is 678\n"
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rscons
 version: !ruby/object:Gem::Version
-  version: 1.4.1
+  version: 1.4.2
 platform: ruby
 authors:
 - Josh Holtrop
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-04-17 00:00:00.000000000 Z
+date: 2014-05-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec-core