Merge branch 'testsuite' into next

complete-run: eliminate dependency on EV

complete-run: explicitly state why we need to overwrite SIGCHLD

tests: eliminate List::MoreUtils

tests: eliminate Try::Tiny

tests: Eliminate IO::Scalar

tests: remove unused Proc::Background

docs/testsuite: explain how socket activation works in i3

Merge branch 'testsuite' into next

tests: move i3test.pm from t/lib to lib/

tests: use a lot higher timeouts

This makes the tests more robust on a machine which is loaded.
Also, it makes the tests run (significantly) longer if anything goes wrong.

tests: don’t overwrite $timeout, kill timer when done

tests: Convert all testcases to use the new launch_with_config API

It now returns a pid directly, not a Proc::Background object.

tests: launch_with_config: use socket activation

tests: SocketActivation: append to the logfile, more comments

tests: SocketActivation: only close() and dup2() if fileno(socket) != 3

tests: Refactor the socket activation into lib/SocketActivation.pm

tests: re-order dependencies in complete-run, make clear which are shipped with Perl

tests: remove unused dependency Proc::Background from complete-run

tests: eliminate dependency on the DateTime module by using POSIX::strftime

tests: make complete-run use POSIX::close(3) instead of reserving a file descriptor when starting

This makes it possible to run complete-run.pl with a "tainted" environment in
which fd 3 (and possibly others) are already present.

Bugfix: Warp pointer to the correct window when using the 'focus' command

Comment in the code makes the fix clear, I think.

Steps to reproduce the issue:
1) On the right monitor, open two terminals (h-split workspace)
2) 'i3 mark foo' on the right terminal
3) Focus the left terminal
4) Switch to the left monitor
5) i3 '[con_mark="foo"] focus'
6) the left window will contain the pointer (and is focused for a short period
   of time, or even permanently due to race conditions)

Merge branch 'master' into next

Merge branch 'fix-i3bar-man'

escape font name

The double-dash gets scrambled by a2x otherwise

Merge branch 'ws_next' into next

Mention 'move workspace next/prev' in the userguide

add a test for the 'move workspace next/prev' command

add "move workspace next" and "move workspace prev"

some factorization would be better here, however I don't really know my
way around bison

rename the internal function to _workspace_show, add a comment to workspace_show_by_name

refactor workspace_show and friends

Merge branch 'msg' into next

Make i3 send arguments as command to a running i3 instance (like i3-msg)

From i3 --help:

If you pass plain text arguments, i3 will interpret them as a command
to send to a currently running i3 (like i3-msg). This allows you to
use nice and logical commands, such as:

i3 border none
i3 floating toggle
i3 kill window

Cleanup i3 --help

Move ipc_send_message and ipc_recv_message to libi3

Make i3-msg and src/ipc.c use it

i3-msg: use smalloc, sstrdup from libi3

move sstrdup, scalloc, smalloc, srealloc to libi3, improve error messages

Include libi3.h (Thanks fernandotcl)

Makefile: include libi3/ in 'dist'

Merge branch 'get-socketpath' into next

Implement i3 --get-socketpath, useful for IPC scripts

In order to not depend on X11 just for getting the socket paths, scripts or
other programs can now use i3 --get-socketpath. Since i3 must be present on the
computer anyways, this saves one dependency :).

i3-msg: use socket_path_from_x11 from libi3

Introduce libi3, an *internal* library to eliminate code duplication

Makefiles: prefix compilation messages (for parallel builds)

makefile: make SUBDIRS constant

remove obsolete comment

Merge branch 'master' into next

Merge branch 'userguide-mark'

userguide: fix typo

Signed-off-by: Michael Walle <michael@walle.cc>

Merge branch 'master' into next

Merge branch 'cloexec-errorlog'

set close-on-exec flag on errorlogfile

Make sure the file descriptor is closed within the child after forking.

Signed-off-by: Michael Walle <michael@walle.cc>

Check for PCRE version and disable PCRE_UCP for <= 8.10

Include i3-sensible-* in 'make dist'

Merge branch 'master' into next

Merge branch 'fix-focus-ipc'

Bugfix: IPC: Correctly dump the 'focus' array

tests: Use AnyEvent::I3’s get_marks (requires AE::I3 >=0.08)

We also don’t bother with timeouts anymore. It’s expected to run the tests with
a sufficiently recent version of i3. The tests will just hang if it doesn’t
work.

