Skip to content

Added option to configure dynamic imports#2682

Merged
jbonofre merged 1 commit into
apache:karaf-4.4.xfrom
CptBartender:improve-bundle-dynamic-imports
Jun 11, 2026
Merged

Added option to configure dynamic imports#2682
jbonofre merged 1 commit into
apache:karaf-4.4.xfrom
CptBartender:improve-bundle-dynamic-imports

Conversation

@CptBartender

@CptBartender CptBartender commented May 27, 2026

Copy link
Copy Markdown
Contributor

Current bundle:dynamic-imports command only allows toggling of dynamic imports. This PR adds an option to configure which packages are imported dynamically, instead of setting all-or-nothing, as well as adds option to explicitly set or unset the option.

@CptBartender CptBartender force-pushed the improve-bundle-dynamic-imports branch from c8a16ba to 4ec3e9c Compare May 28, 2026 21:38
@CptBartender CptBartender marked this pull request as ready for review May 28, 2026 21:38
@jbonofre jbonofre self-requested a review May 29, 2026 06:54
@jbonofre jbonofre merged commit cc1824c into apache:karaf-4.4.x Jun 11, 2026
6 checks passed
@holgerfriedrich

Copy link
Copy Markdown
Contributor

I'm wondering why this PR was against 4.4.x and not main. Did I overlook something?

@jbonofre

Copy link
Copy Markdown
Member

@holgerfriedrich oh right, it should have been on main not karaf-4.4.x. Let me backport.

@CptBartender

CptBartender commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

I'm wondering why this PR was against 4.4.x and not main. Did I overlook something?

This is because I needed that change on 4.4.x and eventually in 4.4.12 release. I don't know the process, and was unsure if I should create a separate PR to main or if the entire karaf-4.4.x branch will eventually be merged. Apologies for the inconvenience.

holgerfriedrich pushed a commit that referenced this pull request Jun 11, 2026
Co-authored-by: CptBartender <fryderyk.wysocki@gmail.com>
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.

3 participants