PengZheng commented on code in PR #476:
URL: https://github.com/apache/celix/pull/476#discussion_r1116797356


##########
libs/framework/include/celix_constants.h:
##########
@@ -158,19 +158,54 @@ extern "C" {
 #define OSGI_FRAMEWORK_IMPORT_LIBRARY CELIX_FRAMEWORK_IMPORT_LIBRARY
 
 /**
- * @brief Celix framework environment property (named 
"org.osgi.framework.storage") specifying the cache
+ * @brief Celix framework environment property (named 
"CELIX_FRAMEWORK_CACHE_DIR") specifying the cache
  * directory used for the bundle caches.
  *
- * If not specified ".cache" is used.
+ * This property is not used if CELIX_FRAMEWORK_CACHE_USE_TMP_DIR=true.
+ *
+ * Default value is ".cache".
+ */
+#define CELIX_FRAMEWORK_FRAMEWORK_CACHE_DIR "CELIX_FRAMEWORK_CACHE_DIR"
+
+//@deprecated use CELIX_FRAMEWORK_FRAMEWORK_CACHE_DIR
+#define CELIX_FRAMEWORK_FRAMEWORK_STORAGE   "CELIX_FRAMEWORK_CACHE_DIR"
+
+//@deprecated use CELIX_FRAMEWORK_FRAMEWORK_CACHE_DIR
+#define OSGI_FRAMEWORK_FRAMEWORK_STORAGE    "org.osgi.framework.storage"
+
+/**
+ * @brief Celix framework environment property (named 
"CELIX_FRAMEWORK_CACHE_USE_TMP_DIR") specifying the
+ * whether to use a tmp directory for the bundle cache dir.
+ *
+ * If set to "true" the cache dir is set to a tmp directory and the default 
for CELIX_FRAMEWORK_DELETE_CACHE_ON_DESTROY

Review Comment:
   We don't have `CELIX_FRAMEWORK_DELETE_CACHE_ON_DESTROY`



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@celix.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to