Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds color output support for tmux terminals #6

Merged
merged 1 commit into from
Mar 26, 2015

Conversation

michaeltanner
Copy link
Contributor

Glog did not recognize the 256 color tmux terminal as "color enabled" via the TerminalSupportsCollor() method. This pull request adds that support.

@ukai ukai added the LGTM label Mar 25, 2015
@ukai ukai self-assigned this Mar 25, 2015
ukai added a commit that referenced this pull request Mar 26, 2015
Adds color output support for tmux terminals
@ukai ukai merged commit 852970d into google:master Mar 26, 2015
durswd pushed a commit to durswd/glog that referenced this pull request Sep 2, 2019
Adds color output support for tmux terminals
GerHobbelt pushed a commit to GerHobbelt/glog that referenced this pull request Aug 9, 2022
This removes email sending functionality as per yugabyte/yugabyte-db#11930.

The relevant gflags remain in place to avoid any issues with clusters that already have the flags set, but no longer do anything.
# Conflicts:
#	src/logging.cc
GerHobbelt pushed a commit to GerHobbelt/glog that referenced this pull request Sep 13, 2023
This removes email sending functionality as per yugabyte/yugabyte-db#11930.

The relevant gflags remain in place to avoid any issues with clusters that already have the flags set, but no longer do anything.
# Conflicts:
#	src/logging.cc

# Conflicts:
#	src/logging.cc
GerHobbelt added a commit to GerHobbelt/glog that referenced this pull request Sep 13, 2023
…it SHA-1: 89752a9 :: Remove email sending functionality (google#6)

- fix compiler error due to merge failure
- fix for our gtest library: `GTEST_OS_*` defines are set to 0 or 1, so no `#ifdef` check but `#if GTEST_OS_...` instead.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants