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

fix windows error running version 8.x #2

Merged
merged 2 commits into from
Sep 22, 2022

Conversation

oryan-block
Copy link

@oryan-block oryan-block commented Sep 20, 2022

Copying over the fix from jakemonO@cfd5ecc

This fixes this error: wix-incubator#189

I ran our build against this using 5.7.x and 8.x version.

File[] soFiles = libDir.listFiles(filter);
for (File file : soFiles) {
Files.createSymbolicLink(Paths.get(baseDir + SEP + "bin" + SEP + file.getName()), Paths.get(file.getPath()));
System.out.println("Symlink " + file.getName());

Choose a reason for hiding this comment

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

Do we need this println anymore?

Copy link
Author

Choose a reason for hiding this comment

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

Could be useful for debug.
I'd replace it with a logger.

Copy link

@abel-eblock abel-eblock left a comment

Choose a reason for hiding this comment

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

🚀

@oryan-block oryan-block merged commit 0f37734 into master Sep 22, 2022
@oryan-block oryan-block deleted the bug/windows-version8-fix branch September 22, 2022 14:59
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.

3 participants