EPICS Controls Argonne National Laboratory

Experimental Physics and
Industrial Control System

2002  2003  2004  2005  2006  2007  2008  2009  2010  2011  2012  2013  2014  2015  2016  2017  2018  2019  <20202021  2022  2023  2024  Index 2002  2003  2004  2005  2006  2007  2008  2009  2010  2011  2012  2013  2014  2015  2016  2017  2018  2019  <20202021  2022  2023  2024 
<== Date ==> <== Thread ==>

Subject: Re: [Merge] ~anj/epics-base/+git/base-7.0:add-makeAPIheader into epics-base:7.0
From: mdavidsaver via Core-talk <core-talk at aps.anl.gov>
To: Andrew Johnson <anj at anl.gov>
Date: Mon, 30 Mar 2020 20:32:04 -0000
I wonder if this idea could be extended to enable $@_CFLAGS and similar.

> 2. MAD wrote: "libComAPI.h still includes shareLib.h due to the unfortunate example ...
> 
> My generated API.h files don't currently include shareLib.h ...

The current version was:

> #ifdef epicsExportSharedSymbols
> #   define interfacePDQ_epicsExportSharedSymbols
> #   undef epicsExportSharedSymbols
> #endif
...
> #ifdef interfacePDQ_epicsExportSharedSymbols
> #   define epicsExportSharedSymbols
> #   include "shareLib.h"
> #endif

So if a header following this pattern didn't otherwise include shareLib.h afterward,
the epicsShare* qualifiers would not be defined.

The fix is to include shareLib.h unconditionally.

I recall hitting instances where this was happening, but can't find them again at present.
I guess we'll find out when something breaks...

-- 
https://code.launchpad.net/~anj/epics-base/+git/base-7.0/+merge/380606
Your team EPICS Core Developers is subscribed to branch epics-base:7.0.

Navigate by Date:
Prev: Re: [Merge] ~anj/epics-base/+git/base-7.0:add-makeAPIheader into epics-base:7.0 mdavidsaver via Core-talk
Next: [Merge] ~bfrk/epics-base:write-filters into epics-base:7.0 Ben Franksen via Core-talk
Index: 2002  2003  2004  2005  2006  2007  2008  2009  2010  2011  2012  2013  2014  2015  2016  2017  2018  2019  <20202021  2022  2023  2024 
Navigate by Thread:
Prev: Re: [Merge] ~anj/epics-base/+git/base-7.0:add-makeAPIheader into epics-base:7.0 mdavidsaver via Core-talk
Next: Re: [Merge] ~anj/epics-base/+git/base-7.0:add-makeAPIheader into epics-base:7.0 mdavidsaver via Core-talk
Index: 2002  2003  2004  2005  2006  2007  2008  2009  2010  2011  2012  2013  2014  2015  2016  2017  2018  2019  <20202021  2022  2023  2024 
ANJ, 06 May 2020 Valid HTML 4.01! · Home · News · About · Base · Modules · Extensions · Distributions · Download ·
· Search · EPICS V4 · IRMIS · Talk · Bugs · Documents · Links · Licensing ·