Learn how easy it is to sync an existing GitHub or Google Code repo to a SourceForge project! See Demo

Close

#55 New Merge Points for struts-config

closed-fixed
nobody
vendor (19)
5
2002-07-10
2002-06-27
Volker Krebs
No

I've added two merge points for Struts 1.1. Thess are
for plug-ins and controllers. This patch is to be
applied over XDoclet-v1-1-2 tag, because it needs
struts-config_1_1.dtd.

After applying it, you will be able to include a file
called
struts-plugins.xml and struts-controllers.xml in your
merge dir. So, the strutsconfigxml task will include
your plugins and controllers in your struts-config.xml
generated file.

This is neccesarry if you want to use the new struts
plugin validation.

Since this was my first patch, I hope everything was
submitted correctly.

Volker

Discussion

  • Volker Krebs
    Volker Krebs
    2002-06-27

    struts_config_xml.j

     
    Attachments
  • Logged In: YES
    user_id=49846

    Hi Volker, and thanks for your patch.

    You did a few things wrong...

    Patches should always be done against the latest CVS
    version, otherwise it's a high chance that they're obsolete and
    therefore impossible to apply. Can you do that (if it's still
    appliccable?)

    Also, a patch should be submitted as a diff, not an entire file,
    so we can see _what_ you changed. Here is info about how
    to produce a diff:

    http://jakarta.apache.org/site/source.html

    Cheers,
    Aslak

     
    • status: open --> open-out-of-date
     
  • Volker Krebs
    Volker Krebs
    2002-07-01

    Logged In: YES
    user_id=569376

    Sorry for the wrong patch submission.
    I've made a diff against the latest CVS, hope this patch is
    ok now.

    Volker

     
  • Volker Krebs
    Volker Krebs
    2002-07-01

    Patch for struts_config_xml.xdt

     
    Attachments
  • Volker Krebs
    Volker Krebs
    2002-07-01

    Logged In: YES
    user_id=569376

    Sorry forgot the file.

     
    • status: open-out-of-date --> closed-fixed
     
  • Logged In: YES
    user_id=102175

    Added in CVS. Thanks for the patch!