Skip to content

chore: Remove package gamin#3030

Merged
polarathene merged 2 commits intodocker-mailserver:masterfrom
polarathene:chore/remove-package-gamin
Jan 29, 2023
Merged

chore: Remove package gamin#3030
polarathene merged 2 commits intodocker-mailserver:masterfrom
polarathene:chore/remove-package-gamin

Conversation

@polarathene
Copy link
Copy Markdown
Member

Description

This appears to have been added to replace the fam package in an early version of DMS with Courier for IMAP instead of Dovecot on an Ubuntu 14.04 base image.

I have tried to reproduce the failure with telnet example (from reference blogpost below), but it logs in successfully without the gamin package. Probably whatever caused it, is no longer relevant to the current image? So this package doesn't seem to provide any value to retain.

References

It is too old to have a PR. The original project author committed to master directly in Sep 2015 referencing this issue and their blogpost to explain it. That is also where the docker / 5000 user was introduced into the image.

Type of change

  • Improvement (non-breaking change that does improve existing functionality)

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • New and existing unit tests pass locally with my changes

This appears to have been added to replace the `fam` package in an early version of DMS with Courier for IMAP instead of Dovecot on an Ubuntu 14.04 base image.

It does not appear to serve a purpose anymore.
@polarathene polarathene added priority/low service/dovecot area/scripts kind/improvement Improve an existing feature, configuration file or the documentation labels Jan 26, 2023
@polarathene polarathene added this to the v12.0.0 milestone Jan 26, 2023
@polarathene polarathene self-assigned this Jan 26, 2023
@polarathene polarathene enabled auto-merge (squash) January 29, 2023 12:37
@polarathene polarathene merged commit 74da019 into docker-mailserver:master Jan 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/scripts kind/improvement Improve an existing feature, configuration file or the documentation priority/low service/dovecot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants