Add better abstraction to distribution building code. #162
Conversation
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
lagergren
force-pushed
the
multi-release
branch
from
2f96702 to
f3e1a58
Compare
|
This is rather trivial change. I moved to copyspec config for the distribution tasks to a common place so we can easily create new distros for new platforms with minimum hassle. I also took the Liberty to clean up some sucky Kotlin code I wrote in q2-2023 before I had fully understood Kotlin. |
ggleyzer
approved these changes
Feb 27, 2024
ggleyzer
changed the title
lagergren
force-pushed
the
multi-release
branch
from
568a779 to
655f86b
Compare
…rate several similar distributions, if we need to, e.g. for different platforms. General build logic cleanup.
…en I wrote the initial ones.
lagergren
force-pushed
the
multi-release
branch
from
cdee897 to
4514973
Compare
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.
Added parameterizable distribution contents so we can modify it for several different Gradle distributions with different classifiers to get a more versatile release pipeline. Haven't actually done any yet, would want to discuss with @cpurdy first.
However, this should be possible to get in as a trivial abstraction raising change.
Also cleaned up several things related to the build, like dead code, unused configurations and so on.