Skip to content

Conversation

@johnpoth
Copy link

@johnpoth johnpoth commented May 3, 2023

DO NOT MERGE, DRAFT!

@cunningt @luigidemasi

As we discussed, this is a PR that demonstrates users installing the three SAP libraries in their Maven repository instead of using a lib folder. So for example using version 3.1.4 for sapjco and 3.1.1 for sapidoc on Linux x86 64 bit the user would have to run three commands:

  1. mvn install:install-file -DgroupId=com.sap.conn.jco -DartifactId=sapjco3 -Dversion=3.1.4 -Dpackaging=jar -Dfile=sapjco3.jar

  2. mvn install:install-file -DgroupId=com.sap.conn.idoc -DartifactId=sapidoc3 -Dversion=3.1.1 -Dpackaging=jar -Dfile=sapidoc3.jar

  3. mvn install:install-file -DgroupId=com.sap.conn.jco -DartifactId=sapjco3 -Dversion=3.1.4 -Dclassifier=linux-x86_64 -Dpackaging=so -Dfile=libsapjco3.so

Some drawbacks compared to using the lib folder strategy:

  1. If the user uses a different version for sapjco or sapidoc, he has to update the pom.xml
  2. The command for the native library is OS dependent so error prone. If a different classifier was used the pom.xml also has to be updated.
  3. The standalone version of the quickstarts (not sure if we should keep them here or not as this is for SpringBoot) uses the Camel Maven Plugin which doesn't have an option to add extra dependencies. We would therefore have to use the lib folder strategy.

Given those drawbacks I think I prefer keeping the lib folder strategy but maybe I'm missing something. We can improve on it as @luigidemasi suggested and use a single lib folder too.

Let me know what you think!

@cunningt
Copy link
Collaborator

cunningt commented May 3, 2023

Nice job - this would be a huge improvement over what we currently have. The only thing I worry about - and this doesn't really fall under the scope of this PR but is something I think we need to figure out when documenting the process - is how to explain to users which platform-dependent library (so, dylib, etc) they need for their particular platform. I guess we only have to worry about supported platforms, but I think we need to do some mapping there of platform to library.

cunningt pushed a commit that referenced this pull request May 15, 2025
* Add k8s master cluster service example (#65)

* [CSB-447] Set UBI8 JDK11 as base image
@johnpoth
Copy link
Author

johnpoth commented Sep 9, 2025

Closing in favor of #67

@johnpoth johnpoth closed this Sep 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants