QDox
  1. QDox
  2. QDOX-15

Separate parser packages into own JAR

    Details

    • Type: Wish Wish
    • Status: Closed Closed
    • Priority: Minor Minor
    • Resolution: Won't Fix
    • Affects Version/s: 1.3
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Number of attachments :
      0

      Description

      My project (and, I'm guessing, many others) only use the parser part of QDox and build their own model. It would be nice to offer just the parser packages as a separate JAR for redistribution with such apps. This is not critical since other apps could just package a subset of QDox themselves, but it would be nice.

        Activity

        Aslak Hellesøy made changes -
        Field Original Value New Value
        Fix Version/s 1.4 [ 10304 ]
        Hide
        Mike Williams added a comment -

        I think that any benefit would be outweighed by the added complexity of distributing an extra jar-file. It's simpler to keep it all together; the model classes really don't add that much overhead.

        So, -1 from me.

        Show
        Mike Williams added a comment - I think that any benefit would be outweighed by the added complexity of distributing an extra jar-file. It's simpler to keep it all together; the model classes really don't add that much overhead. So, -1 from me.
        Hide
        Aslak Hellesøy added a comment -

        I second Mike. What problem are we trying to solve here?

        Show
        Aslak Hellesøy added a comment - I second Mike. What problem are we trying to solve here?
        Aslak Hellesøy made changes -
        Fix Version/s 1.5 [ 10650 ]
        Environment
        Fix Version/s 1.4 [ 10304 ]
        Hide
        Piotr Kaminski added a comment -

        My project builds its own model based on the parser's event stream, and hence doesn't need the QDox model classes and other paraphenelia. Pruning the JAR file reduces its size from ~58K to ~23K (on the version I happen to have checked out), a savings that some may consider irrelevant and others attractive.

        However, since the license appears to permit redistribution of modified QDox JARs and in order to keep the base QDox build simple, I agree that we can drop this request and leave it to projects to subset the JAR as needed.

        Show
        Piotr Kaminski added a comment - My project builds its own model based on the parser's event stream, and hence doesn't need the QDox model classes and other paraphenelia. Pruning the JAR file reduces its size from ~58K to ~23K (on the version I happen to have checked out), a savings that some may consider irrelevant and others attractive. However, since the license appears to permit redistribution of modified QDox JARs and in order to keep the base QDox build simple, I agree that we can drop this request and leave it to projects to subset the JAR as needed.
        Aslak Hellesøy made changes -
        Status Open [ 1 ] Closed [ 6 ]
        Fix Version/s 1.5 [ 10650 ]
        Resolution Won't Fix [ 2 ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Piotr Kaminski
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: