stevedlawrence opened a new issue, #212:
URL: https://github.com/apache/daffodil-sbt/issues/212
The `packageDaffodilBin / javaOptions` setting current defaults to:
```scala
packageDaffodilBin / javaOptions := Seq(
...
),
```
https://github.com/apache/daffodil-sbt/blob/main/src/main/scala/org/apache/daffodil/DaffodilPlugin.scala#L524-L528
By using `:=` it means we ignore the unscoped `javaOptions` setting, which
is not standard SBT behavior. We should change this to use `++=` so that users
can put settings in their build.sbt file like this:
```scala
javaOptions ++= Seq(...)
```
And have it apply to all tasks that fork, including tests (if `fork :=
true`) and packageDaffodilBin. Some daffodil-sbt plugin users might prefer this
to avoid duplication, since often times all things that fork require similar
javaOptions, like additional memory, and so it's easier to just specific it
once as javaOptions.
--
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]
