Bugfix: don't create duplicate virtual targets for searched lib with

the same properties. Thanks to Andre Hentz for finding this bug and
providing the test case.

* new/builtin.jam
  (searched-lib-generator.run): Pass the result through
  'virtual-target.register'.

* test/searched_lib.py: Add new test.


[SVN r18247]

new/builtin.jam

@@ -430,7 +430,7 @@ rule searched-lib-generator ( )
             # attach an action and properties to the target
             a = [ new null-action $(t) : $(property-set) ] ;
             $(t).action $(a) ;
-            return $(t) ;
+            return [ virtual-target.register $(t) ] ;
         }
     }
 }

test/searched_lib.py

@@ -80,4 +80,23 @@ lib l : : <name>l_r <variant>release ;
 
 t.run_build_system("-n")
 
+# A regression test. Two virtual target with the same properties
+# were created for 'l' target, which caused and error to be reported
+# when actualizing targets. The final error is correct, but we should
+# not create two duplicated targets. Thanks to Andre Hentz
+# for finding this bug.
+t.write("project-root.jam", "")
+
+t.write("a.cpp", "")
+
+t.write("Jamfile", """
+project a : requirements <link-runtime>static ;
+
+lib a : a.cpp l ;
+lib l : : <name>l_f ;
+""")
+
+t.run_build_system("-n")
+
+
 t.cleanup()

v2/test/searched_lib.py

@@ -80,4 +80,23 @@ lib l : : <name>l_r <variant>release ;
 
 t.run_build_system("-n")
 
+# A regression test. Two virtual target with the same properties
+# were created for 'l' target, which caused and error to be reported
+# when actualizing targets. The final error is correct, but we should
+# not create two duplicated targets. Thanks to Andre Hentz
+# for finding this bug.
+t.write("project-root.jam", "")
+
+t.write("a.cpp", "")
+
+t.write("Jamfile", """
+project a : requirements <link-runtime>static ;
+
+lib a : a.cpp l ;
+lib l : : <name>l_f ;
+""")
+
+t.run_build_system("-n")
+
+
 t.cleanup()

v2/tools/builtin.jam

@@ -430,7 +430,7 @@ rule searched-lib-generator ( )
             # attach an action and properties to the target
             a = [ new null-action $(t) : $(property-set) ] ;
             $(t).action $(a) ;
-            return $(t) ;
+            return [ virtual-target.register $(t) ] ;
         }
     }
 }