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

Add maven build shortcut to plugin.xml #1956

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

kufnerg
Copy link

@kufnerg kufnerg commented Mar 12, 2025

Removing this shortcut broke the "shift-alt-x + m" shortcut to simply run a preconfigured maven build. Currently the "shift-alt-x + m" shortcut has no effect what so ever.

Removing this shortcut broke the "shift-alt-x + m" shortcut to simply run a preconfigured maven build. Currently the "shift-alt-x + m" shortcut has no effect what so ever.
Copy link
Member

@laeubi laeubi left a comment

Choose a reason for hiding this comment

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

@kufnerg this does not seem the right approach, this adds a new Launch Shortcut just to make a a key combination work, I would expect this to be a keybinding instead of a launch shortcut.

@kufnerg
Copy link
Author

kufnerg commented Mar 12, 2025

@laeubi There may very well be a better solution for the issue ( Maybe you can suggest one ? ), this is just the way I fixed it on my local installation. I'ts been broken for quite some time now and nothing is being done about that. It's just annoying to have to change the plugin.xml in the jar file after every installation of eclipse.

@laeubi
Copy link
Member

laeubi commented Mar 12, 2025

I would go that way, define a keybinding and then run the required actions:

https://www.vogella.com/tutorials/EclipseCommandsKeybindings/article.html

@kufnerg
Copy link
Author

kufnerg commented Mar 12, 2025

screenshot
I don't know. I think the actions suggested in the little popup shown in eclipse after "shift-alt-x" should work out of the box without extra configuration !

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