Skip to content

Update foreman package dependencies - #5735

Merged
ehelms merged 1 commit into
theforeman:rpm/developfrom
ekohl:rpm/update-foreman-deps
Sep 1, 2020
Merged

ehelms merged 1 commit into
theforeman:rpm/developfrom
ekohl:rpm/update-foreman-deps

Conversation

@ekohl

@ekohl ekohl commented Aug 31, 2020

Copy link
Copy Markdown
Member

No description provided.

Comment thread packages/foreman/foreman/foreman.spec Outdated
BuildRequires: %{?scl_prefix}npm(@babel/core) >= 7.7.0
BuildRequires: %{?scl_prefix}npm(@babel/core) < 8.0.0
BuildRequires: %{?scl_prefix}npm(@theforeman/builder) >= 4.11.1
BuildRequires: %{?scl_prefix}npm(@theforeman/builder) >= 4.12.0

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

theforeman/foreman#7896 bumps it to 4.14 so I'm going to update this ahead of time.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Could you revert it back to 4.12.0 to reflect current state? This is mixing two update concerns that will block builds unless we know that the referenced PR is going in really soon.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This doesn't block the build since 4.14.0 is available (#5687). Repoclosure is only complaining about sd_notify which is because we didn't mash.

@ekohl
ekohl force-pushed the rpm/update-foreman-deps branch from 0c246f7 to c2dbf9a Compare August 31, 2020 15:27
@ehelms
ehelms merged commit 52f821f into theforeman:rpm/develop Sep 1, 2020
@ekohl
ekohl deleted the rpm/update-foreman-deps branch September 7, 2020 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants