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().

Merge branch 'i3-input-fmt' into next

i3-input: update manpage

i3-input: implement -F (format) option, deprecate -p (prefix)

This introduces the '-F format' parameter, which takes a format and replaces %s
in it with the user input. An example: The user should enter the target
workspace name. The appropriate i3-input invocation looks like this:

    i3-input -F 'workspace "%s"' -P 'Switch to workspace: '

i3-input: add missing \n in --help

i3-input: reformat main.c

Merge branch 'master' into next

Merge branch 'fix-floating-move'

fix-bug-515: always adjust floating position when moving to another output

Floating con position needs to be adjusted when moving to another
output, regardless of whether or not the target workspace is visible.

Merge branch 'master' into next

Merge branch 'fix-warp-same'

Bugfix: Don’t warp the pointer if it already is on the target output (Thanks cls, pnutzh4x0r)

My testcase was putting a floating window on the left output, but overlapping a
little to the right output. Then switch to a workspace on the right output.

Merge branch 'floating-focus' into next

tests: extend t/35-floating-focus to use focus left/right on floating windows

Remove obsolete code for floating focus

Implement focus switching (focus left/right) for floating windows

Fixes: #475

Also make compilation possible on systems with libev 3

From the source:

    We need ev >= 4 for the following code. Since it is not *that* important
    (it only makes sure that there are no i3-nagbar instances left behind) we
    still support old systems with libev 3.

document the dependency on libev >=4.0 for ev_cleanup (Thanks phnom)

libev 4.00 was released 2010-10-25, so nearly a year ago.

Merge branch 'master' into next

Merge branch 'fix-float-size'

Tests: fix floating size, add a test to check floating size when starting in tiling

Send the correct X11 border_width in faked ConfigureNotifys

Bugfix: Honor the X11 window border in ConfigureRequests

Bugfix: Honor the X11 window border when calculating dimensions for floating windows

Bugfix: Don’t use top border height twice when calculating dimensions for floating windows

fix build in 'next' (Thanks thomasba)

Merge branch 'master' into next

Merge branch 'fix-floating-confreq'

Bugfix: Correctly handle ConfigureRequests for floating windows in multi-monitor setups (Thanks thomasba)

Fixes #519

Bugfix: Don’t fix floating coordinates when reassigning (fixes flickering)

This fixes flickering when moving floating windows between different monitors.

Merge branch 'fix-reload-assignments' into next

Bugfix: Correctly free old assignments when reloading

Fixes #516