<p dir="ltr">Vijay </p>
<p dir="ltr">If MOABconfig.h contains the moab version information then that is sufficient for our needs.</p>
<p dir="ltr">Thanks</p>
<p dir="ltr">Andy </p>
<div class="gmail_extra"><br><div class="gmail_quote">On Aug 21, 2016 1:30 PM, "Vijay S. Mahadevan" <<a href="mailto:vijay.m@gmail.com">vijay.m@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Andy, yes I see that MOABConfig.h is now not installed. This is my<br>
bad. I was trying to make sure MOABConfig.h is not part of the<br>
distribution but my change affected the installations also. Thanks for<br>
verifying this.<br>
<br>
Also, Version.h is removed because all the defines are now available<br>
in MOABConfig.h. If there is a need for the original Version.h still,<br>
please let me know why.<br>
<br>
I'll push a fix to master soon. Thanks,<br>
<br>
Vijay<br>
<br>
On Sat, Aug 20, 2016 at 7:13 PM, Andrew Davis <<a href="mailto:andrew.davis@wisc.edu">andrew.davis@wisc.edu</a>> wrote:<br>
> Also to be clear, Version.h has gone?<br>
><br>
> On Sat, Aug 20, 2016 at 7:10 PM, Andrew Davis <<a href="mailto:andrew.davis@wisc.edu">andrew.davis@wisc.edu</a>> wrote:<br>
>><br>
>> Iulian<br>
>><br>
>> We use Version.h, it just turns out our ifdef we incorrect so it wasn't<br>
>> being used properly. I'll write a test and pop it on in.<br>
>><br>
>> So on our Docker based Ubuntu 14.04 build I consistently see this behavior<br>
>> where moab/MOABConfig.h is not correctly installed. It appears to be made,<br>
>> but not installed with make install. I've also built it on two different<br>
>> linux machines with the same result. It seems make install does not install<br>
>> the header into the include directory?<br>
>><br>
>> Thanks<br>
>><br>
>> Andy<br>
>><br>
>> Thanks<br>
>><br>
>> Andy<br>
>><br>
>> On Fri, Aug 19, 2016 at 4:38 PM, Grindeanu, Iulian R. <<a href="mailto:iulian@mcs.anl.gov">iulian@mcs.anl.gov</a>><br>
>> wrote:<br>
>>><br>
>>> MOABConfig.h should not be part of tar file, it was removed from<br>
>>> distribution. It should not be part of distribution. This change is part of<br>
>>> vijaysm/fix-dist (pull request #254)<br>
>>><br>
>>> I don't know anything that uses moab/Version.h<br>
>>><br>
>>> MOABConfig.h is built when you configure, not when you autoreconf -fi<br>
>>><br>
>>><br>
>>> Iulian<br>
>>> ______________________________<wbr>__<br>
>>> From: <a href="mailto:moab-dev-bounces@mcs.anl.gov">moab-dev-bounces@mcs.anl.gov</a> [<a href="mailto:moab-dev-bounces@mcs.anl.gov">moab-dev-bounces@mcs.anl.gov</a>] on<br>
>>> behalf of Andrew Davis [<a href="mailto:andrew.davis@wisc.edu">andrew.davis@wisc.edu</a>]<br>
>>> Sent: Friday, August 19, 2016 4:17 PM<br>
>>> To: <a href="mailto:moab-dev@mcs.anl.gov">moab-dev@mcs.anl.gov</a><br>
>>> Subject: [MOAB-dev] MOABconfig.h<br>
>>><br>
>>> Hi Folks<br>
>>><br>
>>> Has MOABconfig.h been removed in the master branch? My autotools based<br>
>>> build seems to not have it any more? Also, does anyone know if there are any<br>
>>> tests that rely on moab/Version.h, I've only just noticed but it seems that<br>
>>> for my builds it doesn't actually do anything?<br>
>>><br>
>>> Thanks<br>
>>><br>
>>> Andy<br>
>><br>
>><br>
><br>
</blockquote></div></div>