You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I don't remember for sure, but I might have done this because regardless of platform, we still want to manage all native classifiers for downstream consumers to depend upon should they so choose. (This seems like it should be rare to me, but what do I know, maybe it's helpful for debugging sometimes to depend on the natives-linux-amd64 artifact from a Windows box?)
We need to make a decision about which approach is best: "all platforms always" (our current approach for jogamp natives) or "current platform only" (our current approach for javacpp natives).
The text was updated successfully, but these errors were encountered:
Given the fact that Gradle does not properly resolve these profile-triggered property values when pom-scijava is used as a platform, I think "all platforms always" is probably a better bet, since it is more explicit and reduces reliance on profile activation.
For jogamp artifacts, we currently manually articulate all platforms for jogamp components in the
<dependencyManagement>
section—e.g.:Even though we have nice properties defined for use downstream:
I don't remember for sure, but I might have done this because regardless of platform, we still want to manage all native classifiers for downstream consumers to depend upon should they so choose. (This seems like it should be rare to me, but what do I know, maybe it's helpful for debugging sometimes to depend on the
natives-linux-amd64
artifact from a Windows box?)But regardless of whether this pattern is considered good or not, we are currently inconsistent with how we handle the javacpp component native classifiers, which are each only managed for the current platform.
We need to make a decision about which approach is best: "all platforms always" (our current approach for jogamp natives) or "current platform only" (our current approach for javacpp natives).
The text was updated successfully, but these errors were encountered: