apache / celix

Apache Celix is a framework for C and C++14 to develop dynamic modular software applications using component and in-process service-oriented programming.
https://celix.apache.org/
Apache License 2.0
166 stars 87 forks source link

Building on OSX with brew installed fails #567

Closed schorg closed 1 year ago

schorg commented 1 year ago

The dependency tojansson is missing:

CMake Error at /opt/homebrew/Cellar/cmake/3.26.4/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find jansson (missing: JANSSON_LIBRARY JANSSON_INCLUDE_DIR)

Compiling with clang refuses to compile old-style parameter declarations in C functions defined in bundles/deployment_admin/src/ ioapi.c and miniunz.c :

/Users/GFR2FR/Projects/schorg/celix/bundles/deployment_admin/src/ioapi.c:212:6: error: a function declaration without a prototype is deprecated in all versions of C and is not supported in C2x [-Werror,-Wdeprecated-non-prototype]
void fill_fopen_filefunc (pzlib_filefunc_def)
     ^
1 error generated.
/Users/GFR2FR/Projects/schorg/celix/bundles/deployment_admin/src/miniunz.c:94:6: error: a function declaration without a prototype is deprecated in all versions of C and is not supported in C2x [-Werror,-Wdeprecated-non-prototype]
void change_file_date(filename,dosdate,tmu_date)
     ^
/Users/GFR2FR/Projects/schorg/celix/bundles/deployment_admin/src/miniunz.c:135:5: error: a function declaration without a prototype is deprecated in all versions of C and is not supported in C2x [-Werror,-Wdeprecated-non-prototype]
int mymkdir(dirname)
    ^
/Users/GFR2FR/Projects/schorg/celix/bundles/deployment_admin/src/miniunz.c:149:5: error: a function declaration without a prototype is deprecated in all versions of C and is not supported in C2x [-Werror,-Wdeprecated-non-prototype]
int makedir (newdir)
    ^

I will do a pull-request to correct these errors.

PengZheng commented 1 year ago

Please note that we intend to drop deployment admin in the coming 3.0.0 release. If there are still interests of improving it, don't hesitate to leave a message in #509.