Given GitScm exends GitSCMBackwardCompatibility 
and GitSCMBackwardCompatibility implements Serializable, I though it was 
expecting it to be serializable.

Therefore, I suppose I can't do better 
than 
https://github.com/jenkinsci/git-push-plugin/blob/5c9525da6da7bed8fd70e8a45d62eafd1daad3f1/src/main/java/io/jenkins/plugins/git_push/GitPushStep.java#L71-L71
 
?

On Tuesday, July 18, 2023 at 2:25:49 PM UTC+2 Mark Waite wrote:

> On Sunday, July 16, 2023 at 4:43:18 AM UTC-6 Reda wrote:
>
> Hi everyone,
>
> I'd like to understand how am I supposed to handle *GitSCM* in a 
> *StepExecution*.
>
> Here is the interesting portion of the code : 
> https://github.com/jenkinsci/git-push-plugin/blob/5c9525da6da7bed8fd70e8a45d62eafd1daad3f1/src/main/java/io/jenkins/plugins/git_push/GitPushStep.java#L71-L71
>
> If I remove the *transient* modifier from the *GitScm *attribute, I will 
> get *java.io.NotSerializableException: hudson.util.DescribableList*.
>
> Is GitPushStep already doing that correctly? If yes, what are the 
> consequences of this lack of serialization?
>
>
> GitSCM is not serializable.  It is correct to not attempt to serialize 
> it.  Can you explain why you believe that you need to serialize it? 
>

-- 
You received this message because you are subscribed to the Google Groups 
"Jenkins Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/jenkinsci-dev/24c4b9e8-a464-4097-91a4-057bccccdf7bn%40googlegroups.com.

Reply via email to