-
Notifications
You must be signed in to change notification settings - Fork 5k
Open
Labels
a:featureA new functionalityA new functionalityin:configuration-cacheConfiguration CachingConfiguration Cachingre:comprehensibilityreasonable errors and warnings, clear dsl, mental overloadreasonable errors and warnings, clear dsl, mental overload
Description
Currently, BuildEventsListenerRegistry.onTaskCompletion accepts any provider of any OperationCompletionListener implementation.
However, when the Configuration Cache is enabled, anything but providers of BuildService returned by BuildServiceRegistry.registerIfAbsent results in CC failure (after #15530).
This is inconvenient for several reasons:
- This is a discrepancy between Configuration Cache and Vintage modes, making it harder to adopt CC in an existing build.
- Nothing in the type system highlights this requirement.
Ideally, only CC-supported listeners should be accepted by the registry regarding of CC being enabled. This can also be solved by relaxing the restriction on the listeners.
Metadata
Metadata
Assignees
Labels
a:featureA new functionalityA new functionalityin:configuration-cacheConfiguration CachingConfiguration Cachingre:comprehensibilityreasonable errors and warnings, clear dsl, mental overloadreasonable errors and warnings, clear dsl, mental overload