Skip to content

[herd,test] Include x86_64 catalogue tests in default testing - #1806

Merged
maranget merged 1 commit into
masterfrom
more-x86_64-tests
Apr 23, 2026
Merged

[herd,test] Include x86_64 catalogue tests in default testing#1806
maranget merged 1 commit into
masterfrom
more-x86_64-tests

Conversation

@maranget

@maranget maranget commented Apr 23, 2026

Copy link
Copy Markdown
Member

Basic X86_64 testing was not performed, as pointed out by several comments in PR #1804. Fix this problem by including catalogue X86_64 tests in the default testing make test.

@maranget

This comment was marked as outdated.

@maranget
maranget force-pushed the more-x86_64-tests branch from fc28692 to 18cbb80 Compare April 23, 2026 12:01
@maranget maranget changed the title [herd,test] Include x66_64 catalogue tests in default testing [herd,test] Include x86_64 catalogue tests in default testing Apr 23, 2026

@HadrienRenaud HadrienRenaud left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, thanks @maranget

Maybe we need to be careful with the interactions with #1805, but maybe this one is simpler and could be merged quickly?

@maranget

Copy link
Copy Markdown
Member Author

LGTM, thanks @maranget

Maybe we need to be careful with the interactions with #1805, but maybe this one is simpler and could be merged quickly?

I guess so, especially as PR #1805 is still in draft status.

@relokin

relokin commented Apr 23, 2026

Copy link
Copy Markdown
Member

I agree, please go ahead. It will be easier to add X86_64 to the template test target in #1805.

@diaolo01

Copy link
Copy Markdown
Contributor

Looks good to me!

@maranget
maranget force-pushed the more-x86_64-tests branch from 18cbb80 to a195ce3 Compare April 23, 2026 16:14
@maranget

Copy link
Copy Markdown
Member Author

Thanks @relokin and @diaolo01. Merge on its way.

@maranget
maranget merged commit c85781c into master Apr 23, 2026
3 checks passed
@maranget
maranget deleted the more-x86_64-tests branch April 23, 2026 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants