ENH: remove the watch for the upper case variable name, it breaks the

feature summary, which needs to check for both the upper case and original
case _FOUND variables

Alex

Source/cmFindPackageCommand.cxx

@@ -254,10 +254,6 @@ bool cmFindPackageCommand::InitialPass(std::vector<std::string> const& args)
         upperDir.c_str(),
         cmFindPackageNeedBackwardsCompatibility
         );
-      this->Makefile->GetVariableWatch()->RemoveWatch(
-        upperFound.c_str(),
-        cmFindPackageNeedBackwardsCompatibility
-        );
       }
     else
       {
@@ -267,10 +263,6 @@ bool cmFindPackageCommand::InitialPass(std::vector<std::string> const& args)
         upperDir.c_str(),
         cmFindPackageNeedBackwardsCompatibility
         );
-      this->Makefile->GetVariableWatch()->AddWatch(
-        upperFound.c_str(),
-        cmFindPackageNeedBackwardsCompatibility
-        );
       }
     }
 #endif

Tests/FindPackageTest/CMakeLists.txt

@@ -4,7 +4,7 @@ PROJECT(FindPackageTest)
 FIND_PACKAGE(OpenGL QUIET)
 
 # Look for a package that has no find module and will not be found.
-#FIND_PACKAGE(NotAPackage QUIET)
+FIND_PACKAGE(NotAPackage QUIET)
 
 # Look for a package that has an advanced find module.
 FIND_PACKAGE(VTK QUIET)