PR#50131: Issue 50130 - Building RPMs on RHEL8 fails - 389-ds-base

Marc Muehlfeld • 5 years ago

c1cc0ac

rpm/389-ds-base.spec.in

file modified

+1 -1

		`@@ -409,7 +409,7 @@`

		`# Cockpit branding, and directory and file list`
		`%if 0%{?rhel} > 7`
		`- mv -f %{buildroot}%{_datadir}/cockpit/389-console/src/rhds-banner.html %{buildroot}%{_datadir}/cockpit/389-console/src/banner.html`
		`+ mv -f %{buildroot}%{_datadir}/cockpit/389-console/rhds-banner.html %{buildroot}%{_datadir}/cockpit/389-console/banner.html`
		`%endif`
		`find %{buildroot}%{_datadir}/cockpit/389-console -type d \| sed -e "s@%{buildroot}@@" \| sed -e 's/^/\%dir /' > cockpit.list`
		`find %{buildroot}%{_datadir}/cockpit/389-console -type f \| sed -e "s@%{buildroot}@@" >> cockpit.list`

Description: Due to an incorrect path in a "mv" command in
389-ds-base.spec.in, building RPMs on RHEL8 failed

Reviewed by: ?

looks reasonable to me, @spichugi or @mreynolds are the cockpit experts though :)

LGTM

rebased onto 2deb97b65febf573888089745c89512a3f0a43a8

5 years ago

rebased onto c1cc0ac

5 years ago

Pull-Request has been merged by vashirov

5 years ago

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This pull request has been cloned to Github as issue and is available here:
- https://github.com/389ds/389-ds-base/issues/3190

If you want to continue to work on the PR, please navigate to the github issue,
download the patch from the attachments and file a new pull request.

Thank you for understanding. We apologize for all inconvenience.

Pull-Request has been closed by spichugi

3 years ago

Assignee

None

Tags

No Tags