8313274: [BACKOUT] Relax prerequisites for java.base-jmod target

[jaikiran]

Can I please get a review of this change which proposes to fix https://bugs.openjdk.org/browse/JDK-8313274?

The commit in this PR reverts the commit that was introduced in #14561. Martin, in the 8313274 JBS issue, notes that the failure started happening with that change.

The issue can be consistently reproduced using --with-jobs=1 during bash configure and then triggering a build using make images. After the revert of that commit, the build now passes with both --with-jobs=1 and without the --with-jobs option (in which case it uses 8 jobs on my setup).

The other PR where this commit was introduced noted that:

Fixing this won't impact the build much, but certainly won't hurt either.

which I think meant that the change in that PR is mostly a good to have. Given that it does cause these unexpected failures, the proposal in this PR is to revert that change until we have a better way to do that change (if we want to do it).

tier testing is currently in progress with this change.

---

Progress

• Change must be properly reviewed (1 review required, with at least 1 Reviewer)
• Change must not contain extraneous whitespace
• Commit message must refer to an issue

Issue

• JDK-8313274: [BACKOUT] Relax prerequisites for java.base-jmod target (Sub-task - P1)

Reviewers

• David Holmes (@dholmes-ora - Reviewer)

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk.git pull/15118/head:pull/15118
$ git checkout pull/15118

Update a local copy of the PR:
$ git checkout pull/15118
$ git pull https://git.openjdk.org/jdk.git pull/15118/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 15118

View PR using the GUI difftool:
$ git pr show -t 15118

Using diff file

Download this PR as a diff file:
https://git.openjdk.org/jdk/pull/15118.diff

Webrev

Link to Webrev Comment

[bridgekeeper]

👋 Welcome back jpai! A progress list of the required criteria for merging this PR into master will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

[openjdk]

@jaikiran The following label will be automatically applied to this pull request:

• build

When this pull request is ready to be reviewed, an "RFR" email will be sent to the corresponding mailing list. If you would like to change these labels, use the /label pull request command.

[mlbridge]

Webrevs

• 00: Full (710542d9)

[jaikiran]

tier testing is currently in progress with this change.

tier1, tier2 and tier3 tests have passed with this change.

[dholmes-ora]

Backout looks good but please update the JBS issue as per:
https://openjdk.org/guide/#backing-out-a-change
Thanks.

[openjdk]

@jaikiran This change now passes all automated pre-integration checks.

ℹ️ This project also has non-automated pre-integration requirements. Please see the file CONTRIBUTING.md for details.

After integration, the commit message for the final commit will be:

8313274: [BACKOUT] Relax prerequisites for java.base-jmod target

Reviewed-by: dholmes

You can use pull request commands such as /summary, /contributor and /issue to adjust it as needed.

At the time when this comment was updated there had been 16 new commits pushed to the master branch:

• 53ca75b: 8313312: Add missing classpath exception copyright header
• 87d7e97: 8311031: JTable header border vertical lines are not aligned with data grid lines
• 8248e35: 8313576: GCC 7 reports compiler warning in bundled freetype 2.13.0
• 6d180d5: 8313349: Introduce abstract void HtmlDocletWriter.buildPage()
• bc1d2ea: 8312821: Javac accepts char literal as template
• cff25dd: 8306582: Remove MetaspaceShared::exit_after_static_dump()
• 4ba81f6: 8313368: (fc) FileChannel.size returns 0 on block special files
• c1a3f14: 8312078: [PPC] JcmdScale.java Failing on AIX
• 6446792: 8312617: SIGSEGV in ConnectionGraph::verify_ram_nodes
• b093880: 8313322: RISC-V: implement MD5 intrinsic
• ... and 6 more: https://git.openjdk.org/jdk/compare/e8471f6bbe692a0d1e293f9e09aaa4f32312eb6a...master

As there are no conflicts, your changes will automatically be rebased on top of these commits when integrating. If you prefer to avoid this automatic rebasing, please check the documentation for the /integrate command for further details.

➡️ To integrate this PR with the above commit message to the master branch, type /integrate in a new comment.

[jaikiran]

Thank you David for the review and the pointer to the backout instructions. I've followed those instructions and updated this PR title as well the relevant JBS issues to track this backout.
I'll go ahead and integrate this PR in the next hour or so.

[jaikiran]

/integrate

[openjdk]

Going to push as commit 3c920f9.
Since your change was applied there have been 16 commits pushed to the master branch:

• 53ca75b: 8313312: Add missing classpath exception copyright header
• 87d7e97: 8311031: JTable header border vertical lines are not aligned with data grid lines
• 8248e35: 8313576: GCC 7 reports compiler warning in bundled freetype 2.13.0
• 6d180d5: 8313349: Introduce abstract void HtmlDocletWriter.buildPage()
• bc1d2ea: 8312821: Javac accepts char literal as template
• cff25dd: 8306582: Remove MetaspaceShared::exit_after_static_dump()
• 4ba81f6: 8313368: (fc) FileChannel.size returns 0 on block special files
• c1a3f14: 8312078: [PPC] JcmdScale.java Failing on AIX
• 6446792: 8312617: SIGSEGV in ConnectionGraph::verify_ram_nodes
• b093880: 8313322: RISC-V: implement MD5 intrinsic
• ... and 6 more: https://git.openjdk.org/jdk/compare/e8471f6bbe692a0d1e293f9e09aaa4f32312eb6a...master

Your commit was automatically rebased without conflicts.

[openjdk]

@jaikiran Pushed as commit 3c920f9.

💡 You may see a message that your pull request was closed with unmerged commits. This can be safely ignored.