Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enabled some modes for SVTTestRepo/XML tests #582

Closed
wants to merge 2 commits into from

Conversation

AshithaVijay
Copy link

Some of the modes where disabled in ottawa.csv which caused some XML test to get skipped. To execute the required tests, below modes where enabled in respective platform:

aix_ppc-32 linux_ppc-32 linux_x86-32 linux_ppcle-64_ncr linux_ppc-64_le linux_390-31 zos_390-31
500 500 500 500 500 503 503
501 501 501     505 505
503 503 503     506 551
505 505 505     551 555
506 506 551     555 556
551 551 554     556  
554 555 556     557  
555 554 615     610  
556 556 506        
557 557 555        
610 610 557        
615 615 610        

Copy link
Contributor

@llxia llxia left a comment

Choose a reason for hiding this comment

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

@AshithaVijay @varshm16 and @Edith-Mercy, could you work with each other? You are providing similar PRs.

Related PRs:
#527
#459

Also, reiterate #527 (comment)

I would prefer to be cautious when merging changes in ottawa.csv, as it's difficult to discern what has been altered in the file. Please list which mode on which platform has changed, and why.

Also, please do full testing with the Semeru build.

@llxia
Copy link
Contributor

llxia commented Dec 6, 2024

Is anyone still actively working on this PR? If not, please close it. It's been inactive for ~5 months.

@llxia
Copy link
Contributor

llxia commented Dec 18, 2024

According to #459 (comment), close the issue.

@llxia llxia closed this Dec 18, 2024
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.

2 participants