[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:
relation
is related to MAGNOLIA-3518 Create a Maven plugin to generate mod... Closed
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.

Generated at Mon Feb 12 04:42:23 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.