- Since dbfc8c88, distcheck is run even
  when there is only a commit in po/
- This is because "extends" merges hashes, but not array, so the
  ".make" rules are overriden by the "distcheck" rule
- Use !reference to force the usage of ".make" rules in distcheck (and
  this avoid duplicating the "if" everywhere)

- Since thunar!153,
we have thunar and thunarx documentations, which is not supported by the script
- This break the container build

- Fix #53

This stage is only run for tags and is essentially the same as the
distcheck stage.

- If 4.17.x is available, pick this one
- If 4.16.x is the higher (on component without 4.17 yet), pick it
- "grep" the component name to remove any "xfce-" release tags
- Enhance MR!49

Alternatively we could also pick the latest overall release, but the
downside would be that the container could jump back and forth between
4.17 dev and 4.16 maintenance releases.
git describe --tags `git rev-list --tags --max-count=1` --match "$NAME*"

The static index.html page and artifacts get merged into the newly
added apidocs folder.

- So now we won't generate a broken container anymore is we can't build
  on component

- We don't need to override CFLAGS when we build the CI container, only
  project built WITH the container needs that
- Also, build is broken because of new line in CFLAGS variable

O H authored 4 years ago and Romain Bouvier committed 4 years ago

A number of common coding errors are missed by the compiler.
They turn up only after release once packages are build with full CFLAGS.

Adjust the gitlab CI to catch such errors:

-Wall enables a number of diagnostics

-Wno-deprecated-declarations will hide a bogus warning.
As long as an API exists it can be used. Noone beside a curious developer
will actually take the time to do research and look for the replacment.
In the wild this warning is just noise in the build log.

-Werror=implicit-function-declaration will point out missing includes.
In case such errors are not fixed, the resulting code will assume int
for function arguments and function return values.

-Werror=return-type will point out incorrect return values. The caller
will most likely receive just garbage.

Signed-off-by: Olaf Hering <olaf@aepfle.de>

Also take further measures to make the container slimmer and more portable.
Thanks to Romain and Florian (schuellerf/xfce-test)!

Remove --build and --host since they are only needed for crosscompile.
Native builds don't need those and they will not work in case someone
tries build on 32bit.

Also add envronment variables for libdir and libexecdir so the
debian/ubuntu specific lib(exec)dir can be overridden.

Use all cores from `nproc` rather than hardcode parallel jobs to 8.

Allow overide this via environment variable NPROC:
  `docker build --build-arg NPROC=2 ...`

avoid git warning about 'detached HEAD' state by creating a git branch
for the build.

See also: https://docs.xfce.org/xfce/building#xfce_dependencies_explained