Skip to content

[ITS vertexer] Fix compilation with ITS debug#5383

Merged
davidrohr merged 2 commits intoAliceO2Group:devfrom
mconcas:fix-its-debug-mode
Feb 6, 2021
Merged

[ITS vertexer] Fix compilation with ITS debug#5383
davidrohr merged 2 commits intoAliceO2Group:devfrom
mconcas:fix-its-debug-mode

Conversation

@mconcas
Copy link
Copy Markdown
Collaborator

@mconcas mconcas commented Feb 4, 2021

It should not change anything as the flag is disabled by default.
I additionaly need the {nv, h}cc compiler to know about some dataformat methods also when compiling host code.

@mconcas mconcas force-pushed the fix-its-debug-mode branch 2 times, most recently from 5ece5a0 to 613ecd0 Compare February 4, 2021 18:30
@mconcas mconcas force-pushed the fix-its-debug-mode branch from 613ecd0 to e6263f0 Compare February 4, 2021 18:34
Copy link
Copy Markdown
Collaborator

@davidrohr davidrohr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks OK to me, errors seem unrelated.
Shall I squash and merge, or is more coming?

@davidrohr
Copy link
Copy Markdown
Collaborator

@ktf @TimoWilken : There seem to be several ctests failing randomly in the last days, most of them framework test.
Like in the o2 and the dataflow CI here, but I have also seen this in other PRs.
Is someone working on this?

@ktf
Copy link
Copy Markdown
Member

ktf commented Feb 5, 2021

yes, me. I think it's #5484. One fix committed. Another one coming.

@mconcas
Copy link
Copy Markdown
Collaborator Author

mconcas commented Feb 5, 2021

Looks OK to me, errors seem unrelated.
Shall I squash and merge, or is more coming?

Sorry, missed last part. It's mergeable. Thanks.

@davidrohr davidrohr merged commit ff46de9 into AliceO2Group:dev Feb 6, 2021
@mconcas mconcas deleted the fix-its-debug-mode branch February 17, 2021 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants