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

Update redline library to 1.2.10 #76440

Merged
merged 2 commits into from
Aug 13, 2021

Conversation

breskeby
Copy link
Contributor

The redline team just released version 1.2.10 of the redline
library which contains our fix of the rpm signatures / headers.

Also a PR to update that dependency in the ospackage plugin has been
raised at nebula-plugins/gradle-ospackage-plugin#402

@breskeby breskeby self-assigned this Aug 12, 2021
@breskeby breskeby added :Delivery/Build Build or test infrastructure >enhancement Team:Delivery Meta label for Delivery team v7.15.0 v8.0.0 auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) auto-backport Automatically create backport pull requests when merged labels Aug 12, 2021
@@ -54,9 +54,7 @@ buildscript {
// Be aware that it seems the redline project hasnt been active for a while
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we remove this comment? 😉

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed / updated

dependencySubstitution {
substitute module('org.redline-rpm:redline') using module('com.github.breskeby:redline:9c85270')
}
force 'org.redline-rpm:redline:1.2.10'
Copy link
Contributor

Choose a reason for hiding this comment

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

How come we force it? Does Gradle ship its own, or something like that?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

what that means is that we enforce a particular version (1.2.10). Otherwise redline 1.2.9 would be brought in as a transitive dependency by the nebula.ospackage plugin that has 1.2.9 declared. (There's an open PR on ospackage to fix this).

@breskeby breskeby marked this pull request as ready for review August 13, 2021 10:22
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-delivery (Team:Delivery)

The redline team just released version 1.2.10 of the redline
library which contains our fix of the rpm signatures / headers.

Also a PR to update that dependency in the ospackage plugin has been
raised at nebula-plugins/gradle-ospackage-plugin#402
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
7.x

breskeby added a commit to breskeby/elasticsearch that referenced this pull request Aug 13, 2021
* Update redline library to 1.2.10

The redline team just released version 1.2.10 of the redline
library which contains our fix of the rpm signatures / headers.

Also a PR to update that dependency in the ospackage plugin has been
raised at nebula-plugins/gradle-ospackage-plugin#402

* Update common about enforcing redline 1.2.10
elasticsearchmachine pushed a commit that referenced this pull request Aug 13, 2021
* Update redline library to 1.2.10

The redline team just released version 1.2.10 of the redline
library which contains our fix of the rpm signatures / headers.

Also a PR to update that dependency in the ospackage plugin has been
raised at nebula-plugins/gradle-ospackage-plugin#402

* Update common about enforcing redline 1.2.10
@breskeby breskeby deleted the update-redline-version branch October 13, 2021 01:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Automatically create backport pull requests when merged auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Delivery/Build Build or test infrastructure >enhancement Team:Delivery Meta label for Delivery team v7.15.0 v8.0.0-alpha2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants