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

ModuleCalculator doesn't calculate the modules to build accurately in multi-module maven project #27

Open
samirshaik opened this issue Aug 24, 2019 · 0 comments

Comments

@samirshaik
Copy link

samirshaik commented Aug 24, 2019

The hierarchy of the project is like below:

 /-
     +- Base
        +- Components
           -  cu-alerts-ejb
           -  cu-accttitle-ejb

So you can expect:

  1. module names of cu-*-ejb mentioned in /Base/Components/pom.xml
  2. module name of Component mentioned in /Base/pom.xml

Although change is done only in files inside cu-*-ejb, parent of these modules are also calculated as module to build including all the sub-projects under the parent module. This doesn't appear to be calculated right as we want to build the changed modules and all other modules that consume them as dependencies.

Behavior should have been similar to what would happen with "mvn package -pl :cu-alerts-ejb,cu-accttitle-ejb -amd", but from the reactor output and modules chosen to build it doesn't appear to have done the calculation correctly.

[INFO] ------------------------------------ [INFO] Maven Incremental Module Builder [INFO] Version: 0.2.0 [INFO] ------------------------------------ [INFO] Executing: cmd.exe /X /C "git rev-parse --show-toplevel" [INFO] Working directory: C:\delete\Starlight [INFO] Executing: cmd.exe /X /C "git status --porcelain ." [INFO] Working directory: C:\delete\Starlight [WARNING] Ignoring unrecognized line: ?? .mvn/ [INFO] Changed file: Base/Components/cu-accttitle-ejb/src/main/java/com/mschematic/platform/accttitle/ejb/AcctTitle.java modified [INFO] Changed file: Base/Components/cu-alerts-ejb/src/main/java/com/mschematic/platform/alerts/ejb/AlertLogQueryBuilder.java modified [INFO] Changed Project: com.zenmonics.starlight.connect.base:Base:pom:9.5.0-8.0.0-SNAPSHOT [INFO] Changed Project: com.zenmonics.starlight.connect.base:Components:pom:9.5.0-8.0.0-SNAPSHOT [INFO] Changed Project: com.zenmonics.starlight.connect.base:cu-alerts-ejb:ejb:9.5.0-8.0.0-SNAPSHOT [INFO] Changed Project: com.zenmonics.starlight.connect.base:cu-accttitle-ejb:ejb:9.5.0-8.0.0-SNAPSHOT

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

No branches or pull requests

1 participant