search

IBMStreams / streamsx.dps

The IBMStreams/streamsx.dps GitHub repository is home to the Streams toolkit named DPS (Distributed Process Store)
http://ibmstreams.github.io/streamsx.dps
Other
4 stars 9 forks source link

AT compiler check - move fix path setting #88

Closed streams-toolkits closed 3 years ago

streams-toolkits commented 4 years ago

The ppc64 & ppc64le machines recommend to use AT compiler, but the fix path setting is not good way to specify the compiler path. It should be set by a CXX variable or by the PATH.

markheger commented 4 years ago

com.ibm.streamsx.dps/impl/Makefile has commented the ppc specific part and AT compiler is not hard coded with a fixed path, but ppcenv.sh contains hard-coded path settings to AT compiler.

anouri commented 4 years ago

The Makefile of streamsx.dps has to be corrected. https://github.com/IBMStreams/streamsx.dps/blob/master/Makefile

ZollnaPa commented 4 years ago

The Makefile includes the target 'check-compiler' but it is removed from 'all' target. If needed, the user must make an explicit call of the 'check-complier' target. 

all: check-streams gen-message ${SUBDIRS:%=%.all}
markheger commented 3 years ago

@ZollnaPa can this issue be closed? or is something missing?

nysenthil commented 3 years ago

Hi Mark, It is not clear how many customers are running IBM Streams on PPC. It all depends on that factor if we have a need to keep supporting the newer versions of the DPS toolkit in the PPC environment.

markheger commented 3 years ago

It is more a question about have settings in Makefile or outside Makefile, rather than support or not, if I understand the issue correctly.

streams-toolkits commented 3 years ago

The ant files provide workaround. There is no impact on customer builds.