[MGNLCAT-25] Missing optional dependency to scheduler Created: 29/Apr/11 Updated: 04/Jul/11 Resolved: 17/May/11 |
|
| Status: | Closed |
| Project: | Magnolia Categorization Module |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 1.1.7 |
| Type: | Bug | Priority: | Neutral |
| Reporter: | Daniel Lipp | Assignee: | Daniel Lipp |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Template: |
|
||||||||
| Acceptance criteria: |
Empty
|
||||||||
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||||||
| Bug DoR: |
[ ]*
Steps to reproduce, expected, and actual results filled
[ ]*
Affected version filled
|
||||||||
| Date of First Response: | |||||||||
| Description |
|
The optional dependency to the scheduler module is only defined in the module descriptor but not in the pom. |
| Comments |
| Comment by Jan Haderka [ 17/May/11 ] |
|
The dependency is optional so it should not be in the pom. Module compilation doesn't depend on it, nor does the bundling. The reason for dependency is only to ensure that optional configuration is bootstrapped properly in case the optional module is installed. See ModuleDependencyBootstrapTask for details. |
| Comment by Jan Haderka [ 17/May/11 ] |
|
looking forward to see this plugin done |
| Comment by Jan Haderka [ 17/May/11 ] |
|
Added comment to both pom and module descriptor explaining what this dependency is about. |
| Comment by Magnolia International [ 04/Jul/11 ] |
|
Just for the record, as an answer to Jan's comment above - optional dependencies should also be in the pom (and marked as such, just like Daniel did), to ensure consistency and to help any tool using the pom (not just the build) figure out dependencies. |