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


##########
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"

Review Comment:
   I notice there are still `OSGI_FRAMEWORK_FRAMEWORK_STORAGE` user pubsub/rsa 
tests



-- 
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