Maven 2.x Linkcheck Plugin

Upgrade to Doxia 1.1

Details

  • Type: Task Task
  • Status: Open Open
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: 1.0
  • Fix Version/s: None
  • Labels:
    None
  • Number of attachments :
    1

Description

There are a couple of Doxia 1.0 specific quirks in the report-generation code, in particular several calls to sink.rawText(), which makes the plugin unusable with different sinks (cf the pdf plugin).

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Lukas Theussl added a comment -

Attached patch would break compatibility with maven 2.0.9, I couldn't get it to work with the documented procedure (using maven 2.2.1), see the commented remarks and links. IMO we could just require maven 2.1 as prereq.

Show
Lukas Theussl added a comment - Attached patch would break compatibility with maven 2.0.9, I couldn't get it to work with the documented procedure (using maven 2.2.1), see the commented remarks and links. IMO we could just require maven 2.1 as prereq.
Hide
Permalink
Vincent Siveton added a comment -

agree

Show
Vincent Siveton added a comment - agree
Hide
Permalink
Dennis Lundberg added a comment -

Do we want to apply this patch before or after we release version 1.1?

Show
Dennis Lundberg added a comment - Do we want to apply this patch before or after we release version 1.1?
Hide
Permalink
Lukas Theussl added a comment -

Not necessary for this release IMO, unless you take the time (which I haven't) to test and confirm that it works with the pdf plugin and different maven versions. I'd say we apply the patch right after the release and have the snapshot tested for a while.

Show
Lukas Theussl added a comment - Not necessary for this release IMO, unless you take the time (which I haven't) to test and confirm that it works with the pdf plugin and different maven versions. I'd say we apply the patch right after the release and have the snapshot tested for a while.
Hide
Permalink
Dennis Lundberg added a comment -

That is my preference as well.
I'll go ahead and start the release of 1.1 without this issue.

Show
Dennis Lundberg added a comment - That is my preference as well. I'll go ahead and start the release of 1.1 without this issue.

People

  • Assignee:
    Unassigned
    Reporter:
    Lukas Theussl
Vote (0)
Watch (0)

Dates

  • Created:
    Updated: