hi, as seen in bug #2282, courier-imap apparently needs expect for changing the password from within webmail. The proposed solution was a use dependency on tcl-tk/expect. However this his now been changed into a hard dependency. Is this needed ? I am running courier-imap on a production platform without expect and there are no problems here. Reproducible: Always Steps to Reproduce: 1. 2. 3.
while it was true in older versions that you were totally safe if expect was not present, that is not so true of more recent versions. a USE flag based on tcltk also doesn't make sense, as it isn't a tcltk support, it's support for changing passwords. Either we could have a more logical USE flag AND the upstream code needs some cleanup for the case that expect is not present or should not be used [when it is installed tho] OR we just leave it as it is now.
The tcl-tk/expect is not necesary for net-mail/courier-imap, is only used in mail-mta/courier in webmail Please remove dependency tcl-tk/expect
ffelix: submit a patch for the as a described "upstream code needs some cleanup for the case that expect is not present or should not be used [when it is installed tho]" otherwise I'm inclined to leave this as is, because it doesn't break anything to have expect present.
courier-imap does not have because to use expect, to include it adds it ademas tcl with which the installation is increased unnecessarily. I explain myself: mail-mta/courier includes, smtp, imap and webmail, here if that serious necessary to require expect when it settles with I modulate of webmail. net-mail/courier-imap, single has imap and nonwebmail and for that reason it would not have to require expect
Created attachment 38901 [details, diff] remove expect
I've got a number of other applications that aren't webmail that also use the password changing extension. Your patch does NOT fit the requirements I posted. The patch must add a configure option that allows us to disable it detecting expect.
Created attachment 38996 [details, diff] option tcltk, expect optional
it did not know that other applications to part of webmail of courier existed that they used the option to change the password. you know as they are? another serious option to add the option to want to use tcl
your patch still doesn't do it. Use flags should be descriptive of what they do, and USE=tcl really doesn't describe that it allows changing of passwords, and there are no other suitable USE flags. I said you need to write up something that alters the behavior of the upstream configure file (do it in configure.in), as presently, if expect is present, it uses it, without any regard to wheather or not it should. The ebuild change it entirely secondary to the fix for configure. An entirely alternative route would be to rip out the authsystem.passwd and make it an entirely seperate ebuild.
if serious a solution, but to my the ideas have finished to me: - ( it will already think about something
*** Bug 95624 has been marked as a duplicate of this bug. ***
Obsolete bug, the dependency is no longer there in >=courier-imap-4.0.1, neither it is in courier-authlib. Closing.
