diff --git a/var/spack/repos/builtin/packages/hdf/package.py b/var/spack/repos/builtin/packages/hdf/package.py
index aafb345be6af356f4e3f543184e469e72a9d7ae9..9a44184256453ba5567c85f11ca03c34cc8ac9d1 100644
--- a/var/spack/repos/builtin/packages/hdf/package.py
+++ b/var/spack/repos/builtin/packages/hdf/package.py
@@ -38,13 +38,12 @@ class Hdf(Package):
 
     variant('szip', default=False, description="Enable szip support")
 
-    depends_on("jpeg")
-    depends_on("szip", when='+szip')
-    depends_on("zlib")
+    depends_on('jpeg')
+    depends_on('szip', when='+szip')
+    depends_on('zlib')
 
-    def url_for_version(self, version):
-        return "https://www.hdfgroup.org/ftp/HDF/releases/HDF" + str(
-            version) + "/src/hdf-" + str(version) + ".tar.gz"
+    depends_on('bison', type='build')
+    depends_on('flex',  type='build')
 
     def install(self, spec, prefix):
         config_args = [
@@ -66,4 +65,5 @@ def install(self, spec, prefix):
         configure(*config_args)
 
         make()
-        make("install")
+        make('check')
+        make('install')