Re: [Plplot-devel] Planning for the release of 5.13.0 --Cygwin Ada issue

[Alan W. I. <ir...@be...>]

>> -----Original Message-----
>> From: Alan W. Irwin [mailto:ir...@be...]
>> Sent: Tuesday, August 01, 2017 5:02 AM
>> To: Arjen Markus
>> Cc: PLplot development list
>> Subject: Planning for the release of 5.13.0 --Cygwin Ada issue
>>
>> On 2017-07-31 07:13-0000 Arjen Markus wrote:
>>
>>> Here are some results from my most recent, i.e. yesterday's, testing session.
>>
>> Hi Arjen:
[...]
>> It turns out this is a long-standing issue, see your post from more than a year ago
>>
>> 2016-03-17 Arjen Markus    (21K) RE: comprehensive testing results on Cygwin -
>> Ada build errors
>>
>
> Forgot all about that. The script I use for cdash reports takes care of the extra information needed for Ada, so I thought it worked fine.
>
>> However, I think I have a lot better understanding of the issue now, then I did back
>> then.
>>
> ...
>
>>
>> So to test that possibility, please use the -DNON_TRANSITIVE=ON cmake option
>> for this test to see if (a) you get further with Ada, and (b) if that option causes
>> issues for other parts of our comprehensive tests for the Cygwin platform.  And
>> depending on what combination of (a) and (b) you obtain, I will take it from there.
>>

> Very much the same thing: link errors concerning doubly defined
symbols.

Thanks for that tarball report.  I double-checked with it
that the offending command line was very much shortened by the
above option, i.e., it is now

/usr/bin/gnatmake.exe  -Wl,--enable-auto-import "-aI/cygdrive/d/plplot-svn/plplot-git/examples/ada" "-aI/cygdrive/d/plplot-svn/plplot-git/bindings/ada" "-aL/cygdrive/d/plplot-svn/comprehensive_test_disposeable/shared/noninteractive/build_tree/bindings/ada/CMakeFiles/plplotada.dir" xtraditional13a.adb -cargs  -largs ../../dll/libplplotada.dll.a ../../dll/libplplot.dll.a

with only the two directly needed libraries mentioned by the -cargs
-largs combination of options.

So the implementation of this idea was perfect, but (as you said) the
(bad) result is very much the same, sigh.

So I am going to make the official decision now to not investigate
this issue again until after the release.  Which is good because
it simplifies our pre-release lives considerably. :-)

So document your launch script for Cygwin with a datestamped comment
about why you are disabling Ada, and you are done for now
with this Ada issue on Cygwin.

However, after the release you should build and comprehensively test a
good simple test case (i.e., see cmake/test_ada/README for
instructions about how to do that) on Cygwin, and then I would plan to
use your results for that as the basis of a bug report to Cygwin to
see if the Ada packager for that platform can figure out why your
build of that simple test case is (presumably) going wrong.

I have written up this testing idea in my post-release notes so there
should be no need to remind me about this idea then.  :-)

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
__________________________