Daniel Adam

Results 49 comments of Daniel Adam

@Askidea if I understand correctly what @jkralik told me then this is correct behavior and reflects a change in specification. Though I wasn't able to pinpoint the precise place in...

Hello @Askidea, > Is there something wrong with the build or run environment? `oc_tcp_connection_state` is a new function I've added last week. The Android job is broken, so that's probably...

@Askidea: Thank you! I've updated it and now it also should fix the `oc_tcp_connection_state` linker error.

@Askidea Updated the branch again, hopefully no missing symbols anymore. (Sorry about this, but I have nobody else to test this but you.)

@Askidea Updated the branch again. Thank you for your help again!

@Askidea Nice, I can merge it then and at least fix the android Github job, so thank you for your help! Regarding CTT stuff, I'm not sure I can help...

I think this commit - https://github.com/iotivity/iotivity-lite/commit/d349ec91961d25ba76339f767d752f6af74b381e#diff-82d018ba0de8e0bd03ba54e7695dc4d77777346ce4c14a8247fa1d7091c53b59 - is the culprit. If you check oc_rep.h then new oc_rep_encode_* functions were created and all calls to cbor_encode_* were rewritten to oc_rep_encode_* calls....

@Askidea this should be fixed by https://github.com/iotivity/iotivity-lite/pull/328 . I've also found that we had some JUnit tests for the Java bindings. The tests were commented out, but re-enabled them. So...

The problem seems to be in https://github.com/gojp/goreportcard/blob/fe7635329954dfcb646a25072c24228494537881/handlers/check.go#L24. The call to `Download.Clean` returns base url of the module and this url has the version `/v*` suffix stripped. For example `github.com/ydb-platform/ydb-go-sdk/v3` will...

> This fix may be broke repositories where v2+ versions stored as subdirectory of repo It should work, the goreportcard code uses [Module Proxy](https://go.dev/ref/mod#goproxy-protocol) protocol (see `$base/$module/@v/$version.info` docu) to download...