Use ROOT Sink sysStop instead of stop

This is to prevent that the stop method is invoked twice (once by UpdateAndReset::stop and one by ServiceManager::stop).

While a double call of stop is not a big deal in Gaudi v36, it causes an exception in Gaudi v37 (with the new implementaion of BaseSink).

/cc @sponce @rmatev

added lhcb-gaudi-head label

• [2023-10-19 00:07] Validation started with lhcb-gaudi-head#3718
• [2023-10-19 00:39] Validation started with lhcb-gaudi-head#3718
• [2023-10-19 07:20] Validation started with lhcb-gaudi-head#3719
• [2023-10-19 11:39] Validation started with lhcb-gaudi-head#3720
• [2023-10-19 14:20] Validation started with lhcb-master-mr#9494
• [2023-10-20 00:07] Validation started with lhcb-gaudi-head#3721
• [2023-10-20 00:40] Validation started with lhcb-gaudi-head#3721
• [2023-10-20 10:57] Validation started with lhcb-master-mr#9504

mentioned in merge request LHCb!4322 (merged)

added bug fix cleanup labels

@clemenci Is this supposed to be merged -- ie backwards compatible?

From my tests this is perfectly backward compatible.

merged

mentioned in commit 6735dadd

mentioned in merge request !919 (merged)