Merge pull request #292 from Mizux/master

CMake Update

Rework CMake glog VERSION management.

- Use of Project version properties instead of custom variables
- fix missmatch between VERSION (build version) and SOVERSION (API version)
src: https://cmake.org/cmake/help/latest/prop_tgt/VERSION.html#prop_tgt:VERSION

remove uneeded enable_testing()

CTest module already provides it

Provide glog::glog ALIAs

Useful when using glog as subproject.
Add an explanation at top like in gflags.git CMakeLists.txt

Bazel: make example more portable

Allow using glog/stl_logging.h with Bazel

Fixes #289.

Add workaround for Bazel build on macOS

This works around https://github.com/bazelbuild/bazel/issues/3979,
and so closes #282.

cmake: remove config.h from public headers (closes #238)

Merge pull request #283 from Mizux/master

Use target gflags instead of old VARIABLES

Use target gflags instead of old VARIABLES

Merge pull request #276 from drigz/add-example

Add an example using glog from Bazel

Add an example using glog from Bazel

Merge pull request #277 from drigz/update-gflags

Update gflags to latest master

cmake: use gflags package config instead of a find module

This allows to avoid conflicts between third party gflags find modules floating
around. If a package provide a local version of gflags find module, clients
which locate glog will incorrectly use it. Another problem is a CMake error message
"cmake_policy PUSH without matching POP" in nested find_package calls. In summary,
we need to ensure to use the original gflags package config.

cmake: ensure consumers do not overwrite the required policy

removed obsolete vsprojects

cmake: added a postfix for debug builds (closes #237)

This allows to deploy debug and release builds concurrently.

Merge pull request #281 from sergiud/msvc-error-define

Fixed undeclared identifier error

logging_unittest: fixed undeclared identifier error (fixes #267)

Merge pull request #278 from shinh/contrib

Update AUTHORS and CONTRIBUTORS for PR#232

Update AUTHORS and CONTRIBUTORS for PR#232

Update gflags to latest master

This means that gflags no longer leaks config.h, so I had to fix a bug
in glog.bzl where config.h is generated at the wrong path.

I also switched to the best-practice for depending on git repositories,
ie using http_archive with a mirror.

Merge pull request #232 from qzmfranklin/bazel

Build with Bazel.

Remove the use of bind() from WORKSPACE.

Remove an empty textual_hdrs field.

Hide certain header files from the cc_library().

This commit addresses a few issues:

    1.  No longer leak config.h in a way similar to
            https://github.com/gflags/gflags/issues/233
        The solution of prefixing the path by 'glog_internal' is modified from
            https://github.com/gflags/gflags/issues/234

    2.  No longer expose internal headers.

    3.  Replace PACKAGE_NAME with native.package_name()

    4.  Uers can choose namespaces via the newly added 'namespace' keyword.

    5.  Replace glob with explicitly listing of files.

    6.  Make the genrules more compact using pythonic list construction.

Can build with Bazel now

Build with Bazel.

Merge pull request #261 from pcc/fix3

Compute base addresses from program headers while reading /proc/self/maps.

Compute base addresses from program headers while reading /proc/self/maps.

We previously had logic to compute the base address from program
headers as part of symbolization. The problem is that we need a correct
base address earlier in order to adjust a PC into the image's address
space, as these addresses can appear in unsymbolized output.

There was previously an assumption that only the mapping that
was lowest in the address space did not need to be adjusted. This
assumption is not guaranteed (for example, the kernel may choose to
map an ET_DYN lowest) and in fact turned out to be wrong in binaries
linked with lld because the first mapping is read-only.

The solution is to move the program header reading logic into the
code that reads /proc/self/maps.

There is a change in semantics for clients that install a callback
using the InstallSymbolizeOpenObjectFileCallback function. Any such
clients will need to return a correct base address from the callback
by reading program headers using code similar to that in the function
OpenObjectFileContainingPcAndGetStartAddress.

Merge pull request #106 from dimhotepus/master

Cache strlen outside of cycles (PVS-Studio)

Merge pull request #260 from ukai/shell_escape

Shell escape arguments to /bin/mail.

Shell escape arguments to /bin/mail.

Style fixes for consistency

Merge pull request #245 from DariuszOstolski/issue211

Fix username lookup in case of missing USER environment variable

Merge pull request #254 from DariuszOstolski/issue253

#253: Use MS preprocessor idiom to disable warning

#253: Use MS preprocessor idiom to disable warning

Merge pull request #226 from sergiud/zero-allocation-log-failure

Zero allocation fix

fixed gcc logging failure

Added HAVE_PWD_H to cmake build

Fix username lookup in case of missing USER environment variable

Merge pull request #228 from sergiud/dll-export-fix

Fix for missing exports (fixes #227)

added missing exports (fixes #227)

Merge pull request #225 from jray272/remove-log-every-n-assert

Fix LOG_EVERY_N with clang -Wunused-local-typedef

Remove GOOGLE_GLOG_COMPILE_ASSERT

This compile time assert is no longer used anywhere in glog. Remove
it.

Merge pull request #224 from jray272/run-windows-script

Run src/windows/preprocess.sh to pick up latest logging.h.in changes

Fix LOG_EVERY_N with clang -Wunused-local-typedef

Glog uses a pre-C++11 compile time assert to verify the validity of
the severity parameter for LOG_EVERY_N. Unfortunately, some compilers
will complain about the usage of LOG_EVERY_N with
"-Wunused-local-typedef" due to the way the compile time assert is
constructed. This makes it impossible to use LOG_EVERY_N with this
warning treated as an error.

The fix simply removes the assert entirely. This is safe to do since
you can't put anything invalid into the severity parameters without
generating a compile error elsewhere. This has been safe to do ever
since the GLOG_ prefixes were added as part of 6febec361e.

Fixes #223

Update Windows logging.h based on 2df0ca34aa

Commit changes to src/windows/glog/logging.h that were missed in
2df0ca34aa. Because a change to src/glog/logging.h.in was made,
src/windows/preprocess.sh needed to be run.

Add Jim Ray to CONTRIBUTORS

Merge pull request #218 from andschwa/remove-sln

Update Windows docs

Update Windows readme for CMake

Update gitignore for CMake and Visual Studio

Delete ancient Visual Studio Solution file.

Merge pull request #216 from google/revert-120-master

Revert "configure.ac: Allow user to disable gflags"

Revert "configure.ac: Allow user to disable gflags"

Merge pull request #120 from ribalda/master

configure.ac: Allow user to disable gflags

Merge pull request #209 from sergiud/cmake-icc-fix

cmake: fixed gflags namespace detection (fixes #193)

cmake: fixed gflags namespace detection (fixes #193)

This commit fixes gflags namespace detection using Intel compiler and ensures
the generated try_compile command line does not produce malformed paths.

Merge pull request #153 from sergiud/cmake-gnu-install-dirs

cmake: do not hardcode relative install directories

Merge pull request #152 from sergiud/cmake-project-name

cmake: changed project name from google-glog to glog

Merge pull request #157 from sergiud/cygwin-support

Cygwin support

Merge pull request #158 from sergiud/zero-allocation

[RFC] reduce heap memory allocations to zero

Merge pull request #145 from pixelb/rate-limit-posix-fadvise

rate limit calls to posix_fadvise()

reduce memory allocations to zero

Merge pull request #199 from v1bri/brian-fix-gflags-link

Use gflags ALIAS instead of ${gflags_XXX} vars

Merge pull request #168 from andschwa/windows-stacktrace

Port stack tracing to Windows

Support signal handler on Windows

Link to DbgHelp using pre-processor directive

This method ensure that all users of glog get automatically linked to
the DbgHelp library without needing to set compiler flags.

Export GetStackTrace

Necessary when building with BUILD_SHARED_LIBS=1.

Support symbolizer and demangler on Windows

Support stacktrace on Windows

Copy stacktrace_generic-inl.h for Windows

Add Andrew Schwartzmeyer to authors and contributors

Specify gflags version in CMakeLists.txt

Use gflags ALIAS instead of ${gflags_XXX} vars

The gflags project updated their CMake config last year with a
`gflags` ALIAS target. This can be used instead of the legacy
`${gflags_LIBRARIES}` and `${gflags_INCLUDE_DIRS}` variables. It also
looks cleaner.

Fixes #198

Merge pull request #188 from shinh/remove-configure

Remove files generated by autotools

closes #165

Remove files generated by autotools

for #165. I think not many projects have them in their repo.

Update the version in CMakeLists.txt to 0.3.5

Merge pull request #186 from shinh/backport

Backport internal changes

Add a hack for naive include scanners

Define OS_LINUX only if it's not defined yet

symbolize: Allow 4kB stack consumption on PPC64

x86 stacktrace: Use __builtin_frame_address if possible

Relax test for symbolize

Don't rely on an internal-linkage extern "C" function having an
unmangled name. This isn't required by the ABI, and in fact is not
valid for a conforming compiler(!). Instead, allow symbolization to
produce either a mangled or an unmangled name here.

Use LOG(FATAL) instead of CHECK(false)

Run src/windows/preprocess.sh

glog: release 0.3.5

Merge pull request #179 from renning22/master

CHECK_NOTNULL works with smart pointers when compiled in C++11.

CHECK_NOTNULL works with smart pointers when compiled in C++11.

Merge pull request #169 from mayah/windows-no-return

Add __declspec(noreturn) on Win

Add __declspec(noreturn) on Win

Currently cl.exe doesn't know LOG(FATAL) exits the program. Set
__declspec(noreturn).

Merge pull request #164 from yoshisatoyanagisawa/dcheck_always_on

DCHECK_ALWAYS_ON to make D* enabled under NDEBUG

DCHECK_ALWAYS_ON to make D* enabled under NDEBUG

The macro NDEBUG could be automatically defined for release build on
some build environments (e.g. MSVC).  If we use NDEBUG as a key to
distinguish using DCHECK as CHECK (I call this DCHECK is enabled) or
not, we cannot make DCHECK enabled for release build on such
environments.

Considering people use a program with glog for presubmit testing or
dogfooding, they should need to do release build with DCHECK enabled.

fixed cygwin compilation errors

cmake: do not hardcode relative install directories

cmake: changed project name from google-glog to glog

This fixes the default install location on Windows. Specifically,
find_package works now out-of-the box with the default
CMAKE_INSTALL_PREFIX.

rate limit calls to posix_fadvise()

There can be a large kernel overhead involved in POSIX_FADV_DONTNEED.
There is no point in calling this per item logged, so rate limit
to at most once per 2MiB written.

With a simple test program that logs 100K items at WARNING level:

Before:

  $ time strace -c -e fadvise64 log.test \
    -log_dir=/dev/shm -logtofiles=true -logtostderr=false
  % time     seconds  usecs/call     calls    errors syscall
  ------ ----------- ----------- --------- --------- ----------------
  100.00   12.522509         125     99957           fadvise64
  ------ ----------- ----------- --------- --------- ----------------
  real    0m52.671s
  user    0m2.194s
  sys     0m44.022s

After:

  $ time strace -c -e fadvise64 log.test \
    -log_dir=/dev/shm -logtofiles=true -logtostderr=false

  % time     seconds  usecs/call     calls    errors syscall
  ------ ----------- ----------- --------- --------- ----------------
  100.00    0.000759         152         5           fadvise64
  ------ ----------- ----------- --------- --------- ----------------
  real    0m4.206s
  user    0m1.436s
  sys     0m3.153s

Fixes issue #84

configure.ac: Allow user to disable gflags

Under some circumstances like cross-compilation, the user might not want
to enable support for gflags.

This patch allows support for --without-gflags

Signed-off-by: Ricardo Ribalda Delgado <ricardo.ribalda@gmail.com>

Update github.com/sergiud in AUTHORS/CONTRIBUTORS

Merge pull request #132 from sergiud/cmake-disable-unittests

cmake: allow to disable building unit tests