| Summary: | maven build does not trigger autosave | ||
|---|---|---|---|
| Product: | z_Archived | Reporter: | gverspaij |
| Component: | m2e | Assignee: | Project Inbox <m2e.core-inbox> |
| Status: | CLOSED INVALID | QA Contact: | |
| Severity: | normal | ||
| Priority: | P3 | CC: | amj87.iitr, igor, Olivier_Thomann |
| Version: | unspecified | ||
| Target Milestone: | --- | ||
| Hardware: | All | ||
| OS: | All | ||
| Whiteboard: | |||
|
Description
gverspaij
I don't think this is an Eclipse bug. Maven should read the preference page and trigger a save when the user triggers a build. The bug should be reported in the Maven bugzilla. Moving to m2e Assuming the issue is about RunAs->Maven_Install and similar, these are launch configurations and are NOT expected to follow 'Save automatically before build' preference. These launch configurations ARE expected to honour Run/Debug->Launching->Save_required_dirty_editors preference, which does not work either. MavenLaunchDelegate extends JavaLaunchDelegate and re-uses JDT standard save-on-launch logic, which is implemented in LaunchConfigurationDelegate.saveBeforeLaunch and uses AbstractJavaLaunchConfigurationDelegate.getBuildOrder to determine what files should be saved before launch. MavenLaunchDelegate needs to override #getBuildOrder to return affected workspace projects for launch configurations associated with a workspace project and return null for launch configurations that build projects outside of workspace. Or, even better, override #saveBeforeLaunch and handle the case when launching pom.xml nested inside a workspace project. m2e development team has no immediate plans to work on this unless somebody provides a quality patch with adequate test coverage. closing old/stale bugreports |