[Cmake-commits] CMake branch, next, updated. v2.8.3-1000-g6e99e12

David Cole david.cole at kitware.com
Fri Dec 17 11:28:26 EST 2010


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "CMake".

The branch, next has been updated
       via  6e99e12d11cb88bf691bcbc39f555589053c4c46 (commit)
       via  66e79175327249aa2f0c80f5bd4a360b5b628bff (commit)
      from  dae445947f3c88c154de2effb0a441e44b3f2149 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=6e99e12d11cb88bf691bcbc39f555589053c4c46
commit 6e99e12d11cb88bf691bcbc39f555589053c4c46
Merge: dae4459 66e7917
Author:     David Cole <david.cole at kitware.com>
AuthorDate: Fri Dec 17 11:28:24 2010 -0500
Commit:     CMake Topic Stage <kwrobot at kitware.com>
CommitDate: Fri Dec 17 11:28:24 2010 -0500

    Merge topic 'fix-11533-vs10-custom-build-err' into next
    
    66e7917 VS10: stop build on custom command error (#11533)

diff --cc Source/cmLocalVisualStudioGenerator.cxx
index a044363,8eddc43..9164beb
--- a/Source/cmLocalVisualStudioGenerator.cxx
+++ b/Source/cmLocalVisualStudioGenerator.cxx
@@@ -150,17 -149,27 +150,29 @@@ void cmLocalVisualStudioGenerator::Comp
  }
  
  //----------------------------------------------------------------------------
+ std::string cmLocalVisualStudioGenerator::CheckForErrorLine()
+ {
+   return "if errorlevel 1 goto :VCReportError";
+ }
+ 
+ //----------------------------------------------------------------------------
+ std::string cmLocalVisualStudioGenerator::GetCheckForErrorLine()
+ {
+   return this->CheckForErrorLine();
+ }
+ 
+ //----------------------------------------------------------------------------
  std::string
  cmLocalVisualStudioGenerator
 -::ConstructScript(const cmCustomCommandLines& commandLines,
 -                  const char* workingDirectory,
 +::ConstructScript(cmCustomCommand const& cc,
                    const char* configName,
 -                  bool escapeOldStyle,
 -                  bool escapeAllowMakeVars,
                    const char* newline_text)
  {
 +  const cmCustomCommandLines& commandLines = cc.GetCommandLines();
 +  const char* workingDirectory = cc.GetWorkingDirectory();
 +  cmCustomCommandGenerator ccg(cc, configName, this->Makefile);
 +  RelativeRoot relativeRoot = workingDirectory? NONE : START_OUTPUT;
 +
    // Avoid leading or trailing newlines.
    const char* newline = "";
  
diff --cc Source/cmLocalVisualStudioGenerator.h
index 551b01c,0019bfb..1954ac5
--- a/Source/cmLocalVisualStudioGenerator.h
+++ b/Source/cmLocalVisualStudioGenerator.h
@@@ -31,12 -30,22 +31,19 @@@ class cmLocalVisualStudioGenerator : pu
  public:
    cmLocalVisualStudioGenerator();
    virtual ~cmLocalVisualStudioGenerator();
+ 
    /** Construct a script from the given list of command lines.  */
 -  std::string ConstructScript(const cmCustomCommandLines& commandLines,
 -                              const char* workingDirectory,
 +  std::string ConstructScript(cmCustomCommand const& cc,
                                const char* configName,
 -                              bool escapeOldStyle,
 -                              bool escapeAllowMakeVars,
                                const char* newline = "\n");
  
+   /** Line of batch file text that skips to the end after
+     * a failed step in a sequence of custom commands.
+     */
+   std::string GetCheckForErrorLine();
+ 
  protected:
+   virtual std::string CheckForErrorLine();
  
    /** Construct a custom command to make exe import lib dir.  */
    cmsys::auto_ptr<cmCustomCommand>

http://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=66e79175327249aa2f0c80f5bd4a360b5b628bff
commit 66e79175327249aa2f0c80f5bd4a360b5b628bff
Author:     David Cole <david.cole at kitware.com>
AuthorDate: Fri Dec 17 11:11:55 2010 -0500
Commit:     David Cole <david.cole at kitware.com>
CommitDate: Fri Dec 17 11:11:55 2010 -0500

    VS10: stop build on custom command error (#11533)
    
    In VS9 and previous versions, :VCReportError is the goto label
    to jump to after a failed custom command. It stops the build
    before it tries to go any further.
    
    In VS10, :VCEnd is the correct label to use.
    
    Create a method in the VS generators to provide the correct
    line of script to use for each version of Visual Studio.
    
    For more internal details, search for VCEnd in the
    C:\Program Files\MSBuild directory.

diff --git a/Source/cmLocalVisualStudio10Generator.cxx b/Source/cmLocalVisualStudio10Generator.cxx
index 57d8653..de2a837 100644
--- a/Source/cmLocalVisualStudio10Generator.cxx
+++ b/Source/cmLocalVisualStudio10Generator.cxx
@@ -117,3 +117,9 @@ void cmLocalVisualStudio10Generator
                   "Stored GUID",
                   cmCacheManager::INTERNAL);
 }
+
+//----------------------------------------------------------------------------
+std::string cmLocalVisualStudio10Generator::CheckForErrorLine()
+{
+  return "if errorlevel 1 goto :VCEnd";
+}
diff --git a/Source/cmLocalVisualStudio10Generator.h b/Source/cmLocalVisualStudio10Generator.h
index 5694220..06b8b09 100644
--- a/Source/cmLocalVisualStudio10Generator.h
+++ b/Source/cmLocalVisualStudio10Generator.h
@@ -36,6 +36,10 @@ public:
   virtual void Generate();
   virtual void ReadAndStoreExternalGUID(const char* name,
                                         const char* path);
+
+protected:
+  virtual std::string CheckForErrorLine();
+
 private:
 };
 #endif
diff --git a/Source/cmLocalVisualStudioGenerator.cxx b/Source/cmLocalVisualStudioGenerator.cxx
index ed0b07f..8eddc43 100644
--- a/Source/cmLocalVisualStudioGenerator.cxx
+++ b/Source/cmLocalVisualStudioGenerator.cxx
@@ -149,6 +149,18 @@ void cmLocalVisualStudioGenerator::ComputeObjectNameRequirements
 }
 
 //----------------------------------------------------------------------------
