make httpclient compileOnly since it ships with the OS

Signed-off-by: Andy Scherzinger <info@andy-scherzinger.de>

app/build.gradle

@@ -97,6 +97,8 @@ android {
                 ]
             }
         }
+
+        useLibrary 'org.apache.http.legacy'
     }
 
     dexOptions {
@@ -113,6 +115,7 @@ android {
     packagingOptions {
         exclude 'META-INF/LICENSE.txt'
         exclude 'META-INF/LICENSE'
+        exclude 'META-INF/NOTICE.txt'
         exclude 'META-INF/NOTICE'
         exclude 'META-INF/DEPENDENCIES'
         exclude 'META-INF/rxjava.properties'
@@ -293,6 +296,10 @@ dependencies {
     implementation 'org.osmdroid:osmdroid-android:6.1.10'
     implementation 'fr.dudie:nominatim-api:3.4'
 
+    //noinspection DuplicatePlatformClasses
+    api 'org.apache.httpcomponents:httpclient:4.5.9' // Android comes with its own httpclient
+    // nominatim-api uses httpclient
+
     testImplementation 'junit:junit:4.13.2'
     testImplementation 'org.mockito:mockito-core:3.11.0'
     testImplementation "org.powermock:powermock-core:${powermockVersion}"

app/src/main/java/com/nextcloud/talk/controllers/GeocodingController.kt

@@ -22,7 +22,6 @@ package com.nextcloud.talk.controllers
 
 import android.app.SearchManager
 import android.content.Context
-import android.graphics.drawable.ColorDrawable
 import android.os.Build
 import android.os.Bundle
 import android.text.InputType

build.gradle

@@ -49,8 +49,6 @@ buildscript {
 configurations.all {
     // check for updates every build
     resolutionStrategy.cacheChangingModulesFor 3600, 'seconds'
-    exclude module: 'httpclient'
-    exclude module: 'commons-logging'
 }
 
 allprojects {