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

Ignore exclude lists for jck_custom #3927

Conversation

Mesbah-Alam
Copy link
Contributor

This change makes sure that when jck_custom target is used, we'd ignore all exclude files except the default.

Signed-off-by: Mesbah-Alam Mesbah_Alam@ca.ibm.com

@Mesbah-Alam
Copy link
Contributor Author

Test with jck_custom: Grinder_JCK/571
Test with a regular target: Grinder_JCK/570

@Mesbah-Alam Mesbah-Alam requested a review from llxia August 22, 2022 22:28
@Mesbah-Alam
Copy link
Contributor Author

Test with FIPS: Grinder_FIPS/120

@Mesbah-Alam Mesbah-Alam force-pushed the allow-only-standard-excludes-for-jck-custom branch 6 times, most recently from e9bf133 to 84bb351 Compare August 23, 2022 19:35
Signed-off-by: Mesbah-Alam <Mesbah_Alam@ca.ibm.com>
@Mesbah-Alam Mesbah-Alam force-pushed the allow-only-standard-excludes-for-jck-custom branch from 84bb351 to fec19bf Compare August 23, 2022 19:35
@Mesbah-Alam
Copy link
Contributor Author

Upon conversation with @llxia , updated the PR to not exclude the jtx and kfl files from exclude dir while using jck_custom.

Test:

  • with jck_custom : Grinder_JCK/575
  • with jck_custom + FIPS : Grinder_FIPS/122

@llxia
Copy link
Contributor

llxia commented Aug 24, 2022

There is no FIPS jtx for JDK8. Retest this PR on JDK11:
Grinder_FIPS/125
Grinder_FIPS/124

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.

LGTM

Copy link
Contributor

@smlambert smlambert left a comment

Choose a reason for hiding this comment

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

@smlambert smlambert merged commit e31266c into adoptium:master Aug 24, 2022
@Mesbah-Alam Mesbah-Alam deleted the allow-only-standard-excludes-for-jck-custom branch February 6, 2023 13:53
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.

3 participants