On Sat, Jul 30, 2011 at 07:02:27PM +0200, Julian Taylor wrote:
> reopen 632090
> found 632090 0.8-1
> thanks
> 
> the fix was not merged in 0.8.
> Makefile.am and debian/rules is still wrongly using LDFLAGS for linking.
> 
> The proper fix is still waiting in this pull request:
> https://github.com/moxie0/sslsniff/pull/1
> see:
> https://github.com/moxie0/sslsniff/issues/2
> 

I must have missed something ..

I tried to add -Wl,--as-needed to LDFLAGS and building the package and
it builds fine. Last lines from build are:

mv -f .deps/FirefoxAddonUpdater.Tpo .deps/FirefoxAddonUpdater.Po
g++ -ggdb -g -O2 -lssl -lboost_filesystem-mt -lpthread -lboost_thread-mt 
-llog4cpp -Wl,-z,defs -lcrypto -lboost_system -Wl,--as-needed -o sslsniff 
SSLConnectionManager.o Destination.o FirefoxUpdater.o Logger.o SessionCache.o 
SSLBridge.o HTTPSBridge.o sslsniff.o FingerprintManager.o 
AuthorityCertificateManager.o TargetedCertificateManager.o CertificateManager.o 
HttpBridge.o HttpConnectionManager.o HttpHeaders.o UpdateManager.o OCSPDenier.o 
FirefoxAddonUpdater.o  
{master//sslsniff} $ ++ -ggdb -g -O2 -lssl -lboost_filesystem-mt -lpthread 
-lboost_thread-mt -llog4cpp -Wl,-z,defs -lcrypto -lboost_system -Wl,--as-needed 
-o sslsniff SSLConnectionManager.o Destination.o FirefoxUpdater.o Logger.o 
SessionCache.o SSLBridge.o HTTPSBridge.o sslsniff.o FingerprintManager.o 
AuthorityCertificateManager.o TargetedCertificateManager.o CertificateManager.o 
HttpBridge.o HttpConnectionManager.o HttpHeaders.o UpdateManager.o OCSPDenier.o 
FirefoxAddonUpdater.o

(note the -Wl,--as-needed on the command line).

Do I have something else to check ?

Pierre

Attachment: signature.asc
Description: Digital signature

Reply via email to