Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

RFE: make rpm plugin API public #1536

Closed
2 tasks done
pmatilai opened this issue Feb 12, 2021 · 0 comments · Fixed by #2661
Closed
2 tasks done

RFE: make rpm plugin API public #1536

pmatilai opened this issue Feb 12, 2021 · 0 comments · Fixed by #2661
Labels
API API related design Complicated design issue RFE
Milestone
4.20.0

Comments

@pmatilai
Copy link
Member

pmatilai commented Feb 12, 2021 •
edited
Loading

The plugin API should made public, the current situation is missing half the point of plugins by forcing them to live in rpm proper. This was of course always the goal, but things haven't moved anywhere since the plugin system was introduced. There are things that will need addressing first, at least
@pmatilai pmatilai added RFE API API related labels Feb 12, 2021
@pmatilai pmatilai mentioned this issue Feb 12, 2021
Closed
@pmatilai pmatilai added the design Complicated design issue label Apr 12, 2021
@pmatilai pmatilai added this to RPM Jun 21, 2022
@pmatilai pmatilai moved this to Backlog in RPM Jun 21, 2022
@pmatilai pmatilai added this to the 4.20.0 milestone Nov 17, 2022
@pmatilai pmatilai removed this from the 4.20.0 milestone Jan 26, 2023
@pmatilai pmatilai added this to the 4.20.0 milestone Aug 23, 2023
@pmatilai pmatilai moved this from Backlog to Todo in RPM Aug 28, 2023
pmatilai added a commit to pmatilai/rpm that referenced this issue Sep 13, 2023
@pmatilai
Make the plugin API public
4fae979 
We've procrastinated on making this API public for about ten years now,
and in the meanwhile there has been exactly one disruptive change to
the API. As in, it might've just as well been public all along.

There will always be more things to improve wrt any API, but we're not
going to hold this hostage to one more thing or another anymore. Some
of them we'd like to do before this goes to a stable release (ie 4.20)
but doing this now to kinda enforce this actually happens this time
around, through come hell or high water.

Fixes: rpm-software-management#1536
@pmatilai pmatilai mentioned this issue Sep 13, 2023
Merged
@pmatilai pmatilai moved this from Todo to In Review in RPM Sep 13, 2023
@pmatilai pmatilai mentioned this issue Sep 13, 2023
Open
@pmatilai pmatilai moved this from In Review to In Progress in RPM Sep 13, 2023
@pmatilai pmatilai moved this from In Progress to Done in RPM Sep 13, 2023
pmatilai added a commit that referenced this issue Sep 13, 2023
@pmatilai
Make the plugin API public
959b4e4 
We've procrastinated on making this API public for about ten years now,
and in the meanwhile there has been exactly one disruptive change to
the API. As in, it might've just as well been public all along.

There will always be more things to improve wrt any API, but we're not
going to hold this hostage to one more thing or another anymore. Some
of them we'd like to do before this goes to a stable release (ie 4.20)
but doing this now to kinda enforce this actually happens this time
around, through come hell or high water.

Fixes: #1536
@pmatilai pmatilai added highlight Release highlight and removed highlight Release highlight labels Mar 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
API API related design Complicated design issue RFE
Projects
Public plugin API
Status: No status
RPM
Status: Done
Milestone
4.20.0
Development

Successfully merging a pull request may close this issue.

Make the plugin API public pmatilai/rpm
1 participant
@pmatilai