Skip to content
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

Cleaning dependencies repositories #885

Merged

Conversation

amottier
Copy link
Contributor

I'm trying to build Knowage-Server from source using latest stable version of Maven (3.9.4). But since Maven 3.8.1 HTTP repositories are blocked by default.

So in this pull request I tried to update all HTTP repositories to their equivalent HTTPS versions.

For some of them I took the opportunity to use more "official" repositories.

For http://repo.bidlink.cn/nexus/content/repositories/ repository that is used to download com.jamon:fdsapi artifact the HTTPS URL (https://repo.bidlink.cn/nexus/content/repositories/) provides invalid certificate. I tried to look for an alternative repository but didn't find any. I opened an issue on a related project to get help to identify an official repository and also to identify where the source code is hosted. Due to this issue I'm opening this pull request as a draft.

I have read the CLA Document and I hereby sign the CLA

Update repositories to use HTTPS instead of HTTP to avoid issue with Maven version 3.8.1 or newer (see https://maven.apache.org/docs/3.8.1/release-notes.html).

Remove jaspersoft.artifactoryonline.com repository. JasperReports artifacts are availble from Maven Central (e.g. https://repo1.maven.org/maven2/net/sf/jasperreports/jasperreports/6.1.0/jasperreports-6.1.0.jar) so getting them from there. For artifacts not availble on Maven Central, use jaspersoft repositories available at https://jaspersoft.jfrog.io.

Use a different repository for Mondrian. Previous repository (http://nexus.pentaho.org/content/groups/omni) answers with an error 503. The new repository (https://repo.orl.eng.hitachivantara.com/artifactory/pnt-mvn/) is the one included in https://github.com/pentaho/mondrian/blob/5dbad92040d455291f63e2cb4acdb0416ff04884/pom.xml#L86C12-L86C72. Also the new repository allow usage of HTTPS.

Remove http://repo.bidlink.cn/nexus/content/repositories as HTTPS provides invalid certificate. An [issue](stevensouza/jamonapi#29) has been open to identify official repository (and source code location).
@davide-zerbetto
Copy link

Thanks a lot for your input @amottier !!
Let's wait a bit for a response on com.jamon:fdsapi dependency...

@amottier
Copy link
Contributor Author

@davide-zerbetto I'm not really optimistic about getting a reply to stevensouza/jamonapi#29

Would it be ok with you if I you if I get fdsapi-1.2.jar from https://github.com/stevensouza/jamonapi/blob/master/jamon_war/src/main/webapp/WEB-INF/lib/fdsapi-1.2.jar, published it on OW2 Nexus repository (https://repository.ow2.org/nexus/) and update this pull request to get it from there?

Of course I think the best long term solution would be to figure out a way to remove the dependency on fdsapi-1.2.jar as the project seems to be no longer actively developed.

@amottier
Copy link
Contributor Author

amottier commented Oct 2, 2023

@davide-zerbetto as I didn't get a response to my issue stevensouza/jamonapi#29 I upload the artifact to OW2 Nexus repository for 3rd party libraries: https://repository.ow2.org/nexus/content/repositories/3rd-party/

Would it be ok with you to get the dependency from there?

@davide-zerbetto
Copy link

@amottier it would be ok to get library from ow2 repo, but sources should be there along with binary. Could you please publish source code as well? You know: having an open source project, we should also provide sources for the libraries

@amottier
Copy link
Contributor Author

amottier commented Oct 5, 2023

The thing is I don't know where the source code of this specific version is hosted.

I checked the content of the repository you are currently using (http://repo.bidlink.cn/nexus/content/repositories/public/com/fdsapi/fdsapi/1.2/) looking for a jar or zip file with the sources but can't find it.

Where did you currently get the source code?

Also, as far as I know, as you don't do any modification on this library you don't have obligation to distribute the source code, you only need to make sure you include the license (BSD for fdsapi). But I might be wrong here.

com.jamon:fdsapi was previously get from a repository that wasn't available in HTTPS and seems rather unofficial.
Now get com.jamon:fdsapi from OW2 third party repository. The library upload on OW2 repository was download from a GitHub project created by the developer of library (https://github.com/stevensouza/jamonapi/blob/master/jamon_war/src/main/webapp/WEB-INF/lib/fdsapi-1.2.jar).
@amottier amottier marked this pull request as ready for review October 30, 2023 08:41
@amottier
Copy link
Contributor Author

@davide-zerbetto I did the change to get fdsapi-1.2.jar from OW2 third party repository. Is it ok for you to merge this pull request as is?

@davide-zerbetto
Copy link

@amottier regarding if it is mandatory for a dependency to have its sources publicly available, I'm not sure too, I think there is no unanimous consensus about it. Let's say that we prefer to have only dependencies with publicly available sources and, when possible, we try to remove the ones that do not meet this requirement.
For now, I'll merge you PR as it is.
Thanks.

@amottier
Copy link
Contributor Author

@davide-zerbetto thanks a lot, I'll just rebase my commit to avoid any conflict.

@davide-zerbetto davide-zerbetto merged commit 28bc9f8 into KnowageLabs:master Nov 20, 2023
2 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Nov 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants