Skip to content

[MSHARED-1024] Replace deprecated code - #2

Merged
asfgit merged 1 commit into
masterfrom
MSHARED-1024
Feb 1, 2022
Merged

[MSHARED-1024] Replace deprecated code#2
asfgit merged 1 commit into
masterfrom
MSHARED-1024

Conversation

@michael-o

Copy link
Copy Markdown
Member

This closes #2

asfgit pushed a commit that referenced this pull request Jan 24, 2022
* Stop using the deprecated Codehaus Doxia Sink, use Maven Doxia Sink since the
  Codehaus one extends from Maven code should continue to work.
* Don't use expressions, use XML constructs

This closes #2
@michael-o
michael-o marked this pull request as ready for review January 25, 2022 20:23
@michael-o
michael-o requested a review from hboutemy January 25, 2022 20:23
@asfgit asfgit closed this in 35fc61e Feb 1, 2022
@asfgit
asfgit merged commit 35fc61e into master Feb 1, 2022
@michael-o
michael-o deleted the MSHARED-1024 branch February 1, 2022 21:03
@basil

basil commented Jul 29, 2022

Copy link
Copy Markdown

FYI this breaks binary compatibility with https://github.com/kohsuke/maven-jellydoc-plugin and https://github.com/jenkinsci/maven-stapler-plugin:

jenkins-infra/helpdesk#3049 (comment)

But I suppose we are long past the point where such binary compatibility could reasonably be expected. I will try to update the consumers.

@michael-o

Copy link
Copy Markdown
Member Author

FYI this breaks binary compatibility with https://github.com/kohsuke/maven-jellydoc-plugin and https://github.com/jenkinsci/maven-stapler-plugin:

jenkins-infra/helpdesk#3049 (comment)

But I suppose we are long past the point where such binary compatibility could reasonably be expected. I will try to update the consumers.

That's true, but is a major and we have to remove at some point. Those methods were deprecated for a long time.

@basil

basil commented Jul 29, 2022

Copy link
Copy Markdown

Was this breaking change documented in any release notes?

@michael-o

Copy link
Copy Markdown
Member Author

I need to revert my previous statement because I had another release in memory. Yes, this change was partially wrong and I haven't anticipated people still using 10 years old deprecated interfaces. It should have happened only in a new major version. Although I am still surprised that code even being deprecated and the IDE tells you the successor is still not changed. BTW, same happened with Asciidoctor Maven Plugin. Sorry for the inconvenience.

@michael-o

Copy link
Copy Markdown
Member Author

Would you prefer a patch release with a revert? This code will go away in 4.0.0 anyway.

@basil

basil commented Jul 29, 2022

Copy link
Copy Markdown

Thanks Michael, that would definitely be appreciated as otherwise I need to go obtaining write access to two ancient Maven plugins to modernize them and perform new releases, which I would rather not rush to do.

@michael-o

Copy link
Copy Markdown
Member Author

Thanks Michael, that would definitely be appreciated as otherwise I need to go obtaining write access to two ancient Maven plugins to modernize them and perform new releases, which I would rather not rush to do.

That would only be an interims solution because with Maven Site Plugin 4 they will fail anyway. Please file an issue with MSHARED and link this conversation to it.

@jira-importer

Copy link
Copy Markdown

Resolve #29

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants