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

Fixed problem with XDG shortcuts https://groups.google.com/forum/#!to… #726

Merged
merged 1 commit into from Nov 14, 2020

Conversation

ThibLind
Copy link

Dear IzPack developers,

Some time ago, I reported two problems (one on Linux and one on Windows) with shortcuts, please see https://groups.google.com/forum/#!topic/izpack-user/UrLck2vG-6U . This is the fix for the problem under Linux. If you accept this patch, then I will submit the other patch for the Windows problem.

Sincerely,

Thibault LINDECKER

PS: Thank you for this great free software!

@reinhapa
Copy link
Member

@ThibLind could you update you're feature branch with the latest changes?

@ThibLind
Copy link
Author

Hello,

Thank you for your response. It's done.

Sincerely,

Thibault LINDECKER

@reinhapa reinhapa requested a review from akuhtz November 12, 2020 08:17
@reinhapa reinhapa added the bug label Nov 12, 2020
Copy link

@akuhtz akuhtz left a comment

Choose a reason for hiding this comment

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

The PR is not rebased but the only change is the check for not empty startMenuShortcuts and this LGTM.

@reinhapa
Copy link
Member

@ThibLind can you rebase your change in order to be merged?

@ThibLind
Copy link
Author

All is good for you now ?

@akuhtz
Copy link

akuhtz commented Nov 12, 2020

👍 Thank you.

@reinhapa
Copy link
Member

@akuhtz have you already opened an issue?

@akuhtz
Copy link

akuhtz commented Nov 13, 2020

@akuhtz
Copy link

akuhtz commented Nov 13, 2020

@ThibLind Would you update the patch with the fix for Windows? That would be less work for us 😉

@reinhapa
Copy link
Member

@akuhtz is there any work left to do before merge?

@akuhtz
Copy link

akuhtz commented Nov 13, 2020

@reinhapa Yes, there is a fix for Windows OS in the referenced google groups thread. I have not seen this earlier unfortunately. But maybe merge this PR now and have another for the Windows issue.

@reinhapa reinhapa merged commit df1e45f into izpack:master Nov 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants