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

Move FontForge.app to /Applications? #12517

Closed
davelab6 opened this issue Jul 10, 2015 · 23 comments
Closed

Move FontForge.app to /Applications? #12517

davelab6 opened this issue Jul 10, 2015 · 23 comments

Comments

@davelab6
Copy link
Contributor

FontForge.app needs to be moved to /Applications to run; when it is installed with cask and first run, it pops up a CocoaDialog message asking the user to do this, and then opens the pwd of the app's current location. It would be better for the cask script to do this directly.

A symlink will not work.

Is that allowed?

cc @DomT4

@DomT4
Copy link
Member

DomT4 commented Jul 10, 2015

Afraid I have no control over the Cask's choices. It's possible to do, easily, but I don't know how it'll square with the Cask removal/installation process.

Usually Homebrew will not override a target that already exists, which would make reinstalls or future updates break without removing the application first (And we've moving towards a sandbox that will actively prevent formulae installing/removing things in $HOME and such); I don't know about the Cask.

Their zap :delete implementation is nifty and I suspect capable of handling that, but I don't know their boundaries on what they find acceptable.

There's an ongoing discussion on this sort of topic in #3888.

@vitorgalvao
Copy link
Member

To be clear, the user has to move it manually, FontForge won’t offer to do it, correct?

@davelab6
Copy link
Contributor Author

To be clear, the user has to move it manually, FontForge won’t offer to do it, correct?

Yes, at the moment. Is it possible for FontForge to do itself on first run?

In the next release there will be a DMG, so cask will have to move the app out of the dmg, and thus that could be a good moment to do this.

cc @larsenwork

@vitorgalvao
Copy link
Member

Is it possible for FontForge to do itself on first run?

Yes, multiple apps do it. It isn’t related to us but the app itself, though.

In the next release there will be a DMG, so cask will have to move the app out of the dmg, and thus that could be a good moment to do this.

No, not really. The container it is in has nothing to do with it. The issue is deeper than that, and I’ll start a needed discussion on that soon.

@davelab6
Copy link
Contributor Author

davelab6 commented Jul 10, 2015 via email

@vitorgalvao
Copy link
Member

Check casks with suppress_move_to_applications. It’s a very common technique, and the message usually looks like this.

@larsenwork
Copy link

@davelab6 not sure how this is related to the dmg that we'll use once we get the building working again? The new installer in the dmg already copies FontForge.app to /Applications

@davelab6
Copy link
Contributor Author

On 10 July 2015 at 12:48, Andreas Larsen notifications@github.com wrote:

The new installer in the dmg already copies FontForge.app to /Applications

The user has to do that by hand though, right?

@larsenwork
Copy link

No, double click on the installer that automatically copies ff to /app

@davelab6
Copy link
Contributor Author

davelab6 commented Jul 10, 2015 via email

@larsenwork
Copy link

dunno - what is cask used for (I've never used it)?

@jawshooah
Copy link
Contributor

Is it a .pkg installer? If so, then yes, Cask can handle that just fine.

@vitorgalvao
Copy link
Member

@jawshooah Testing it now, it’s an app that does something on first launch. It also requires X11, so we need to add that dependency.

@larsenwork Command-line package manager for GUI apps.

@jawshooah
Copy link
Contributor

@vitorgalvao Right, but from the discussion above it sounded like they were changing the installer format. Is that the case, @davelab6, @larsenwork?

@vitorgalvao vitorgalvao added bug Issue describing a reproducible bug. cask awaiting user reply Issue needs response from a user. labels Jul 25, 2015
@n370
Copy link
Contributor

n370 commented Aug 2, 2015

screen shot 2015-08-02 at 11 22 08

@vitorgalvao
Copy link
Member

Being tracked in #12858.

Will no longer be an issue after #13201.

@tjnycum
Copy link
Contributor

tjnycum commented Aug 13, 2015

FontForge.app needs to be moved to /Applications to run

Why is that? That seems like rather bad behavior.

@scribblemaniac
Copy link
Contributor

@tjnycum Nobody here developed the application, so nobody here could possibly answer your question.

@tjnycum
Copy link
Contributor

tjnycum commented Aug 13, 2015

@scribblemaniac: davelab6, the OP and person whom I was quoting, seems to be heavily involved in that project.

@scribblemaniac
Copy link
Contributor

Yes sorry, my bad.

@davelab6
Copy link
Contributor Author

@tjnycum Fontforge is 100% bad behaviour. It is a 15 year old C application with its own python, and it is brimming with magic to make it run on Mac OS X. One of the consequences is hardcoded paths that begin /Applications/FontForge.app/...

If you want to fix it, you are welcome, but you'll be working full time for a year.

@davelab6
Copy link
Contributor Author

they were changing the installer format

Believe it when you see it :)

@n370
Copy link
Contributor

n370 commented Aug 13, 2015

Thanks @vitorgalvao. I'll be following the discussion on #12858 and #13201. Good luck.

@miccal miccal removed bug Issue describing a reproducible bug. cask labels Dec 23, 2016
@Homebrew Homebrew locked and limited conversation to collaborators May 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

9 participants