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

dev branch: 1.8.0 installation creates a .net shortcut #479

Closed
jjw24 opened this issue Jun 19, 2021 · 7 comments · Fixed by #500
Closed

dev branch: 1.8.0 installation creates a .net shortcut #479

jjw24 opened this issue Jun 19, 2021 · 7 comments · Fixed by #500
Labels
bug Something isn't working
Milestone

Comments

@jjw24
Copy link
Member

jjw24 commented Jun 19, 2021

this shortcut is created everytime 1.8.0 is installed:
image

@taooceros any ideas?

@jjw24 jjw24 added the bug Something isn't working label Jun 19, 2021
@taooceros
Copy link
Member

2021-06-17 00:53:27> ApplyReleasesImpl: Squirrel Enabled Apps: []
2021-06-17 00:53:27> ApplyReleasesImpl: No apps are marked as Squirrel-aware! Going to run them all
2021-06-17 00:53:27> ApplyReleasesImpl: About to create shortcuts for createdump.exe, rootAppDir C:\Users\1\AppData\Local\FlowLauncher
2021-06-17 00:53:27> ApplyReleasesImpl: Creating shortcut for createdump.exe => C:\Users\1\AppData\Roaming\Microsoft\Windows\Start Menu\Programs\Microsoft Corporation\Microsoft® .NET.lnk
2021-06-17 00:53:27> ApplyReleasesImpl: About to save shortcut: C:\Users\1\AppData\Roaming\Microsoft\Windows\Start Menu\Programs\Microsoft Corporation\Microsoft® .NET.lnk (target C:\Users\1\AppData\Local\FlowLauncher\createdump.exe, workingDir C:\Users\1\AppData\Local\FlowLauncher\app-1.7.2, args , toastActivatorCSLID acc29e55-6fa9-5c52-8861-f900e093184f)
2021-06-17 00:53:27> ApplyReleasesImpl: Creating shortcut for createdump.exe => C:\Users\1\Desktop\Microsoft® .NET.lnk
2021-06-17 00:53:27> ApplyReleasesImpl: About to save shortcut: C:\Users\1\Desktop\Microsoft® .NET.lnk (target C:\Users\1\AppData\Local\FlowLauncher\createdump.exe, workingDir C:\Users\1\AppData\Local\FlowLauncher\app-1.7.2, args , toastActivatorCSLID acc29e55-6fa9-5c52-8861-f900e093184f)

1 similar comment
@taooceros
Copy link
Member

2021-06-17 00:53:27> ApplyReleasesImpl: Squirrel Enabled Apps: []
2021-06-17 00:53:27> ApplyReleasesImpl: No apps are marked as Squirrel-aware! Going to run them all
2021-06-17 00:53:27> ApplyReleasesImpl: About to create shortcuts for createdump.exe, rootAppDir C:\Users\1\AppData\Local\FlowLauncher
2021-06-17 00:53:27> ApplyReleasesImpl: Creating shortcut for createdump.exe => C:\Users\1\AppData\Roaming\Microsoft\Windows\Start Menu\Programs\Microsoft Corporation\Microsoft® .NET.lnk
2021-06-17 00:53:27> ApplyReleasesImpl: About to save shortcut: C:\Users\1\AppData\Roaming\Microsoft\Windows\Start Menu\Programs\Microsoft Corporation\Microsoft® .NET.lnk (target C:\Users\1\AppData\Local\FlowLauncher\createdump.exe, workingDir C:\Users\1\AppData\Local\FlowLauncher\app-1.7.2, args , toastActivatorCSLID acc29e55-6fa9-5c52-8861-f900e093184f)
2021-06-17 00:53:27> ApplyReleasesImpl: Creating shortcut for createdump.exe => C:\Users\1\Desktop\Microsoft® .NET.lnk
2021-06-17 00:53:27> ApplyReleasesImpl: About to save shortcut: C:\Users\1\Desktop\Microsoft® .NET.lnk (target C:\Users\1\AppData\Local\FlowLauncher\createdump.exe, workingDir C:\Users\1\AppData\Local\FlowLauncher\app-1.7.2, args , toastActivatorCSLID acc29e55-6fa9-5c52-8861-f900e093184f)

@taooceros
Copy link
Member

It seems that we need to mark the enabled app for squirrel, or else it will create shortcut for every .exe exist in the package.

@taooceros
Copy link
Member

Squirrel/Squirrel.Windows#1644 this is a bug, so a potential workaround is to remove the createdump.exe since most user won't need to use it.

@jjw24
Copy link
Member Author

jjw24 commented Jun 19, 2021

ok cool thanks for looking into this @taooceros

@taooceros
Copy link
Member

dotnet/runtime#43716 We may not be able to remove the createdump.exe. Someone has made a workaround for squirrel. Maybe we can create a custom squirrel to do that.

@taooceros taooceros added this to the 1.8.0 milestone Jun 21, 2021
@taooceros
Copy link
Member

I check the issue again. It is possible to remove the createdump.exe if we remove it from dep.json. I will create a pr for doing that soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants