James jSieve
  1. James jSieve
  2. JSIEVE-24

Include build time libraries (javacc, javamail, activation) in the source distribution

    Details

    • Type: Task Task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.3
    • Component/s: None
    • Labels:
      None

      Description

      We cannot ship javacc 4.0 because of licensing restrictions:
      ------------------
      You acknowledge that this software is not designed, licensed or
      intended for use in the design, construction, operation or maintenance
      of any nuclear facility.
      ------------------

      We also removed javamail/activation from the stage folder because we cannot ship their poms and they are simply used to run tests.

      There is a javacc-4.1d1 release (4 Jun 2008) that is licensed under the standard BSD license. I gave it a try and our tests pass. I had to change one thing because jjtree 4.1 creates jj files in a different folder.

      Index: build.xml
      ===================================================================

      • <arg line="-output_directory=$ {build.src}/org/apache/jsieve/parser/generated ${build.src}

        /sieve.jj"/>
        + <arg line="-output_directory=$

        {build.src}/org/apache/jsieve/parser/generated ${build.src}

        /org/apache/jsieve/parser/generated/sieve.jj"/>

      • <arg line="-output_directory=$ {build.src}/org/apache/jsieve/parser/generated/address ${build.src}

        /AddressListParser.jj"/>
        + <arg line="-output_directory=$

        {build.src}/org/apache/jsieve/parser/generated/address ${build.src}

        /org/apache/jsieve/parser/generated/address/AddressListParser.jj"/>
        ===================================================================

        Activity

        Stefano Bagnara created issue -
        Robert Burrell Donkin made changes -
        Field Original Value New Value
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Closed [ 6 ]
        Robert Burrell Donkin made changes -
        Assignee Robert Burrell Donkin [ robertburrelldonkin ]

          People

          • Assignee:
            Robert Burrell Donkin
            Reporter:
            Stefano Bagnara
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development