From bb6a97a8f3e4e046442eaf8814b30d9c9995bfd2 Mon Sep 17 00:00:00 2001 From: Boris Kolpackov Date: Tue, 30 Aug 2016 16:50:01 +0200 Subject: Build infrastructure update --- .gitignore | 12 ++++++++++-- INSTALL | 1 + build/bootstrap.build | 20 +++++++++++++------- build/root.build | 8 ++------ buildfile | 7 ++++++- hello/buildfile | 23 +++++++++++++++++++---- hello/export | 36 ++++++++++++++++++++++++++++++++++++ hello/hello | 21 +++++++++++++++++++-- hello/hello.cxx | 5 ++--- manifest | 1 + tests/build/root.build | 4 ++-- 11 files changed, 111 insertions(+), 27 deletions(-) create mode 100644 INSTALL create mode 100644 hello/export diff --git a/.gitignore b/.gitignore index e062bf3..01994ef 100644 --- a/.gitignore +++ b/.gitignore @@ -1,5 +1,13 @@ -*.o +# Compiler/linker output. +# *.d +*.o +*.obj *.so +*.dll *.a -core +*.lib +*.exp +*.exe +*.exe.dlls/ +*.exe.manifest diff --git a/INSTALL b/INSTALL new file mode 100644 index 0000000..485b10f --- /dev/null +++ b/INSTALL @@ -0,0 +1 @@ +Just use build2, bro. diff --git a/build/bootstrap.build b/build/bootstrap.build index d0ea1f7..612be77 100644 --- a/build/bootstrap.build +++ b/build/bootstrap.build @@ -1,15 +1,21 @@ project = libhello + +using build@0.4.0-a1 + version = 1.1.1 -revision = 0 +abi_major = 1 +abi_minor = 1 +abi_patch = 1 +abi_prerelease = false -using config -using dist -using test -using install +revision = 0 dist.package = $project-$version if ($revision != 0) -{ dist.package += +$revision -} + +using config +using dist +using test +using install diff --git a/build/root.build b/build/root.build index ebe16d5..4f70114 100644 --- a/build/root.build +++ b/build/root.build @@ -1,10 +1,6 @@ +cxx.std = 11 + using cxx hxx{*}: extension = cxx{*}: extension = cxx - -cxx.std = 11 - -# Don't install tests. -# -tests/: install = false diff --git a/buildfile b/buildfile index f846f89..a416648 100644 --- a/buildfile +++ b/buildfile @@ -1,3 +1,8 @@ d = hello/ tests/ -./: $d doc{version} file{manifest} +./: $d doc{INSTALL version} file{manifest} include $d + +# Don't install tests or the INSTALL file. +# +dir{tests/}: install = false +doc{INSTALL}@./: install = false diff --git a/hello/buildfile b/hello/buildfile index c18fa98..8807f1b 100644 --- a/hello/buildfile +++ b/hello/buildfile @@ -1,11 +1,26 @@ -import libs = libformat%lib{format} -import libs += libprint%lib{print} +import int_libs = libformat%lib{format} +import imp_libs = libprint%lib{print} -lib{hello}: {hxx cxx}{hello} $libs +lib{hello}: {hxx cxx}{hello} hxx{export} $imp_libs $int_libs + +# For pre-releases use the complete version to make sure they cannot be used +# in place of another pre-release or the final version. +# +if $abi_prerelease + lib{hello}: bin.lib.version = @-$version +else + lib{hello}: bin.lib.version = @-$abi_major.$abi_minor cxx.poptions += -I$src_root +obja{*}: cxx.poptions += -DLIBHELLO_STATIC_BUILD +objs{*}: cxx.poptions += -DLIBHELLO_SHARED_BUILD + lib{hello}: cxx.export.poptions = -I$src_root +liba{hello}: cxx.export.poptions += -DLIBHELLO_STATIC +libs{hello}: cxx.export.poptions += -DLIBHELLO_SHARED + +lib{hello}: cxx.export.libs = $int_libs # Install into the hello/ subdirectory of, say, /usr/include/. # -install.include = $install.include/hello +install.include = $install.include/hello/ diff --git a/hello/export b/hello/export new file mode 100644 index 0000000..18f4968 --- /dev/null +++ b/hello/export @@ -0,0 +1,36 @@ +// file: hello/export -*- C++ -*- + +#pragma once + +// Normally we don't export class templates (but do complete specializations), +// inline functions, and classes with only inline member functions. Exporting +// classes that inherit from non-exported/imported bases (e.g., std::string) +// will end up badly. The only known workarounds are to not inherit or to not +// export. Also, MinGW GCC doesn't like seeing non-exported function being +// used before their inline definition. The workaround is to reorder code. In +// the end it's all trial and error. + +#if defined(LIBHELLO_STATIC) // Using static. +# define LIBHELLO_EXPORT +#elif defined(LIBHELLO_STATIC_BUILD) // Building static. +# define LIBHELLO_EXPORT +#elif defined(LIBHELLO_SHARED) // Using shared. +# ifdef _WIN32 +# define LIBHELLO_EXPORT __declspec(dllimport) +# else +# define LIBHELLO_EXPORT +# endif +#elif defined(LIBHELLO_SHARED_BUILD) // Building shared. +# ifdef _WIN32 +# define LIBHELLO_EXPORT __declspec(dllexport) +# else +# define LIBHELLO_EXPORT +# endif +#else +// If none of the above macros are defined, then we assume we are being used +// by some third-party build system that cannot/doesn't signal the library +// type. Note that this fallback works for both static and shared but in case +// of shared will be sub-optimal compared to having dllimport. +// +# define LIBHELLO_EXPORT // Using static or shared. +#endif diff --git a/hello/hello b/hello/hello index 4f85114..75a47a0 100644 --- a/hello/hello +++ b/hello/hello @@ -4,8 +4,25 @@ #include +#include + +#include + namespace hello { - void - say (const std::string& name); + // If you compare this interface to version 1.0, then you will notice that + // while it is API/source-compatible (the signature of say() is unchanged) + // it is not ABI/binary-compatible (say() is now inline). + // + // Notice also that inline say() calls format() which means libformat is an + // interface dependency of libhello. + + LIBHELLO_EXPORT void + say_formatted (const std::string& message); + + inline void + say (const std::string& name) + { + say_formatted (format ("Hello", name)); + } } diff --git a/hello/hello.cxx b/hello/hello.cxx index 46d9f6e..8fdd5a6 100644 --- a/hello/hello.cxx +++ b/hello/hello.cxx @@ -2,7 +2,6 @@ #include -#include #include using namespace std; @@ -10,8 +9,8 @@ using namespace std; namespace hello { void - say (const string& n) + say_formatted (const string& m) { - print (format ("Hello", n)); + print (m); } } diff --git a/manifest b/manifest index 2e5bf06..4871ec1 100644 --- a/manifest +++ b/manifest @@ -11,5 +11,6 @@ goal is to show a canonical build2/bpkg project/package. url: http://www.example.org/libhello email: hello-users@example.org requires: c++11 +requires: build2 >= 0.4.0 depends: libformat [1.0.0 2.0.0-); compatible with libformat-1.X.Y depends: libprint [1.0.0 2.0.0-); compatible with libprint-1.X.Y diff --git a/tests/build/root.build b/tests/build/root.build index 4bf103e..1f7c350 100644 --- a/tests/build/root.build +++ b/tests/build/root.build @@ -1,10 +1,10 @@ +cxx.std = 11 + using cxx hxx{*}: extension = cxx{*}: extension = cxx -cxx.std = 11 - # Every exe{} in this subproject is by default a test. # exe{*}: test = true -- cgit v1.1