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

IsolatedRemoteDevModeMain Mutable Jar error message #34230

Closed
rjtmahinay opened this issue Jun 21, 2023 · 2 comments · Fixed by #34258
Closed

IsolatedRemoteDevModeMain Mutable Jar error message #34230

rjtmahinay opened this issue Jun 21, 2023 · 2 comments · Fixed by #34258
Labels
area/housekeeping Issue type for generalized tasks not related to bugs or enhancements
Milestone

Comments

@rjtmahinay
Copy link
Contributor

rjtmahinay commented Jun 21, 2023

Description

Misleading error message when JAR type is not Mutable Jar in IsolatedRemoteDevModeMain.

At Line 93:

 if (!start.getJar().getType().equalsIgnoreCase(PackageConfig.BuiltInType.MUTABLE_JAR.getValue())) {
                    throw new RuntimeException(
                            "remote-dev can only be used with mutable applications generated with the fast-jar format");
}

Should it replace fast-jar format to mutable-jar format right?

Seeing also that in PackageConfig, mutable-jar is treated as fast-jar. But not quite sure if it should be fast-jar format since there's a dedicated FAST_JAR.

    public boolean isFastJar() {
        return type.equalsIgnoreCase(PackageConfig.BuiltInType.JAR.getValue()) ||
                type.equalsIgnoreCase(PackageConfig.BuiltInType.FAST_JAR.getValue()) ||
                type.equalsIgnoreCase(PackageConfig.BuiltInType.MUTABLE_JAR.getValue());
    }

Implementation ideas

Replace message to: "remote-dev can only be used with mutable applications generated with the mutable-jar format"

@rjtmahinay rjtmahinay added the area/housekeeping Issue type for generalized tasks not related to bugs or enhancements label Jun 21, 2023
@geoand
Copy link
Contributor

geoand commented Jun 22, 2023

I can understand why this confusing, but technically it is correct.

But maybe we should just change the message to "remote-dev can only be used with mutable applications i.e, using the mutable-jar package type"

@rjtmahinay
Copy link
Contributor Author

@geoand Agree on adding the specific package type. Will open a PR on this.

rjtmahinay added a commit to rjtmahinay/quarkus that referenced this issue Jun 22, 2023
rjtmahinay added a commit to rjtmahinay/quarkus that referenced this issue Jun 22, 2023
rjtmahinay added a commit to rjtmahinay/quarkus that referenced this issue Jun 22, 2023
@quarkus-bot quarkus-bot bot added this to the 3.3 - main milestone Jun 23, 2023
michelle-purcell pushed a commit to michelle-purcell/quarkus that referenced this issue Jun 23, 2023
gsmet pushed a commit to gsmet/quarkus that referenced this issue Jun 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/housekeeping Issue type for generalized tasks not related to bugs or enhancements
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants