On 2015-03-19 14:38-0700 Alan W. Irwin wrote:

> Hi Andrew (again):
>
> I just discovered this issue was a regression compared to 5.10.0.  So
> I will git bisect it to see which commit first created this issue, and
> get back to you at that point if I cannot figure out the problem
> introduced by that commit.

According to git bisect the issue first appeared for 6f5dc7b546f05c79

That's one of my commits which is a small but rather subtle symbol visibility 
change given by the
following diff:

software@raven> git diff 6f5dc7b546f05c79^ 6f5dc7b546f05c79
diff --git a/include/pldll.h.in b/include/pldll.h.in
index bdf5f7e..f818b64 100644
--- a/include/pldll.h.in
+++ b/include/pldll.h.in
@@ -131,7 +131,10 @@
    #define PLDLLIMPEXP_PLPLOT_MODULE_DATA( type )    PLDLLIMPORT type
  #endif

-#if defined ( plplotqt_EXPORTS )
+// Note for the case when ENABLE_DYNDRIVERS is not defined, the
+// libplplot build specifically includes bindings/qt-gui/plqt.cpp (see
+// cmake/modules/qt.cmake).
+#if defined ( plplotqt_EXPORTS ) || ( !defined ( ENABLE_DYNDRIVERS ) && 
defined ( plplot_EXPORTS ) )
    #define PLDLLIMPEXP_QT    PLDLLEXPORT
    #define PLDLLIMPEXP_QT_DATA( type )    PLDLLEXPORT type
  #else

So the difference means that for the case where ENABLE_DYNDRIVERS is
not #defined, then for _both_ the build of libplplotqt and the build
of libplplot, PLDLLIMPEXP_QT and PLDLLIMPEXP_QT_DATA are both defined
(which makes complete sense to me since some the qt device driver code
gets into libplplot for the nondynamic devices case). So I don't see
anything wrong with the above change at all, but if anyone spots a
problem, please let me know.  Meanwhile, I will keep investigating by
seeing whether temporarily reverting the above change for master tip
solves the segfault issue (confirming the above change is
the only source of the problem) and by investigations with nm.

Alan
__________________________
Alan W. Irwin

Astronomical research affiliation with Department of Physics and Astronomy,
University of Victoria (astrowww.phys.uvic.ca).

Programming affiliations with the FreeEOS equation-of-state
implementation for stellar interiors (freeeos.sf.net); the Time
Ephemerides project (timeephem.sf.net); PLplot scientific plotting
software package (plplot.sf.net); the libLASi project
(unifont.org/lasi); the Loads of Linux Links project (loll.sf.net);
and the Linux Brochure Project (lbproject.sf.net).
__________________________

Linux-powered Science
__________________________

------------------------------------------------------------------------------
Dive into the World of Parallel Programming The Go Parallel Website, sponsored
by Intel and developed in partnership with Slashdot Media, is your hub for all
things parallel software development, from weekly thought leadership blogs to
news, videos, case studies, tutorials and more. Take a look and join the 
conversation now. http://goparallel.sourceforge.net/
_______________________________________________
Plplot-devel mailing list
Plplot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/plplot-devel

Reply via email to