jsr-333
  1. jsr-333
  2. JSR_333-72

Deal with the "PHP in a JSR" issue

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      We have received some feedback concerning the fact that there is PHP code inside the JSR package and that this somehow violates the rules of the JCP. I do not think that it does violate the rules but we may need to thow in some disclaimers here and there like "this is only additional documentaion, any similarlity to actual PHP code is entirely coincidental"

        Activity

        Peeter Piegaze created issue -
        Hide
        Peeter Piegaze added a comment - - edited

        Ok, so after some discussion with Werner Kiel, who is involved in the JCP, we will take his advice and remove the actual php directory from the spec zip. Instead we will reference the github URL in the PHPCR section of the spec doc.

        Show
        Peeter Piegaze added a comment - - edited Ok, so after some discussion with Werner Kiel, who is involved in the JCP, we will take his advice and remove the actual php directory from the spec zip. Instead we will reference the github URL in the PHPCR section of the spec doc.
        Hide
        Peeter Piegaze added a comment -

        PHPCR is now referenced from within the spec document, but no PHP code is included in the actual spec package.

        Show
        Peeter Piegaze added a comment - PHPCR is now referenced from within the spec document, but no PHP code is included in the actual spec package.
        Peeter Piegaze made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Peeter Piegaze
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: