Skip to content

Commit

Permalink
Merge branch 'feature/gcc-cleanup' into develop
Browse files Browse the repository at this point in the history
  • Loading branch information
grafikrobot committed Aug 22, 2017
2 parents 5377199 + fb11681 commit b487b1b
Show file tree
Hide file tree
Showing 89 changed files with 597 additions and 519 deletions.
2 changes: 0 additions & 2 deletions .travis.yml
Expand Up @@ -27,5 +27,3 @@ script:
- ./bootstrap.sh --with-toolset=${TOOLSET}
- cd test
- DO_DIFF=diff python test_all.py ${TOOLSET} ${TEST_ALL_EXTRAS}
- DO_DIFF=diff python library_chain.py --verbose ${TOOLSET} ${TEST_ALL_EXTRAS}
- DO_DIFF=diff python searched_lib.py --verbose ${TOOLSET} ${TEST_ALL_EXTRAS}
17 changes: 16 additions & 1 deletion src/build/virtual-target.jam
Expand Up @@ -794,7 +794,9 @@ class action
# rules.
.action on $(actual-targets) = $(__name__) ;

indirect.call $(self.action-name) $(actual-targets)
#indirect.call $(self.action-name) $(actual-targets)
# : $(self.actual-sources) : [ $(properties).raw ] ;
execute $(self.action-name) $(actual-targets)
: $(self.actual-sources) : [ $(properties).raw ] ;

# Since we set up the creating action here, we set up the action for
Expand Down Expand Up @@ -865,6 +867,19 @@ class action
{
return $(property-set) ;
}

# Execute the action rule on the given targets, sources, and properties.
# Since this does the final call to the engine action rule this takes
# engine level targets and raw properties. One could override this, for
# example, to set additional variables on hte target that might be
# difficult to determine just using toolset flags.
# Note, you must call this base rule when overriding as otherwise the
# actions will not execute and the engine will not run commands.
#
rule execute ( action-name targets + : sources * : properties * )
{
indirect.call $(action-name) $(targets) : $(sources) : $(properties) ;
}
}


Expand Down
2 changes: 1 addition & 1 deletion src/tools/clang-darwin.jam
Expand Up @@ -66,7 +66,7 @@ rule init ( version ? : command * : options * )

common.handle-options clang-darwin : $(condition) : $(command) : $(options) ;

gcc.init-link-flags clang-darwin darwin $(condition) ;
gcc.init-link-flags clang darwin $(condition) ;

}

Expand Down
2 changes: 1 addition & 1 deletion src/tools/clang-linux.jam
Expand Up @@ -60,7 +60,7 @@ rule init ( version ? : command * : options * ) {

common.handle-options clang-linux : $(condition) : $(command) : $(options) ;

gcc.init-link-flags clang-linux gnu $(condition) ;
gcc.init-link-flags clang linux $(condition) ;
}

###############################################################################
Expand Down
7 changes: 7 additions & 0 deletions src/tools/common.jam
Expand Up @@ -208,6 +208,13 @@ rule check-init-parameters ( toolset requirement * : * )
}
sig = $(sig)$(value:E="")- ;
}
# We also need to consider requirements on the toolset as we can
# configure the same toolset multiple times with different options that
# are selected with the requirements.
if $(requirement)
{
sig = $(sig)$(requirement:J=,) ;
}
if $(sig) in $(.all-signatures)
{
local message =
Expand Down

0 comments on commit b487b1b

Please sign in to comment.