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
Update CI infrastructure #6703
Update CI infrastructure #6703
Conversation
Closes #6458 |
Closes #6652 |
From what I could see in the logs, the errors seem to originate in the use of reflection. Maybe merging this PR might be a good opportunity to fix this? |
I just want to add, that I don't need this on my system (gentoo) with OpenJDK+OpenJFX (11.0.7). Maybe it is because of they way it is installed (I use gentoos ebuilds from the main repo, with a modified one for openjfx to enable building webkit) |
On Tue, 23 Jun 2020 at 10:47, SpeedProg ***@***.***> wrote:
I just want to add, that I don't need this on my system with
OpenJDK+OpenJFX (gentoo). Maybe it is because of they way it is installed
(I use gentoos ebuilds from the main repo, with a modified one for openjfx
to enable building webkit)
Are you referring to running the released version, or about building from
sources? (I am referring to the second case.)
|
All the output I referred to corresponds to a |
Not sure what the building from sources refers to, but my OpenJDK+OpenJFX is built from soruce and building XMage from source works with current master (and like old masters since the Java 11 patches were merged).
|
Hmm.... Interesting.
Could you please pass me the exact hash for master? I will try again...
|
@SpeedProg I forgot to add: Thank you for your constructive reply! :-) |
I mean if this change fixes it for you, it should maybe just be added as a dependency, since apparently openjfx is not really part of a standard JDK anymore so it kind of is a dependency now. |
Thanks again! I'll try it and let you know. |
Hm I took a look and how debian does the openjfx install in bullseye and apparently it only installs the built .jar files into some libs folder. |
On Tue, 23 Jun 2020 at 15:49, SpeedProg notifications@github.com wrote:
That's precisely the reason for all this :-)
Sure. If there is one thing we all have learned from this, is that Gentoo somehow bundles openjfx into openjdk, and that Debian does not treat openjfx as part of openjdk :-) |
Stepping back to look at the big picture a bit, is OpenJFX still worth all the trouble its constantly causing? As far as I can tell, literally the only thing it's used for is displaying release notes as HTML. a) Is there really no other way to do that in modern Java besides JFX? |
@apetresc Personally, I think the real problem here is depending on a proprietary/privative Java package (Oracle's). I do agree in that using OpenJFX solely for displaying release notes as HTML is... a little bit over the top, to say the least ;-)
No idea. Perhaps, there is.
Not my call. I have repeatedly asked for permission to do simple things, such as labeling issues, and did not get many constructive answers. In the few cases where I actually got some, they were written in something that, while aiming at being English, is so incomprehensible a simple conversation becomes impossible. (Don't take me wrong: I do value the effort of striving to communicate in a language one does not fully master - I am not a native English speaker myself either. That being said, objectively speaking, within the context of international projects such as this one, it is very difficult to communicate properly if the few counterparts that actually answer your messages do not even minimally understand the project's de facto lingua franca...) We are in 2020; Java 8 (LTS) was released in 2014; Java 11 (LTS), in 2018; the next LTS version of the language is due in 2021. BTW, @apetresc : to which project were you referring as "Magic Arena"? |
It would be nice to get some feedback about how fix the "Verify" project to build (it is the only one I am unable to build). Here's the log for a clean compitation of this branch rebased onto current master (543ac20).
|
It's outdated logs from mtgjson issue (someday site was broken by outdated backup and returns wrong data, so verify tests are failed). Find and delete all mtgjson zip files from project folder and run it again. |
Thanks for your feedback, @JayDi85 ! I deleted the .zip files from Mage.Verify/, and tried again after rebasing (again) onto current master (now at 1db5235). However, I still get the same error... I tried this twice: with and without updating the project dependencies. Afterwards, I checked mtgjson.com. It seems they have switched to MTGJSON v5. @JayDi85 Could you please try to reproduce this? You just need to checkout this branch, and do a |
@SpeedProg Could you try reproducing the error I describe above, please? |
Current code works with latest mtgjson (v5). Show your error logs here. |
@JayDi85 Please read attached log. Please note I am using OpenJDK 11.
To reproduce it, |
I can see h2database errors. Try to delete all |
@allentiak I checked out your compile-with-openjdk branch and deleted *.json.zip files in Mage.Verify and everything in Mage.Verify/db and afterwards maven package ran fine and reported everything as success
|
Thanks for your feedbak, @JayDi85. Maybe It works now! Build log: mvn_install--1db523518b338af1c591e41841af077f17ac703c.log Notes: |
@SpeedProg try to change download threads to 1-2 -- it can help with scryfall IP ban for too much images download. |
@SpeedProg OK. I have moved the dependency updates to another branch. I will propose the deps updates in a new PR. I have also rebased this PR on current master (8c7d7f0). @JayDi85 I have updated the details of this PR. I think this is ready to be merged now.
Cool!
No problem. AFAIK, there hasn't been any breaking change since OpenJDK9. |
@SpeedProg BTW: Thank you very much for your help on this! |
@SpeedProg w.r.t. the dependencies, I have just opened #7128. |
@JayDi85 Is there anything else I should do in order for this to be merged? Just to recap:
[*]: EA: early access (currently, OpenJDK 16) |
|
@JayDi85 Thanks for your answer. Considering that both items 1. and 2. have already been successfully completed (by me and @SpeedProg)... |
Just rebased against current master (9f7a873). |
@JayDi85 BTW: NetBeans' latest version is 12.1. |
@JayDi85 |
Just rebased against current master (fec9da5). |
@JayDi85 could we do something for this? |
@t-my yes, it will be released soon with new launcher (magefree/Launcher#14) |
You can Compiler xmage in openjdk11 without these patches if you habe openjfx jmods properly installed. Wich seems ti happen if you usw the buildscript to install it. |
I managed to compile it today, but I can't run yet because of JFX, which I'll try to patch out over weekend. |
It has been over a year, can we merge this in now? |
These changes are no longer needed as the javafx dependency has been removed. However I can't get the current head to compile with openjdk 11 or 18 on my Mac as it produces a different ClassNotFoundException.
|
… OpenJDK version This will help ensuring XMage can be buit with upcoming OpenJDK versions. This is an unofficial support: the whole build will not break if the EA job fails. Note: EA = early access.
@Alex-Vasile Thanks for taking this over. I'm happy that, finally, someone with merging power starts paying this kind of improvements some attention. |
Will merge once Tarvis CI is back up and running. It seems to have disappeared from the project... |
@LevelX2 @jeffwadsworth @JayDi85
This allows compiling XMage 1.4.43V0 under OpenJDK 11+
Tested with both currently supported OpenJDK versions: 11 (LTS) and 14.
All projects but one build fine. The only exception is Verify (the last one). I would appreciate help with that.
Output and logs below:
Output
Logs
openjdk-11.0.7+10-post-Debian-3--mage.verify.VerifyCardDataTest.txt
openjdk-11.0.7+10-post-Debian-3--TEST-mage.verify.VerifyCardDataTest.xml.txt
openjdk-14.0.1+7-Debian-1--TEST-mage.verify.VerifyCardDataTest.xml.txt
openjdk-14.0.1+7-Debian-1--mage.verify.VerifyCardDataTest.txt