Bug 199129

Summary: [PATCH] sysutils/synergy avoid -march=native producing broken packages for other CPUs
Product: Ports & Packages Reporter: Adam McDougall <ebay>
Component: Individual Port(s)Assignee: Kevin Lo <kevlo>
Status: Closed Overcome By Events    
Severity: Affects Some People Keywords: patch
Priority: --- Flags: bugzilla: maintainer-feedback? (kevlo)
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Avoid -march= in ext/cryptopp none

Description Adam McDougall 2015-04-02 23:13:45 UTC
Created attachment 155137 [details]
Avoid -march= in ext/cryptopp

Recently I reinstalled my desktop and discovered synergy would coredump on Illegal instruction:
kernel: pid 32211 (synergys), uid 21281: exited on signal 4 (core dumped)

I compile my own packages with poudriere and recently switched to a newer server.  The synergy package I installed was from a newer Intel CPU that supported optimizations that my older desktop does not.  These optimizations were being applied to the bundled ext/cryptopp subdir in the synergy sources.  Upstream I believe they are replacing it with openssl in the moderate future.

Server: CPU: Intel(R) Xeon(R) CPU E5-2430 v2 @ 2.50GHz (2500.05-MHz K8-class CPU)
Desktop: CPU: Intel(R) Core(TM) i5 CPU         660  @ 3.33GHz (3325.08-MHz K8-class CPU)

I found in the compile logs that it was using -march=native for the bundled cryptopp which I understand is undesirable for ports to use without being specifically asked for (and I am not asking for it).  I made a patch that I am using with success in poudriere.  I am submitting a patch for the port that should be suitable for committing and it has been compile tested.  I remembered to bump PORTREVISION this time.
Comment 1 Adam McDougall 2016-01-17 15:53:59 UTC
This patch is obsolete since synergy was upgraded to 1.7.x in July 2015. Closing