Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add support for sbt-assembly #152

Merged
merged 2 commits into from
Jun 19, 2020
Merged

add support for sbt-assembly #152

merged 2 commits into from
Jun 19, 2020

Conversation

somdoron
Copy link
Contributor

No description provided.

scalaVersion := "2.12.7"

enablePlugins(ReproducibleBuildsPlugin)
enablePlugins(AssemblyPlugin)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Apparently I have to enable the Assembly for the reproducible hook to work, although the AssemblyPlugin is an auto plugin?

Copy link
Owner

Choose a reason for hiding this comment

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

That is indeed pretty weird.

Without this explicit call the assembly plugin was enabled, but indeed assembly called the original task instead of the 'reproducible' one.

Putting a $ pause in the scripted test right before the failing check, started an interactive sbt shell and did inspect assembly. Sure enough this confirmed assembly was the original assembly from AssemblyPlugin.scala:57.

I wonder if it's some sort of ordering or scoping thing?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I can confirm that changing the order here matters, it won't work if the order was:

enablePlugins(AssemblyPlugin)
enablePlugins(ReproducibleBuildsPlugin)

@somdoron somdoron changed the title WIP: add support for sbt-assembly add support for sbt-assembly Jun 17, 2020
@somdoron somdoron marked this pull request as ready for review June 17, 2020 17:57
@somdoron
Copy link
Contributor Author

@raboof aside from the enablePlugin issue this is ready on my side. Let me know if you have any comments.

@raboof
Copy link
Owner

raboof commented Jun 18, 2020

@somdoron what do you think about somdoron#1 ?

@somdoron
Copy link
Contributor Author

Amazing, thanks

Copy link
Owner

@raboof raboof left a comment

Choose a reason for hiding this comment

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

Nice contribution, thanks!

@raboof raboof merged commit 2217a04 into raboof:master Jun 19, 2020
@raboof
Copy link
Owner

raboof commented Jun 19, 2020

refs #127

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.

2 participants