Merge branch 'nagbar-wrap' into next

Introduce the i3-sensible-{pager,editor,terminal} scripts

The former two provide fallbacks in case $PAGER or $EDITOR is not set (which
might be more common than you think, because they have to be set in
~/.xsession, not in the shell configuration!) while the latter tries to launch
a terminal emulator. The scripts are most prominently used in i3-nagbar, which
alerts the user when the configuration is broken for some reason. Also,
i3-sensible-terminal is used in the default configuration.

This commit does not rely on the shell supporting ${PAGER:-less} anymore, which
is not the case for 'fish'.

docs/testsuite: a few corrections by fernandotcl

s/This adds up quickly/Delays add up quickly (Thanks mxf)

Merge branch 'testsuite' into next

Add docs/testsuite

tests: make sure to leave no tempfiles behind

tests: Bugfix: 11-goto.t: use mktemp for generating a random mark, not base64

The base64 string could contain / and + which is treated specially since we
implemented PCRE support :)

tests: lib/i3test: Use //= instead of unless exists $args{key} (Thanks mxf)

tests: refactor t/58-wm_take_focus to use wait_for_event

tests: lib/i3test: Remove open_standard_window, introduce open_window

open_window has a better API than open_standard_window. It uses named
parameters and supplies default values for everything you don’t specify. This
way, you can use every feature which X11::XCB::Window supports.

tests: complete-run: remove debugging messages

tests: complete-run: display time i3 took for starting up

tests: complete_run: directly use X11::XCB instead of ::Connection

This saves about 0.5s wallclock time due to not starting up Moose/Mouse.
This is worthwhile when you develop a new feature and you are often invoking
complete_run for one specific test.

tests: use wait_for_{map,unmap} to eliminate more sleep()s

tests: lib/i3test: provide wait_for_map and wait_for_unmap

These functions should be used instead of calling wait_for_event directly when
waiting for MAP_NOTIFY or UNMAP_NOTIFY

tests: implement sync_with_i3 and use it instead of sleep()

Also use open_standard_window() in a few more places where appropriate

lib/i3test.pm: reformat exports list

Implement the I3_SYNC client protocol

This is mainly useful for the testsuite. The tests can wait until i3 processed
all X11 events and then continue. This eliminates sleep() calls which leads to
a more robust and faster testsuite.

lib/i3test.pm: Don’t sleep(0.25), but wait until the window was mapped

This makes it faster and less racey

Merge branch 'master' into next

Merge branch 'fix-dragging-fullscreen'

Skip dragging of floating cons in fullscreen mode

Merge branch 'master' into next

Merge branch 'fix-close-focus'

Bugfix: Correctly revert focus to other floating windows when closing a floating window

Uncovered by the testsuite \o/

Merge branch 'master' into next

Merge branch 'fix-variables-cfg'

Bugfix: use bufcopy instead of buf when boundary checking (Thanks thomasba)

Also replace the useless (bufcopy + (next - bufcopy)) with next

Merge branch 'master' into next

Merge branch 'fix-variables-cfg'

A bit more boundary-checking when replacing variables.

Makes valgrind happy

Bugfix: Correctly split key/value when parsing variables (Thanks xeen)

update docs/multi-monitor to include the output names and refer to the force_xinerama config directive

mention the force_xinerama configfile directive in the userguide

Add force_xinerama configuration option

The configuration option does the same as the commandline parameter, except
it can be easily set by the user (e.g. you are using KDM and can't start a
session through ~/.xsession).

Signed-off-by: Michael Walle <michael@walle.cc>

Merge branch 'master' into next

Merge branch 'fix-i3bar-x11'

i3bar: Bugfix: Check if the X11 connection is unavailable

This fixes the condition where the i3 socket for some reason did not produce an
error, but the X server exited (earlier than i3?) and the left-over i3bar
process would consume 100% CPU.

How to reproduce the problem:
1) Start ./testcases/Xdummy :8
2) Start DISPLAY=:8 i3bar -s <socket path to i3 on :0>
3) Kill the Xdummy

Merge branch 'master' into next

Merge branch 'fix-focus'

Bugfix: Only change focus in tree_close() when the container was focused before

Fixes: #441
I’m not sure if that’s really all there is to it. Seems a bit too simple :).
We’ll see if I missed some corner cases in the next few days…