Package mime-support declared by another module (Puppetlabs/apache)

Issue #1 resolved
Bren Briggs created an issue

Output:

Error: Duplicate declaration: Package[mime-support] is already declared in file /etc/puppet/modules/apache/manifests/mod/mime.pp:19; cannot redeclare at /etc/puppet/modules/amazon_s3/manifests/install.pp:19

Comments (7)

  1. Bren Briggs reporter

    Closing because I am a foolish person who does not properly read documentation. However, it is still my opinion that support for this parameter could be improved.

  2. Michael Speth

    @briggsb Can you please suggest how we can improve the documentation to help you with this problem?

  3. Bren Briggs reporter

    Sending a PR your way in about 15-20 min. The parameter for include_mime_package was broken for some reason, I think it was puppet refusing to let me redefine the variable. I am adding a params.pp class and evaluating that.

    Thanks for the quick reply!

  4. Bren Briggs reporter

    I would also request that since puppetlabs/apache is such a popular module, the caveat about the mime-support be placed in the readme so it gets more visibility. I can submit a PR for that as well if you like.

  5. Michael Speth

    The mime-support is in the readme ... I forgot to upload that version to the forge. I just did that now :P

  6. Bren Briggs reporter

    Awesome! Looking forward to your thoughts on the PR as well. Thanks again for your prompt responses. Love this module.

    -- Briggs

  7. Michael Speth

    Thank you for the pull request! I have made a comment on the PR. Can you fix the error and resubmit? Thank you for your contribution!

  8. Log in to comment