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

Create temporaty directories at the begining of uninstall #645

Closed
wants to merge 1 commit into from
Closed

Create temporaty directories at the begining of uninstall #645

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Mar 23, 2017

Since commit 38c6689 temporary directories are no longer created at package
install time. Instead they're created at server install time.
Some steps in uninstall also assume that temporary direcories exist. Creating
the directories in the begining of server uninstall ensure that the uninstall
will go through.

https://pagure.io/freeipa/issue/6715

Since commit 38c6689 temporary directories are no longer created at package
install time. Instead they're created at server install time.
Some steps in uninstall also assume that temporary direcories exist. Creating
the directories in the begining of server uninstall ensure that the uninstall
will go through.

https://pagure.io/freeipa/issue/6715
@MartinBasti MartinBasti self-assigned this Mar 27, 2017
@MartinBasti
Copy link
Contributor

Not the nicest but good enough for now

@MartinBasti MartinBasti added the ack Pull Request approved, can be merged label Mar 27, 2017
@ghost ghost added the pushed Pull Request has already been pushed label Mar 28, 2017
@ghost
Copy link
Author

ghost commented Mar 28, 2017

master:

  • 3dcd342 Create temporaty directories at the begining of uninstall
    ipa-4-5:

  • c0a3957 Create temporaty directories at the begining of uninstall

@ghost ghost closed this Mar 28, 2017
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
1 participant