kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #21494
Re: [PATCH 1/4] Use CMake high-level facility for visibility
I do not think so.
Right now, the nightly packages use something like this for CMake settings:
KiCad CMake Settings: -DDEFAULT_INSTALL_PATH=/Library/Application
Support/kicad
-DCMAKE_C_COMPILER=/usr/bin/clang
-DCMAKE_CXX_COMPILER=/usr/bin/clang++
-DCMAKE_OSX_SYSROOT=/Users/wolf/projects/kicad-mac-packaging/MacOSX10.7.sdk
-DCMAKE_OSX_DEPLOYMENT_TARGET=10.7
-DwxWidgets_CONFIG_EXECUTABLE=../wx/wx-bin/bin/wx-config
-DKICAD_SCRIPTING=ON
-DKICAD_SCRIPTING_MODULES=ON
-DKICAD_SCRIPTING_WXPYTHON=ON
-DPYTHON_EXECUTABLE=/usr/local/bin/python
-DPYTHON_SITE_PACKAGE_PATH=/Users/wolf/projects/kicad-mac-packaging/wx/wx-bin/lib/python2.7/site-packages
-DCMAKE_INSTALL_PREFIX=../bin
-DCMAKE_BUILD_TYPE=Release
Many of the paths are dynamically generated.
(If there are other settings I *should* be using, please let me know.)
On Wed, Dec 2, 2015 at 9:18 AM, Wayne Stambaugh <stambaughw@xxxxxxxxx>
wrote:
> No hurry. The stable release OSX package has priority. I just need
> confirmation on OSX builds that the patch wont break anything. One
> other question: do you use the KICAD_BUILD_STATIC or KICAD_BUILD_DYNAMIC
> options for your build configuration? I can't see where they were used
> outside of OSX_DEP_BUILDER code I removed so I want to remove them if
> possible.
>
> On 12/2/2015 10:13 AM, Adam Wolf wrote:
> > I can but I need to get the 4.0.0 build out first.
> >
> > If someone else can do it sooner than end of this week/weekend, please
> do :)
> >
> > On Wed, Dec 2, 2015 at 8:30 AM, Wayne Stambaugh <stambaughw@xxxxxxxxx
> > <mailto:stambaughw@xxxxxxxxx>> wrote:
> >
> > Would one of our OSX devs please confirm this doesn't break anything
> on
> > OSX builds before I commit it.
> >
> > Thanks,
> >
> > Wayne
> >
> > On 12/1/2015 2:14 AM, Simon Richter wrote:
> > >
> > > CMake 3.0 defines two new variables,
> > >
> > > * CMAKE_CXX_VISIBILITY_PRESET and
> > > * CMAKE_VISIBILITY_INLINES_HIDDEN
> > >
> > > to control whether symbols not explicitly tagged for export are
> > implicitly
> > > exported. Because only version 3.3 and following also applies that
> to
> > > static libraries when in 3.3 mode, compatibility code is added as
> > well.
> > >
> > > When the minimum required version is bumped to 3.3, this code
> becomes
> > > obsolete and a warning is displayed that the compatibility code
> > should be
> > > removed as well.
> > > ---
> > > CMakeLists.txt | 32
> > ++++++++++++++++++++++++--------
> > > CMakeModules/PerformFeatureChecks.cmake | 2 --
> > > 2 files changed, 24 insertions(+), 10 deletions(-)
> > >
> > >
> > >
> > > _______________________________________________
> > > Mailing list: https://launchpad.net/~kicad-developers
> > > Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> > <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> > > Unsubscribe : https://launchpad.net/~kicad-developers
> > > More help : https://help.launchpad.net/ListHelp
> > >
> >
> > _______________________________________________
> > Mailing list: https://launchpad.net/~kicad-developers
> > Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> > <mailto:kicad-developers@xxxxxxxxxxxxxxxxxxx>
> > Unsubscribe : https://launchpad.net/~kicad-developers
> > More help : https://help.launchpad.net/ListHelp
> >
> >
>
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help : https://help.launchpad.net/ListHelp
>
Follow ups
References