readme: use cmake -S instead of undocumented -H

undocumented -H for source directory was replaced
by documented -S and -H is now an alias to --help

easy-builder: improvements

- handle arguments with spaces properly
- provide fallback for nproc
- replace cmake undocumented -H with -S
- do not require to change directory

bikeshedding: minor readme fix

q3map2/smokinguns: fix .tex writing, <3 @vloup

Thanks ElBarto (@vloup) for reporting this

bikeshedding: reduce diff noise with Garux tree

see:
https://github.com/Garux/netradiant-custom/commit/4292115ef741ff066b382c094f8c7814afaccf59#diff-386c03feac6f15ccc15c4dab7bb97320R342

that's not fair

q3map2: pad with zero, not with null pointers, ref #160

see:

https://gitlab.com/xonotic/netradiant/-/merge_requests/160#note_287938815

radiant/buildmenu: rename ".[ExecutableType]" as "[ExecutableExt]"

- introduce GDEF_OS_EXE_EXT
- no need for RADIANT_EXECUTABLE anymore
- keep "[ExecutableType]" parsing but it looks useless

radiant/buildmenu: fixup, ref #138

radiant/buildmenu: better executable extension strip, ref #138

this fixes issue #138 on my end

bikeshedding: some touchup on library-bundler

Merge branch 'missingarg' into 'master'

q3map2: fix segfault when missing bspfile arg on light stage

See merge request xonotic/netradiant!161

q3map2: fix segfault when missing bspfile arg on light stage

Merge branch 'memoryfixes' into 'master'

q3map2: fix memory errors and misuses

See merge request xonotic/netradiant!160

q3map2: use safe_malloc0 when safe_malloc is followed by a memset to 0 with the same size

q3map2: zero the verts array on initialization, verts are tested to not be null in FindMetaVertex()

q3map: magic string, know about the asctime's trailing \n

q3map: magic string, no closing parenthesis

tools/quake2: use same strupr other tools use

tools/hd2data: strupr is windows only, define it elsewhere

some compiler may ask if strstr would have to be used instead, which is wrong

q3map2/vis: do not read outside of script before detecting end of script

autoexpand: add an alternative way to autoexpand while zeroing the allocated memory

use it to allocate shader lump, without that, garbage from uninitialized memory may be written
after the ending \0 of strings

bsp lump write: pad with zeros, not with random unitialized memory data

q3map2/vfs: free temporary allocated dirlist when not used

LoadPNGBuffer: use safe_malloc0 to allocate a zeored imate buffer

valgrind reports there may be issues if this is not initialized

introduce safe_malloc0, also 0 the memory before usage

note called safe_calloc because calloc does not have the same syntax
and this safe_malloc0 uses malloc syntax

unnecessary SAFE_MALLOC ifdef: always defined

valgrind: Conditional jump or move depends on uninitialised value

fix a buffer overflow due to a string not being large enough to store ending \0

Merge branch 'easy-builder' into 'master'

introduce easy-builder script

See merge request xonotic/netradiant!157

update readme

add simple easy-builder script

fail library-bundler script on error

enable crunch if available

crn_rgba: use “algorithm” header to avoid std::min/max being unknown

avoid this error on some systems (Clang on macOS or FreeBSD):

netradiant/libs/crnrgba/crn_rgba.cpp:93:38: error: no
      member named 'max' in namespace 'std'
    const crn_uint32 blocks_x = std::max(1U, (ti.m_width + 3) >> 2);

Merge branch 'illwieckz/gcclt7' into 'master'

Fix compiling using GCC older than 7, obsolete !143, fix #122, <3 @zturtleman, bananovenc

Closes #122

See merge request xonotic/netradiant!159

Fix compiling using GCC older than 7, obsolete !143, fix #122, <3 @zturtleman, bananovenc

Merge branch 'fixpng' into 'master'

radiant: redo 6011367 (!102), fix alpha channel for tRNS cheap transparency PNG

See merge request xonotic/netradiant!158

radiant: redo 6011367, fix alpha channel for tRNS cheap transparency PNG

Merge branch 'packonce' into 'master'

q3map2/light: add option to only store lightmaps on very last bounce, q3map2/bsp: add fast meta option

See merge request xonotic/netradiant!156

q3map2/bsp: introduce -fastmeta option

set lightmap and surface verts and indexes values like quake3, known to be fast

