Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

.exe file on Windows is 0KB, breaking the shortcut created on the desktop #6313

Closed
luixxiul opened this issue Dec 20, 2016 · 23 comments
Closed

Comments

@luixxiul
Copy link
Contributor

Describe the issue you encountered:
.exe file on Windows is 0KB, breaking the shortcut created on the desktop
clipboard01

Expected behavior:

  • Platform (Win7, 8, 10? macOS? Linux distro?): Windows 10

  • Brave Version: 0.13.0 Preview5

  • Steps to reproduce:

    1. Uninstall and install Brave
  • Any related issues:

@luixxiul
Copy link
Contributor Author

the installation also removed the browser from the default browser setting:

clipboard01

@bsclifton
Copy link
Member

bsclifton commented Dec 20, 2016

@luixxiul the default browser setting being lost is a known issue. Unfortunately, it's been present for as long as I've been contributing to Brave (more info at #5246; originally opened with #2054)

@luixxiul
Copy link
Contributor Author

@bsclifton the thing is not "being lost", the browser is not even listed there :-o which I think a new issue.

@bsclifton
Copy link
Member

@luixxiul yikes-

I'm curious... where do your shortcuts point to? (desktop shortcut and/or taskbar pinned shortcut)

@luixxiul
Copy link
Contributor Author

@bsclifton I'm seeing the same thing as @srirambv does; #6322 (comment)

@bsclifton
Copy link
Member

It seems shortcut is pointing at the new Brave.exe... which is 0 bytes. Besides this breaking the app's icon, this may be causing the default app not showing Brave.

Great screencap, courtesy of @srirambv:
shortcut

@luixxiul luixxiul removed this from the 0.13.0 milestone Dec 22, 2016
@bbondy bbondy added this to the 0.13.0 milestone Dec 22, 2016
@bbondy
Copy link
Member

bbondy commented Dec 22, 2016

This was previously triaged with release/blocking and also in 0.13.0, why was the milestone and label removed @luixxiul ? Thanks!

@luixxiul
Copy link
Contributor Author

my bad, that's my mistake. sorry :-(

@bbondy
Copy link
Member

bbondy commented Dec 23, 2016

np!

@bbondy
Copy link
Member

bbondy commented Dec 28, 2016

There's a couple problems here, our fork doesn't have the stub executable which causes one problem. I'm reverting to electron-winstaller to fix that.

But then that uncovers a bigger issue which is with us not having a squirrel aware application anymore. This makes it so squirrel events aren't sent to our exe and also makes it so our main app executable points to braveDefaults.exe instead. Squirrel has special handling if there are no squirrel aware applications to create a shortcut (by the same name apparently!) for each exe you have.

Bug was introduced here:
brave/muon@14d85d9#diff-e1251fb68363e10586ee5cfda3ee78baL83

Testing out a fix now.

@bbondy bbondy closed this as completed in e583f7b Dec 29, 2016
@luixxiul
Copy link
Contributor Author

It seems that the issue was not really solved.

On Appdata\Local\brave\

clipboard01

The shortcut on Desktop links to Appdata\Local\brave\BraveDefaults.exe

@luixxiul luixxiul reopened this Dec 30, 2016
@bbondy
Copy link
Member

bbondy commented Dec 30, 2016

Are you sure this isn't left over from before? I'm sure it is solved for me.

@bbondy
Copy link
Member

bbondy commented Dec 30, 2016

BraveDefaults.exe there isn't an issue btw, does it create shortcuts that are usable on your desktop?

@bbondy
Copy link
Member

bbondy commented Dec 30, 2016

Also does it create default registry keys for default browser status on install? And on win7 prompt for UAC? If so I don't think this file causes any harm.

@luixxiul
Copy link
Contributor Author

Manually I deleted AppData\local\brave and reinstalled it as usual, with no UAC prompt (I've disabled it).

Need I uninstall from Control Panel and reinstall it?

@bbondy
Copy link
Member

bbondy commented Dec 30, 2016

just make sure that with UAC on it will prompt for it on install on Win7, Win8 doesn't need to. And that the desktop shortcut works. If so then this should be closed.

@luixxiul
Copy link
Contributor Author

On Windows 10, after uninstalling, deleting the folder and reinstalling, the shortcut on Desktop is still broken.

I clicked AppData\local\brave\Brave.exe, AppData\local\brave\app-0.13.0\Brave.exe and AppData\local\brave\app-0.13.0\BraveDefaults.exe (which is not 0kb), but the browser did not start.

Is this a different issue maybe?

@bsclifton
Copy link
Member

bsclifton commented Dec 30, 2016

@luixxiul this may be a different issue; we have had reports of the app not working after launch on community.brave.com

@luixxiul
Copy link
Contributor Author

OK so closing it for now.

@luixxiul
Copy link
Contributor Author

Test plan: available on the 1st post.

@bsclifton
Copy link
Member

added release-notes/exclude since this was introduced w/ 0.13.0 (specifically, using the muon-winstaller over electron-winstaller)

@luixxiul luixxiul reopened this Jan 17, 2017
@bbondy bbondy closed this as completed in 0bd01d0 Jan 17, 2017
@bbondy
Copy link
Member

bbondy commented Jan 17, 2017

@luixxiul please retest with next preview, should be fixed for preview10.

@luixxiul
Copy link
Contributor Author

I confirmed this was fixed on Preview10 :-)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants