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

Fewer modules are built with GIB than without it when -rf is used #247

Closed
famod opened this issue Sep 2, 2020 · 4 comments · Fixed by #288
Closed

Fewer modules are built with GIB than without it when -rf is used #247

famod opened this issue Sep 2, 2020 · 4 comments · Fixed by #288
Labels

Comments

@famod
Copy link
Member

famod commented Sep 2, 2020

Observed by @maschoene:

  • 4 modules A, B, C, D
  • D needs B & C
  • B & C need A
  • A and B are changed
  • -rf :B only builds B & D, but B, C and D are expected

Without GIB, B, C and D are built.

@famod famod added the bug label Sep 2, 2020
@famod famod changed the title Fewer modules are built with GIB than without it when-rf is used Fewer modules are built with GIB than without it when -rf is used Sep 2, 2020
@famod
Copy link
Member Author

famod commented Sep 2, 2020

I need to check this but I guess no downstream is calculated for A because it is "deselected" via -rf.

@gastaldi I think this will also affect the -pl single-project use case in some of the Quarkus CI jobs since this is also "deselecting" modules (all except the selected ones).

@famod famod self-assigned this Sep 6, 2020
@famod
Copy link
Member Author

famod commented Sep 6, 2020

@maschoene

Hm, could not reproduce it:

[INFO] Scanning for projects...
[INFO] gitflow-incremental-builder 3.11.3 starting...
[INFO] ------------------------------------------------------------------------
[INFO] Changed Artifacts:
[INFO]
[INFO] bar.A (but deselected)
[INFO] bar.B
[INFO]
[INFO] gitflow-incremental-builder exiting...
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Build Order:
[INFO]
[INFO] bar.B                                                              [jar]
[INFO] bar.C                                                              [jar]
[INFO] bar.D                                                              [jar]

I'll have a look at the original scenario (our internal project). Something must be different.

@gastaldi Good news, we just a need a new property that disables all "convenience features" for "selected projects" to make -pl work like we discussed.
So for now it seems the two issues do not have a common cause.

@famod
Copy link
Member Author

famod commented Sep 7, 2020

To be clarified after @maschoene has returned from PTO.

FTR: Maven itself (without GIB) does not build C in the mentioned internal project (which is far more complex than my simple reproducer).

@famod famod removed their assignment Sep 13, 2020
@famod famod added this to the release-3.12.1 milestone Dec 16, 2020
@famod
Copy link
Member Author

famod commented Dec 16, 2020

Turns out that this got fixed as a side effect of #288.

@famod famod linked a pull request Dec 16, 2020 that will close this issue
@famod famod closed this as completed Dec 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant