-
Notifications
You must be signed in to change notification settings - Fork 225
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
Cleaning dependencies repositories #885
Conversation
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).
Thanks a lot for your input @amottier !! |
@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 Of course I think the best long term solution would be to figure out a way to remove the dependency on |
@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? |
@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 |
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).
@davide-zerbetto I did the change to get |
@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. |
@davide-zerbetto thanks a lot, I'll just rebase my commit to avoid any conflict. |
I'm trying to build Knowage-Server from source using latest stable version of Maven (
3.9.4
). But since Maven3.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