Hi Jody,

I computed a quick list of package names outside of the 'library' that need to be moved to avoid the split module issues.  The attached files have that output.

This won't necessarily sort all the split modules, but it'd be a handy next step.

Cheers,

Jim

On 10/25/2018 2:49 AM, Jody Garnett wrote:
Updated spreadsheet plan, reviewed with team, and sent the updated proposal <https://github.com/geotools/geotools/wiki/Restructure-GeoTools-into-Jigsaw-modules> to geotools-devel
- repackage plan is nice and solid
- splitting up gt-main is technically feasible - asking geotools-devel if we should do so

Initial pull request is here: https://github.com/geotools/geotools/pull/2142

1) redistribute gt-api classes to gt-metadata and gt-main
- Initial classes moved to gt-metadata successfully
- api change: org.geotools.decorate --> org.geotools.util.decorate
- note: DirectPosition3D had to move to gt-referencing (to preserve a package visibility "friend" relationship
- everything else moved to gt-main as planned
- gt-api successfully removed, and gt-main pom.xml description updated
- recording refactoring script and placed into user guide for later

Tomorrow: Continuing on with the core library tomorrow morning, moving on to plugins and extensions tomorrow afternoon. Goal is to finish up on Thursday, and apply these changes to geowebcache and geoserver projects Friday morning.

How to help?
- Love to hear how EMF upgrade is going (to see if our models can track api change) - Need a sample application ready to test using geotools on the module path



--
Jody Garnett




_______________________________________________
Geoserver-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

app-schema/app-schema/src/main/java/org/geotools/feature/AppSchemaAttributeBuilder.java
app-schema/app-schema/src/main/java/org/geotools/feature/AppSchemaFeatureFactoryImpl.java
app-schema/app-schema/src/main/java/org/geotools/filter/AsMultiGeometryFunctionExpression.java
app-schema/app-schema/src/main/java/org/geotools/filter/expression/FormatDateTimezoneFunction.java
app-schema/app-schema/src/main/java/org/geotools/filter/expression/ToDirectPositionFunction.java
app-schema/app-schema/src/main/java/org/geotools/filter/expression/ToEnvelopeFunction.java
app-schema/app-schema/src/main/java/org/geotools/filter/expression/ToLineStringFunction.java
app-schema/app-schema/src/main/java/org/geotools/filter/expression/ToPointFunction.java
app-schema/app-schema/src/main/java/org/geotools/filter/expression/ToXlinkHrefFunction.java
app-schema/app-schema/src/main/java/org/geotools/filter/expression/VocabFunction.java
app-schema/app-schema/src/main/java/org/geotools/filter/expression/XmlXPathPropertyAccessorFactory.java
app-schema/app-schema/src/main/java/org/geotools/filter/FilterFactoryImplNamespaceAware.java
app-schema/app-schema/src/main/java/org/geotools/filter/FilterFactoryImplReportInvalidProperty.java
app-schema/app-schema/src/main/java/org/geotools/filter/IDFunctionExpression.java
app-schema/app-schema/src/main/java/org/geotools/filter/NestedAttributeExpression.java
app-schema/app-schema/src/main/java/org/geotools/jdbc/JdbcMultipleValueEncoder.java
app-schema/app-schema/src/main/java/org/geotools/jdbc/JoiningJDBCFeatureSource.java
app-schema/app-schema/src/main/java/org/geotools/jdbc/NamespaceAwareAttributeRenameVisitor.java
app-schema/app-schema/src/main/java/org/geotools/jdbc/NestedFilterToSQL.java
app-schema/app-schema/src/main/java/org/geotools/jdbc/WrappedFilterToSql.java
app-schema/app-schema/src/main/java/org/geotools/util/IndexQueryUtils.java
app-schema/app-schema/src/main/java/org/geotools/util/InterpolationProperties.java
app-schema/app-schema/src/main/java/org/geotools/util/XmlXpathUtilites.java
app-schema/app-schema/src/test/java/org/geotools/filter/BBoxTest.java
app-schema/app-schema/src/test/java/org/geotools/filter/expression/AppSchemaFeaturePropertyAccessorTest.java
app-schema/app-schema/src/test/java/org/geotools/filter/expression/FormatDateTimezoneFunctionTest.java
app-schema/app-schema/src/test/java/org/geotools/filter/GeometryFunctionsTest.java
app-schema/app-schema/src/test/java/org/geotools/filter/IDFunctionExpressionTest.java
app-schema/app-schema/src/test/java/org/geotools/filter/VocabFunctionsTest.java
app-schema/app-schema/src/test/java/org/geotools/test/AppSchemaTestSupport.java
app-schema/app-schema/src/test/java/org/geotools/util/ComplexAttributeConverterFactoryTest.java
app-schema/app-schema/src/test/java/org/geotools/util/IndexQueryUtilsTest.java
app-schema/app-schema/src/test/java/org/geotools/util/InterpolationPropertiesTest.java
app-schema/app-schema-resolver/src/main/java/org/geotools/xml/AppSchemaConfiguration.java
app-schema/app-schema-resolver/src/main/java/org/geotools/xml/AppSchemaLocationResolver.java
app-schema/app-schema-resolver/src/main/java/org/geotools/xml/AppSchemaValidator.java
app-schema/app-schema-resolver/src/main/java/org/geotools/xml/AppSchemaXSD.java
app-schema/app-schema-resolver/src/main/java/org/geotools/xml/AppSchemaXSDRegistry.java
app-schema/app-schema-resolver/src/test/java/org/geotools/xml/AppSchemaConfigurationTest.java
app-schema/app-schema-resolver/src/test/java/org/geotools/xml/AppSchemaValidatorDemo.java
app-schema/app-schema-resolver/src/test/java/org/geotools/xml/AppSchemaValidatorTest.java
app-schema/sample-data-access/src/main/java/org/geotools/data/SampleDataAccess.java
app-schema/sample-data-access/src/main/java/org/geotools/data/SampleDataAccessData.java
app-schema/sample-data-access/src/main/java/org/geotools/data/SampleDataAccessFactory.java
app-schema/sample-data-access/src/main/java/org/geotools/data/SampleDataAccessFeatureCollection.java
app-schema/sample-data-access/src/main/java/org/geotools/data/SampleDataAccessFeatureIterator.java
app-schema/sample-data-access/src/main/java/org/geotools/data/SampleDataAccessFeatureSource.java
app-schema/sample-data-access/src/test/java/org/geotools/data/SampleDataAccessDataTest.java
app-schema/sample-data-access/src/test/java/org/geotools/data/SampleDataAccessFactoryTest.java
app-schema/sample-data-access/src/test/java/org/geotools/data/SampleDataAccessTest.java
brewer/src/main/java/org/geotools/Builder.java
brewer/src/main/java/org/geotools/filter/AndBuilder.java
brewer/src/main/java/org/geotools/filter/expression/AddBuilder.java
brewer/src/main/java/org/geotools/filter/expression/ChildExpressionBuilder.java
brewer/src/main/java/org/geotools/filter/expression/DivideBuilder.java
brewer/src/main/java/org/geotools/filter/expression/ExpressionBuilder.java
brewer/src/main/java/org/geotools/filter/expression/FunctionBuilder.java
brewer/src/main/java/org/geotools/filter/expression/LiteralBuilder.java
brewer/src/main/java/org/geotools/filter/expression/MultiplyBuilder.java
brewer/src/main/java/org/geotools/filter/expression/NilBuilder.java
brewer/src/main/java/org/geotools/filter/expression/PropertyNameBuilder.java
brewer/src/main/java/org/geotools/filter/expression/SubtractBuilder.java
brewer/src/main/java/org/geotools/filter/FilterBuilder.java
brewer/src/main/java/org/geotools/filter/IdBuilder.java
brewer/src/main/java/org/geotools/filter/SubFilterBuilder.java
brewer/src/test/java/org/geotools/filter/expression/ExpressionBuilderTest.java
brewer/src/test/java/org/geotools/filter/FilterBuilderTest.java
complex/src/main/java/org/geotools/feature/type/ComplexFeatureTypeFactoryImpl.java
complex/src/main/java/org/geotools/feature/type/ComplexFeatureTypeImpl.java
complex/src/main/java/org/geotools/feature/type/UniqueNameFeatureTypeFactoryImpl.java
complex/src/main/java/org/geotools/feature/type/UniqueNameFeatureTypeImpl.java
complex/src/main/java/org/geotools/filter/expression/FeaturePropertyAccessorFactory.java
complex/src/test/java/org/geotools/data/ComplexTestData.java
complex/src/test/java/org/geotools/filter/expression/FeaturePropertyAccessorTest.java
complex/src/test/java/org/geotools/filter/function/AttributeCountFunctionTest.java
wms/src/main/java/org/geotools/data/ows/CRSEnvelope.java
wms/src/main/java/org/geotools/data/ows/Layer.java
wms/src/main/java/org/geotools/data/ows/LayerDescription.java
wms/src/main/java/org/geotools/data/ows/MultithreadedHttpClient.java
wms/src/main/java/org/geotools/data/ows/StyleImpl.java
wms/src/main/java/org/geotools/data/ows/WMSCapabilities.java
wms/src/main/java/org/geotools/data/ows/WMSRequest.java
wms/src/main/java/org/geotools/map/WMSCoverageReader.java
wms/src/main/java/org/geotools/map/WMSLayer.java
wms/src/main/java/org/geotools/map/WMSMapLayer.java
wms/src/test/java/org/geotools/data/ows/LayerTest.java
wms/src/test/java/org/geotools/data/ows/MockHttpClient.java
wms/src/test/java/org/geotools/data/ows/MockHttpResponse.java
wms/src/test/java/org/geotools/data/ows/MultithreadedHttpClientTest.java
wms/src/test/java/org/geotools/map/WMSCoverageReaderTest.java
wms/src/test/java/org/geotools/map/WMSLayerTest.java
wmts/src/main/java/org/geotools/map/WMTSCoverageReader.java
wmts/src/main/java/org/geotools/map/WMTSMapLayer.java
wmts/src/test/java/org/geotools/map/WMTSCoverageReaderTest.java
wmts/src/test/java/org/geotools/map/WMTSMapLayerTest.java
xsd/xsd-core/src/main/java/org/geotools/xml/AbstractComplexBinding.java
xsd/xsd-core/src/main/java/org/geotools/xml/AbstractComplexEMFBinding.java
xsd/xsd-core/src/main/java/org/geotools/xml/AbstractSimpleBinding.java
xsd/xsd-core/src/main/java/org/geotools/xml/AttributeInstance.java
xsd/xsd-core/src/main/java/org/geotools/xml/Binding.java
xsd/xsd-core/src/main/java/org/geotools/xml/BindingFactory.java
xsd/xsd-core/src/main/java/org/geotools/xml/BindingWalkerFactory.java
xsd/xsd-core/src/main/java/org/geotools/xml/ComplexBinding.java
xsd/xsd-core/src/main/java/org/geotools/xml/ComplexEMFBinding.java
xsd/xsd-core/src/main/java/org/geotools/xml/Configuration.java
xsd/xsd-core/src/main/java/org/geotools/xml/DOMParser.java
xsd/xsd-core/src/main/java/org/geotools/xml/ElementInstance.java
xsd/xsd-core/src/main/java/org/geotools/xml/EMFUtils.java
xsd/xsd-core/src/main/java/org/geotools/xml/Encoder.java
xsd/xsd-core/src/main/java/org/geotools/xml/EncoderConfiguration.java
xsd/xsd-core/src/main/java/org/geotools/xml/EncoderDelegate.java
xsd/xsd-core/src/main/java/org/geotools/xml/EnumSimpleBinding.java
xsd/xsd-core/src/main/java/org/geotools/xml/InstanceComponent.java
xsd/xsd-core/src/main/java/org/geotools/xml/Node.java
xsd/xsd-core/src/main/java/org/geotools/xml/OptionalComponentParameter.java
xsd/xsd-core/src/main/java/org/geotools/xml/Parser.java
xsd/xsd-core/src/main/java/org/geotools/xml/ParserDelegate.java
xsd/xsd-core/src/main/java/org/geotools/xml/ParserDelegate2.java
xsd/xsd-core/src/main/java/org/geotools/xml/ParserNamespaceSupport.java
xsd/xsd-core/src/main/java/org/geotools/xml/PropertyExtractor.java
xsd/xsd-core/src/main/java/org/geotools/xml/PullParser.java
xsd/xsd-core/src/main/java/org/geotools/xml/SchemaIndex.java
xsd/xsd-core/src/main/java/org/geotools/xml/SchemaLocationResolver.java
xsd/xsd-core/src/main/java/org/geotools/xml/SchemaLocator.java
xsd/xsd-core/src/main/java/org/geotools/xml/Schemas.java
xsd/xsd-core/src/main/java/org/geotools/xml/SimpleBinding.java
xsd/xsd-core/src/main/java/org/geotools/xml/SimpleContentComplexEMFBinding.java
xsd/xsd-core/src/main/java/org/geotools/xml/StreamingParser.java
xsd/xsd-core/src/main/java/org/geotools/xml/Text.java
xsd/xsd-core/src/main/java/org/geotools/xml/TextInstance.java
xsd/xsd-core/src/main/java/org/geotools/xml/XML.java
xsd/xsd-core/src/main/java/org/geotools/xml/XMLConfiguration.java
xsd/xsd-core/src/main/java/org/geotools/xml/XmlConverterFactory.java
xsd/xsd-core/src/main/java/org/geotools/xml/XSD.java
xsd/xsd-core/src/main/java/org/geotools/xml/XSDParserDelegate.java
xsd/xsd-core/src/stub/java/org/geotools/resources/Utilities.java
xsd/xsd-core/src/stub/java/org/geotools/util/Converters.java
xsd/xsd-core/src/stub/java/org/geotools/util/logging/Logging.java
xsd/xsd-core/src/test/java/org/geotools/xml/DateConversionTimezoneTest.java
xsd/xsd-core/src/test/java/org/geotools/xml/EncoderTest.java
xsd/xsd-core/src/test/java/org/geotools/xml/FacetTest.java
xsd/xsd-core/src/test/java/org/geotools/xml/ParserNamespaceSupportTest.java
xsd/xsd-core/src/test/java/org/geotools/xml/ParserTest.java
xsd/xsd-core/src/test/java/org/geotools/xml/PullParserTest.java
xsd/xsd-core/src/test/java/org/geotools/xml/SchemasTest.java
xsd/xsd-filter/src/main/java/org/geotools/filter/FilterParsingUtils.java
xsd/xsd-gml3/src/test/java/org/geotools/referencing/crs/EPSGCRSAuthorityFactory.java
xsd/xsd-ows/src/main/java/org/geotools/ows/OWS.java
xsd/xsd-ows/src/main/java/org/geotools/ows/OWSConfiguration.java
xsd/xsd-ows/src/test/java/org/geotools/ows/OWSTestSupport.java
epsg-extension/src/main/java/org/geotools/referencing/factory/epsg/EsriExtension.java
epsg-extension/src/main/java/org/geotools/referencing/factory/epsg/UnnamedExtension.java
epsg-extension/src/test/java/org/geotools/referencing/factory/epsg/EpsgFallbackTest.java
epsg-extension/src/test/java/org/geotools/referencing/factory/epsg/EsriExtensionTest.java
epsg-extension/src/test/java/org/geotools/referencing/factory/epsg/UnnamedExtensionTest.java
epsg-hsql/src/main/java/org/geotools/referencing/factory/epsg/DatabaseCreationScript.java
epsg-hsql/src/main/java/org/geotools/referencing/factory/epsg/FactoryUsingHSQL.java
epsg-hsql/src/main/java/org/geotools/referencing/factory/epsg/HsqlDialectEpsgFactory.java
epsg-hsql/src/main/java/org/geotools/referencing/factory/epsg/HsqlDialectEpsgMediator.java
epsg-hsql/src/main/java/org/geotools/referencing/factory/epsg/HsqlEpsgDatabase.java
epsg-hsql/src/main/java/org/geotools/referencing/factory/epsg/SqlScriptReader.java
epsg-hsql/src/main/java/org/geotools/referencing/factory/epsg/ThreadedHsqlEpsgFactory.java
epsg-hsql/src/test/java/org/geotools/referencing/CrsCreationDeadlockTest.java
epsg-hsql/src/test/java/org/geotools/referencing/EsriLookupTest.java
epsg-hsql/src/test/java/org/geotools/referencing/factory/epsg/DefaultFactoryTest.java
epsg-hsql/src/test/java/org/geotools/referencing/factory/epsg/DialectEpsgMediatorTest.java
epsg-hsql/src/test/java/org/geotools/referencing/factory/epsg/HsqlDialectEpsgFactoryTest.java
epsg-hsql/src/test/java/org/geotools/referencing/factory/epsg/HsqlDialectEpsgMediatorConnectionLeakTest.java
epsg-hsql/src/test/java/org/geotools/referencing/factory/epsg/HsqlDialectEpsgMediatorStressTest.java
epsg-hsql/src/test/java/org/geotools/referencing/factory/epsg/HsqlDialectEpsgMediatorTest.java
epsg-hsql/src/test/java/org/geotools/referencing/factory/epsg/LongitudeFirstFactoryOverrideTest.java
epsg-hsql/src/test/java/org/geotools/referencing/factory/epsg/OperationFactoryTest.java
epsg-hsql/src/test/java/org/geotools/referencing/factory/epsg/PropertyDumper.java
epsg-hsql/src/test/java/org/geotools/referencing/factory/epsg/ThreadedHsqlEpsgFactoryTest.java
epsg-hsql/src/test/java/org/geotools/referencing/factory/epsg/ThreadedTransformTest.java
epsg-hsql/src/test/java/org/geotools/referencing/factory/OrderedAxisAuthorityFactoryTest.java
epsg-hsql/src/test/java/org/geotools/referencing/factory/URN_EPSG_Test.java
epsg-hsql/src/test/java/org/geotools/referencing/HSQLCRSTest.java
epsg-postgresql/src/main/java/org/geotools/referencing/factory/epsg/ThreadedPostgreSQLEpsgFactory.java
epsg-wkt/src/main/java/org/geotools/referencing/crs/EPSGCRSAuthorityFactory.java
epsg-wkt/src/test/java/org/geotools/referencing/crs/CRSTest.java
epsg-wkt/src/test/java/org/geotools/referencing/crs/EPSGTest.java
epsg-wkt/src/test/java/org/geotools/referencing/crs/URNEPSGTest.java
jdbc/jdbc-oracle/src/main/java/org/geotools/filter/function/FilterFunction_sdonn.java
referencing3D/src/main/java/org/geotools/referencing/operation/transform/EarthGravitationalModel.java
referencing3D/src/main/java/org/geotools/referencing/operation/transform/VerticalTransform.java
referencing3D/src/test/java/org/geotools/referencing/operation/transform/EarthGravitationalModelTest.java
shapefile/src/test/java/org/geotools/data/directory/DataStoreCacheTest.java
shapefile/src/test/java/org/geotools/data/directory/DirectoryDataStoreFactoryTest.java
shapefile/src/test/java/org/geotools/data/directory/DirectoryDataStoreTest.java
shapefile/src/test/java/org/geotools/data/directory/DirectoryTestSupport.java
svg/src/main/java/org/geotools/renderer/style/RenderableSVG.java
svg/src/main/java/org/geotools/renderer/style/RenderableSVGCache.java
svg/src/main/java/org/geotools/renderer/style/SVGGraphicFactory.java
svg/src/main/java/org/geotools/renderer/style/SVGMarkFactory.java
svg/src/test/java/org/geotools/renderer/lite/DrawSVGTest.java
svg/src/test/java/org/geotools/renderer/lite/DrawTest.java
svg/src/test/java/org/geotools/renderer/lite/MetaBufferOrderTest.java
svg/src/test/java/org/geotools/renderer/style/SVGFactoryFinderTest.java
svg/src/test/java/org/geotools/renderer/style/SVGGraphicFactoryTest.java
svg/src/test/java/org/geotools/renderer/style/SVGMarkFactoryTest.java
app-schema/webservice/src/main/java/org/geotools/data/ows/FeatureSetDescription.java
app-schema/webservice/src/main/java/org/geotools/data/ows/WSCapabilities.java
app-schema/webservice/src/main/java/org/geotools/filter/AsXpathFunctionExpression.java
dxf/src/main/java/org/geotools/data/GeometryType.java
dxf/src/main/java/org/geotools/data/GeometryTypeMap.java
epsg-oracle/src/main/java/org/geotools/referencing/factory/epsg/FactoryUsingOracleSQL.java
epsg-oracle/src/main/java/org/geotools/referencing/factory/epsg/OracleDialectEpsgFactory.java
epsg-oracle/src/main/java/org/geotools/referencing/factory/epsg/OracleDialectEpsgMediator.java
epsg-oracle/src/main/java/org/geotools/referencing/factory/epsg/ThreadedOracleEpsgFactory.java
epsg-oracle/src/test/java/org/geotools/referencing/CRSOnlineTest.java
epsg-oracle/src/test/java/org/geotools/referencing/factory/epsg/DefaultFactoryOnlineTest.java
epsg-oracle/src/test/java/org/geotools/referencing/factory/epsg/OperationFactoryOnlineTest.java
epsg-oracle/src/test/java/org/geotools/referencing/factory/epsg/OracleDialectEpsgFactoryOnlineTest.java
epsg-oracle/src/test/java/org/geotools/referencing/factory/epsg/OracleDialectEpsgMediatorConnectionLeakOnlineTest.java
epsg-oracle/src/test/java/org/geotools/referencing/factory/epsg/OracleDialectEpsgMediatorOnlineStressTest.java
epsg-oracle/src/test/java/org/geotools/referencing/factory/epsg/OracleDialectEpsgMediatorOnlineTest.java
epsg-oracle/src/test/java/org/geotools/referencing/factory/epsg/OracleDialectEpsgMediatorStarvationOnlineStressTest.java
epsg-oracle/src/test/java/org/geotools/referencing/factory/OrderedAxisAuthorityFactoryOnlineTest.java
epsg-oracle/src/test/java/org/geotools/referencing/factory/URN_EPSG_OnlineTest.java
geometry/src/test/java/org/geotools/geometry/CompatibleCRSBuilderTest.java
jts-wrapper/src/main/java/org/geotools/geometry/jts/GeometryUtils.java
jts-wrapper/src/main/java/org/geotools/geometry/jts/JTSGeometry.java
jts-wrapper/src/main/java/org/geotools/geometry/jts/JTSUtils.java
process-raster/src/main/java/org/geotools/util/CRSConverterFactory.java
process-raster/src/main/java/org/geotools/util/InterpolationConverterFactory.java
process-raster/src/test/java/org/geotools/util/CRSConverterFactortTest.java
process-raster/src/test/java/org/geotools/util/InterpolationConverterFactoryTest.java
wfs-ng/src/main/java/org/geotools/data/store/WFSContentComplexFeatureCollection.java
wfs-ng/src/main/java/org/geotools/feature/collection/ComplexFeatureIteratorImpl.java
wps/src/main/java/org/geotools/data/ows/AbstractWPS.java
wps/src/main/java/org/geotools/data/ows/AbstractWPSGetCapabilitiesResponse.java
wps/src/main/java/org/geotools/data/ows/package-info.java
wps/src/main/java/org/geotools/data/ows/ProcessDescription.java
wps/src/main/java/org/geotools/data/ows/WPSCapabilities.java
wps/src/main/java/org/geotools/data/ows/WPSRequest.java
_______________________________________________
Geoserver-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geoserver-devel

Reply via email to