Develop: Resolve issue 2070 with new depends_on clause in the openspeedshop package file. (#2136)

* Update the krell institute products to use the latest features of spack for building on cluster platforms.

* Address travis error messages and resubmit the pull request.

* Update the contents of openspeedshop package.py so it passes the flake8 tests.

* Fix flake8 error-whitespack issue in mrnet package.py file.

* Add updates based on spack reviewer feedback.

* More fixes based on comments from reviewers.  Switch using extend to using append, remove additional setting of PATH and LD_LIBRARY_PATH that should not be required due to RPATH.

* More review related changes.  Update MPIOption.append lines and take out xercesc references.

* Create a base options function for common openspeedshop base cmake options to reduce redundencies.

* Add libxml2+python depends on to get around issues with the libxml2 package file.

var/spack/repos/builtin/packages/openspeedshop/package.py

@@ -120,7 +120,7 @@ class Openspeedshop(Package):
     depends_on("sqlite")
     depends_on("boost@1.50.0:")
     depends_on("dyninst@9.1.0:")
-    depends_on("python")
+    depends_on("libxml2+python")
     depends_on("qt@3.3.8b+krellpatch")
 
     # Dependencies only for the openspeedshop offline package.