+std::string cmLocalVisualStudioGenerator::CheckForErrorLine()
+{
+  return "if errorlevel 1 goto :VCReportError";
+}
+
+//----------------------------------------------------------------------------
+std::string cmLocalVisualStudioGenerator::GetCheckForErrorLine()
+{
+  return this->CheckForErrorLine();
+}
+
+//----------------------------------------------------------------------------
 std::string
 cmLocalVisualStudioGenerator
 ::ConstructScript(const cmCustomCommandLines& commandLines,
@@ -237,7 +249,7 @@ cmLocalVisualStudioGenerator
     // sequence.
     //
     script += newline_text;
-    script += "if errorlevel 1 goto VCReportError";
+    script += this->GetCheckForErrorLine();
     }
 
   return script;
diff --git a/Source/cmLocalVisualStudioGenerator.h b/Source/cmLocalVisualStudioGenerator.h
index 6034b22..0019bfb 100644
--- a/Source/cmLocalVisualStudioGenerator.h
+++ b/Source/cmLocalVisualStudioGenerator.h
@@ -30,6 +30,7 @@ class cmLocalVisualStudioGenerator : public cmLocalGenerator
 public:
   cmLocalVisualStudioGenerator();
   virtual ~cmLocalVisualStudioGenerator();
+
   /** Construct a script from the given list of command lines.  */
   std::string ConstructScript(const cmCustomCommandLines& commandLines,
                               const char* workingDirectory,
@@ -38,7 +39,13 @@ public:
                               bool escapeAllowMakeVars,
                               const char* newline = "\n");
 
+  /** Line of batch file text that skips to the end after
+    * a failed step in a sequence of custom commands.
+    */
+  std::string GetCheckForErrorLine();
+
 protected:
+  virtual std::string CheckForErrorLine();
 
   /** Construct a custom command to make exe import lib dir.  */
   cmsys::auto_ptr<cmCustomCommand>

-----------------------------------------------------------------------

Summary of changes:
 Source/cmLocalVisualStudio10Generator.cxx |    6 ++++++
 Source/cmLocalVisualStudio10Generator.h   |    4 ++++
 Source/cmLocalVisualStudioGenerator.cxx   |   14 +++++++++++++-
 Source/cmLocalVisualStudioGenerator.h     |    7 +++++++
 4 files changed, 30 insertions(+), 1 deletions(-)


hooks/post-receive
-- 
CMake


More information about the Cmake-commits mailing list