Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 8930 - fetchmail 6.1.0 doesn't compile with ssl USE variable
Summary: fetchmail 6.1.0 doesn't compile with ssl USE variable
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: x86 Linux
: High major (vote)
Assignee: Nick Hadaway
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-10-09 08:57 UTC by Holger Benl
Modified: 2003-02-04 19:42 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Holger Benl 2002-10-09 08:57:55 UTC
when I try to merge it, it fails with an error message about the patch
not being applicable. Well, no big surprise there - patching a file that
has been autogenerated by a configure script doesn't sound like a very
good idea.
Comment 1 Holger Benl 2002-10-09 09:39:53 UTC
Actually, while it might compile on some systems with "ssl" enabled,
it will certainly not compile on systems with "ssl" disabled.
To fix this, replace

  use ssl && \
  patch -p1 < ${FILESDIR}/${P}-gentoo.diff || die "patch failed"

with

  use ssl && \
  ( patch -p1 < ${FILESDIR}/${P}-gentoo.diff || die "patch failed" )

in the ebuild.
This doesn't solve the problem I'm having with "ssl" enabled, but at
least I can now compile it with "ssl" disabled.
Comment 2 SpanKY gentoo-dev 2002-10-10 00:27:33 UTC
using ( ) like that wont accomplish what you want ... 
 
( ) spawns another shell and when you use that 'die' in there, it kills the 
spawned shell while the parent shell keeps on emerging ... 
Comment 3 Holger Benl 2002-10-10 05:46:35 UTC
Hm, OK, that was the first time I used () in a shell script...
So I guess that this one would be right:

  use ssl && \
  { patch -p1 < ${FILESDIR}/${P}-gentoo.diff || die "patch failed" }
Comment 4 Holger Benl 2002-10-10 06:05:47 UTC
Just looked at the Changelog for fetchmail and saw that the ssl patch
was only needed to remove a warning message during compilation!?
Since patch-a-file-generated-by-configure is a very flaky construction,
I'd suggest that this patch be removed altogether (unless someone has
the know-how required to patch the configure script itself and not
break anything).
Comment 5 Nick Hadaway 2002-10-10 06:22:42 UTC
I got rid of that naughty patch.  Sorry about that.  I'll have a better method 
implemented eventually. 
Comment 6 Scott Robbins 2002-10-10 16:41:01 UTC
I had the problem with emerging fetchmail this morning and looked at this 
bug--figured that it would probably be fixed by the time I got home this 
evening (about 5:30 EDT).  

Tested it a few minutes ago, and yes, the emerge goes without problem.

Thanks, and as always to the Gentoo developers, it's great having a distro 
where serious problems get fixed so quickly.

Sincerely,

Scott Robbins