BUG: Fix ALL_BUILD ordering enforcement

The previous change to make ALL_BUILD come first among targets did not
account for comparing the target name against itself.  This led to an
invalid ordering of the target set.  This change fixes it.

1 files changed, 4 insertions, 4 deletions

diff --git a/Source/cmGlobalVisualStudio7Generator.cxx b/Source/cmGlobalVisualStudio7Generator.cxx
index 02dc39b65f..e43afc0175 100644
--- a/Source/cmGlobalVisualStudio7Generator.cxx
+++ b/Source/cmGlobalVisualStudio7Generator.cxx
@@ -730,14 +730,14 @@ cmGlobalVisualStudio7Generator::TargetCompare
 ::operator()(cmTarget const* l, cmTarget const* r)
 {
   // Make sure ALL_BUILD is first so it is the default active project.
-  if(strcmp(l->GetName(), "ALL_BUILD") == 0)
-    {
-    return true;
-    }
   if(strcmp(r->GetName(), "ALL_BUILD") == 0)
     {
     return false;
     }
+  if(strcmp(l->GetName(), "ALL_BUILD") == 0)
+    {
+    return true;
+    }
   return strcmp(l->GetName(), r->GetName()) < 0;
 }