[MAGNOLIA-5901] Remove dependency on javax.mail Created: 29/Aug/14 Updated: 20/Jul/15 Resolved: 17/Jul/15 |
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | build |
| Affects Version/s: | 5.0 |
| Fix Version/s: | 5.3.10 |
| Type: | Bug | Priority: | Major |
| Reporter: | Roman Kovařík | Assignee: | Evzen Fochr |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | quickwin, support | ||
| Remaining Estimate: | 1h | ||
| Time Spent: | 0.5h | ||
| 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 |
<dependency>
<groupId>javax.mail</groupId>
<artifactId>mail</artifactId>
<version>1.4.1</version>
</dependency>
is probably a remnant from splitting main project and mail module for Magnolia 5 and overrides dependency specified by edit: note that this only affects dependency management. magnolia_main itself did NOT depend on javax.mail (at least not 5.4), but by using scope:import of main, one could easily end up with 1.4.1 instead of the version they need. |
| Comments |
| Comment by Evzen Fochr [ 02/Jul/15 ] |
|
mail, main and ce-bundle builds without error, so lets see what breaks later. |
| Comment by Magnolia International [ 17/Jul/15 ] |
|
The fix on master reintroduced dependency (management) to commons-lang:2.4 which we explicitly removed in 5.4. Check your diffs before merging/committing/pushing ! |
| Comment by Jan Haderka [ 17/Jul/15 ] |
|
This ticket is for 5.3.10, there's linked clone for 5.4.1 and that should not be integrated yet according to jira. |