Cocoon 3
  1. Cocoon 3
  2. COCOON3-58

The org.apache.cocoon.optional.pipeline.components.sax.jaxb.JAXBGenerator is incomplete

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.0.0-alpha-3
    • Fix Version/s: 3.0.0-alpha-3
    • Component/s: cocoon-optional
    • Labels:
      None

      Description

      There is still a minor issue in the "Plural Stemmer/Inflator" issue[1] to pluralize the entity names.
      It would be great resolve it before releasing the alpha-3

      [1] http://markmail.org/message/snl3vdq4nh3dxldo
      1. COCOON3-58.patch
        16 kB
        Simone Tripodi

        Activity

        Hide
        Simone Tripodi added a comment -
        Look like the Jersey's Inflector has a new place[1] and released under ASL2.0 :)
        how do we want to proceed, importing the code - and adapt it according to C3 needs - or adding the dependency?

        [1] https://inflector.dev.java.net/
        Show
        Simone Tripodi added a comment - Look like the Jersey's Inflector has a new place[1] and released under ASL2.0 :) how do we want to proceed, importing the code - and adapt it according to C3 needs - or adding the dependency? [1] https://inflector.dev.java.net/
        Hide
        Francesco Chicchiriccò added a comment -
        In the meanwhile the Jersey project has been moved - due to java.net reorganization - to [1].

        I wasn't' able to find the inflector you were talking about above as a separate file, but I believe it should be contained in one of the JARs indicated in [2] and available at java.net maven repository [3].

        About license, Jersey is under the CDDL+GPL license [4]: is that a problem for cocoon-optional?

        [1] http://jersey.java.net/
        [2] http://jersey.java.net/nonav/documentation/latest/chapter_deps.html
        [3] http://download.java.net/maven/2/com/sun/jersey/
        [4] http://jersey.java.net/contribute/
        Show
        Francesco Chicchiriccò added a comment - In the meanwhile the Jersey project has been moved - due to java.net reorganization - to [1]. I wasn't' able to find the inflector you were talking about above as a separate file, but I believe it should be contained in one of the JARs indicated in [2] and available at java.net maven repository [3]. About license, Jersey is under the CDDL+GPL license [4]: is that a problem for cocoon-optional? [1] http://jersey.java.net/ [2] http://jersey.java.net/nonav/documentation/latest/chapter_deps.html [3] http://download.java.net/maven/2/com/sun/jersey/ [4] http://jersey.java.net/contribute/
        Hide
        Simone Tripodi added a comment -
        The attached patch contains a possible implementation for a complete PluralStemmer.
        Before I commit, can you please check it to verify eventually *legal issues* we could have with Oracle?
        Implementations are different but patterns have extracted from the Inflector - I don't think they could claim TM on regexp patterns, can they?
        Thanks in advance!
        Show
        Simone Tripodi added a comment - The attached patch contains a possible implementation for a complete PluralStemmer. Before I commit, can you please check it to verify eventually *legal issues* we could have with Oracle? Implementations are different but patterns have extracted from the Inflector - I don't think they could claim TM on regexp patterns, can they? Thanks in advance!
        Hide
        Francesco Chicchiriccò added a comment -
        I am not at all a legal expert but if the "possible TM issue" is limited to regexp patterns, I don't think there could be anything to claim about.

        +1 for fix :-)
        Show
        Francesco Chicchiriccò added a comment - I am not at all a legal expert but if the "possible TM issue" is limited to regexp patterns, I don't think there could be anything to claim about. +1 for fix :-)
        Hide
        Bertrand Delacretaz added a comment -
        See http://apache.org/legal/resolved.html for info about CDDL-licensed libraries
        Show
        Bertrand Delacretaz added a comment - See http://apache.org/legal/resolved.html for info about CDDL-licensed libraries
        Hide
        Simone Tripodi added a comment -
        Salut Bertrand :)
        what worries me is that

        1) Jersey is licensed under CDDL+GPL2
        2) Even if their and my implementations look similar, they are different (and honestly I don't see a different way to do it) and I assume regexp patterns cannot be protected by copyrights... am I wrong?
        3) I didn't import their code as 3rd part library, but reimplemented only one feature of their superset; if they can claim that C3 code is "copied" from Jersey, I assume cannot be released under ASL2.0...

        Any suggestion? Many thanks in advance for your help!!!
        Show
        Simone Tripodi added a comment - Salut Bertrand :) what worries me is that 1) Jersey is licensed under CDDL+GPL2 2) Even if their and my implementations look similar, they are different (and honestly I don't see a different way to do it) and I assume regexp patterns cannot be protected by copyrights... am I wrong? 3) I didn't import their code as 3rd part library, but reimplemented only one feature of their superset; if they can claim that C3 code is "copied" from Jersey, I assume cannot be released under ASL2.0... Any suggestion? Many thanks in advance for your help!!!
        Hide
        Simone Tripodi added a comment -
        fixed on /trunk, see r1087839
        Show
        Simone Tripodi added a comment - fixed on /trunk, see r1087839

          People

          • Assignee:
            Simone Tripodi
            Reporter:
            Simone Tripodi
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development