The next day sonar commit

Merge remote-tracking branch 'github/master'

MC: warn the users when reaching max-depth with DPOR

Github Action : add mattermost notification (#377)

* try to notify mattermost on jarfile creation

* test2

* test3

* more error messages

* actually send the message on build failure

ChangeLog for non-linear resource sharing

Host: non-linear constraints

Allows the use of non-linear constraints for CPU resources.

Add API in s4u::Host::set_sharing_policy
- LINEAR(default, shared resources) or NONLINEAR

Example: examples/cpp/exec-cpu-nonlinear

s4u::Link: avoid set_sharing_policy==WIFI

Implements sharing policy for split-duplex links.
Replicate config for each up/down link

Link: Non-linear constraints

Add non-linear constraints to links.
Similar to disks, user can use a callback to dynamically change the
link's capacity based on the number of flows sharing it.
Only traffic explicit generated by the user is considered as active
flow, so cross-traffic communications aren't taken into account.

Extend s4u::Link::sharing_policy
- New type: NONLINEAR

Example: examples/cpp/network-nonlinear/

Disk: Non-linear contraints

Users can now define a callback to set the capacity of the resource
during the system solve. The callback receives the current capacity and
number of activities/flows currently sharing the resource.

The capacity is defined once at the beginning of the system solve.

- Disk:
  - Add API to set sharing_policy by type: read/write/read-write
  - Example: io-degradation: simulates the degradation in read/write
operation on disks.

Merge branch 'fix-smpi-install-dirs' into 'master'

cmake: fix smpi install dirs (absolute arg)

See merge request simgrid/simgrid!67

cmake: fix smpi install dirs (absolute arg)

Previous code assumed that the following variables were relative paths
- CMAKE_INSTALL_INCLUDEDIR
- CMAKE_INSTALL_LIBDIR

However, using absolute paths there is totally legal in CMake,
and doing so broke installed paths in smpicc/smpicxx.

Fix uses the CMAKE_INSTALL_FULL_<dir> variant instead,
which should work in this case and generally be more robust.

FULL variants exist since at least cmake-3.0.0 so it should be available
everywhere (cmake-3.0.0 was released on June 2014).

References:
- https://cmake.org/cmake/help/latest/command/install.html
- https://cmake.org/cmake/help/latest/module/GNUInstallDirs.html

doc: Add some recent release notes as a lighthouse to the recent developments

Define macro XBT_ATTRIB_DEPRECATED_v333.

Remove deprecated features for next release (3.29).

Start the dev of the 3.29 cycle

Post-release NEWS improvement

forgot to update the python package version number

Netzone::get_children() is actually not deprecated.

Move its declaration outside of #ifndef DOXYGEN.

Typos.

Release v3.28

Fix make distcheck

doc: Fix a subtle formatting error that is not reported by tools

Small improvements to the changelog and doc

I think the changelog is now complete

proposal for a NEWS file for the upcoming release

try to further improve docs/find-missing

Remove useless includes from the public headers

doc: praise for 'discovery reports' as a way to contribute back to the project

also, reword and shorten some other sections in the same file

remove 2 files with no useful content to scavenge from the old doc

Hide some deprecated symbols from doxygen

port find-missing.ignore from autodoxy to breathe

Use == for string comparison [ci-skip].

Simplify, and avoid nested SmpiBenchGuard when rastro_resolution is called by rastro_timestamp.

[sonar] Less implicit conversions.

Add network factors in changelog

doc: fix an issue spotted by find-missing.py

docs: properly report typedefs that are missing in the doc

Adapt docs/find-missing to breathe now that autodoxy is gone

Fix another bunch of warnings in doc generation.

Use 'console' syntax highlighting for shell typescripts.

Various corrections in docs.

Missing const.

Fix debug message better [ci-skip]

Remove dead entries in 'docs/find-missing.ignore'.
Kill unused file 'ignored_symbols'.

Instr: for consistency, rename 'father' to 'parent' here too.

NetZone: rename the remaining occurrence of 'father' to 'parent'.

Cosmetics around deprecated functions.

Kill deprecated private field ActorImpl::userdata_ (use Actor::get/set_data instead).

This is on the *Impl side, and does not need to wait for the deprecation quarantine.

Make comment match the declaration in Engine.hpp.

Vector::at() is good enough to check bounds.

Reduce number of unnecessary copies of 'args'.

Make EngineImpl::tasksTemp a local variable.

Simplify function calls (avoid explicit std::vector<double>).

Either use the variant taking a scalar, or use an initialization list.

Make single argument constructor LinkInRoute::LinkInRoute(const Link*) explicit.

Fix empty #ifndef..#endif. Annotate helper function as deprecated.

SplitDuplexLinkImpl always have SharingPolicy::SPLITDUPLEX. Use a sdingle assert in Link::get_impl().

doc: autodoxy is gone, so remove a workaround for it

Cosmetics in ChangeLog (mostly reorder and rewrap)

small doc improvement

Use reference to const for large input parameters.

Add 'const' for sonar.

Add informative comment near implementation for deprecated functions.

Attribute 'virtual' is redundant with 'override'.

It's an internal error if get_impl returns null. Add an assert to be sure.

(and hope to please fbinfer)

Changelog: C++ interface

Sonar variety

Try to fix opensuse, gcc 11.0.0 build

Merge branch 'split_link_impl' into 'master'

Split-Duplex: s4u support

See merge request simgrid/simgrid!66

Split-Duplex: new management

1. Huge commit to add support to Split-Duplex in S4U.
  - New interface: s4u::SplitDuplexLink

2. Allows its creation through the C++ interface without manually creating
the link UP and DOWN.
  - NetZone::create_split_duplex_link

3. Changes in NetZone::add_route to allow the description of the
direction in C++ interface
  - LinkInRoute: wrap around Link* to indicate the direction
(UP/DOWN/NONE) for the link.
  - UP/DOWN are used to split-duplex links
  - NONE for other types (shared, wifi, fat-pipe)

4. Fix use of split-duplex links in symmetric routes in XML too.
     <route src="alice" dst="bob" symmetrical="YES">
       <link_ctn id="link1" direction="UP"/>
     </route>
  Now this code works, it will add the link1_DOWN as route to
bob->alice. In the past, it would add link1_UP for bob->alice too.

Don't mix LMM sharing policy with S4U.

Add another enum for maxmin sharing policy
In the maxmin system, only FATPIPE and SHARED make sense.

Don't mix C with C++ API here.

This is in the Tutorial section and should be kept simple.

Fix another bunch of dead links in docs.

More trivial fixes for docs.

Remove reference to Mutex/Semaphore ctor and dtor, since they are not public.

Increase requirements for breathe.

Disallow sphinx>=4.0 for now: it's breaking on CI

Exception occurred:
  File /usr/lib/python3/dist-packages/docutils/nodes.py, line 625, in __getitem__
    return self.attributes[key]
KeyError: 'domain'

Try to increase sphinx requirements.

Match version currently available in Debian testing and unstable.

Minor fixes in doc.

Two more const methods spotted by sonar.

More int -> unsigned long.

Don't fail when there's no error!

Fix a few warnings when building doc (mostly related to doxygen).

Correctly remove generated files.

The added "rm" command for s4u-exec-ptasks prevents use of ADD_TESH_FACTORIES
(it chokes on "--cfg=contexts/factory:...").

Get back |br| in XML doc

Fix my previous mistake removing them

This is an unsigned integer.

GraphNodeData::graph_id_ is an unsigned long.

Type consistency.

This is a test for emptiness.

[sonar] Avoid C-style array.

Parameter 'fd' is always -1 for xbt_mheap_new. Kill dead code.

Several improvements in doc [ci-skip]

Declare method const.

Fix type for variable.

Use std::unique_ptr and remove explicit destructor.

Pass the real pointer to the englobing MC class for the event callback.

(avoid accidental upcast)

Use ssize_t for Exec::wait_any and Io::wait_any too.

Use ssize_t as return type for Comm::wait_any.

Use ssize_t as return type for Comm::test_any.

Update .mailmap.

Two more s/pointer/reference/ for std::vector parameter.

Prefer a reference for first parameter of {test,wait}_{all,any}.

Add a test in teshsuite for Comm::wait_all_for.