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

cmake: allow to disable building unit tests

This commit introduces a CMake `BUILD_TESTING` boolean option that
allows to disable unit tests. This is especially useful in superbuilds
where typically only the main library is built. By default, this option
is enabled.

Merge pull request #127 from MinGW-caffe/master

try to avoid the error "conflicting declaration 'typedef DWORD pthread_t'" etc. in MinGW

Add MiniLight to AUTHORS/CONTRIBUTORS

Merge pull request #123 from yying/master

Set sinks_ to NULL after deletion in LogDestination::DeleteLogDestinations

Add Andy Ying to AUTHORS/CONTRIBUTORS

Merge pull request #116 from pcc/fix2

Fix autotools build.

Merge pull request #98 from willyd/relocatable

Added $<BUILD_INTERFACE> to gflags include directory

Add Guillaume Dumont to AUTHORS/CONTRIBUTORS

Merge pull request #129 from sergiud/cmake-PIC

cmake: enable position independent code (PIC)

Merge pull request #130 from sergiud/cmake-warning-fix

cmake: fixed clang warning propagation to gcc

cmake: fixed clang warning propagation to gcc

cmake: enable position independent code (PIC)

try to avoid the error "conflicting declaration 'typedef DWORD
pthread_t'" etc.

NULL sinks_ after deletion to prevent dangling pointer

Add Peter Collingbourne <pcc@google.com> to CONTRIBUTORS

Merge pull request #115 from pcc/fix1

symbolize: Calculate a module's zero VA using program headers.

Fix autotools build.

It looks like commit 3c49b93 modified the auto-generated file src/config.h.in
to add a definition of macro GOOGLE_GLOG_DLL_DECL. One of the autotools
reverts this change upon running "make", causing the build to fail when a
source file includes demangle.h.

To fix the problem, revert the change to src/config.h.in and include
glog/logging.h from demangle.h which provides a definition of that macro.

symbolize: Calculate a module's zero VA using program headers.

Previously we were using a module's "start address", i.e. the
address at which the module's executable region was mapped, as the
zero virtual address, i.e. the address from which the DSO's virtual
addresses are calculated. This works fine for DSOs created by the
bfd and gold linkers, which will emit a PT_LOAD directive into the
program header which loads the executable region at virtual address
(p_vaddr) and file offset (p_offset) 0.

However, the lld linker may place a read-only region before the
executable region, meaning that both p_vaddr and p_offset for the
executable region are non-zero. This means that any symbols resolved
by the symbolizer are resolved to an incorrect virtual address. To
correctly calculate the address corresponding to virtual address zero,
we need to take into account p_vaddr and p_offset.

Specifically, the calculation starts with the "base address", i.e. the
start address minus the file offset. To get from the base address to
virtual address zero, we first add p_offset. This gives us the mapped
address of the start of the segment, or in other words the mapped
address corresponding to the virtual address of the segment. (Note
that this is distinct from the start address, as p_offset is not
guaranteed to be page aligned.) We then subtract p_vaddr, which takes
us to virtual address zero.

Merge pull request #101 from yasushi-saito/master

Allow permission line in /proc/self/map to be "rwx", not just "r-x".

Merge pull request #91 from abdasgupta/master

Added support for PowerPC.

Merge pull request #82 from sergiud/cmake-imported-target

Better support and documentation for CMake projects that consume glog

Allow permission line in /proc/self/map to be "rwx", not just "r-x". At
least in Ubuntu14, a binary file that's writable by the process owner
has permission "rwx", not "r-x".

Added $<BUILD_INTERFACE> to gflags include to avoid including the gflags include dir in the glog-targets.cmake

Merge pull request #96 from mayah/update-windows

Update src/windows/glog/logging.h to ToT

Run src/windows/preprocess.sh

src/windows/glog/logging.h is not generated by the current source.
Let me run src/windows/preprocess.sh and update the source.

Added support for PowerPC.

added CMake usage instructions

cmake: allow to refer to imported glog target as glog::glog

use namespace GFLAGS_NAMESPACE instead namespace gflags

only signalhandler_unittest.cc uses "using namespace gflags".
others use "using namespace GFLAGS_NAMESPACE", so
signalhandler_unittest.cc does the same way.

fixes #62

win: use _fdopen instead of fdopen

The POSIX function fdopen is deprecated.
Use the ISO C++ conformant _fdopen instead.
https://msdn.microsoft.com/library/ms235351.aspx

fixes #73

win: FAILED macro can't be used with HANDLE.

FAILED macro is for HRESULT, not for HANDLE.
FindFirstFile returns INVALID_HANDLE_VALUE when error or not found.
https://msdn.microsoft.com/library/windows/desktop/aa364418(v=vs.85).aspx

fixes #79

avoid calling new/malloc in signalhandler.

FlushLogFilesUnsafe would be called in FailureSignalHandler,
so should avoid calling new/malloc, which are not signal safe.

fixes #78

Merge pull request #74 from yoshisatoyanagisawa/fix_sigabrt_action

Reset SIGABRT action only if FailureSignalHandler is installed.

Reset SIGABRT action only if FailureSignalHandler is installed.

When I set my own signal handler to SIGABRT, it did not executed
with CHECK.  That is because SIGABRT handler is reset to default
just before glog calls abort.
Let me make it reset only if the handler is what glog installed
i.e. FailureSignalHandler.

Merge pull request #58 from ruslo/fix.include

Fix missing public include directory

Merge pull request #56 from rhencke/fix-test

Fix double-free in unit test on Windows.

Merge pull request #51 from flandr/pthread-cmake-build

Link libpthread when necessary in CMake builds

Fix missing public include directory

Path "<prefix>/include" will not be added to the headers search of dependent
targets without specifying INSTALL_INTERFACE, i.e. compilation
will fail with error:

  "fatal error: 'glog/logging.h' file not found".

Add pthread library to linker list when needed

Fix double-free in unit test on Windows.

The LOG_SYSRESULT refers to result twice.  Since, in this test,
result expands to FindClose(handle), the handle was being freed twice.

Merge pull request #49 from sergiud/cmake-visibility

Fixed warning caused by policy CMP0063 introduced in CMake 3.3

Merge pull request #47 from sergiud/master

Allow to disable multithreading support using CMake

Add Teddy Reed to AUTHORS/CONTRIBUTORS

Merge pull request #26 from theopolis/feature-mode-flag

[#23] Add logfile_mode to control logfile permissions

cmake: fixed warning caused by policy CMP0063

[#23] Add logfile_mode to control logfile permissions

cmake: allow to disable multithreading support

This commit additionally enables threading for win32 by default.

Merge pull request #35 from bsilver8192/fix-mocklog-unused-arguments

Fix mocklog unused arguments

Merge pull request #34 from bsilver8192/fix-macro-redefinition

Fix redefinition of _XOPEN_SOURCE.

Add tomange to AUTHORS/CONTRIBUTORS

Merge pull request #20 from UbimoLTD/patch-1

Don't call RAW_VLOG with locking vmodule_lock

SetVLOGLevel locks vmodule_lock however RAW_VLOG at the end also implicitly locks it via InitVLOG3__.
This causes deadlock/segfault.

fixes #29

Merge pull request #12 from ukai/fix_leak

delete LogDestination::sinks_ at DeleteLogDestinations().

fixes #8

Add Sergiu Dotenco to AUTHORS/CONTRIBUTORS