From 7f11536704d9f971cf5c8b0a2b490ccc6af58588 Mon Sep 17 00:00:00 2001 From: Alexander Neundorf Date: Wed, 9 May 2007 08:25:45 -0400 Subject: [PATCH] ENH: now target names can be used in add_custom_command() and add_custom_target() as COMMAND, and cmake will recognize them and replace them with the actual output path of these executables. Also the dependency will be added automatically. Test included. ENH: moved TraceVSDependencies() to the end of GlobalGenerator::Configure(), so it is done now in one central place Alex --- Source/cmCustomCommand.h | 4 +- Source/cmGlobalGenerator.cxx | 18 +- Source/cmGlobalGenerator.h | 2 + Source/cmGlobalUnixMakefileGenerator3.cxx | 2 +- Source/cmGlobalVisualStudio6Generator.cxx | 5 +- Source/cmGlobalVisualStudio7Generator.cxx | 3 +- Source/cmGlobalXCodeGenerator.cxx | 169 +++++++++++------- Source/cmGlobalXCodeGenerator.h | 9 + Source/cmLocalGenerator.cxx | 39 ++++ Source/cmLocalGenerator.h | 14 +- Source/cmLocalUnixMakefileGenerator3.cxx | 6 +- Source/cmLocalVisualStudio6Generator.cxx | 96 +++++----- Source/cmLocalVisualStudio6Generator.h | 6 +- Source/cmLocalVisualStudio7Generator.cxx | 60 +++---- Source/cmLocalVisualStudio7Generator.h | 10 +- Source/cmLocalVisualStudioGenerator.cxx | 7 +- Source/cmLocalVisualStudioGenerator.h | 1 + Source/cmMakefile.cxx | 12 -- Source/cmTarget.cxx | 54 +++++- Source/cmTarget.h | 11 +- Templates/DLLHeader.dsptemplate | 8 +- Templates/EXEHeader.dsptemplate | 8 +- Templates/EXEWinHeader.dsptemplate | 8 +- Templates/UtilityHeader.dsptemplate | 8 +- Templates/staticLibHeader.dsptemplate | 8 +- Tests/CustomCommand/CMakeLists.txt | 25 +++ .../GeneratorInExtraDir/CMakeLists.txt | 6 + Tests/CustomCommand/main.cxx | 6 + 28 files changed, 400 insertions(+), 205 deletions(-) create mode 100644 Tests/CustomCommand/GeneratorInExtraDir/CMakeLists.txt create mode 100644 Tests/CustomCommand/main.cxx diff --git a/Source/cmCustomCommand.h b/Source/cmCustomCommand.h index 5d59f0a97..f7c6b5eee 100644 --- a/Source/cmCustomCommand.h +++ b/Source/cmCustomCommand.h @@ -69,8 +69,8 @@ public: void SetEscapeAllowMakeVars(bool b); /** set get the used status of the command */ - void SetUsed() { this->Used = true;}; - bool IsUsed() { return this->Used;}; + void SetUsed() { this->Used = true;} + bool IsUsed() const { return this->Used;} private: std::vector Outputs; diff --git a/Source/cmGlobalGenerator.cxx b/Source/cmGlobalGenerator.cxx index a0e66d326..16913086b 100644 --- a/Source/cmGlobalGenerator.cxx +++ b/Source/cmGlobalGenerator.cxx @@ -718,6 +718,13 @@ void cmGlobalGenerator::Configure() // at this point this->LocalGenerators has been filled, // so create the map from project name to vector of local generators this->FillProjectMap(); + // now create project to target map + // This will make sure that targets have all the + // targets they depend on as part of the build. + this->FillProjectToTargetMap(); + // now trace all dependencies + this->TraceDependencies(); + if ( !this->CMakeInstance->GetScriptMode() ) { this->CMakeInstance->UpdateProgress("Configuring done", -1); @@ -1036,6 +1043,13 @@ const char* cmGlobalGenerator::GetLinkerPreference(const char* lang) return "None"; } +void cmGlobalGenerator::TraceDependencies() +{ + for (unsigned int i = 0; i < this->LocalGenerators.size(); ++i) + { + this->LocalGenerators[i]->TraceDependencies(); + } +} void cmGlobalGenerator::FillProjectMap() { @@ -1057,10 +1071,6 @@ void cmGlobalGenerator::FillProjectMap() } while (lg); } - // now create project to target map - // This will make sure that targets have all the - // targets they depend on as part of the build. - this->FillProjectToTargetMap(); } diff --git a/Source/cmGlobalGenerator.h b/Source/cmGlobalGenerator.h index 65fe7159e..0c0cd20f6 100644 --- a/Source/cmGlobalGenerator.h +++ b/Source/cmGlobalGenerator.h @@ -202,6 +202,8 @@ protected: void FillProjectMap(); bool IsExcluded(cmLocalGenerator* root, cmLocalGenerator* gen); void FillProjectToTargetMap(); + // Calls TraceVSDependencies() on all targets + void TraceDependencies(); void CreateDefaultGlobalTargets(cmTargets* targets); cmTarget CreateGlobalTarget(const char* name, const char* message, const cmCustomCommandLines* commandLines, diff --git a/Source/cmGlobalUnixMakefileGenerator3.cxx b/Source/cmGlobalUnixMakefileGenerator3.cxx index b4e05ede8..068cc517b 100644 --- a/Source/cmGlobalUnixMakefileGenerator3.cxx +++ b/Source/cmGlobalUnixMakefileGenerator3.cxx @@ -1056,7 +1056,7 @@ cmGlobalUnixMakefileGenerator3 { cmTarget *result; cmLocalUnixMakefileGenerator3 *lg3; - + // first check the same dir as the current target lg3 = static_cast (target.GetMakefile()->GetLocalGenerator()); diff --git a/Source/cmGlobalVisualStudio6Generator.cxx b/Source/cmGlobalVisualStudio6Generator.cxx index 3b905c03e..3829b3e09 100644 --- a/Source/cmGlobalVisualStudio6Generator.cxx +++ b/Source/cmGlobalVisualStudio6Generator.cxx @@ -208,9 +208,8 @@ void cmGlobalVisualStudio6Generator static_cast(generators[i]) ->GetCreatedProjectNames(); cmTargets &tgts = generators[i]->GetMakefile()->GetTargets(); - cmTargets::iterator l = tgts.begin(); - for(std::vector::iterator si = dspnames.begin(); - l != tgts.end(); ++l) + std::vector::iterator si = dspnames.begin(); + for(cmTargets::iterator l = tgts.begin(); l != tgts.end(); ++l) { // special handling for the current makefile if(mf == generators[0]->GetMakefile()) diff --git a/Source/cmGlobalVisualStudio7Generator.cxx b/Source/cmGlobalVisualStudio7Generator.cxx index afe96e718..67fd7a3a1 100644 --- a/Source/cmGlobalVisualStudio7Generator.cxx +++ b/Source/cmGlobalVisualStudio7Generator.cxx @@ -294,8 +294,7 @@ void cmGlobalVisualStudio7Generator // than one dsp could have been created per input CMakeLists.txt file // for each target cmTargets &tgts = generators[i]->GetMakefile()->GetTargets(); - cmTargets::iterator l = tgts.begin(); - for(; l != tgts.end(); ++l) + for (cmTargets::iterator l = tgts.begin(); l != tgts.end(); ++l) { // special handling for the current makefile if(mf == generators[0]->GetMakefile()) diff --git a/Source/cmGlobalXCodeGenerator.cxx b/Source/cmGlobalXCodeGenerator.cxx index 7e6d506e2..580aa5b3d 100644 --- a/Source/cmGlobalXCodeGenerator.cxx +++ b/Source/cmGlobalXCodeGenerator.cxx @@ -577,16 +577,17 @@ cmGlobalXCodeGenerator::CreateXCodeTargets(cmLocalGenerator* gen, { continue; } + + if(cmtarget.GetType() == cmTarget::UTILITY || + cmtarget.GetType() == cmTarget::GLOBAL_TARGET) + { + targets.push_back(this->CreateUtilityTarget(cmtarget)); + } if(cmtarget.GetType() == cmTarget::UTILITY || cmtarget.GetType() == cmTarget::GLOBAL_TARGET || cmtarget.GetType() == cmTarget::INSTALL_FILES || cmtarget.GetType() == cmTarget::INSTALL_PROGRAMS) { - if(cmtarget.GetType() == cmTarget::UTILITY || - cmtarget.GetType() == cmTarget::GLOBAL_TARGET) - { - targets.push_back(this->CreateUtilityTarget(cmtarget)); - } continue; } @@ -849,6 +850,7 @@ std::string cmGlobalXCodeGenerator::ExtractFlag(const char* flag, } return retFlag; } + //---------------------------------------------------------------------------- void cmGlobalXCodeGenerator::AddCommandsToBuildPhase(cmXCodeObject* buildphase, @@ -872,8 +874,31 @@ cmGlobalXCodeGenerator::AddCommandsToBuildPhase(cmXCodeObject* buildphase, return; } + // collect multiple outputs of custom commands into a set + // which will be used for every configuration std::map multipleOutputPairs; - + for(std::vector::const_iterator i = commands.begin(); + i != commands.end(); ++i) + { + cmCustomCommand const& cc = *i; + if(!cc.GetCommandLines().empty()) + { + const std::vector& outputs = cc.GetOutputs(); + if(!outputs.empty()) + { + // If there are more than one outputs treat the + // first as the primary output and make the rest depend on it. + std::vector::const_iterator o = outputs.begin(); + std::string primaryOutput = this->ConvertToRelativeForMake(o->c_str()); + for(++o; o != outputs.end(); ++o) + { + std::string currentOutput =this->ConvertToRelativeForMake(o->c_str()); + multipleOutputPairs[currentOutput] = primaryOutput; + } + } + } + } + std::string dir = this->CurrentMakefile->GetCurrentOutputDirectory(); dir += "/CMakeScripts"; cmSystemTools::MakeDirectory(dir.c_str()); @@ -883,7 +908,48 @@ cmGlobalXCodeGenerator::AddCommandsToBuildPhase(cmXCodeObject* buildphase, makefile += "_"; makefile += name; makefile += ".make"; - cmGeneratedFileStream makefileStream(makefile.c_str()); + + for (std::vector::const_iterator currentConfig= + this->CurrentConfigurationTypes.begin(); + currentConfig!=this->CurrentConfigurationTypes.end(); + currentConfig++ ) + { + this->CreateCustomRulesMakefile(makefile.c_str(), + target, + commands, + currentConfig->c_str(), + multipleOutputPairs); + } + + std::string cdir = this->CurrentMakefile->GetCurrentOutputDirectory(); + cdir = this->ConvertToRelativeForXCode(cdir.c_str()); + std::string makecmd = "make -C "; + makecmd += cdir; + makecmd += " -f "; + makecmd += this->ConvertToRelativeForMake((makefile+"$CONFIGURATION").c_str()); + if(!multipleOutputPairs.empty()) + { + makecmd += " cmake_check_multiple_outputs"; + } + makecmd += " all"; + cmSystemTools::ReplaceString(makecmd, "\\ ", "\\\\ "); + buildphase->AddAttribute("shellScript", + this->CreateString(makecmd.c_str())); +} + +//---------------------------------------------------------------------------- +void cmGlobalXCodeGenerator +::CreateCustomRulesMakefile(const char* makefileBasename, + cmTarget& target, + std::vector + const & commands, + const char* configName, + const std::map& multipleOutputPairs + ) +{ + std::string makefileName=makefileBasename; + makefileName+=configName; + cmGeneratedFileStream makefileStream(makefileName.c_str()); if(!makefileStream) { return; @@ -909,7 +975,7 @@ cmGlobalXCodeGenerator::AddCommandsToBuildPhase(cmXCodeObject* buildphase, o != outputs.end(); ++o) { makefileStream - << "\\\n\t" << this->ConvertToRelativeForMake(o->c_str()); + << "\\\n\t" << this->ConvertToRelativeForMake(o->c_str()); } } else @@ -939,21 +1005,9 @@ cmGlobalXCodeGenerator::AddCommandsToBuildPhase(cmXCodeObject* buildphase, const std::vector& outputs = cc.GetOutputs(); if(!outputs.empty()) { - // There is at least one output. If there is more than one treat the - // first as the primary output and make the rest depend on it. - std::vector::const_iterator o = outputs.begin(); + // There is at least one output, start the rule for it std::string primary_output = - this->ConvertToRelativeForMake(o->c_str()); - for(++o; o != outputs.end(); ++o) - { - std::string current_output = - this->ConvertToRelativeForMake(o->c_str()); - makefileStream << current_output << ": " - << primary_output << "\n"; - multipleOutputPairs[current_output] = primary_output; - } - - // Start the rule for the primary output. + this->ConvertToRelativeForMake(outputs.begin()->c_str()); makefileStream << primary_output << ": "; } else @@ -962,11 +1016,11 @@ cmGlobalXCodeGenerator::AddCommandsToBuildPhase(cmXCodeObject* buildphase, makefileStream << tname[&cc] << ": "; } for(std::vector::const_iterator d = - cc.GetDepends().begin(); + cc.GetDepends().begin(); d != cc.GetDepends().end(); ++d) { if(!this->FindTarget(this->CurrentProject.c_str(), - d->c_str())) + d->c_str())) { // if the depend is not a target but // is a full path then use it, if not then @@ -974,7 +1028,7 @@ cmGlobalXCodeGenerator::AddCommandsToBuildPhase(cmXCodeObject* buildphase, if(cmSystemTools::FileIsFullPath(d->c_str())) { makefileStream << "\\\n" << this - ->ConvertToRelativeForMake(d->c_str()); + ->ConvertToRelativeForMake(d->c_str()); } } else @@ -989,12 +1043,14 @@ cmGlobalXCodeGenerator::AddCommandsToBuildPhase(cmXCodeObject* buildphase, // Add each command line to the set of commands. for(cmCustomCommandLines::const_iterator cl = - cc.GetCommandLines().begin(); + cc.GetCommandLines().begin(); cl != cc.GetCommandLines().end(); ++cl) { // Build the command line in a single string. const cmCustomCommandLine& commandLine = *cl; - std::string cmd2 = commandLine[0]; + std::string cmd2 = this->CurrentLocalGenerator + ->GetRealLocation(commandLine[0].c_str(), configName); + cmSystemTools::ReplaceString(cmd2, "/./", "/"); cmd2 = this->ConvertToRelativeForMake(cmd2.c_str()); std::string cmd; @@ -1011,13 +1067,13 @@ cmGlobalXCodeGenerator::AddCommandsToBuildPhase(cmXCodeObject* buildphase, if(escapeOldStyle) { cmd += (this->CurrentLocalGenerator - ->EscapeForShellOldStyle(commandLine[j].c_str())); + ->EscapeForShellOldStyle(commandLine[j].c_str())); } else { cmd += (this->CurrentLocalGenerator-> - EscapeForShell(commandLine[j].c_str(), - escapeAllowMakeVars)); + EscapeForShell(commandLine[j].c_str(), + escapeAllowMakeVars)); } } makefileStream << "\t" << cmd.c_str() << "\n"; @@ -1025,38 +1081,31 @@ cmGlobalXCodeGenerator::AddCommandsToBuildPhase(cmXCodeObject* buildphase, } } - // Add a rule to deal with multiple outputs of custom commands. + // Add rules to deal with multiple outputs of custom commands. if(!multipleOutputPairs.empty()) { - makefileStream << - "\n" - "cmake_check_multiple_outputs:\n"; + makefileStream << + "\n# Dependencies of multiple outputs to their primary outputs \n"; + for(std::map::const_iterator o = - multipleOutputPairs.begin(); o != multipleOutputPairs.end(); ++o) + multipleOutputPairs.begin(); o != multipleOutputPairs.end(); ++o) + { + makefileStream << o->first << ": " << o->second << "\n"; + } + + makefileStream << + "\n" + "cmake_check_multiple_outputs:\n"; + for(std::map::const_iterator o = + multipleOutputPairs.begin(); o != multipleOutputPairs.end(); ++o) { makefileStream << "\t@if [ ! -f " - << o->first << " ]; then rm -f " - << o->second << "; fi\n"; + << o->first << " ]; then rm -f " + << o->second << "; fi\n"; } } - - std::string cdir = this->CurrentMakefile->GetCurrentOutputDirectory(); - cdir = this->ConvertToRelativeForXCode(cdir.c_str()); - std::string makecmd = "make -C "; - makecmd += cdir; - makecmd += " -f "; - makecmd += this->ConvertToRelativeForMake(makefile.c_str()); - if(!multipleOutputPairs.empty()) - { - makecmd += " cmake_check_multiple_outputs"; - } - makecmd += " all"; - cmSystemTools::ReplaceString(makecmd, "\\ ", "\\\\ "); - buildphase->AddAttribute("shellScript", - this->CreateString(makecmd.c_str())); } - //---------------------------------------------------------------------------- void cmGlobalXCodeGenerator::CreateBuildSettings(cmTarget& target, cmXCodeObject* buildSettings, @@ -2406,20 +2455,6 @@ cmGlobalXCodeGenerator::OutputXCodeProject(cmLocalGenerator* root, cmMakefile* mf = (*g)->GetMakefile(); std::vector sourceGroups = mf->GetSourceGroups(); cmTargets &tgts = mf->GetTargets(); - // Call TraceVSDependencies on all targets - for(cmTargets::iterator l = tgts.begin(); - l != tgts.end(); l++) - { - // INCLUDE_EXTERNAL_MSPROJECT command only affects the workspace - // so don't build a projectfile for it - if ((l->second.GetType() != cmTarget::INSTALL_FILES) - && (l->second.GetType() != cmTarget::INSTALL_PROGRAMS) - && (strncmp(l->first.c_str(), "INCLUDE_EXTERNAL_MSPROJECT", 26) != 0)) - { - cmTarget& target = l->second; - target.TraceVSDependencies(target.GetName(), mf); - } - } // now for all custom commands that are not used directly in a // target, add them to all targets in the current directory or // makefile diff --git a/Source/cmGlobalXCodeGenerator.h b/Source/cmGlobalXCodeGenerator.h index e9091e42d..4a0821745 100644 --- a/Source/cmGlobalXCodeGenerator.h +++ b/Source/cmGlobalXCodeGenerator.h @@ -106,6 +106,15 @@ private: std::vector const & commands, const char* commandFileName); + + void CreateCustomRulesMakefile(const char* makefileBasename, + cmTarget& target, + std::vector const & commands, + const char* configName, + const std::map& + multipleOutputPairs + ); + cmXCodeObject* FindXCodeTarget(cmTarget*); // create cmXCodeObject from these functions so that memory can be managed // correctly. All objects created are stored in this->XCodeObjects. diff --git a/Source/cmLocalGenerator.cxx b/Source/cmLocalGenerator.cxx index 05ba04d71..ec78dafeb 100644 --- a/Source/cmLocalGenerator.cxx +++ b/Source/cmLocalGenerator.cxx @@ -52,6 +52,7 @@ cmLocalGenerator::cmLocalGenerator() this->UseRelativePaths = false; this->Configured = false; this->EmitUniversalBinaryFlags = true; + this->IsMakefileGenerator = false; this->RelativePathsConfigured = false; this->PathConversionsSetup = false; } @@ -135,6 +136,30 @@ void cmLocalGenerator::ConfigureFinalPass() this->Makefile->ConfigureFinalPass(); } +void cmLocalGenerator::TraceDependencies() +{ + // Generate the rule files for each target. + cmTargets& targets = this->Makefile->GetTargets(); + for(cmTargets::iterator t = targets.begin(); t != targets.end(); ++t) + { + // INCLUDE_EXTERNAL_MSPROJECT command only affects the workspace + // so don't build a projectfile for it + if ((t->second.GetType() != cmTarget::INSTALL_FILES) + && (t->second.GetType() != cmTarget::INSTALL_PROGRAMS) + && (t->second.GetType() != cmTarget::INSTALL_DIRECTORY) + && (strncmp(t->first.c_str(), "INCLUDE_EXTERNAL_MSPROJECT", 26) != 0) + && (t->second.GetPropertyAsBool("IMPORTED") == false)) + { + std::string projectFilename; + if (this->IsMakefileGenerator == false) // only use of this variable + { + projectFilename=t->second.GetName(); + } + t->second.TraceVSDependencies(projectFilename, this->Makefile); + } + } +} + void cmLocalGenerator::GenerateTestFiles() { if ( !this->Makefile->IsOn("CMAKE_TESTING_ENABLED") ) @@ -1986,6 +2011,20 @@ std::string cmLocalGenerator::GetRealDependency(const char* inName, return name; } +//---------------------------------------------------------------------------- +std::string cmLocalGenerator::GetRealLocation(const char* inName, + const char* config) +{ + std::string outName=inName; + // Look for a CMake target with the given name. + cmTarget* target = this->GlobalGenerator->FindTarget(0, inName); + if ((target!=0) && (target->GetType()==cmTarget::EXECUTABLE)) + { + outName = target->GetLocation( config ); + } + return outName; +} + //---------------------------------------------------------------------------- void cmLocalGenerator::AddSharedFlags(std::string& flags, const char* lang, diff --git a/Source/cmLocalGenerator.h b/Source/cmLocalGenerator.h index 651988960..a9f56822c 100644 --- a/Source/cmLocalGenerator.h +++ b/Source/cmLocalGenerator.h @@ -42,7 +42,7 @@ public: /** * Generate the makefile for this directory. */ - virtual void Generate() {}; + virtual void Generate() {} /** * Process the CMakeLists files for this directory to fill in the @@ -50,6 +50,11 @@ public: */ virtual void Configure(); + /** + * Calls TraceVSDependencies() on all targets of this generator. + */ + virtual void TraceDependencies(); + /** * Perform any final calculations prior to generation */ @@ -137,6 +142,11 @@ public: the source directory of this generator. This should only be used for dependencies of custom commands. */ std::string GetRealDependency(const char* name, const char* config); + + /** Translate a command as given in CMake code to the location of the + executable if the command is the name of a CMake executable target. + If that's not the case, just return the original name. */ + std::string GetRealLocation(const char* inName, const char* config); ///! for existing files convert to output path and short path if spaces std::string ConvertToOutputForExisting(const char* p); @@ -298,6 +308,8 @@ protected: bool IgnoreLibPrefix; bool Configured; bool EmitUniversalBinaryFlags; + // A type flag is not nice. It's used only in TraceDependencies(). + bool IsMakefileGenerator; // Hack for ExpandRuleVariable until object-oriented version is // committed. std::string TargetImplib; diff --git a/Source/cmLocalUnixMakefileGenerator3.cxx b/Source/cmLocalUnixMakefileGenerator3.cxx index 2b8f3466f..a48bebbaa 100644 --- a/Source/cmLocalUnixMakefileGenerator3.cxx +++ b/Source/cmLocalUnixMakefileGenerator3.cxx @@ -54,6 +54,7 @@ cmLocalUnixMakefileGenerator3::cmLocalUnixMakefileGenerator3() this->NativeEchoCommand = "@echo "; this->NativeEchoWindows = true; this->MakeCommandEscapeTargetTwice = false; + this->IsMakefileGenerator = true; } //---------------------------------------------------------------------------- @@ -112,7 +113,7 @@ void cmLocalUnixMakefileGenerator3::Generate() if (tg) { this->TargetGenerators.push_back(tg); - t->second.TraceVSDependencies(empty, this->Makefile); +// t->second.TraceVSDependencies(empty, this->Makefile); tg->WriteRuleFiles(); } } @@ -885,7 +886,8 @@ cmLocalUnixMakefileGenerator3 { // Build the command line in a single string. const cmCustomCommandLine& commandLine = *cl; - std::string cmd = commandLine[0]; + std::string cmd = GetRealLocation(commandLine[0].c_str(), + this->ConfigurationName.c_str()); if (cmd.size()) { cmSystemTools::ReplaceString(cmd, "/./", "/"); diff --git a/Source/cmLocalVisualStudio6Generator.cxx b/Source/cmLocalVisualStudio6Generator.cxx index 24a381afd..b3ae6e219 100644 --- a/Source/cmLocalVisualStudio6Generator.cxx +++ b/Source/cmLocalVisualStudio6Generator.cxx @@ -116,16 +116,6 @@ void cmLocalVisualStudio6Generator::OutputDSPFile() { this->AddDSPBuildRule(l->second); } - - // INCLUDE_EXTERNAL_MSPROJECT command only affects the workspace - // so don't build a projectfile for it - if ((l->second.GetType() != cmTarget::INSTALL_FILES) - && (l->second.GetType() != cmTarget::INSTALL_PROGRAMS) - && (strncmp(l->first.c_str(), "INCLUDE_EXTERNAL_MSPROJECT", 26) != 0)) - { - cmTarget& target = l->second; - target.TraceVSDependencies(target.GetName(), this->Makefile); - } } // build any targets @@ -490,23 +480,8 @@ void cmLocalVisualStudio6Generator } if (command) { - std::string script = - this->ConstructScript(command->GetCommandLines(), - command->GetWorkingDirectory(), - command->GetEscapeOldStyle(), - command->GetEscapeAllowMakeVars(), - "\\\n\t"); - std::string comment = - this->ConstructComment(*command, - "Building Custom Rule $(InputPath)"); - if(comment == "") - { - comment = ""; - } const char* flags = compileFlags.size() ? compileFlags.c_str(): 0; - this->WriteCustomRule(fout, source.c_str(), script.c_str(), - comment.c_str(), command->GetDepends(), - command->GetOutputs(), flags); + this->WriteCustomRule(fout, source.c_str(), *command, flags); } else if(!compileFlags.empty() || !objectNameDir.empty()) { @@ -603,16 +578,29 @@ void cmLocalVisualStudio6Generator ::WriteCustomRule(std::ostream& fout, const char* source, - const char* command, - const char* comment, - const std::vector& depends, - const std::vector& outputs, + const cmCustomCommand& command, const char* flags) { + std::string comment = + this->ConstructComment(command, "Building Custom Rule $(InputPath)"); + if(comment == "") + { + comment = ""; + } + // Write the rule for each configuration. std::vector::iterator i; for(i = this->Configurations.begin(); i != this->Configurations.end(); ++i) { + + std::string script = + this->ConstructScript(command.GetCommandLines(), + command.GetWorkingDirectory(), + i->c_str(), + command.GetEscapeOldStyle(), + command.GetEscapeAllowMakeVars(), + "\\\n\t"); + if (i == this->Configurations.begin()) { fout << "!IF \"$(CFG)\" == " << i->c_str() << std::endl; @@ -627,8 +615,9 @@ cmLocalVisualStudio6Generator } // Write out the dependencies for the rule. fout << "USERDEP__HACK="; - for(std::vector::const_iterator d = depends.begin(); - d != depends.end(); ++d) + for(std::vector::const_iterator d=command.GetDepends().begin(); + d != command.GetDepends().end(); + ++d) { // Lookup the real name of the dependency in case it is a CMake target. std::string dep = this->GetRealDependency(d->c_str(), i->c_str()); @@ -639,26 +628,28 @@ cmLocalVisualStudio6Generator fout << "# PROP Ignore_Default_Tool 1\n"; fout << "# Begin Custom Build -"; - if(comment && *comment) + if(!comment.empty()) { - fout << " " << comment; + fout << " " << comment.c_str(); } fout << "\n\n"; - if(outputs.empty()) + if(command.GetOutputs().empty()) { fout << source << "_force : \"$(SOURCE)\" \"$(INTDIR)\" \"$(OUTDIR)\"\n\t"; - fout << command << "\n\n"; + fout << script.c_str() << "\n\n"; } else { - for(std::vector::const_iterator o = outputs.begin(); - o != outputs.end(); ++o) + for(std::vector::const_iterator o = + command.GetOutputs().begin(); + o != command.GetOutputs().end(); + ++o) { // Write a rule for every output generated by this command. fout << this->ConvertToOptionallyRelativeOutputPath(o->c_str()) << " : \"$(SOURCE)\" \"$(INTDIR)\" \"$(OUTDIR)\"\n\t"; - fout << command << "\n\n"; + fout << script.c_str() << "\n\n"; } } fout << "# End Custom Build\n\n"; @@ -768,6 +759,7 @@ void cmLocalVisualStudio6Generator::SetBuildType(BuildType b, // look for custom rules on a target and collect them together std::string cmLocalVisualStudio6Generator::CreateTargetRules(cmTarget &target, + const char* configName, const char * /* libName */) { std::string customRuleCode = ""; @@ -808,6 +800,7 @@ cmLocalVisualStudio6Generator::CreateTargetRules(cmTarget &target, } customRuleCode += this->ConstructScript(cr->GetCommandLines(), cr->GetWorkingDirectory(), + configName, cr->GetEscapeOldStyle(), cr->GetEscapeAllowMakeVars(), "\\\n\t"); @@ -822,6 +815,7 @@ cmLocalVisualStudio6Generator::CreateTargetRules(cmTarget &target, } customRuleCode += this->ConstructScript(cr->GetCommandLines(), cr->GetWorkingDirectory(), + configName, cr->GetEscapeOldStyle(), cr->GetEscapeAllowMakeVars(), "\\\n\t"); @@ -850,6 +844,7 @@ cmLocalVisualStudio6Generator::CreateTargetRules(cmTarget &target, } customRuleCode += this->ConstructScript(cr->GetCommandLines(), cr->GetWorkingDirectory(), + configName, cr->GetEscapeOldStyle(), cr->GetEscapeAllowMakeVars(), "\\\n\t"); @@ -1274,7 +1269,14 @@ void cmLocalVisualStudio6Generator // are there any custom rules on the target itself // only if the target is a lib or exe - std::string customRuleCode = this->CreateTargetRules(target, libName); + std::string customRuleCodeRelease + = this->CreateTargetRules(target, "RELEASE", libName); + std::string customRuleCodeDebug + = this->CreateTargetRules(target, "DEBUG", libName); + std::string customRuleCodeMinSizeRel + = this->CreateTargetRules(target, "MINSIZEREL", libName); + std::string customRuleCodeRelWithDebInfo + = this->CreateTargetRules(target, "RELWITHDEBINFO", libName); std::ifstream fin(this->DSPHeaderTemplate.c_str()); if(!fin) @@ -1307,8 +1309,6 @@ void cmLocalVisualStudio6Generator } cmSystemTools::ReplaceString(line, "OUTPUT_LIBNAME_EXPORTS", exportSymbol.c_str()); - cmSystemTools::ReplaceString(line, "CMAKE_CUSTOM_RULE_CODE", - customRuleCode.c_str()); cmSystemTools::ReplaceString(line, "CMAKE_MFC_FLAG", mfcFlag); if(target.GetType() == cmTarget::STATIC_LIBRARY ) @@ -1337,6 +1337,18 @@ void cmLocalVisualStudio6Generator cmSystemTools::ReplaceString(line, "CM_MULTILINE_OPTIMIZED_LIBRARIES", libMultiLineOptimizedOptions.c_str()); #endif + + // Substitute the rules for custom command. When specifying just the + // target name for the command the command can be different for + // different configs + cmSystemTools::ReplaceString(line, "CMAKE_CUSTOM_RULE_CODE_RELEASE", + customRuleCodeRelease.c_str()); + cmSystemTools::ReplaceString(line, "CMAKE_CUSTOM_RULE_CODE_DEBUG", + customRuleCodeDebug.c_str()); + cmSystemTools::ReplaceString(line, "CMAKE_CUSTOM_RULE_CODE_MINSIZEREL", + customRuleCodeMinSizeRel.c_str()); + cmSystemTools::ReplaceString(line, "CMAKE_CUSTOM_RULE_CODE_RELWITHDEBINFO", + customRuleCodeRelWithDebInfo.c_str()); // Substitute the real output name into the template. cmSystemTools::ReplaceString(line, "OUTPUT_NAME_DEBUG", diff --git a/Source/cmLocalVisualStudio6Generator.h b/Source/cmLocalVisualStudio6Generator.h index 3b789f8f3..7ac84a43e 100644 --- a/Source/cmLocalVisualStudio6Generator.h +++ b/Source/cmLocalVisualStudio6Generator.h @@ -84,10 +84,7 @@ private: void AddDSPBuildRule(cmTarget& tgt); void WriteCustomRule(std::ostream& fout, const char* source, - const char* command, - const char* comment, - const std::vector& depends, - const std::vector& outputs, + const cmCustomCommand& command, const char* flags); void AddUtilityCommandHack(cmTarget& target, int count, std::vector& depends, @@ -95,6 +92,7 @@ private: void WriteGroup(const cmSourceGroup *sg, cmTarget target, std::ostream &fout, const char *libName); std::string CreateTargetRules(cmTarget &target, + const char* configName, const char *libName); void ComputeLinkOptions(cmTarget& target, const char* configName, const std::string extraOptions, diff --git a/Source/cmLocalVisualStudio7Generator.cxx b/Source/cmLocalVisualStudio7Generator.cxx index c40ca4298..6e2f1be01 100644 --- a/Source/cmLocalVisualStudio7Generator.cxx +++ b/Source/cmLocalVisualStudio7Generator.cxx @@ -124,16 +124,6 @@ void cmLocalVisualStudio7Generator::OutputVCProjFile() { this->AddVCProjBuildRule(l->second); } - - // INCLUDE_EXTERNAL_MSPROJECT command only affects the workspace - // so don't build a projectfile for it - if ((l->second.GetType() != cmTarget::INSTALL_FILES) - && (l->second.GetType() != cmTarget::INSTALL_PROGRAMS) - && (strncmp(l->first.c_str(), "INCLUDE_EXTERNAL_MSPROJECT", 26) != 0)) - { - cmTarget& target = l->second; - target.TraceVSDependencies(target.GetName(), this->Makefile); - } } for(cmTargets::iterator l = tgts.begin(); @@ -623,7 +613,7 @@ void cmLocalVisualStudio7Generator::WriteConfiguration(std::ostream& fout, << "\t\t\t/>\n"; } - this->OutputTargetRules(fout, target, libName); + this->OutputTargetRules(fout, configName, target, libName); this->OutputBuildTool(fout, configName, target); fout << "\t\t\n"; } @@ -1141,17 +1131,8 @@ void cmLocalVisualStudio7Generator fout << "\t\t\t\tRelativePath=\"" << d << "\">\n"; if (command) { - // Construct the entire set of commands in one string. - std::string script = - this->ConstructScript(command->GetCommandLines(), - command->GetWorkingDirectory(), - command->GetEscapeOldStyle(), - command->GetEscapeAllowMakeVars()); - std::string comment = this->ConstructComment(*command); const char* flags = compileFlags.size() ? compileFlags.c_str(): 0; - this->WriteCustomRule(fout, source.c_str(), script.c_str(), - comment.c_str(), command->GetDepends(), - command->GetOutputs(), flags); + this->WriteCustomRule(fout, source.c_str(), *command, flags); } else if(compileFlags.size() || additionalDeps.length() || objectName.size() || excludedFromBuild) @@ -1228,12 +1209,11 @@ void cmLocalVisualStudio7Generator void cmLocalVisualStudio7Generator:: WriteCustomRule(std::ostream& fout, const char* source, - const char* command, - const char* comment, - const std::vector& depends, - const std::vector& outputs, + const cmCustomCommand& command, const char* compileFlags) { + std::string comment = this->ConstructComment(command); + // Write the rule for each configuration. std::vector::iterator i; std::vector *configs = @@ -1251,14 +1231,21 @@ WriteCustomRule(std::ostream& fout, << "\t\t\t\t\tAdditionalOptions=\"" << this->EscapeForXML(compileFlags) << "\"/>\n"; } + + std::string script = + this->ConstructScript(command.GetCommandLines(), + command.GetWorkingDirectory(), + i->c_str(), + command.GetEscapeOldStyle(), + command.GetEscapeAllowMakeVars()); fout << "\t\t\t\t\tEscapeForXML(comment) << "\"\n" + << this->EscapeForXML(comment.c_str()) << "\"\n" << "\t\t\t\t\tCommandLine=\"" - << this->EscapeForXML(command) << "\"\n" + << this->EscapeForXML(script.c_str()) << "\"\n" << "\t\t\t\t\tAdditionalDependencies=\""; - if(depends.empty()) + if(command.GetDepends().empty()) { // There are no real dependencies. Produce an artificial one to // make sure the rule runs reliably. @@ -1272,8 +1259,10 @@ WriteCustomRule(std::ostream& fout, else { // Write out the dependencies for the rule. - for(std::vector::const_iterator d = depends.begin(); - d != depends.end(); ++d) + for(std::vector::const_iterator d = + command.GetDepends().begin(); + d != command.GetDepends().end(); + ++d) { // Get the real name of the dependency in case it is a CMake target. std::string dep = this->GetRealDependency(d->c_str(), i->c_str()); @@ -1283,7 +1272,7 @@ WriteCustomRule(std::ostream& fout, } fout << "\"\n"; fout << "\t\t\t\t\tOutputs=\""; - if(outputs.empty()) + if(command.GetOutputs().empty()) { fout << source << "_force"; } @@ -1291,8 +1280,9 @@ WriteCustomRule(std::ostream& fout, { // Write a rule for the output generated by this command. const char* sep = ""; - for(std::vector::const_iterator o = outputs.begin(); - o != outputs.end(); ++o) + for(std::vector::const_iterator o = command.GetOutputs().begin(); + o != command.GetOutputs().end(); + ++o) { fout << sep << this->ConvertToXMLOutputPathSingle(o->c_str()); sep = ";"; @@ -1323,6 +1313,7 @@ void cmLocalVisualStudio7Generator::WriteVCProjEndGroup(std::ostream& fout) // look for custom rules on a target and collect them together void cmLocalVisualStudio7Generator ::OutputTargetRules(std::ostream& fout, + const char* configName, cmTarget &target, const char * /*libName*/) { @@ -1350,6 +1341,7 @@ void cmLocalVisualStudio7Generator std::string script = this->ConstructScript(cr->GetCommandLines(), cr->GetWorkingDirectory(), + configName, cr->GetEscapeOldStyle(), cr->GetEscapeAllowMakeVars()); fout << this->EscapeForXML(script.c_str()).c_str(); @@ -1379,6 +1371,7 @@ void cmLocalVisualStudio7Generator std::string script = this->ConstructScript(cr->GetCommandLines(), cr->GetWorkingDirectory(), + configName, cr->GetEscapeOldStyle(), cr->GetEscapeAllowMakeVars()); fout << this->EscapeForXML(script.c_str()).c_str(); @@ -1408,6 +1401,7 @@ void cmLocalVisualStudio7Generator std::string script = this->ConstructScript(cr->GetCommandLines(), cr->GetWorkingDirectory(), + configName, cr->GetEscapeOldStyle(), cr->GetEscapeAllowMakeVars()); fout << this->EscapeForXML(script.c_str()).c_str(); diff --git a/Source/cmLocalVisualStudio7Generator.h b/Source/cmLocalVisualStudio7Generator.h index b98379905..1863ff9c3 100644 --- a/Source/cmLocalVisualStudio7Generator.h +++ b/Source/cmLocalVisualStudio7Generator.h @@ -89,8 +89,8 @@ private: std::string EscapeForXML(const char* s); std::string ConvertToXMLOutputPath(const char* path); std::string ConvertToXMLOutputPathSingle(const char* path); - void OutputTargetRules(std::ostream& fout, cmTarget &target, - const char *libName); + void OutputTargetRules(std::ostream& fout, const char* configName, + cmTarget &target, const char *libName); void OutputBuildTool(std::ostream& fout, const char* configName, cmTarget& t); void OutputLibraries(std::ostream& fout, @@ -104,12 +104,10 @@ private: const char* group, const char* filter); void WriteVCProjEndGroup(std::ostream& fout); + void WriteCustomRule(std::ostream& fout, const char* source, - const char* command, - const char* comment, - const std::vector& depends, - const std::vector& outputs, + const cmCustomCommand& command, const char* extraFlags); void WriteTargetVersionAttribute(std::ostream& fout, cmTarget& target); diff --git a/Source/cmLocalVisualStudioGenerator.cxx b/Source/cmLocalVisualStudioGenerator.cxx index 8ca677e01..1d494c2ab 100644 --- a/Source/cmLocalVisualStudioGenerator.cxx +++ b/Source/cmLocalVisualStudioGenerator.cxx @@ -111,6 +111,7 @@ std::string cmLocalVisualStudioGenerator ::ConstructScript(const cmCustomCommandLines& commandLines, const char* workingDirectory, + const char* configName, bool escapeOldStyle, bool escapeAllowMakeVars, const char* newline_text) @@ -152,13 +153,15 @@ cmLocalVisualStudioGenerator // Start with the command name. const cmCustomCommandLine& commandLine = *cl; + std::string commandName = this->GetRealLocation(commandLine[0].c_str(), + configName); if(!workingDirectory) { - script += this->Convert(commandLine[0].c_str(),START_OUTPUT,SHELL); + script += this->Convert(commandName.c_str(),START_OUTPUT,SHELL); } else { - script += this->Convert(commandLine[0].c_str(),NONE,SHELL); + script += this->Convert(commandName.c_str(),NONE,SHELL); } // Add the arguments. diff --git a/Source/cmLocalVisualStudioGenerator.h b/Source/cmLocalVisualStudioGenerator.h index 825947449..db03f8dfd 100644 --- a/Source/cmLocalVisualStudioGenerator.h +++ b/Source/cmLocalVisualStudioGenerator.h @@ -41,6 +41,7 @@ protected: /** Construct a script from the given list of command lines. */ std::string ConstructScript(const cmCustomCommandLines& commandLines, const char* workingDirectory, + const char* configName, bool escapeOldStyle, bool escapeAllowMakeVars, const char* newline = "\n"); diff --git a/Source/cmMakefile.cxx b/Source/cmMakefile.cxx index c1a65d7e9..4cc265f21 100644 --- a/Source/cmMakefile.cxx +++ b/Source/cmMakefile.cxx @@ -578,18 +578,6 @@ cmMakefile::AddCustomCommandToTarget(const char* target, ti->second.GetPostBuildCommands().push_back(cc); break; } - - // Add dependencies on commands CMake knows how to build. - for(cmCustomCommandLines::const_iterator cli = commandLines.begin(); - cli != commandLines.end(); ++cli) - { - std::string cacheCommand = *cli->begin(); - if(const char* knownTarget = - this->GetCacheManager()->GetCacheValue(cacheCommand.c_str())) - { - ti->second.AddUtility(knownTarget); - } - } } } diff --git a/Source/cmTarget.cxx b/Source/cmTarget.cxx index 85f3d6e0b..9f9926673 100644 --- a/Source/cmTarget.cxx +++ b/Source/cmTarget.cxx @@ -408,6 +408,31 @@ void cmTarget::SetMakefile(cmMakefile* mf) } } + +void +cmTarget::checkForTargetsAsCommand(const std::vector& commands) +{ + for ( std::vector::const_iterator cli = commands.begin(); + cli != commands.end(); + ++cli ) + { + for ( cmCustomCommandLines::const_iterator cit=cli->GetCommandLines().begin(); + cit!=cli->GetCommandLines().end(); + ++cit ) + { + std::string command = *cit->begin(); + // see if we can find a target with this name + cmTarget* t = this->Makefile->GetLocalGenerator()-> + GetGlobalGenerator()->FindTarget ( 0, command.c_str() ); + if ( ( t ) && ( t->GetType() ==cmTarget::EXECUTABLE ) ) + { + this->AddUtility ( command.c_str() ); + } + } + } +} + + void cmTarget::TraceVSDependencies(std::string projFile, cmMakefile *makefile) { @@ -504,6 +529,11 @@ void cmTarget::TraceVSDependencies(std::string projFile, } } } + + checkForTargetsAsCommand(this->GetPreBuildCommands()); + checkForTargetsAsCommand(this->GetPreLinkCommands()); + checkForTargetsAsCommand(this->GetPostBuildCommands()); + while (!srcFilesToProcess.empty()) { // is this source the output of a custom command @@ -533,9 +563,28 @@ void cmTarget::TraceVSDependencies(std::string projFile, srcFilesQueued.insert(temp); } } + + // check if commands for this custom commands are names of targets and + // if that's the case add these targets as dependencies + std::vector automaticTargetDepends; + for(cmCustomCommandLines::const_iterator it= + outsf->GetCustomCommand()->GetCommandLines().begin(); + it!=outsf->GetCustomCommand()->GetCommandLines().end(); + ++it) + { + const std::string& currentCommand = (*it)[0]; + // see if we can find a target with this name + cmTarget* t = this->Makefile->GetLocalGenerator()-> + GetGlobalGenerator()->FindTarget(0, currentCommand.c_str()); + if (( t) && (t->GetType()==cmTarget::EXECUTABLE)) + { + automaticTargetDepends.push_back(currentCommand); + } + } + outsf->GetCustomCommand()->AppendDepends(automaticTargetDepends); + // add its dependencies to the list to check - unsigned int i; - for (i = 0; i < outsf->GetCustomCommand()->GetDepends().size(); ++i) + for (unsigned int i = 0; i < outsf->GetCustomCommand()->GetDepends().size(); ++i) { const std::string& fullName = outsf->GetCustomCommand()->GetDepends()[i]; @@ -590,6 +639,7 @@ void cmTarget::TraceVSDependencies(std::string projFile, } } } + } // finished with this SF move to the next srcFilesToProcess.pop(); diff --git a/Source/cmTarget.h b/Source/cmTarget.h index 5cda102f5..215d879be 100644 --- a/Source/cmTarget.h +++ b/Source/cmTarget.h @@ -55,7 +55,7 @@ public: void SetType(TargetType f, const char* name); ///! Set/Get the name of the target - const char* GetName() {return this->Name.c_str();} + const char* GetName() const {return this->Name.c_str();} ///! Set the cmMakefile that owns this target void SetMakefile(cmMakefile *mf); @@ -180,7 +180,7 @@ public: /** * Trace through the source files in this target and add al source files - * that they depend on, used by the visual studio generators + * that they depend on, used by all generators */ void TraceVSDependencies(std::string projName, cmMakefile *mf); @@ -291,6 +291,13 @@ private: const LibraryID& lib, const LibraryID& dep); + /* + * Checks the prebuild, prelink and postbuild custom commands for known + * targets and adds them to the dependencies. + */ + void checkForTargetsAsCommand(const std::vector& commands); + + /** * Emits the library \a lib and all its dependencies into link_line. * \a emitted keeps track of the libraries that have been emitted to diff --git a/Templates/DLLHeader.dsptemplate b/Templates/DLLHeader.dsptemplate index 5b31e08e8..29ae8e7d7 100644 --- a/Templates/DLLHeader.dsptemplate +++ b/Templates/DLLHeader.dsptemplate @@ -74,7 +74,7 @@ LINK32=link.exe # ADD LINK32 /nologo /dll TARGET_VERSION_FLAG /machine:I386 /out:"OUTPUT_DIRECTORY\Release/OUTPUT_NAME_RELEASE" TARGET_IMPLIB_FLAG_RELEASE CM_MULTILINE_OPTIONS_RELEASE -CMAKE_CUSTOM_RULE_CODE +CMAKE_CUSTOM_RULE_CODE_RELEASE !ELSEIF "$(CFG)" == "OUTPUT_LIBNAME - Win32 Debug" @@ -106,7 +106,7 @@ LINK32=link.exe # ADD LINK32 /nologo /dll TARGET_VERSION_FLAG /debug /machine:I386 /out:"OUTPUT_DIRECTORY\Debug/OUTPUT_NAME_DEBUG" /pdbtype:sept TARGET_IMPLIB_FLAG_DEBUG CM_MULTILINE_OPTIONS_DEBUG -CMAKE_CUSTOM_RULE_CODE +CMAKE_CUSTOM_RULE_CODE_DEBUG !ELSEIF "$(CFG)" == "OUTPUT_LIBNAME - Win32 MinSizeRel" @@ -140,7 +140,7 @@ LINK32=link.exe # ADD LINK32 /nologo /dll TARGET_VERSION_FLAG /machine:I386 /out:"OUTPUT_DIRECTORY\MinSizeRel/OUTPUT_NAME_MINSIZEREL" TARGET_IMPLIB_FLAG_MINSIZEREL CM_MULTILINE_OPTIONS_MINSIZEREL -CMAKE_CUSTOM_RULE_CODE +CMAKE_CUSTOM_RULE_CODE_MINSIZEREL !ELSEIF "$(CFG)" == "OUTPUT_LIBNAME - Win32 RelWithDebInfo" @@ -172,7 +172,7 @@ LINK32=link.exe # ADD LINK32 /nologo /dll TARGET_VERSION_FLAG /debug /machine:I386 /pdbtype:sept /out:"OUTPUT_DIRECTORY\RelWithDebInfo/OUTPUT_NAME_RELWITHDEBINFO" TARGET_IMPLIB_FLAG_RELWITHDEBINFO CM_MULTILINE_OPTIONS_RELWITHDEBINFO -CMAKE_CUSTOM_RULE_CODE +CMAKE_CUSTOM_RULE_CODE_RELWITHDEBINFO !ENDIF diff --git a/Templates/EXEHeader.dsptemplate b/Templates/EXEHeader.dsptemplate index 90faf7784..57c5b5f0b 100644 --- a/Templates/EXEHeader.dsptemplate +++ b/Templates/EXEHeader.dsptemplate @@ -72,7 +72,7 @@ LINK32=link.exe # ADD LINK32 /out:"OUTPUT_DIRECTORY\Release\OUTPUT_NAME_RELEASE" TARGET_IMPLIB_FLAG_RELEASE CM_MULTILINE_OPTIONS_RELEASE -CMAKE_CUSTOM_RULE_CODE +CMAKE_CUSTOM_RULE_CODE_RELEASE !ELSEIF "$(CFG)" == "OUTPUT_LIBNAME - Win32 Debug" @@ -104,7 +104,7 @@ LINK32=link.exe # ADD LINK32 /out:"OUTPUT_DIRECTORY\Debug\OUTPUT_NAME_DEBUG" TARGET_IMPLIB_FLAG_DEBUG CM_MULTILINE_OPTIONS_DEBUG -CMAKE_CUSTOM_RULE_CODE +CMAKE_CUSTOM_RULE_CODE_DEBUG !ELSEIF "$(CFG)" == "OUTPUT_LIBNAME - Win32 MinSizeRel" # PROP BASE Use_MFC CMAKE_MFC_FLAG @@ -133,7 +133,7 @@ LINK32=link.exe # ADD LINK32 /out:"OUTPUT_DIRECTORY\MinSizeRel\OUTPUT_NAME_MINSIZEREL" TARGET_IMPLIB_FLAG_MINSIZEREL CM_MULTILINE_OPTIONS_MINSIZEREL -CMAKE_CUSTOM_RULE_CODE +CMAKE_CUSTOM_RULE_CODE_MINSIZEREL !ELSEIF "$(CFG)" == "OUTPUT_LIBNAME - Win32 RelWithDebInfo" @@ -163,7 +163,7 @@ LINK32=link.exe # ADD LINK32 /out:"OUTPUT_DIRECTORY\RelWithDebInfo\OUTPUT_NAME_RELWITHDEBINFO" TARGET_IMPLIB_FLAG_RELWITHDEBINFO CM_MULTILINE_OPTIONS_RELWITHDEBINFO -CMAKE_CUSTOM_RULE_CODE +CMAKE_CUSTOM_RULE_CODE_RELWITHDEBINFO !ENDIF diff --git a/Templates/EXEWinHeader.dsptemplate b/Templates/EXEWinHeader.dsptemplate index 51c7f155a..5ca0f91a8 100644 --- a/Templates/EXEWinHeader.dsptemplate +++ b/Templates/EXEWinHeader.dsptemplate @@ -74,7 +74,7 @@ LINK32=link.exe # ADD LINK32 /out:"OUTPUT_DIRECTORY\Release\OUTPUT_NAME_RELEASE" CM_MULTILINE_OPTIONS_RELEASE -CMAKE_CUSTOM_RULE_CODE +CMAKE_CUSTOM_RULE_CODE_RELEASE !ELSEIF "$(CFG)" == "OUTPUT_LIBNAME - Win32 Debug" @@ -106,7 +106,7 @@ LINK32=link.exe # ADD LINK32 /out:"OUTPUT_DIRECTORY\Debug\OUTPUT_NAME_DEBUG" CM_MULTILINE_OPTIONS_DEBUG -CMAKE_CUSTOM_RULE_CODE +CMAKE_CUSTOM_RULE_CODE_DEBUG !ELSEIF "$(CFG)" == "OUTPUT_LIBNAME - Win32 MinSizeRel" # PROP BASE Use_MFC CMAKE_MFC_FLAG @@ -135,7 +135,7 @@ LINK32=link.exe # ADD LINK32 /out:"OUTPUT_DIRECTORY\MinSizeRel\OUTPUT_NAME_MINSIZEREL" CM_MULTILINE_OPTIONS_MINSIZEREL -CMAKE_CUSTOM_RULE_CODE +CMAKE_CUSTOM_RULE_CODE_MINSIZEREL !ELSEIF "$(CFG)" == "OUTPUT_LIBNAME - Win32 RelWithDebInfo" @@ -167,7 +167,7 @@ LINK32=link.exe # ADD LINK32 /out:"OUTPUT_DIRECTORY\RelWithDebInfo\OUTPUT_NAME_RELWITHDEBINFO" CM_MULTILINE_OPTIONS_RELWITHDEBINFO -CMAKE_CUSTOM_RULE_CODE +CMAKE_CUSTOM_RULE_CODE_RELWITHDEBINFO !ENDIF diff --git a/Templates/UtilityHeader.dsptemplate b/Templates/UtilityHeader.dsptemplate index 681ba74e8..4ad57d551 100644 --- a/Templates/UtilityHeader.dsptemplate +++ b/Templates/UtilityHeader.dsptemplate @@ -42,7 +42,7 @@ MTL=midl.exe # PROP Intermediate_Dir "Release" # PROP Target_Dir "" -CMAKE_CUSTOM_RULE_CODE +CMAKE_CUSTOM_RULE_CODE_RELEASE !ELSEIF "$(CFG)" == "OUTPUT_LIBNAME - Win32 Debug" @@ -57,7 +57,7 @@ CMAKE_CUSTOM_RULE_CODE # PROP Intermediate_Dir "Debug" # PROP Target_Dir "" -CMAKE_CUSTOM_RULE_CODE +CMAKE_CUSTOM_RULE_CODE_DEBUG !ELSEIF "$(CFG)" == "OUTPUT_LIBNAME - Win32 MinSizeRel" @@ -72,7 +72,7 @@ CMAKE_CUSTOM_RULE_CODE # PROP Intermediate_Dir "MinSizeRel" # PROP Target_Dir "" -CMAKE_CUSTOM_RULE_CODE +CMAKE_CUSTOM_RULE_CODE_MINSIZEREL !ELSEIF "$(CFG)" == "OUTPUT_LIBNAME - Win32 RelWithDebInfo" @@ -87,7 +87,7 @@ CMAKE_CUSTOM_RULE_CODE # PROP Intermediate_Dir "RelWithDebInfo" # PROP Target_Dir "" -CMAKE_CUSTOM_RULE_CODE +CMAKE_CUSTOM_RULE_CODE_RELWITHDEBINFO !ENDIF diff --git a/Templates/staticLibHeader.dsptemplate b/Templates/staticLibHeader.dsptemplate index b20efcc18..9ef1822d7 100644 --- a/Templates/staticLibHeader.dsptemplate +++ b/Templates/staticLibHeader.dsptemplate @@ -67,7 +67,7 @@ LIB32=link.exe -lib # ADD BASE LIB32 /nologo # ADD LIB32 /nologo /out:"OUTPUT_DIRECTORY\Release/OUTPUT_NAME_RELEASE" CM_STATIC_LIB_ARGS -CMAKE_CUSTOM_RULE_CODE +CMAKE_CUSTOM_RULE_CODE_RELEASE !ELSEIF "$(CFG)" == "OUTPUT_LIBNAME - Win32 Debug" @@ -95,7 +95,7 @@ LIB32=link.exe -lib # ADD BASE LIB32 /nologo # ADD LIB32 /nologo /out:"OUTPUT_DIRECTORY\Debug/OUTPUT_NAME_DEBUG" CM_STATIC_LIB_ARGS -CMAKE_CUSTOM_RULE_CODE +CMAKE_CUSTOM_RULE_CODE_DEBUG !ELSEIF "$(CFG)" == "OUTPUT_LIBNAME - Win32 MinSizeRel" @@ -124,7 +124,7 @@ LIB32=link.exe -lib # ADD BASE LIB32 /nologo # ADD LIB32 /nologo /out:"OUTPUT_DIRECTORY\MinSizeRel/OUTPUT_NAME_MINSIZEREL" CM_STATIC_LIB_ARGS -CMAKE_CUSTOM_RULE_CODE +CMAKE_CUSTOM_RULE_CODE_MINSIZEREL !ELSEIF "$(CFG)" == "OUTPUT_LIBNAME - Win32 RelWithDebInfo" @@ -152,7 +152,7 @@ LIB32=link.exe -lib # ADD BASE LIB32 /nologo # ADD LIB32 /nologo /out:"OUTPUT_DIRECTORY\RelWithDebInfo/OUTPUT_NAME_RELWITHDEBINFO" CM_STATIC_LIB_ARGS -CMAKE_CUSTOM_RULE_CODE +CMAKE_CUSTOM_RULE_CODE_RELWITHDEBINFO !ENDIF diff --git a/Tests/CustomCommand/CMakeLists.txt b/Tests/CustomCommand/CMakeLists.txt index 83d2de233..fb7b714e7 100644 --- a/Tests/CustomCommand/CMakeLists.txt +++ b/Tests/CustomCommand/CMakeLists.txt @@ -167,6 +167,31 @@ TARGET_LINK_LIBRARIES(CustomCommand GeneratedHeader) # the CustomCommand executable really needs doc1.h ADD_DEPENDENCIES(CustomCommand TDocument) +############################################################################## +# Test for using just the target name as executable in the COMMAND +# section. Has to be recognized and replaced by CMake with the output +# actual location of the executable. +# Additionally the generator is created in an extra subdir after the +# ADD_CUSTOM_COMMAND() is used. +# +# Test the same for ADD_CUSTOM_TARGET() + +ADD_CUSTOM_COMMAND(OUTPUT ${CMAKE_CURRENT_BINARY_DIR}/generated_extern.cxx + COMMAND generator_extern ${CMAKE_CURRENT_BINARY_DIR}/generated_extern.cxx + ) + +ADD_EXECUTABLE(CustomCommandUsingTargetTest main.cxx ${CMAKE_CURRENT_BINARY_DIR}/generated_extern.cxx ) + +ADD_CUSTOM_TARGET(RunTarget + COMMAND generator_extern ${CMAKE_CURRENT_BINARY_DIR}/run_target.cxx + ) + +ADD_CUSTOM_COMMAND(TARGET CustomCommandUsingTargetTest POST_BUILD + COMMAND dummy_generator ${CMAKE_CURRENT_BINARY_DIR}/generated_dummy.cxx) + +ADD_SUBDIRECTORY(GeneratorInExtraDir) + + ############################################################################## # Test non-trivial command line arguments in custom commands. SET(EXPECTED_ARGUMENTS) diff --git a/Tests/CustomCommand/GeneratorInExtraDir/CMakeLists.txt b/Tests/CustomCommand/GeneratorInExtraDir/CMakeLists.txt new file mode 100644 index 000000000..e838e7b08 --- /dev/null +++ b/Tests/CustomCommand/GeneratorInExtraDir/CMakeLists.txt @@ -0,0 +1,6 @@ +# add the executable which will be used for generating files +ADD_EXECUTABLE(generator_extern ../generator.cxx) +SET_TARGET_PROPERTIES(generator_extern PROPERTIES OUTPUT_NAME the_external_generator) + +# add an executable which will be called from ADD_CUSTOM_COMMAND( ... POST_BUILD) +ADD_EXECUTABLE(dummy_generator ../generator.cxx) diff --git a/Tests/CustomCommand/main.cxx b/Tests/CustomCommand/main.cxx new file mode 100644 index 000000000..600e751dd --- /dev/null +++ b/Tests/CustomCommand/main.cxx @@ -0,0 +1,6 @@ +extern int generated(); + +int main() +{ + return generated(); +}