Fix genconf on macOS

Not sure if this is the ideal solution, but for some reason (likely the .dylib suffix ending) this warning is triggered and the .confdb files are empty on macOS. This fixes the issue (tested on: macos13, clang14, arm)

changed the description

added 1 commit

• a0635538 - pre-commit fixes

Compare with previous version

Can you quote an example of these warning messages you get, please?

It's the one from the line above:

[warning] library [GaudiCommonSvc] exposes factory [SequentialOutputStream] which is declared in [] !!
[warning] library [GaudiCommonSvc] exposes factory [StoreExplorerAlg] which is declared in [] !!
[warning] library [GaudiCommonSvc] exposes factory [StoreSnifferAlg] which is declared in [] !!
[warning] library [GaudiCommonSvc] exposes factory [TagCollectionStream] which is declared in [] !!

Hi @vavolkl, I took a moment to look at this issue.

I believe the problem comes from getDSONameFor which is only implemented if _GNU_SOURCE is defined, which might not be the case on MacOS.

From what I can gather from MacOS man pages on the web, the code used on Linux should work on MacOS out of the box. Can you try to adapt the #ifdef condition to see if it fixes your problem?

changed milestone to %v36r11

changed milestone to %v36r12

changed milestone to %v36r13

@vavolkl, can you check my suggestion?

added 89 commits

• a0635538...27a178af - 88 commits from branch gaudi:master
• 285c5b0d - macos: fix for genconf

Compare with previous version

Thanks @clemenci, I could only do some preliminary tests due to the other macos issue concerning SIP, but I can confirm this indeed fixes the genconf issue.

added 1 commit

• feee1c00 - pre-commit fixes

Compare with previous version

added lhcb-gaudi-head label

assigned to @clemenci

• [2023-04-19 00:04] Validation started with lhcb-gaudi-head#3536
• [2023-04-19 00:12] Validation started with lhcb-run2-gaudi-head#497
• [2023-04-20 00:05] Validation started with lhcb-gaudi-head#3537
• [2023-04-20 00:24] Validation started with lhcb-run2-gaudi-head#498
• [2023-04-21 00:06] Validation started with lhcb-gaudi-head#3538
• [2023-04-21 00:14] Validation started with lhcb-run2-gaudi-head#499
• [2023-04-22 00:06] Validation started with lhcb-gaudi-head#3539
• [2023-04-23 00:04] Validation started with lhcb-gaudi-head#3540
• [2023-04-23 00:12] Validation started with lhcb-run2-gaudi-head#500
• [2023-04-24 00:05] Validation started with lhcb-gaudi-head#3541
• [2023-04-24 00:15] Validation started with lhcb-run2-gaudi-head#501

approved this merge request

mentioned in commit 769abd2a

merged