Skip to content
This repository has been archived by the owner on Nov 7, 2020. It is now read-only.

import 1.4.9 form https://github.com/x-stream/xstream/tree/XSTREAM_1_4_9 #6

Closed
wants to merge 2 commits into from
Closed

import 1.4.9 form https://github.com/x-stream/xstream/tree/XSTREAM_1_4_9 #6

wants to merge 2 commits into from

Conversation

ndeloof
Copy link
Contributor

@ndeloof ndeloof commented Sep 29, 2016 •
edited
Loading

@reviewbybees

@oleg-nenashev
Copy link
Member

If you do not mind, I'm going to cut the release without this fix, and then we can start reviewing it

@ndeloof
Copy link
Contributor Author

ndeloof commented Sep 30, 2016

@oleg-nenashev as you prefer. I'm more worried we accept PR about changes that are already fixed upstream (#4 as a trivial sample)
In the meantime I've proposed to backport some changes made to XStream for jenkins usage : x-stream/xstream#70. Codebase has minor divergence, I'd like we don't rely on a fork if possible

@ndeloof
Copy link
Contributor Author

ndeloof commented Oct 5, 2016

closing, will be better to address this from https://github.com/jenkinsci/xstream-fork

@ndeloof ndeloof closed this Oct 5, 2016
@batmat batmat mentioned this pull request Mar 2, 2017
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ndeloof @oleg-nenashev