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

mail-filter/spamassassin: remove unused file #9744

Closed
wants to merge 1 commit into from

Conversation

mm1ke
Copy link
Contributor

@mm1ke mm1ke commented Aug 31, 2018

Hi,

This PR removes a unused file from:
mail-filter/spamassassin

Please review

@gentoo-bot
Copy link

Pull Request assignment

Areas affected: ebuilds
Packages affected: mail-filter/spamassassin

mail-filter/spamassassin: @Philippe23, bug[at]mejor.pl, @orlitzky, @gentoo/proxy-maint

No bugs to link found. If your pull request references any of the Gentoo bug reports, please add appropriate GLEP 66 tags to the commit message and ping us to reset the assignment.

If you do not receive any reply to this pull request, please open or link a bug to attract the attention of maintainers.

In order to force reassignment and/or bug reference scan, please append [please reassign] to the pull request title.

@gentoo-bot gentoo-bot added assigned PR successfully assigned to the package maintainer(s). no bug found No Bug/Closes found in the commits. labels Aug 31, 2018
@Philippe23
Copy link
Contributor

I agree with @mm1ke, that file does not look like it's used anymore.

Same with 3.4.1-spamd.conf, 3.4.1-spamd.init, nor spamassassin.service.conf from what I see.

@monsieurp
Copy link
Member

Thanks for the feedback. @Philippe23 can we punt it then?

@Philippe23
Copy link
Contributor

@monsieurp I removed all of them locally and tested spamassassin-3.4.1-r21.ebuild and it works fine and double-checked that it's not referenced in the other spamassassin ebuilds.

I vote nuke it/accept the PR.

@mm1ke
Copy link
Contributor Author

mm1ke commented Sep 1, 2018

@Philippe23
Thanks for checking, i've updated the PR and added the other files too.

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2018-09-01 18:36 UTC
Newest commit scanned: caeaddb
Status: ✅ good

No issues found

@gentoo-bot gentoo-bot closed this in a560f07 Sep 4, 2018
@mm1ke mm1ke deleted the spamassassin-lAcujrLE branch January 29, 2019 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assigned PR successfully assigned to the package maintainer(s). no bug found No Bug/Closes found in the commits.
Projects
None yet
5 participants