Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make Wonder go to 11+6 #944

Closed
wants to merge 6 commits into from
Closed

Make Wonder go to 11+6 #944

wants to merge 6 commits into from

Conversation

hugithordarson
Copy link
Contributor

@hugithordarson hugithordarson commented Mar 26, 2021

I've been running this without problems for a couple of days now, a couple of eyes on it wouldn't hurt before it's merged.

I did not upgrade the JDK version in the "release.yml" github action to keep it compatible with 7.2. Or is that perhaps not neccessary? Not sure of the mechanics of how action configuration is handled when building older versions (does the action definiton file/jdk version come from the branch being built?)

@paulhoadley
Copy link
Contributor

Nice work @hugith.

a couple of eyes on it wouldn't hurt before it's merged.

As potentially the only remaining user of ERProfiling.framework, I probably need to volunteer for this. The first problem is that I don't even have Java 11 set up in development yet. Two alternatives:

  1. give me a little time to get set up and check out the ERProfiling.framework change; or
  2. ram it through now and I'll test it later since apparently no one else uses it anyway.

What do you think?

@hugithordarson
Copy link
Contributor Author

hugithordarson commented Mar 27, 2021

@paulhoadley As far as I'm concerned there's no big hurry, so go ahead and test at your leisure :).

Before merging I'm also wondering about the consequences for the WOCommunity Jenkins builds. Jenkins probably needs to be configured with a Java 11 JDK so it can build this thing. Also; the Wonder7 job there is building from master, meaning downloads for the Ant People would silently move to Java 11. We might want to have separate stable (7.2 on java 8) and master (java 11) builds there? @maiksd is probably the right person to provide input on that topic.

@maiksd
Copy link
Contributor

maiksd commented Mar 27, 2021

Before merging I'm also wondering about the consequences for the WOCommunity Jenkins builds. Jenkins probably needs to be configured with a Java 11 JDK so it can build this thing. Also; the Wonder7 job there is building from master, meaning downloads for the Ant People would silently move to Java 11. We might want to have separate stable (7.2 on java 8) and master (java 11) builds there? @maiksd is probably the right person to provide input on that topic.

Jenkins already has 8 and 11 installed and configured, so that's just a matter of job configuration. But yes, the Wonder7 job uses the master branch, we should separate that before moving on. But I feel this warrants a proper discussion on the mailing list first?

@hugithordarson
Copy link
Contributor Author

Jenkins already has 8 and 11 installed and configured, so that's just a matter of job configuration.

Awesome!

But yes, the Wonder7 job uses the master branch, we should separate that before moving on. But I feel this warrants a proper discussion on the mailing list first?

OK, should I start that discussion or should we first discuss what the proposal is?

@paulhoadley
Copy link
Contributor

Jenkins already has 8 and 11 installed and configured, so that's just a matter of job configuration. But yes, the Wonder7 job uses the master branch, we should separate that before moving on.

Here's another idea: we could start abandoning Jenkins in favour of GitHub Actions—or is the issue that we still need to support legacy Ant-based builds for the framework build products? My motivation here is just to try and reduce the number of things @maiksd needs to maintain, and Jenkins would be a nice one to ditch.

Yet another idea would be to draw an even brighter line in the sand. Wonder 8.0 would be:

  • Java 11
  • Maven only

Jenkins could stick around for a while to build any Wonder 7 branches, and at some point fade away. Thoughts? Too harsh?

@hugithordarson
Copy link
Contributor Author

hugithordarson commented Mar 28, 2021

@paulhoadley You probably know where I stand on the Maven only idea, I think it would be fantastic and would make life easier. I believe most of us who currently work on Wonder use maven, and Ant users shouldn't be affected since the built frameworks will work fine with Ant. Can't hurt to propose it.

I'm up for GitHub Actions too if they do everything we need. The thing I like most about Jenkins is basically @maiksd :)

@maiksd
Copy link
Contributor

maiksd commented Mar 28, 2021

That Jenkins is one of three that I'm maintaining. I wouldn't mind it going away, but that wouldn't actually free me from having it in my mind in general, so no big deal keeping it really. And besides Wonder, it also serves WOLips, which would need to be migrated too. And we couldn't just abandon it and then remain unable to publish minor fixes in Wonder 7 should they be needed, could we?

And I think there are still a lot of devs stuck on ant. I feel half of this community has problems moving away from what used to work for them for many years, and I'm not comfortable forcing them to.

@paulhoadley
Copy link
Contributor

And besides Wonder, it also serves WOLips, which would need to be migrated too.

Yeah, I did think of WOLips as soon as I hit send on that one...

And we couldn't just abandon it and then remain unable to publish minor fixes in Wonder 7 should they be needed, could we?

I hear what you're saying. I don't have a strong view on abandoning Ant, but let me push the argument a little further. Wonder isn't the vibrant open source project it used to be. There are fewer and fewer active committers, fewer mailing list participants every year. The resource that is "people who can and do work on Wonder" is stretched thinner than it ever has been. Given that picture, how long are we obliged to support backwards compatibility (with JDKs, build systems)? Can we ever draw a line in the sand?

These are rhetorical questions for thinking about, and aren't directed specifically at you, @maiksd!

@hprange
Copy link
Contributor

hprange commented Jan 16, 2022

Moving Wonder to Java 11 would be great. After publishing the next 7.x release, we could create a new branch wonder_7 and merge this pull request to master. What do you think?

@paulhoadley
Copy link
Contributor

I'm all for moving ahead. Call master 8.0-SNAPSHOT at that point?

@paulhoadley
Copy link
Contributor

Actually—hold on. At this point, why stop at 11? Should we put in the effort for 17? @hugith points out in #943 that (at least) ERProfiling.framework would be problematic. But we can't stop for a single framework.

@hugithordarson
Copy link
Contributor Author

hugithordarson commented Jan 17, 2022

I definitely support going for 17, I only picked 11 since it was the latest LTS when I did this. The API changes required for ERProfiling aren't really a problem if it's a full move to JDK 17, without the need to maintain compatibility with older JDKs.

I've upgraded the pull request to try it out, and the build runs fine on my computer (fails on GitHub due to the expired cert on maven.wocommunity.org though).

@hugithordarson hugithordarson changed the title Make Wonder go to 11 Make Wonder go to 11+6 Jan 18, 2022
@hugithordarson
Copy link
Contributor Author

hugithordarson commented Oct 15, 2022

I'm going to close this PR since I believe the changes made in it were added by @nullterminated in #994

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.

None yet

4 participants