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.

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

Change trento-premium to be obsolete in the spec #53

Merged
merged 1 commit into from
Jul 11, 2022

Conversation

rtorrero
Copy link
Contributor

This PR simply changes the relationship in the spec file with the old trento-premium package so that it's now marked as Obsolete. This should reduce the possibility of users installing the older package without noticing that it has been obsoleted.

Copy link
Member

@fabriziosestito fabriziosestito left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!

Copy link

@ajaeger ajaeger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AFAIR this is not enough, see the linked documents.

packaging/suse/trento-agent.spec Outdated Show resolved Hide resolved
@rtorrero rtorrero force-pushed the obsolete-old-package branch from c867117 to fc711c4 Compare July 11, 2022 15:04
@@ -33,7 +33,9 @@ BuildRequires: golang(API) = 1.16
Requires: golang-github-prometheus-node_exporter
Provides: %{name} = %{version}-%{release}
Provides: trento = %{version}-%{release}
Provides: trento-premium = %{version}-%{release}
Obsoletes: trento < %{version}-%{release}
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now that I think about this, I suspect we should have probably not used the macro here to fix the (deprecated) provided version to an specific version.

@rtorrero rtorrero merged commit ef65c78 into main Jul 11, 2022
@arbulu89 arbulu89 added the enhancement New feature or request label Jul 14, 2022
@stefanotorresi stefanotorresi deleted the obsolete-old-package branch August 17, 2022 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

Successfully merging this pull request may close these issues.

5 participants