Fix #335: Get --package working with Gradle 7 by replacing the capsule plugin #370
Conversation
This file contains 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
…est as it's not needed.
|
Great job - thanks again! Are you able to still work on tests, cache, and a message with the location of the created package? Please let me know. If you can:
|
|
Time is getting a little tight now, so I'm not sure when I can get to it. (Feel free to add to the PR yourself.) Thanks for all the feedback. |
|
Okay - I understand: life 😀 I will leave this PR open for about 2 weeks. If you have some spare time, you are welcome to continue working on PR. If anyone else wants to work on it - please add a comment here. Otherwise, I will merge the PR into the 4.1 branch and fix the remaining issues later. |
aartiPl
added a commit
that referenced
this pull request
Aug 26, 2022
* Cleanup of env variables * Fixed indentation and generation of Gradle files for IntelliJ * Re-enabled idea tests * Improved runConfiguration * Support for idea project creation: correct handling for duplicated file names. * IntelliJ support fixed * Formatting of wrapper for scripts. * Fix for OsType comparison logic (#357) * build: installation tester * build: update installer.yml * build: added ability to start GitHub action build manually. * bugfix: fix for OSTYPE linux-musl which is reported on Alpine Linux. * make KOTLIN_HOME required * feature: osHandler for operations outside of Java ecosystem * feature: added nicer debug information * refactor: split of config to smaller pieces * test: split building and cleaning (faster builds for most cases) * cleanup: overall cleanups * feature: Migrated kscript to os independent paths (OsPath) * feature: KScript seems to work on a plain Windows console. * Fix #335: Get --package working with Gradle 7 by replacing the capsule plugin (#370) * Fix #335: Remove the need for the Gradle capsule plugin. * Additional fixes for packaging (better cache support) * Improved Templates.kt * Added Kotlin integration tests for packaging and simple tests * Added suites for different OSes * Renamed MAC to MACOS constant. * Fixed error in StreamGobbler in ProcessRunner * Added empty source case handling * Better implementation of OsPath.leaf and OsPath.root * Added integration tests * Improved testing tools (matchers) * Cleanup, bug fixes and small improvements. * Removed old test suite in bash * Updated TestsReadme.md * Replace .kscript dir with OS-specific dirs (#323) (#374) * Refactoring to allow deprecation. * Deprecation of old features (annotations based on comments) * Deprecation report * Updated doc. Co-authored-by: holgerbrandl@gmail.com <holgerbrandl@users.noreply.github.com> Co-authored-by: Vinay Sajip <vinay_sajip@yahoo.co.uk> Co-authored-by: meztihn <6940823+meztihn@users.noreply.github.com
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 has now been merged with the
kscript_4.1branch.