Skip to content

Prototype / early discussion for enabling incremental build for installFeature#811

Draft
scottkurz wants to merge 1 commit intoOpenLiberty:mainfrom
scottkurz:incr-experiment-1
Draft

Prototype / early discussion for enabling incremental build for installFeature#811
scottkurz wants to merge 1 commit intoOpenLiberty:mainfrom
scottkurz:incr-experiment-1

Conversation

@scottkurz
Copy link
Contributor

@scottkurz scottkurz commented Apr 7, 2023

This is an attempt to advance a bit the idea in #649 in public. So I'll link with the "Fixes #649" comment

This oversimplifies the full scope of the installFeature task.

It uses the install lib/features as the output dir and it uses the io.openliberty.tools.gradle.tasks.AbstractFeatureTask.getSpecifiedFeatures(String) as the input (which doesn't capture all possible cases, I know).

Putting it out here to help the discussion.

EXTRA NOTE

Though not really related, also noting I regularly (with and without this PR) see a re-run of libertyCreate with message:

Task :libertyCreate
C:\git\guide-gradle-intro\finish\build\wlp\usr\servers\defaultServer\configDropins\overrides\liberty-plugin-variable-config.xml file deleted before processing plugin configuration.

Signed-off-by: Scott Kurz <skurz@us.ibm.com>
@scottkurz scottkurz force-pushed the incr-experiment-1 branch from 582ffe8 to d597cbe Compare April 7, 2023 14:39
@scottkurz scottkurz changed the title Doesn't work...output directory deleted Prototype / early discussion for enabling incremental build for installFeatre Apr 7, 2023
@scottkurz scottkurz changed the title Prototype / early discussion for enabling incremental build for installFeatre Prototype / early discussion for enabling incremental build for installFeature Apr 7, 2023
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.

Some tasks do not support up-to-date checks

1 participant