You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Hudson (Jira)" <ji...@apache.org> on 2022/04/24 08:10:00 UTC

[jira] [Commented] (MNG-7386) ModelMerger$MergingList is not serializable

    [ https://issues.apache.org/jira/browse/MNG-7386?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17527092#comment-17527092 ] 

Hudson commented on MNG-7386:
-----------------------------

Build failed in Jenkins: Maven » Maven TLP » maven » MNG-7457 #2

See https://ci-maven.apache.org/job/Maven/job/maven-box/job/maven/job/MNG-7457/2/

> ModelMerger$MergingList is not serializable
> -------------------------------------------
>
>                 Key: MNG-7386
>                 URL: https://issues.apache.org/jira/browse/MNG-7386
>             Project: Maven
>          Issue Type: Bug
>          Components: Core
>    Affects Versions: 3.6.3, 3.8.4
>            Reporter: Kostiantyn Liutovych
>            Assignee: Guillaume Nodet
>            Priority: Minor
>             Fix For: 3.8.5, 4.0.0-alpha-1, 4.0.0
>
>
> Hello!
> [Spotless Maven plugin|https://github.com/diffplug/spotless] serializes {{org.apache.maven.model.Plugin}} instances to fingerprint plugin's configuration. Serialization fails for Maven 3.6.3 with:
> {code}
> java.io.NotSerializableException: org.apache.maven.model.merge.ModelMerger$MergingList
> {code}
> when plugin configuration comes from {{pluginManagement}}. Class {{org.apache.maven.model.Plugin}} implements {{java.io.Serializable}}, however nested class {{org.apache.maven.model.merge.ModelMerger$MergingList}} does not.
> Would it be possible to make {{MergingList}} serializable or make {{Plugin#dependencies}} field always hold a serializable collection?
> Related issue for the Spotless Maven plugin: https://github.com/diffplug/spotless/issues/1073 and PR with a workaround https://github.com/diffplug/spotless/pull/1074.
> Thank you!



--
This message was sent by Atlassian Jira
(v8.20.7#820007)