Regards,
Hantsy Bai
Self-employed consultant, fullstack developer, agile coach, freelancer/remote worker
GitHub: https://github.com/hantsy
Twitter: https://twitter.com/@hantsy
Medium: https://medium.com/@hantsy--
Codehaus Cargo - https://codehaus-cargo.github.io/
---
You received this message because you are subscribed to the Google Groups "Codehaus Cargo" group.
To unsubscribe from this group and stop receiving emails from it, send an email to codehaus-carg...@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/codehaus-cargo/CAMq3Qk0z_CCy2vKdQ6_84bo-PRxdq6NJC2a_u8E80YWtGAFJeg%40mail.gmail.com.
Regards,
Hantsy Bai
Self-employed consultant, fullstack developer, agile coach, freelancer/remote worker
GitHub: https://github.com/hantsy
Twitter: https://twitter.com/@hantsy
Medium: https://medium.com/@hantsyHi Hantsy
Good catch, it seems the Jetty deplorer’s context.xml capability didn't have a single testcase and hence the EE11 implementation being incomplete was never caught 😮
I added https://github.com/codehaus-cargo/cargo/blob/master/core/samples/java/src/test/java/org/codehaus/cargo/sample/java/jetty/JettyWarContextXmlTestCase.java now, and of course a bunch of commits to fix the actual bug too. Let's see what the CI says 😉
Regards
S. Ali Tokmen https://ali.tokmen.com/ https://contact.ali.tokmen.com/
--
Codehaus Cargo - https://codehaus-cargo.github.io/
---
You received this message because you are subscribed to the Google Groups "Codehaus Cargo" group.
To unsubscribe from this group and stop receiving emails from it, send an email to codehaus-carg...@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/codehaus-cargo/CAMq3Qk2aGjXa7uWFoNiLyjdj_NWpG4m7C83LTwFaV4GqaRh%2BDA%40mail.gmail.com.
Hi Hantsy
This turned out more complex than initially assessed, I now created https://codehaus-cargo.atlassian.net/browse/CARGO-1650 and added links to the different commits.
Let's see what the results from the CI build https://codehaus-cargo.semaphoreci.com/workflows/fdefca52-c67b-4891-9ce2-b8f51ed17a11?pipeline_id=4aa51bc5-3158-4433-b37b-789398f66d9c will show!
Regards
S. Ali Tokmen https://ali.tokmen.com/ https://contact.ali.tokmen.com/
Hi Hantsy
As per https://codehaus-cargo.github.io/cargo/Jetty+12.x.html, you also need to set JettyPropertySet.MODULES (in other words, cargo.jetty.modules) so it has ee11. It would hence most likely have to be set to server,ext,http,ee11-annotations,ee11-plus,ee11-jsp,ee11-deploy
Regards
S. Ali Tokmen https://ali.tokmen.com/ https://contact.ali.tokmen.com/
To view this discussion visit https://groups.google.com/d/msgid/codehaus-cargo/d01b5844-605f-4d35-817d-dad53aef646an%40googlegroups.com.
Regards,
Hantsy Bai
Self-employed consultant, fullstack developer, agile coach, freelancer/remote worker
GitHub: https://github.com/hantsy
Twitter: https://twitter.com/@hantsy
Medium: https://medium.com/@hantsyYou received this message because you are subscribed to a topic in the Google Groups "Codehaus Cargo" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/codehaus-cargo/Muxbd4680UU/unsubscribe.
To unsubscribe from this group and all its topics, send an email to codehaus-carg...@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/codehaus-cargo/581dd685-e48f-43e8-a7df-488a206471f9%40alishomepage.com.
Hi Hantsy
Thanks for your feedback, this is more a question of "philosophy" than actual development...
As per https://jetty.org/docs/jetty/12.1/operations-guide/begin/index.html whoever uses Jetty must give the list of services - Hence Codehaus Cargo also requesting the same.
What we could do, of course, is to make the services "auto adapt" if not explicitly set in Codehaus Cargo.
What would be your thoughts?
Regards
S. Ali Tokmen https://ali.tokmen.com/ https://contact.ali.tokmen.com/
To view this discussion visit https://groups.google.com/d/msgid/codehaus-cargo/CAMq3Qk1N9iOXfByM-aimi74H4mpqFpMquUapWhtkrHQUmnY1PA%40mail.gmail.com.
Regards,
Hantsy Bai
Self-employed consultant, fullstack developer, agile coach, freelancer/remote worker
GitHub: https://github.com/hantsy
Twitter: https://twitter.com/@hantsy
Medium: https://medium.com/@hantsyTo view this discussion visit https://groups.google.com/d/msgid/codehaus-cargo/92230ea1-6e9a-4ef6-ac2f-fa20b86325a0%40alishomepage.com.
Hi Hantsy
Fair enough, I added
https://codehaus-cargo.atlassian.net/browse/CARGO-1651 and checked
in
https://github.com/codehaus-cargo/cargo/commit/971215751bc4fb3a2168f799f37063cd90209d28,
all looks good. This will be part of Codehaus Cargo 1.10.25 when
it is released.
Regards
S. Ali Tokmen https://ali.tokmen.com/ https://contact.ali.tokmen.com/
To view this discussion visit https://groups.google.com/d/msgid/codehaus-cargo/CAMq3Qk394R%2BiJB7ML%3D43pFY5BOHQsNYf_go_HKYVEdjtLFWfxQ%40mail.gmail.com.