Re: [Java related] packaging Italian ID card middleware

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Thank you for understanding my reaction...

As for the Java side: what Eclipse does under the hood is simply invoking its own compiler and passing it a classpath and source files. It isn't difficult to manually compile the files using javac and the bundled libraries:
```
cd CIEID
java_sources=$(find src -name -type f '*.java')
libs=$(echo $(find lib -name -type f '*.jar') | tr ' ' ':')
javac -cp "${libs}" -d classes ${java_sources}
```
This creates a `classes` directory with the compiled classes using bundled libraries.

I took a look at the bundled libraries and some half of them are definitely packaged in Fedora (and maintained by us). The ones with capital letters are unknown to me though.

If the Java files were to be compiled using the system libraries, the ${libs} argument would have to be built by searching for standard locations for jars: (/usr/share/java/...).

On 18. 7. 2024 18:12, Cristian Le via devel wrote:
Hi Marian,

Thanks for the comments. We all had our turns being horrified by this project. The more you look at it the more you want to use bleach as eye-drops. Germano is trying to write an issue to at least try to steer the upstream project in a more sensible directions.

On the CMake side, I gave some comments on what are the minimal changes that are required. On Java side though, I don't know what the build process is. Vitaly mentioned that java environment is tricky with things like Kotlin compiler being removed, so that's why we are asking for some second-opinions on this. The project seems to use only plain java, and taking a quick look at the dependencies, it doesn't seem that they are all packaged, but are there among them any red flags that would be a complete deal-breaker?

Probably for the Java build it would be better to use Maven. Are there other pointers of what upstream developers should look into/implement on the Java side?

Re: Sergio, so far it seems all Italian, Portuguese and Estonian are using different infrastructures.

On 2024/07/18 17:31, Marián Konček wrote:
Hi, I am a member of Fedora Java package maintainers.

I see several problems with this project:
1. *.jar, *.a, *.o files not only do not belong into Fedora sources, they should not even be in *any* GitHub repository unless they are used for testing. Binary components are not sources. 2. Requiring graphical Eclipse to build some Java code is a heavy overkill, you should instead add a standard Java build system like Maven or add CMake targets to build Java code using standard tools like `javac` and `jar`. 3. generated files like *.d, *.mk should also not be in a source repository, unless there is a good reason for them. 4. You will do the others a great service if you use the English language in GitHub commits and READMEs. Open-source software is not there for you but for the whole world. 5. (minor) The project is missing a README information what it actually is about. All I could read about is that it is "some middleware for Linux".

You should really take a look at other more standard projects on GitHub, how they handle builds and library dependencies.

On 18. 7. 2024 16:42, Germano Massullo wrote:
Hello, I discussed the feasibility of packaging
https://github.com/italia/cie-middleware-linux
in Fedora devel Matrix channel with Cristian Le and xvitaly. With the help of Cristian Le I wrote the following [1] draft that will be used to open an upstream ticket to request various improvements to make it possible to include the software in the Fedora repository. We ended up needing the comment from Java package maintainers, if they see any other no-go. I am in particular concerned about https://docs.fedoraproject.org/en-US/packaging-guidelines/Java/#_pre_built_dependencies

Thank you

[1]: https://germano.fedorapeople.org/canc/cie_middleware.md

--
Marián Konček

--
_______________________________________________
devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/devel@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Users]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]

  Powered by Linux