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

Fixes #5793: Add the rudder-agent utilities to rudder-agent #531

Merged
merged 1 commit into from Nov 26, 2014
Merged

Fixes #5793: Add the rudder-agent utilities to rudder-agent #531

merged 1 commit into from Nov 26, 2014

Conversation

Kegeruneku
Copy link

# Initial promises
cp -r %{_sourcedir}/initial-promises %{buildroot}%{rudderdir}/share/
cp -r %{SOURCE13}/* %{buildroot}%{rudderdir}/share/initial-promises/
Copy link
Member

Choose a reason for hiding this comment

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

This will fail because the initial-promises dir doesn't exist. But beyond that, it's a unrelated change. Please remove it from this PR and make a separate ticket for it.

If something breaks because of this change, we'll never think to look back here.

Copy link
Member

Choose a reason for hiding this comment

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

I see now that the mkdir was done in the main spec file. But the same logic still applies, this is not a related change, but it does change the way this package builds.

Copy link
Author

Choose a reason for hiding this comment

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

'kay !

Copy link
Member

Choose a reason for hiding this comment

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

Great. Please remove and self-merge.

Kegeruneku pushed a commit that referenced this pull request Nov 26, 2014
…der_agent_utilities

Fixes #5793: Add the rudder-agent utilities to rudder-agent
@Kegeruneku Kegeruneku merged commit 41c30af into Normation:branches/rudder/3.0 Nov 26, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants