xAODTriggerCnv+xAODTauCnv: cmake fixes

Library dependency fixes.

Define interface libraries for exported headers.

This merge request affects 2 packages:

• Event/xAOD/xAODTauCnv
• Event/xAOD/xAODTriggerCnv

Adding @tamartin ,@martindl ,@akraszna ,@adbailey ,@xiaozhon as watchers

added EDM Tau Trigger master review-pending-level-1 labels

CI Result FAILURE (hash 1bca582b)

	Athena	AthSimulation	AnalysisBase	AthGeneration
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 15266]

Note: Unit test failure also seen in the nightlies http://atlas-computing.web.cern.ch/atlas-computing/links/distDirectory/gitwww/GITWebArea/nightlies/master/2020-06-10T0447/AnalysisBase/x86_64-centos7-gcc8-opt/AnalysisBase/UT/xAODTriggerCnv.log.html

This is now on my radar.

added 1 commit

• a810c525 - remove redundant atlas_install_headers

Compare with previous version

Thanks. Should be fixed now.

This merge request affects 2 packages:

• Event/xAOD/xAODTauCnv
• Event/xAOD/xAODTriggerCnv

Adding @tamartin ,@martindl ,@akraszna ,@adbailey ,@xiaozhon as watchers

CI Result FAILURE (hash a810c525)

	Athena	AthSimulation	AnalysisBase	AthGeneration
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 15331]

Changing label to review-user-action-required until all threads are resolved. Please switch back when ready.

Xiaozhong (L1)

added review-user-action-required label and removed review-pending-level-1 label

added review-pending-level-1 label and removed review-user-action-required label

The failing unit test is a known issue and the changes are fine. Approving.

Jason (L1)

added review-approved label and removed review-pending-level-1 label

resolved all threads

merged

mentioned in commit 80283e3f

added sweep:ignore label