Continuum

Patch to add JBoss support

Details

  • Type: Wish Wish
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 1.1-beta-3
  • Component/s: None
  • Labels:
    None
  • Complexity:
    Intermediate
  • Patch Submitted:
    Yes
  • Number of attachments :
    3

Description

I've been bold enough to prepare a little patch that adds explicit
support for JBoss to the Continuum WAR.

The patch adds a jboss-web.xml file (this should be totally harmless)
and adds two <resource-ref>s to web.xml. I get the impression that
this latter change is harmless as well, or would it break other app
servers?

Anyway, here's the patch. I hope it can get applied, otherwise I'll
just keep on doing it by hand. With this applied the only thing
left to do is add a derby-ds.xml file to the JBoss deployment
directory. And make sure JBoss has a Derby driver, of course.

  1. continuum_jboss_2.patch
    01/Mar/07 2:36 PM
    3 kB
    thierry lach
  2. continuum_jboss_2.patch
    22/Feb/07 3:55 PM
    3 kB
    thierry lach
  3. continuum_jboss.patch
    14/Feb/07 7:04 PM
    1 kB
    Hilco Wijbenga

Activity

Hide
thierry lach added a comment -

I've added to the patch a little bit.

It contains a fix so that email can get sent.

It also contains a new module to create an ear file for the war. This is only activated through a profile named "ear", so to create the ear file you must "mvn -Pear .....". The context root is defined in the main pom and can be overridden using -Dcontinuum.ear.contextRoot=whateverYouWantItToBe.

Show
thierry lach added a comment - I've added to the patch a little bit. It contains a fix so that email can get sent. It also contains a new module to create an ear file for the war. This is only activated through a profile named "ear", so to create the ear file you must "mvn -Pear .....". The context root is defined in the main pom and can be overridden using -Dcontinuum.ear.contextRoot=whateverYouWantItToBe.
Hide
thierry lach added a comment -

OK this patch file might work a little better - I didn't realize that creating a patch through Eclipse put full path names into the patch file.

Show
thierry lach added a comment - OK this patch file might work a little better - I didn't realize that creating a patch through Eclipse put full path names into the patch file.
Hide
Emmanuel Venisse added a comment -

Thierry,

I'll can look at your patch if you attach continuum-ear too

Show
Emmanuel Venisse added a comment - Thierry, I'll can look at your patch if you attach continuum-ear too
Hide
Emmanuel Venisse added a comment -

Patch applied without continuum-ear profile

Show
Emmanuel Venisse added a comment - Patch applied without continuum-ear profile

People

Vote (2)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: