search

kingsfordgroup / sailfish

Rapid Mapping-based Isoform Quantification from RNA-Seq Reads
http://www.cs.cmu.edu/~ckingsf/software/sailfish
GNU General Public License v3.0
124 stars 45 forks source link

PPC64le: Make fails on Jellyfish. Need --without-sse #94

Open jlost opened 8 years ago

jlost commented 8 years ago

I'm trying to get Sailfish to work on an IBM POWER8 system (PPC64le RedHat). Sailfish fails during make due to SSE-specific instructions in Jellyfish. Tackling the Jellyfish issue (https://github.com/gmarcais/Jellyfish/issues/63) separately, one workaround was to configure Jellyfish with: --without-sse.

Alter the CMake script to check for SSE on its own and provide the necessary flag, or provide a flag for an existing Jellyfish installation. The end result should be that sailfish is able to pass this particular build step when building on PPC64le, either automatically or by specifying an external Jellyfish installation.

Millak commented 6 years ago

Somewhere between jellyfish-2.2.3 and 2.2.10 jellyfish was patched. This fix neither adds a check for SSE nor allows for an existing jellyfish installation, but it does fix the compiling. If jellyfish must remain at 2.2.3 I can prepare a different fix.

rafaeldelucena commented 5 years ago

@jlost this was already fixed, right?

jlost commented 9 months ago

It looks like #110 would fix it, but I no longer have access to the hardware to verify. At any rate, updating patch versions of libraries is usually a good idea.