[pypy-dev] cppyy comments/questions

[wlavrijsen at lbl.gov]

Hi Uwe,

> rules in translator/platform/posix.py places $(LDFLAGSEXTRA) before $(OBJECTS)
> ('$(TARGET)', '$(OBJECTS)', '$(CC_LINK) $(LDFLAGSEXTRA) -o $@ $(OBJECTS) 
> $(LIBDIRS) $(LIBS) $(LINKFILES) $(LDFLAGS)'),

ah, but that then just means that I should not use the 'link_extra' keyword in
the ExternalCompilationInfo object (see cppyy/capi/reflex_capi.py) for -lReflex,
but instead it should be part of $(LIBS), i.e. keyword 'libraries'.

Will check that in shortly, after running a full translation.

But again, the ECI is going to go away soon.

Thanks,
      Wim

--- a/pypy/module/cppyy/capi/reflex_capi.py     Mon Aug 13 17:18:12 2012 -0700
+++ b/pypy/module/cppyy/capi/reflex_capi.py     Tue Aug 14 09:48:58 2012 -0700
@@ -35,7 +35,7 @@
      include_dirs=[incpath] + rootincpath,
      includes=["reflexcwrapper.h"],
      library_dirs=rootlibpath,
-    link_extra=["-lReflex"],
+    libraries=["Reflex"],
      use_cpp_linker=True,
  )

-- 
WLavrijsen at lbl.gov    --    +1 (510) 486 6411    --    www.lavrijsen.net