Skip to content
This repository has been archived by the owner on Nov 30, 2017. It is now read-only.

Can't move when app is download from web #1

Open
runningdemo opened this issue Jan 1, 2017 · 14 comments
Open

Can't move when app is download from web #1

runningdemo opened this issue Jan 1, 2017 · 14 comments
Assignees
Labels

Comments

@runningdemo
Copy link

runningdemo commented Jan 1, 2017

When package an electron app on local machine, it works.
But when I upload the app to a web server and downloaded to Downloads folder it doesn't work. The password popup show correctly though.

Try this sample app.
https://dl.dropboxusercontent.com/u/85873187/gethello/Stack-1.0.0-mac.zip

@tommoor
Copy link
Owner

tommoor commented Jan 1, 2017

Hey @liaa - it does appear that this example isn't working every time but I'm not sure if it's related to coming from a remote server or not…

Could you share the code snippet you used - preferably the entire main.js file if possible? Also make sure that you have logging on any errors that come out of the process - there is a likely some sort of file system or permissions error being produced.

@tommoor tommoor added the bug label Jan 1, 2017
@tommoor tommoor self-assigned this Jan 1, 2017
@zone117x
Copy link

This is due to the Gatekeeper / translocation "security feature" as discussed here: potionfactory/LetsMove#56

@tommoor
Copy link
Owner

tommoor commented Jan 30, 2017

Thanks @zone117x are you seeing this issue? I'd definitely like to get a fix in for that

tommoor added a commit that referenced this issue Feb 12, 2017
@davej
Copy link

davej commented Feb 15, 2017

Any update @tommoor? I'd prefer to package my app with the 'move to applications folder' dialog over a dmg with drag to apps, but this issue makes it look like a dmg is probably more reliable.

Thanks for the great work.

@zone117x
Copy link

According to that thread there seems to be a way to delete the app from the original Download folder location using AppleScript potionfactory/LetsMove#56 (comment)

I played around with it a bit but couldn't get it to work - I don't know applescript.

@tommoor
Copy link
Owner

tommoor commented Feb 15, 2017

@davej there is work in progress on the develop branch. I'm not sure how long it will take to get to a releasable state - totally depends on what time I get.

Even if you decide to go the dmg route this is still will still be a good module to include otherwise users can double click the application on the mounted read-only DMG which is about as bad as launching it from the Downloads folder!

@davej
Copy link

davej commented Feb 15, 2017

Thanks for your work, much appreciated.

I'm also having issues with my app where it doesn't relaunch properly but remains in the menubar when I use electron-lets-move. I haven't had time to debug it yet but I'll create a new issue when I do that.

@djalmaaraujo
Copy link

djalmaaraujo commented Apr 7, 2017

I don't think it's because is download from web. I just packed my app and on sierra and is not moving the application when I click to move.

@runningdemo
Copy link
Author

@djalmaaraujo Yes, it's not because of download from the web. It's because of the downloaded file is in the ~/Downloads folder.

@djalmaaraujo
Copy link

@tommoor Can you give us some estimation on how to approach on this? I noticed your commits in the develop branch.

@bengotow
Copy link

Hey folks—it looks like this is still an issue on Mac OS High Sierra. It's really odd (just from a general user standpoint) that this fails after the admin password dialog.

image

Based on that LetsMove issue, it seems like it's possible to use AppleScript to strip the quarantine flag from the application (which... presumably would allow it to be moved? I'm not sure.) It seems like a real hack though!

@tommoor
Copy link
Owner

tommoor commented Oct 11, 2017

Yo @bengotow not sure if you noticed, but they included a new API in Electron to achieve this so this module has become a little redundant. I don't know if it's any better at dealing with this edge case though…

electron/electron#10142

@davej
Copy link

davej commented Oct 11, 2017

@bengotow: moving to app folder is available in Electron core API now. Have you tried that? Or are you using this because you're pinned to an earlier version of Electron?

@bengotow
Copy link

Thanks folks! Will check that out. 👍

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

No branches or pull requests

6 participants