Hi all,
On Mon, Jul 29, 2013 at 3:03 AM, Fred Hansen <[email protected]> wrote: > Versions of DateConverter,java are in three (3) packages: jempbox.impl, > pdfbox.util, and xmpbox. They have diverged as updates have been made to > one or the other. > > Date Converter is used in these modules: > jempbox.xmp.ResourceEvent.java > jempbox.xmp.XMPSchema.java > pdfbox.cos.COSDictionary.java > pdfbox.pdmodel.fdf.FDFAnnotation.java > preflight.metadata.SynchronizedMetaDataValidation.java > xmpbox.type.DateType.java > > The smallest library is xmpbox, so let's consider it as a home for > DateConverter. Preflight already imports xmpbox. Pdfbox requires jempbox, > so importing xmpbox into jempbox will satisfy both. Since xmpbox defines > some xmp features, it seems likely that jempbox could take further > advantage of xmpbox. > > Conclusion and recommendation: have a single copy of DateConverter in > xmpbox. > > I partially agree with the conclusion. Having a single copy of DateConverter is the good conclusion but I am not sure xmpbox is the right place for it. It will create a dependency between xmpbox and pdfbox where there is not today. One of the goal of the 2.0.0 release is to break PDFBox in different modules with few dependencies if possible to prevent too big footprints in different use cases. In the package org.apache.pdfbox.util, there are some other classes that could be used in different modules (XMLUtil, StringUtil as examples). An other idea is to create a specific little module with theses classes... but it is a new module ... So, what are the opinions ? > [...] > > Fred Hansen > Guillaume Bailleul
