Skip to content

Conversation

jroper
Copy link
Member

@jroper jroper commented Aug 26, 2024

This follows on from #7470, to include all sources, not just managed and unmanaged, in the source jar, along with all resources (previously only unmanaged resources were included).

This means that if, for whatever crazy reason, someone does modify the sources task to include additional sources or filter out sources, rather than using the managed or unmanaged source mechanisms, their changes will still be reflected in the source jar. It ensures consistency with both compile and doc, which both also depend on sources rather than manually gathering unmanagedSources and managedSources themselves.

This follows on from sbt#7470, to include all sources, not just managed and
unmanaged, in the source jar, along with all resources (previously only
unmanaged resources were included).

This means that if, for whatever crazy reason, someone does modify the
`sources` task to include additional sources or filter out sources, rather than
using the managed or unmanaged source mechanisms, their changes will still be
reflected in the source jar.
Copy link
Member

@adpi2 adpi2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@eed3si9n eed3si9n left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@eed3si9n eed3si9n merged commit ee7a9ae into sbt:1.10.x Aug 26, 2024
10 checks passed
@jroper jroper deleted the package-src branch August 26, 2024 22:32
raboof added a commit to raboof/sbt-apache-sonatype that referenced this pull request Sep 22, 2024
Fixes mdedetrich#39

In sbt 1.10.2 the managed resources are automatically included in the
packageSrc (sbt/sbt#7630), and doing it again
explicitly causes errors.
mdedetrich pushed a commit to mdedetrich/sbt-apache-sonatype that referenced this pull request Sep 23, 2024
Fixes #39

In sbt 1.10.2 the managed resources are automatically included in the
packageSrc (sbt/sbt#7630), and doing it again
explicitly causes errors.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants