[Octave-bug-tracker] [bug #51758] [octave-forge] (interval) fails to build on non-x86 archs

[Colin Macdonald]

Follow-up Comment #2, bug #51758 (project octave):

It doesn't work for me.

I cherry-picked that patch.  But I always get --enable-sse2, independent of
what is in /proc/cpuinfo.

For example, I has findstring for "WTF" instead of "sse2":

HAS_SSE2       =$(findstring WTF,$(shell cat /proc/cpuinfo))
ifdef HAS_SSE2
CONF_FLAG_SSE2 ?=--enable-sse2
endif


And I *still* get --enable-sse2.  Perhaps HAS_SSE2 is defined but empty: [1]
says "if it occurs, the value is find; otherwise, the value is empty."

[1] https://www.gnu.org/software/make/manual/html_node/Text-Functions.html

Seems to me we should just have:

CONF_FLAG_SSE2 =$(findstring sse2,$(shell cat /proc/cpuinfo))


But I don't know how to support "?=" with such (which I guess was
the reason for the "ifdef HAS_SSE2" stuff.)


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?51758>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/