Repository navigation
Conversation
- build on ubuntu / java 8 only - use a matrix for os [ubuntu-latest, macos-latest, windows-latest] and java [11, 17, 21, 25] - make dist/bin/jython executable - get a bit more more output out of test-jython.sh - upload the junit test results and add a junit test report
They should not interfere with ant regrtest .yml, which waits for the build output to have been uploaded.
This has nothing to do with the changes in the last months. They fail for a much longer time, but never were caught by ci pipelines.
- make sure test.compile.dir exists before starting junit - include org.python.generate package in jython-dev jar - add jython-test.jar and jython-dev.jar to the test classpath - show stdout and stderr of javatests - add the target 'cleanWithoutDist' for local testing of the -built targets
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is an informational draft pull request, not intended to be merged as is into master.
The goal is to show that the "promise" of the upcoming release:
is really met.
The pipeline builds once on ubuntu with java 8, uploads the
dist/folder for later use and then exercises ant tests on a matrix:os: [ubuntu-latest, macos-latest, windows-latest]java-version: [11, 17, 21, 25]The results can be found here: Cafino#2
Parts of this PR might be useful for the future, especially the ones allowing tests to be run on a true "
dist/only" setup.For local reproduction, I used the following commands:
ant cleanantant cleanWithoutDistant test-ci-built(or any single-builttarget)Necessary tweaks for green matrix runs are:
Indexertests on windows (they fail since ages, but never popped up on ci)socketandselecttests (they turned out to be very flaky on GitHub)