net-mail/uw-mailutils-2007e currently builds with SSL and PAM disabled, regardless of whether they were requested by the USE flags. This patch enables SSL and PAM if the USE flags requested them. Reproducible: Always Steps to Reproduce:
Created attachment 196544 [details, diff] Patch against uw-mailutils-2007e.ebuild
Perhaps it would be better if you * declared sslflags as local * quoted sslflags in the make statement. Thanks for the patch.
Making sslflags local is a good idea -- thanks for the suggestion. I don't want to quote sslflags though, because it's a string that can contain more than one make setting, and I want make to see them as separate args.
Created attachment 196690 [details, diff] Updated patch to make $sslflags local
(In reply to comment #3) > I don't want to quote sslflags though, because it's a string that can contain > more than one make setting, and I want make to see them as separate args. Two points: * passing unquoted areguments is not safe enough * SSLINCLUDE and SSLIB directives errors out when compiling mtest.c further down the emerge. Please check the attachment for an alternative.
Created attachment 196719 [details, diff] patch to uw-mailutils-2007e adding pam and ssl USE flags
Created attachment 196720 [details, diff] revize ssl dirs
I tested your new patches, and they work for me. Thanks!
+*uw-mailutils-2007e-r1 (06 May 2011) + + 06 May 2011; Eray Aslan <eras@gentoo.org> +uw-mailutils-2007e-r1.ebuild, + +files/uw-mailutils-ssl.patch: + add pam and ssl USE flags - bugs 129729 276401 +