Skip to content

8365533: Remove outdated jdk.internal.javac package export to several modules from java.base #26776

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

Closed
wants to merge 2 commits into from
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 1 addition & 7 deletions src/java.base/share/classes/module-info.java
Original file line number Diff line number Diff line change
Expand Up @@ -154,13 +154,7 @@
// module declaration be annotated with jdk.internal.javac.ParticipatesInPreview
exports jdk.internal.javac to
java.compiler,
jdk.compiler,
jdk.incubator.vector, // participates in preview features
jdk.jartool, // participates in preview features
jdk.jdeps, // participates in preview features
jdk.jfr, // participates in preview features
jdk.jlink, // participates in preview features
jdk.jshell; // participates in preview features
jdk.compiler;
exports jdk.internal.access to
java.desktop,
java.logging,
Expand Down
5 changes: 1 addition & 4 deletions src/jdk.jartool/share/classes/module-info.java
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2014, 2023, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2014, 2025, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
Expand All @@ -23,8 +23,6 @@
* questions.
*/

import jdk.internal.javac.ParticipatesInPreview;

/**
* Defines tools for manipulating Java Archive (JAR) files,
* including the <em>{@index jar jar tool}</em> and
Expand All @@ -49,7 +47,6 @@
* @moduleGraph
* @since 9
*/
@ParticipatesInPreview
module jdk.jartool {
requires jdk.internal.opt;

Expand Down
3 changes: 0 additions & 3 deletions src/jdk.jdeps/share/classes/module-info.java
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,6 @@
* questions.
*/

import jdk.internal.javac.ParticipatesInPreview;

/**
* Defines tools for analysing dependencies in Java libraries and programs,
* including the <em>{@index jdeps jdeps tool}</em>,
Expand Down Expand Up @@ -62,7 +60,6 @@
* @moduleGraph
* @since 9
*/
@ParticipatesInPreview
module jdk.jdeps {
requires java.compiler;
requires jdk.compiler;
Expand Down
5 changes: 1 addition & 4 deletions src/jdk.jfr/share/classes/module-info.java
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2014, 2019, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2014, 2025, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
Expand All @@ -23,15 +23,12 @@
* questions.
*/

import jdk.internal.javac.ParticipatesInPreview;

/**
* Defines the API for JDK Flight Recorder.
*
* @moduleGraph
* @since 9
*/
@ParticipatesInPreview
module jdk.jfr {
exports jdk.jfr;
exports jdk.jfr.consumer;
Expand Down
5 changes: 1 addition & 4 deletions src/jdk.jlink/share/classes/module-info.java
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2015, 2022, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2015, 2025, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
Expand All @@ -23,8 +23,6 @@
* questions.
*/

import jdk.internal.javac.ParticipatesInPreview;

/**
* Defines the <em>{@index jlink jlink tool}</em> tool for creating run-time
* images, the <em>{@index jmod jmod tool}</em> tool for creating and manipulating
Expand Down Expand Up @@ -53,7 +51,6 @@
* @moduleGraph
* @since 9
*/
@ParticipatesInPreview
module jdk.jlink {
requires jdk.internal.opt;
requires jdk.jdeps;
Expand Down
1 change: 0 additions & 1 deletion src/jdk.jshell/share/classes/jdk/jshell/JShellConsole.java
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@
import java.io.PrintWriter;
import java.io.Reader;
import java.nio.charset.Charset;
import jdk.internal.javac.PreviewFeature;

/**
* An interface providing functionality for {@link java.io.Console} in the user's snippet.
Expand Down
5 changes: 1 addition & 4 deletions src/jdk.jshell/share/classes/module-info.java
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2015, 2024, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2015, 2025, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
Expand All @@ -23,8 +23,6 @@
* questions.
*/

import jdk.internal.javac.ParticipatesInPreview;

/**
* Provides the <em>{@index jshell jshell tool}</em> tool for evaluating
* snippets of Java code, and defines a JDK-specific API for modeling and
Expand Down Expand Up @@ -65,7 +63,6 @@
* @moduleGraph
* @since 9
*/
@ParticipatesInPreview
module jdk.jshell {
requires java.logging;
requires jdk.compiler;
Expand Down