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

Remove unused template #125

Merged
merged 1 commit into from Aug 1, 2018

Conversation

smortex
Copy link
Member

@smortex smortex commented Aug 1, 2018

While converting ERB files to EPP, I found what seems to be an unused template.

I am not sure if it is better to keep this template and convert it to epp so that users who want to have a custom template can use it to get inspired, or simply drop it? Maybe moving this to the documentation may make sense?

It seems to be very specific to some particular configuration…

Please tell me what you think about it!

@zachfi
Copy link
Contributor

zachfi commented Aug 1, 2018

Oh! Let's drop it. I think this would have been added to have a job to backup the catalog. I've never hit the situation where I needed to have a catalog job specifically, but I think the Bacula project recommends it. I've moved directors and databases many times in the last few years without ever needing to rely on this functionality, mostly because I have good database backups. Anyway, if the idea ever comes up again, we can re-implement. Thanks for the PR!

@zachfi zachfi merged commit c2341d5 into voxpupuli:master Aug 1, 2018
@smortex smortex deleted the remove-unused-template branch August 2, 2018 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants