Ship it with older releases #6

Closed
PVince81 opened this Issue Dec 22, 2016 · 10 comments

Projects

None yet

2 participants

@PVince81
Contributor
PVince81 commented Dec 22, 2016 edited
  • test if it works with 9.0
  • test if it works with 9.1
  • ship with 9.0
  • ship with 9.1

@PhilippSchaffrath is it ready for testing ? Can you confirm that the code is compatible with older versions ? If not we might need to do backport branches with adjusted code.

@PVince81 PVince81 added this to the 9.1.4 milestone Dec 22, 2016
@phisch phisch was assigned by PVince81 Dec 22, 2016
@phisch
Contributor
phisch commented Dec 22, 2016 edited

I'll test it.

@PVince81: i would like to get #5 in, can you review it?

@PVince81
Contributor

It's merged.

Any update on the backports ?

@phisch
Contributor
phisch commented Jan 19, 2017

Works with 9.1, not with 9.0 though, creating branch for 9.0.

@PVince81
Contributor

Is there a PR for the 9.1 backport ?

Let me know when backports are done so I can update the release scripts to include the app in the next release.

@phisch
Contributor
phisch commented Jan 19, 2017

Just managed to get it working with 9.0.

9.1+ works with master, creating identical branches for 9.1, 9.2... seems bad. How do we handle this usually @PVince81?

Branches incomming.

@PVince81
Contributor

Hmm, indeed. We never had this kind of situation before.

If you tested it with 9.1 then let's just create a stable9.1 and stable9 branches from master then

@phisch
Contributor
phisch commented Jan 19, 2017

Branch stable9 works with 9.0, stable9.1 works with everything > 9.1. There is a good chance that stable9 works with older versions too in case we need this.

@PVince81
Contributor

Any update on the backports ? Else move to 9.1.5

@PVince81 PVince81 assigned PVince81 and unassigned phisch Jan 24, 2017
@PVince81
Contributor

As discussed, I'll update the release scripts.
To be tested in the tarball release.

@PVince81
Contributor

It's in the tarballs now, closing

@PVince81 PVince81 closed this Jan 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment