-
Notifications
You must be signed in to change notification settings - Fork 29
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.
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
deb: refine fluentd-apt-source #515
Conversation
Even though modified /etc/apt/sources.list.d/fluentd.sources to v5.
fluentd-apt-source package can be removed because it is transitional package.
|
* Drop transitional apt-transport-https package * https support has been moved into the apt package. * Renamed to fluent-apt-source * It also provides transitional fluentd-apt-source package. Signed-off-by: Kentaro Hayashi <[email protected]>
Signed-off-by: Kentaro Hayashi <[email protected]>
Provides transitional package. Signed-off-by: Kentaro Hayashi <[email protected]>
Signed-off-by: Kentaro Hayashi <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
I think the following codes should be fixed too.
- https://github.com/kenhys/fluent-package-builder/blob/fluent-apt-source/.github/workflows/apt.yml#L43-L47
- https://github.com/kenhys/fluent-package-builder/blob/fluent-apt-source/.gitignore
- https://github.com/kenhys/fluent-package-builder/blob/fluent-apt-source/fluent-package/apt/install-test.sh#L28-L46
- https://github.com/kenhys/fluent-package-builder/blob/fluent-apt-source/fluent-package/apt/systemd-test/update-from-v4.sh#L9-L15
Signed-off-by: Kentaro Hayashi <[email protected]>
Thanks, I've fixed them. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! LGTM!
I have confirmed that I can install the new package correctly in the environment where the old fluentd-apt-source
is already installed.
TODO: try |
If it removes the transitional package entry and use Before (transitional):
After (no transitional, use provides)
|
As the former behavior is preferred, so stay as is. |
Thanks for your check! |
Drop transitional apt-transport-https package
Renamed to fluent-apt-source