Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tiledb_unit unused variable build errors on NixOS: tiledb 1.5.1 #1369

Closed
Rakesh4G opened this issue Aug 22, 2019 · 2 comments
Closed

tiledb_unit unused variable build errors on NixOS: tiledb 1.5.1 #1369

Rakesh4G opened this issue Aug 22, 2019 · 2 comments

Comments

@Rakesh4G
Copy link

Rakesh4G commented Aug 22, 2019

We are trying to build tiledb on NixOS. We are able to build basic shared library but are facing issues in building tiledb_unit tests.
Related to: Intel-HLS/TileDB#81

main PR is at : NixOS/nixpkgs#67187

/build/TileDB-7236bb2/test/src/unit-capi-consolidation.cc: In member function 'void ConsolidationFx::read_dense_vector()':
/build/TileDB-7236bb2/test/src/unit-capi-consolidation.cc:1859:12: error: unused variable 'subarray' [-Werror=unused-variable]
   uint64_t subarray[] = {1, 410};
            ^~~~~~~~

/build/TileDB-7236bb2/test/src/unit-capi-consolidation.cc:2933:7: error: unused variable 'has_key' [-Werror=unused-variable]
   int has_key;
       ^~~~~~~
cc1plus: all warnings being treated as errors
make[7]: *** [test/CMakeFiles/tiledb_unit.dir/build.make:167: test/CMakeFiles/tiledb_unit.dir/src/unit-capi-consolidation.cc.o] Error 1

The logs are attached here.
tiledb-unit-error.txt

Please suggest what should be done to get rid off this error.

Thanks.

@Rakesh4G Rakesh4G changed the title tiledb_unit build errors on NixOS: tiledb 1.5.1 tiledb_unit unused variable build errors on NixOS: tiledb 1.5.1 Aug 22, 2019
@tdenniston
Copy link
Contributor

As mentioned in NixOS/nixpkgs#67187, unless there is a reason otherwise, we recommend you update to the latest stable release of TileDB, version 1.6.2. Now that major release 1.6 is available, bugfixes and support for older releases (such as 1.5.1) will be minimal.

@stavrospapadopoulos
Copy link
Member

Closing this for now, as we are working on adding TileDB 1.6.2 to NixOS instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants