stevedlawrence commented on code in PR #812:
URL: https://github.com/apache/daffodil/pull/812#discussion_r923613630
##########
build.sbt:
##########
@@ -57,6 +57,7 @@ lazy val io = Project("daffodil-io",
file("daffodil-io")).configs(
lazy val runtime1 = Project("daffodil-runtime1",
file("daffodil-runtime1")).configs(IntegrationTest)
.dependsOn(io, lib % "test->test", udf, macroLib
% "compile-internal, test-internal")
.settings(commonSettings, usesMacros)
+ .settings(libraryDependencies ++=
Dependencies.exi)
Review Comment:
Agreed. We could maybe take advantage of optional dependencies for this.
That way runtime1 would depend on all of infoset subprojects, but a subset of
the less common ones would be marked as optional--any API users that want to
use an optional one would need to explicitly add them. This would make it
trivial to change the "core" set of infoset inputters/outputters by just
marking some as required vs optional.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
