-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
FINERACT-2108: Fix gradle build error #4034
FINERACT-2108: Fix gradle build error #4034
Conversation
3e14a4d
to
5fe579f
Compare
Hi @HarimbolaSantatra , we addressed this by adding Apache Archiva to the repositories section of the buildscript. That is:
The above approach was adopted because we weren't sure that "io.github.rburgst:okhttp-digest:1.21" is an in-place replacement for "com.burgstaller:okhttp-digest:1.10". |
@ufonduo which lib use it, can not found it in project |
Required by: org.asciidoctor.jvm.gems:org.asciidoctor.jvm.gems.gradle.plugin:3.3.2 > org.asciidoctor:asciidoctor-gradle-jvm-gems:3.3.2 > com.github.jruby-gradle:jruby-gradle-core-plugin:2.0.2 > io.github.http-builder-ng:http-builder-ng-okhttp:1.0.3 |
How about:
... shorter... one-liner... |
@ufonduo It is the official replacement, mentioned in this issue, at github.com/rburgst and at mvnrepository. |
@vidakovic There's a deprecation warning:
With |
@HarimbolaSantatra @vidakovic I believe the mentioned dependency was already removed from Fineract. I dont think there is anything to do here. |
I can confirm that the current develop branch is successfully building the jar file. The affected version was 1.9 and 1.8.4, so maybe it was fixed along the way. |
I would say we should have this fix for those versions only then ;) |
I think we need some sort of confirmation and we can close this PR |
@HarimbolaSantatra guys, anything happening here? |
@galovics This PR addresses a dependency problem linked with |
@HarimbolaSantatra I can confirm the dependency has been removed. I'll close this PR. |
Description
As mentioned in the Jira ticket,
com.burgstaller:okhttp-digest:1.10
is no longer available on Maven Central nor any other repositories.The full error:
Credit:
Checklist
Write the commit message as per https://github.com/apache/fineract/#pull-requests
Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.
Create/update unit or integration tests for verifying the changes made.
Follow coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions.
Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/legacy-docs/apiLive.htm with details of any API changes
Submission is not a "code dump". (Large changes can be made "in repository" via a branch. Ask on the developer mailing list for guidance, if required.)