Bug 194511 - [PATCH] Make mail/claws-mail's libetpan dependency optional
Summary: [PATCH] Make mail/claws-mail's libetpan dependency optional
Status: Closed Rejected
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Pawel Pekala
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-10-21 10:37 UTC by Fabian Keil
Modified: 2014-12-14 16:32 UTC (History)
0 users

See Also:
fk: maintainer-feedback? (pawel)


Attachments
Make mail/claws-mail's libetpan dependency optional (3.36 KB, patch)
2014-10-21 10:37 UTC, Fabian Keil
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Fabian Keil 2014-10-21 10:37:24 UTC
Created attachment 148536 [details]
Make mail/claws-mail's libetpan dependency optional

mail/claws-mail's libetpan dependency is only required for NNTP and IMAP,
the attached patch makes it optional.
Comment 1 Bugzilla Automation freebsd_committer 2014-10-21 10:37:24 UTC
Auto-assigned to maintainer pawel@FreeBSD.org
Comment 2 Pawel Pekala freebsd_committer 2014-10-25 14:47:48 UTC
While this looks ok from technical point of view I'm wondering what's
motivation for this option. As a maintainer I try to keep available
options to minimum - less things to can go wrong and less testing on
each update.
Can you elaborate on why you need this option? IMAP seems to be
essential these days.
Comment 3 Fabian Keil 2014-10-25 15:53:46 UTC
I build ports from source and prefer not to compile and install stuff I don't actually want to use.

Having the libetpan option in the official ports tree is not essential for me, I use local patches anyway. I mainly submitted the patch based on the assumption that it might be useful for other users as well.

I agree that the majority of Claws-Mail users are unlikely to care about this option, but that's probably true for other options as well.
Comment 4 Pawel Pekala freebsd_committer 2014-12-14 12:38:16 UTC
Sorry but I decided against including your patch. While you are right
that there some options similar to what you propose I kept them 
for backward compatibility when taking over maintainership of
claws-mail ports from netchild@.
Comment 5 Fabian Keil 2014-12-14 16:32:48 UTC
No problem, I understand your point of view.