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…

Merge branch 'master' into next

Merge branch 'fix-floating-move'

Bugfix: Avoid out of bounds coordinates when moving floating windows (Thanks eeemsi)

This commit makes the coordinates proportional when moving floating windows.
That is, if you have a window at the bottom of your 1920 px monitor and move it
to your 800 px monitor, it will be at the bottom of the 800 px monitor (and not
out of bounds).

Merge branch 'role-criterion' into next

docs/userguide: document the window_role criterion

t/65-for_window: add tests for window_role

Implement the window_role criterion (checks WM_WINDOW_ROLE)

Closes: #446
This is handy for matching specific windows of a multi-window application, for
example only Pidgin’s buddy list window.

tests: also make launch_with_config() log to LOGPATH

Merge branch 'master' into next

Merge branch 'fix-output-focus'

Bugfix: Use con_orientation(). Fixes switching between different outputs

Fixes: #518
For stacking containers, direct access using con->orientation is wrong, we need
to use con_orientation().