use lower case kethereum dependencies

[uport-project]

use lower case kethereum dependencies

[mirceanis]

In kethereum 0.75 there was still some confusion about the walleth namespace vs komputing
The next version of kotlin-did-jwt should remove any reference to the walleth namespace.
Which version are you on?

You can try using did_jwt_version = "29aa080e" and implementation "com.github.komputing.kethereum:wallet:$kethereum_version" with the version being kethereum_version = "0.76.1" to check if it fixes it for you, or point to a project that's manifesting the errors so I can test on my end.

[mirceanis]

Thanks for reporting this.
I used the camel case names to match the repository names of Komputing/KEthereum.
It seems that jitpack sees them as separate artifacts but gradle either doesn't or gets confused.
To correctly fix this we'll need to update kotlin-common and kmnid too since they also use the camel case names.

[friedger]

This is the message I get from gradle:

Duplicate class org.kethereum.ValuesKt found in modules model-0.75.1.jar (com.github.komputing.KEthereum:model:0.75.1) and model-0.75.1.jar (com.github.walleth.kethereum:model:0.75.1)
Duplicate class org.kethereum.crypto.ConvertersKt found in modules crypto-0.75.1.jar (com.github.komputing.KEthereum:crypto:0.75.1) and crypto-0.75.1.jar (com.github.walleth.kethereum:crypto:0.75.1)

I don't see any upper case for kotlin-common and kmnid

[friedger]

I still get the error with the updated version. The project is https://github.com/blockstack/app-android

[friedger]

A work around would be to use KEtherum in my project's depedencies.

Now, it is a question of convention: show dependencies in build.gradle be lower case only?

[mirceanis]

I did not realize this was an issue before so perhaps we should adopt this convention.
I guess this is a bigger problem on systems where files are case insensitive, so to avoid problems like that we'll keep everything lowercase here as well.

We already have a dependency refactoring in the works:
#22

Stay tuned for the next release which should ameliorate these import issues.