halapi: Disable dependency checking between hal-api

Temporarily disable it. For now, it is unclear that dependency checking
is necessary. Because, those hal-api header and manifest.xml aren't
managed by ACR process so they could be transformed anyhow, breaking
dependency. Currently, our policy has no specification that force them
to think about and comply with dependency among hal-apis.

Change-Id: Iab1fc46674e4af0e0b6771c630d6e1a093dda8f9
Signed-off-by: Youngjae Cho <y0.cho@samsung.com>

diff --git a/src/hal-api-compatibility-checker.c b/src/hal-api-compatibility-checker.c
index b544db27e26a4bd2aefda7e8e57483635fd35b8c..9423093eeffefab0bcad7ee4bcf89bc30bbc4d37 100644 (file)
--- a/src/hal-api-compatibility-checker.c
+++ b/src/hal-api-compatibility-checker.c
@@ -149,7 +149,12 @@ static void make_compatibility_manifest_info(void *data, void *user_data)
        halcc_hal *hal;
        const char *hal_name = NULL;
        int manifest_major, manifest_minor;
-       halcc_dependency_state_e state;
+       /**
+        * FIXME:
+        * Temporarily disable dependency checking. See the below FIXME.
+        *
+        * halcc_dependency_state_e state;
+        */
        int ret;
        const char *error_desc = NULL;
        struct compatibility_info *manifest_info = NULL;
@@ -180,11 +185,20 @@ static void make_compatibility_manifest_info(void *data, void *user_data)
                return;
        }
 
-
-       if (halcc_hal_get_dependency_state(hal, &state) != 0) {
-               error_desc = "Failed to resolve dependency";
-               goto out;
-       }
+       /**
+        * FIXME:
+        * Temporarily disable it. For now, it is unclear that
+        * this dependency checking is necessary. Because, those hal-api
+        * header and manifest.xml aren't managed by ACR process so they
+        * could be transformed anyhow, breaking dependency. Currently,
+        * our policy has no such specification that force them to think
+        * about and comply with dependency among hal-apis.
+        *
+        * if (halcc_hal_get_dependency_state(hal, &state) != 0) {
+        *      error_desc = "Failed to resolve dependency";
+        *      goto out;
+        * }
+        */
 
        if (halcc_hal_get_version(hal, &manifest_major, &manifest_minor, NULL) < 0) {
                error_desc = "Invalid manifest version";
@@ -364,7 +378,17 @@ static int load_hal_manifest(struct compatibility_info *manifest_info, int load_
        if (ret < 0)
                goto out;
 
-       halcc_manifest_validate_hal_dependency(manifest);
+       /**
+        * FIXME:
+        * Temporarily disable it. For now, it is unclear that
+        * this dependency checking is necessary. Because, those hal-api
+        * header and manifest.xml aren't managed by ACR process so they
+        * could be transformed anyhow, breaking dependency. Currently,
+        * our policy has no such specification that force them to think
+        * about and comply with dependency among hal-apis.
+        *
+        * halcc_manifest_validate_hal_dependency(manifest);
+        */
 
        halcc_manifest_foreach_hal(manifest, make_compatibility_manifest_info, manifest_info);