From 56714a49d4bb4b70799e857a24983e3002b4f112 Mon Sep 17 00:00:00 2001
From: Ken Martin <ken.martin@kitware.com>
Date: Fri, 4 May 2001 09:56:07 -0400
Subject: [PATCH] fixed custom command rule

---
 Source/cmDSPMakefile.cxx | 52 +++++++++++++++++++++-------------------
 Source/cmDSPWriter.cxx   | 52 +++++++++++++++++++++-------------------
 2 files changed, 54 insertions(+), 50 deletions(-)

diff --git a/Source/cmDSPMakefile.cxx b/Source/cmDSPMakefile.cxx
index 56fb4206d..5333d8866 100644
--- a/Source/cmDSPMakefile.cxx
+++ b/Source/cmDSPMakefile.cxx
@@ -268,35 +268,37 @@ void cmDSPMakefile::WriteDSPFile(std::ostream& fout,
       // Tell MS-Dev what the source is.  If the compiler knows how to
       // build it, then it will.
       fout << "SOURCE=" << source.c_str() << "\n\n";
-      
-      // Loop through every custom command generating code from the
-      // current source.
-      // build up the depends and outputs and commands 
-      cmSourceGroup::CommandFiles totalCommand;
-      std::string totalCommandStr;
-      for(cmSourceGroup::Commands::const_iterator c = commands.begin();
-          c != commands.end(); ++c)
+      if (!commands.empty())
         {
-        totalCommandStr += "\n\t";
-        totalCommandStr += c->first;
-        totalCommand.Merge(c->second);
-        }      
-      // Create a dummy file with the name of the source if it does
-      // not exist
-      if(totalCommand.m_Outputs.empty())
-        { 
-        std::string dummyFile = m_Makefile->GetStartOutputDirectory();
-        dummyFile += "/";
-        dummyFile += source;
-        if(!cmSystemTools::FileExists(dummyFile.c_str()))
+        // Loop through every custom command generating code from the
+        // current source.
+        // build up the depends and outputs and commands 
+        cmSourceGroup::CommandFiles totalCommand;
+        std::string totalCommandStr;
+        for(cmSourceGroup::Commands::const_iterator c = commands.begin();
+            c != commands.end(); ++c)
           {
-          std::ofstream fout(dummyFile.c_str());
-          fout << "Dummy file created by cmake as unused source for utility command.\n";
+          totalCommandStr += "\n\t";
+          totalCommandStr += c->first;
+          totalCommand.Merge(c->second);
+          }      
+        // Create a dummy file with the name of the source if it does
+        // not exist
+        if(totalCommand.m_Outputs.empty())
+          { 
+          std::string dummyFile = m_Makefile->GetStartOutputDirectory();
+          dummyFile += "/";
+          dummyFile += source;
+          if(!cmSystemTools::FileExists(dummyFile.c_str()))
+            {
+            std::ofstream fout(dummyFile.c_str());
+            fout << "Dummy file created by cmake as unused source for utility command.\n";
+            }
           }
+        this->WriteCustomRule(fout, source.c_str(), totalCommandStr.c_str(), 
+                              totalCommand.m_Depends, 
+                              totalCommand.m_Outputs);
         }
-      this->WriteCustomRule(fout, source.c_str(), totalCommandStr.c_str(), 
-                            totalCommand.m_Depends, 
-                            totalCommand.m_Outputs);
       fout << "# End Source File\n";
       }
     
diff --git a/Source/cmDSPWriter.cxx b/Source/cmDSPWriter.cxx
index 56fb4206d..5333d8866 100644
--- a/Source/cmDSPWriter.cxx
+++ b/Source/cmDSPWriter.cxx
@@ -268,35 +268,37 @@ void cmDSPMakefile::WriteDSPFile(std::ostream& fout,
       // Tell MS-Dev what the source is.  If the compiler knows how to
       // build it, then it will.
       fout << "SOURCE=" << source.c_str() << "\n\n";
-      
-      // Loop through every custom command generating code from the
-      // current source.
-      // build up the depends and outputs and commands 
-      cmSourceGroup::CommandFiles totalCommand;
-      std::string totalCommandStr;
-      for(cmSourceGroup::Commands::const_iterator c = commands.begin();
-          c != commands.end(); ++c)
+      if (!commands.empty())
         {
-        totalCommandStr += "\n\t";
-        totalCommandStr += c->first;
-        totalCommand.Merge(c->second);
-        }      
-      // Create a dummy file with the name of the source if it does
-      // not exist
-      if(totalCommand.m_Outputs.empty())
-        { 
-        std::string dummyFile = m_Makefile->GetStartOutputDirectory();
-        dummyFile += "/";
-        dummyFile += source;
-        if(!cmSystemTools::FileExists(dummyFile.c_str()))
+        // Loop through every custom command generating code from the
+        // current source.
+        // build up the depends and outputs and commands 
+        cmSourceGroup::CommandFiles totalCommand;
+        std::string totalCommandStr;
+        for(cmSourceGroup::Commands::const_iterator c = commands.begin();
+            c != commands.end(); ++c)
           {
-          std::ofstream fout(dummyFile.c_str());
-          fout << "Dummy file created by cmake as unused source for utility command.\n";
+          totalCommandStr += "\n\t";
+          totalCommandStr += c->first;
+          totalCommand.Merge(c->second);
+          }      
+        // Create a dummy file with the name of the source if it does
+        // not exist
+        if(totalCommand.m_Outputs.empty())
+          { 
+          std::string dummyFile = m_Makefile->GetStartOutputDirectory();
+          dummyFile += "/";
+          dummyFile += source;
+          if(!cmSystemTools::FileExists(dummyFile.c_str()))
+            {
+            std::ofstream fout(dummyFile.c_str());
+            fout << "Dummy file created by cmake as unused source for utility command.\n";
+            }
           }
+        this->WriteCustomRule(fout, source.c_str(), totalCommandStr.c_str(), 
+                              totalCommand.m_Depends, 
+                              totalCommand.m_Outputs);
         }
-      this->WriteCustomRule(fout, source.c_str(), totalCommandStr.c_str(), 
-                            totalCommand.m_Depends, 
-                            totalCommand.m_Outputs);
       fout << "# End Source File\n";
       }