equivalent to:
-maxlightmapverts 64 -maxsurfaceverts 999 -maxsurfaceindexes 6000

q3map2/bsp: introduce -maxsurfacevertices, rename -mv and -mi

- introduce -maxsurfaceverts (was missing as command line option)
- rename -mv to -maxlightmapvertices
- rename -mi to -maxsurfaceindexes

q3map2/light: introduce -nobouncestore

when storing computed lightmap on each bounce, user can interrupt compilation and
get working files, but it spends allocation time (which is slow and single-threaded

with this option, user can decide to only allocate lightmaps on the very final
bounce, it means it can't be interrupted, but it can save a lot of time

q3map2/light: rename -fastallocate to -fastlightmapsearch

Merge branch 'deprecate-makefile' into 'master'

deprecate the legacy Makefile

See merge request xonotic/netradiant!138

makefile: remind future ourselves that conftest.cpp belongs to the legacy Makefile

makefile: move conftest.cpp back right to legacy Makefile since it's required by it

makefile: deprecate the legacy Makefile

requires make to be called this way:

make I_KNOW_MAKEFILE_IS_DEPRECATED=1

Merge branch 'restart' into 'master'

do not prompt for game when restarting after switching game, <3 @Garux

See merge request xonotic/netradiant!155

import some improvements from garux tree

see https://github.com/Garux/netradiant-custom/commit/1a18246a1fcb08ad15da046382684f7754d6aae1

fix cancel button

do not remember game was switched when selected at startup, <3 @Garux

do not prompt for game when restarting after switching game, <3 @Garux

Merge branch 'cleanup' into 'master'

some cleanup to make the repository cooler

See merge request xonotic/netradiant!146

readme: lots of improvements

- compatibility matrix
- Ubuntu build requirements
- a lot of rewrites

cleanup: make appveyor file an hidden dot file

cleanup: move old stuff to oldstuff/

also move in the old COMPILING file that was deleted in 8ac6cfa

We may decide later what to do with them

Merge branch 'bundle' into 'master'

more bundling fixes and improvements, <3 @SpiKe @Calinou

See merge request xonotic/netradiant!154

bundle: make linux bundle that is known to work on both Ubuntu, Arch and Fedora, <3 @SpiKe @Calinou

bundle: fix some minor mistakes (had no bad consequences)

produced the same bundle, but with error messages at build time, and some actions uselessly done more than once

Merge branch 'bundle' into 'master'

last-minute bundling fixes, <3 @SpiKe

See merge request xonotic/netradiant!153

cmake: fix a mistake introduced while revamping bundling code, <3 @SpiKe

bundle: do not bundle libstdc++, pango, and some remaining X11 stuff on Linux

- it's a bad idea to bundle libstdc++ as it may conflict in non-obvious manner
  (like failing on X11 stuff)
- pango is known to conflict with newer distros
- no need to bundle libICE and libSM since we expect user to have a working x11 install

Merge branch 'bundle' into 'master'

factorize the bundling code then implement linux library bundling

See merge request xonotic/netradiant!151

bundle: do not bundle gtk and other deps on linux

- gtk is not easily bundlable on linux because it looks for harcoded system path to optional
  shared libraries like image codecs, theme engines, sound notification system, etc.
  so expect user to install gtk first
- since we ask user to instal gtk, we can also ask them to install gtkglext,
  which is likely to pull gtk itself, x11 and gl dependencies
- old fontconfig does not work correctly if newer fontconfig configuration is installed
- if gtk and fontconfig is installed, freetype is

bundle: more linux bundling

bundle: linux shared lib bundling

- always rely on up-to-date x11 and gl libraries, bundling them will break on future distros

bundle: a bunch of improvements

bundle: chmod +x library-bundler

cmake: BUNDLE_LIBRARIES disables FHS_INSTALL

bikeshedding: spacing

Merge branch 'floating' into 'master'

floating window: use HINT_UTILITY on Windows but normal window everywhere else, <3 @SpiKe, ref !140 !147

See merge request xonotic/netradiant!152

floating window: use HINT_UTILITY on Windows but normal window everywhere else, ref !140 !147

GDK_WINDOW_TYPE_HINT_UTILITY would have been a good option for any OS but KDE is known to be broken (such window is not put in focus).

We must use GDK_WINDOW_TYPE_HINT_UTILITY on Windows to be sure the window is resizeable and to avoid the infinite minimize-restore loop.

<3 @SpiKe

Merge branch 'kingpin' into 'master'

add Kingpin gamepack

See merge request xonotic/netradiant!149

add Kingpin gamepack

Merge branch 'q3map2-cmdline-help' into 'master'

q3map2: add more to q3map2 -help and fix some issues with command-line processing

See merge request xonotic/netradiant!150

q3map2: add missing help for q3map2 stage arguments

Move -keeplights help from -light stage to -bsp. Add other ~40 arguments
that were missing from help. My main focus was on the -convert stage
but I tried to document the rest as well. Some descriptions are copied
from message when enabling the option.

q3map2: fix crash if command-line argument is missing

Running `q3map2 -fs_forbiddenpath -v mapname.map` would crash because
-v gets replaced with NULL in main().

Running `q3map2 -threads` would crash because missing next argument
for number of threads.

q3map2: fix not displaying default args for some q3map2 stages

Fix not displaying default arguments, and possibly segfaulting, when
running q3map2 -bsp, -analyze, -convert, -exportents, -export, or
-import.

Merge branch 'radiant-cmdline-null-mapname' into 'master'

radiant: fix command-line error message for non-map filename

See merge request xonotic/netradiant!148

radiant: fix command-line error message for non-map filename

mapname wasn't set so "(null)" was printed.

Fixes 6ba43753 ("radiant: fix command-line opening on windows").

Merge branch 'floating' into 'master'

make floating windows resizable on Windows, ref !140

See merge request xonotic/netradiant!147

make floating windows resizable on Windows, ref !140

“hint menu” windows are resizeable on Linux but not on Windows,
that's why it's better to use the “hint utility” type instead.

Merge branch 'readme-no-redownload' into 'master'

add how to avoid redownloading gamepacks to README.md

See merge request xonotic/netradiant!145

add how to avoid redownloading gamepacks to README.md

Merge branch 'bintarget' into 'master'

add a "binaries" target, fix #135, <3 @zturtleman

Closes #135

See merge request xonotic/netradiant!144

add a "binaries" target, fix #135, <3 @zturtleman

add a "binaries" target to be able to rebuild everything without
redownloading gamepacks

Merge branch 'cleanup' into 'master'

some clean up

Closes #134

See merge request xonotic/netradiant!142

rename game_packs target to gamepacks

radiant target is now named netradiant, fix #134

also mention the BUNDLE_LIBRARIES cmake option, ref #133

delete the COMPILING file as it is heavily outdated

- macos instructions were not edited in 4 years
- linux and windows instructions were not edited in 9 years

and almost everything in this file is now obsolete or wrong

build instructions in README.md file are meant to be up to date

Merge branch 'restart' into 'master'

make radiant able to restart itself when required, ask user to

See merge request xonotic/netradiant!141

bikeshedding: some wording

radiant: make restart notification a question

thanks to latched variable mechanism
people can chose to not restart immediately

radiant: restart on game change

ask user to restart radiant on game change

radiant: restart on latched preference change

radiant: code to make radiant able to restart itself

- save preferences
- check for map being modified (this asks user for saving if not yet saved)
- check for a map being currently edited
- start a new instance, with the current map as parameter if exists
- quit current instance if new instance started correctly

radiant: get own radiant executable file path

previous code was getting it to compute app_path, lib_path and data_path
but was not keeping the value

radiant: fix command-line opening on windows

clean-up windows path, especially use '/' instead of '\'
as directory separator since since radiant is hardcoded
to use an lookup for '/' anywhere.

Merge branch 'floating' into 'master'

gtk2: do not make floating windows minimizable

See merge request xonotic/netradiant!140

gtk2: do not make floating windows minimizable

First, this feature leads to an horrible behavior on
Windows where the application ends in a loop of endless
minimization and restoration, probably because the window
manager minimizes or restores the whole application once
a floating window in minimized or restored, leading to a
race condition between all floating windows, some being
minimized and others being restored at the same time,
triggering the minimization and the restoration of the
others, and so on. It's difficult to say such issue
will never happen on other OS or with some window manager.

Second, those floating windows are expected to be displayed
or hidden using menu or keyboard shortcut, it's a design
choice. Then the OS-specific way to minimize/restore them
is superflous and less efficient.

Finally, the mainframe is not created as a floating window
so the user minimizes the application by minimizing the
mainframe.