Sean Leather

Set default compiler to clang on apple hosts

5

The change in this PR (615c53bf23b3a026b90a613c6364dd6ade869cd1) _seems_ to be the right choice to make. `clang`/`clang++` should be considered the default on macOS, and it allows the `ToolFamily` to be chosen...

Improve cross-compiling support

6

I ran into a cross-compiling issue where `cmake` used the host's `CMAKE_SYSTEM_PROCESSOR` instead of the target's and incorrectly evaluated a condition in the `CMakeLists.txt` that led to some unsupported compiler...

Figure out why the GitHub Actions are failing. In particular, look at the [7.10 job](https://github.com/spl/dlist/runs/3470146381?check_suite_focus=true).

* The `bench` job needs to be faster, and it seems like installing dependencies is what slows it down. * See [Dead simple cross-platform GitHub Actions for Haskell](https://kodimensional.dev/github-actions).

This will probably be a simple addition to mention `make docker` and using `DOCKER_BUILDKIT=1` along with the minimum `docker` version requirement.

``` $ ./terminusdb db create test $ ./terminusdb query admin/test 'sum(1,2)' [32] woql_compile:sum_list([1,2]) [31] ''(woql_compile:(...,...)) [30] call(woql_compile:(...,...)) at /Users/leather/.local/lib/swipl/boot/init.pl:499 [29] catch(woql_compile:call(...),error(instantiation_error,_12020),woql_compile:(...,...)) at /Users/leather/.local/lib/swipl/boot/init.pl:561 [27] ''('') [26] findall_loop(_12096,'',_12100,[]) at /Users/leather/.local/lib/swipl/boot/bags.pl:99 [25]...

I believe we're using the existing (locally stored) remote commit graph when deciding whether to push or not. I think it would be better to fetch the latest remote commit...

Other CLI commands (`push`, `pull`, `fetch`) allow you to pass the remote name, but `clone` does not. We should add `--remote` to `clone` to be consistent.

```sh $ curl -i http://localhost:6363/api/schema/_system HTTP/1.1 500 Internal Server Error Date: Mon, 14 Feb 2022 09:51:10 GMT Connection: Keep-Alive Content-Type: application/json Content-Length: 1551 { "api:message":"Error: read_access_malformed_context(query_context{all_witnesses:false,authorization:anonymous,bindings:[],default_collection:system_descriptor{},files:[],filter:type_filter{types:[instance]},prefixes:_25338{'@base':\"terminusdb://system/data/\",'@schema':\"http://terminusdb.com/schema/system#\",'@type':'Context',api:'http://terminusdb.com/schema/api#',owl:'http://www.w3.org/2002/07/owl#',rdf:'http://www.w3.org/1999/02/22-rdf-syntax-ns#',rdfs:'http://www.w3.org/2000/01/rdf-schema#',sys:'http://terminusdb.com/schema/sys#',vio:'http://terminusdb.com/schema/vio#',woql:'http://terminusdb.com/schema/woql#',xdd:'http://terminusdb.com/schema/xdd#',xsd:\"http://www.w3.org/2001/XMLSchema#\"},selected:[],system:transaction_object{descriptor:system_descriptor{},inference_objects:[],instance_objects:[read_write_obj{descriptor:system_graph{type:instance},read:,write:_25178}],schema_objects:[read_write_obj{descriptor:system_graph{type:schema},read:,write:_25208}]},transaction_objects:[transaction_object{descriptor:system_descriptor{},inference_objects:[],instance_objects:[read_write_obj{descriptor:system_graph{type:instance},read:,write:_25276}],schema_objects:[read_write_obj{descriptor:system_graph{type:schema},read:,write:_25306}]}],update_guard:_25100,write_graph:system_graph{type:instance}})", "api:status":"api:server_error" } ```

Update documentation for open_descriptor

6

In `src/core/transaction/descriptor.pl`, `open_descriptor/5` is documented as `det`, but multiple uses of it are wrapped with `do_or_die`, which, I believe, should only be necessary with predicates that are not deterministic. *...