Cargo
  1. Cargo
  2. CARGO-917

Update documentation for "merge.xml" in uberwar goal

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.1.0
    • Component/s: Documentation
    • Labels:
      None
    • Complexity:
      Intermediate
    • Tested on JDKs:
      1.4, 5.0. 6.0
    • Number of attachments :
      1

      Description

      My merge file is like this

      <?xml version="1.0" encoding="UTF-8"?>
      <uberwar>
        <wars>
          <war>com.tty.tragid:Tragid</war>
          <war>QuartersUI:QuartersUI</war>
        </wars>
        <webXml>
          <contextParams>
            <strategy name="ChooseByName">
              <default>
                <strategy name="Preserve"/>
              </default>
              <choice name="contextConfigLocation">
                <strategy name="NodeMerge">
                  <context-param>
                    <param-name>$left:/param-name</param-name>
                    <param-value>$left:/param-value, $right:/param-value</param-value>
                  </context-param>
                </strategy>
              </choice>
            </strategy>
          </contextParams>
        </webXml>
      </uberwar>
      

      I had a look at the source code and it seems to me that webXML node doesnt get processed at all, because the processing is happening in doMerge(...) which happens only for <merges/> nodes.

      Is this a known issue? I am stuck with this issue and cannot use the uberwar goal because of this reason.

      1. merge.xml
        1 kB
        Anton Khitrenovich

        Issue Links

          Activity

          Hide
          Savas Ali Tokmen added a comment -

          OK

          Well, no dates set yet; but we see the end for many tasks so I'm checking

          Perhaps in a few weeks

          Show
          Savas Ali Tokmen added a comment - OK Well, no dates set yet; but we see the end for many tasks so I'm checking Perhaps in a few weeks
          Hide
          Savas Ali Tokmen added a comment -

          Hi again

          Any updates possible, preferably before the end of the month?

          Cheers

          Show
          Savas Ali Tokmen added a comment - Hi again Any updates possible, preferably before the end of the month? Cheers
          Hide
          Anton Khitrenovich added a comment -

          Documentation updated (http://docs.codehaus.org/display/CARGO/Merging+WAR+files). Please review before closing this issue...

          Show
          Anton Khitrenovich added a comment - Documentation updated ( http://docs.codehaus.org/display/CARGO/Merging+WAR+files ). Please review before closing this issue...
          Hide
          Anton Khitrenovich added a comment -

          @Ali: Just opened CARGO-968 and CARGO-969 for refactoring, probably for Cargo 1.2.0...

          Show
          Anton Khitrenovich added a comment - @Ali: Just opened CARGO-968 and CARGO-969 for refactoring, probably for Cargo 1.2.0...
          Hide
          Savas Ali Tokmen added a comment -

          Looks good to me, Anton I've rearranged some minor style stuff, and also changed the target version for the two new issues.

          Thank you, and please feel free to close this ticket if you also believe it's good.

          Show
          Savas Ali Tokmen added a comment - Looks good to me, Anton I've rearranged some minor style stuff, and also changed the target version for the two new issues. Thank you, and please feel free to close this ticket if you also believe it's good.

            People

            • Assignee:
              Anton Khitrenovich
              Reporter:
              Vaishakh Nadgir
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: