Maven 2.x Eclipse Plugin

manifest property usage is only for ogsi maifests

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 2.5
  • Fix Version/s: 2.5.1
  • Component/s: OSGi, Manifest, WTP support
  • Labels:
    None
  • Environment:
    any
  • Patch Submitted:
    Yes
  • Number of attachments :
    1

Description

the manifest property of the eclipse plugin is only for the osgi writer and not for the wtp manifest, because the wtp manifest is a special case that will not be included in the maven build just in the eclipse classpath. The problem is that the property has a default value and by that deacivates the WTP classpath!

included a patch for the 2.5 release, including some renaming so it won't happen again.

please release a 2.5.1 version with this patch!

  1. Text File
    manifest.patch
    26/Mar/08 5:02 AM
    6 kB
    Richard van Nieuwenhoven

Issue Links

is depended upon by

Bug - A problem which impairs or prevents the functions of the product. MECLIPSE-438 manifest location hard coded to src/main/webapp in 2.5.1

  • Major - Major loss of function.
  • Open - The issue is open and ready for the assignee to start work on it.

Activity

Ascending order - Click to sort in descending order
Arnaud Heritier made changes -
Field Original Value New Value
Assignee Arnaud Heritier [ aheritier ]
Arnaud Heritier made changes -
Fix Version/s 2.5.1 [ 14136 ]
Arnaud Heritier made changes -
Summary maifest property usage is only for ogsi maifests manifest property usage is only for ogsi maifests
Hide
Permalink
Arnaud Heritier added a comment -

Patch applied. thx. Sorry for the inconvenience. I deployed a new SNAPSHOT : 2.5.1-20080326.235643-1

Show
Arnaud Heritier added a comment - Patch applied. thx. Sorry for the inconvenience. I deployed a new SNAPSHOT : 2.5.1-20080326.235643-1
Arnaud Heritier made changes -
Status Open [ 1 ] Closed [ 6 ]
Resolution Fixed [ 1 ]
Hide
Permalink
Richard van Nieuwenhoven added a comment -

just tested and it works, thanks! now to the vote.

Show
Richard van Nieuwenhoven added a comment - just tested and it works, thanks! now to the vote.
Arnaud Heritier made changes -
Link This issue is depended upon by MECLIPSE-438 [ MECLIPSE-438 ]

People

  • Assignee:
    Arnaud Heritier
    Reporter:
    Richard van Nieuwenhoven
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: