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

Installation of MacOS apps fail when app already exists #203

Closed
2 tasks done
marcoesters opened this issue May 6, 2024 · 1 comment · Fixed by #205
Closed
2 tasks done

Installation of MacOS apps fail when app already exists #203

marcoesters opened this issue May 6, 2024 · 1 comment · Fixed by #205
Assignees
Labels
type::bug describes erroneous operation, use severity::* to classify the type

Comments

@marcoesters
Copy link
Contributor

Checklist

  • I added a descriptive title
  • I searched open reports and couldn't find a duplicate

What happened?

On Windows and Linux, shortcuts are overwritten when they already exist. On MacOS, menuinst exits with an error because it cannot create the Resources directory inside the app. This is due to the following lines: https://github.com/conda/menuinst/blob/main/menuinst/platforms/osx.py#L107-L108

for path in paths:
    path.mkdir(parents=True, exist_ok=False)

The MacOS app should be removed before installing a new one to make the behavior consistent.

Conda Info

No response

Conda Config

No response

Conda list

No response

Additional Context

No response

@marcoesters marcoesters added the type::bug describes erroneous operation, use severity::* to classify the type label May 6, 2024
@marcoesters marcoesters self-assigned this May 6, 2024
@marcoesters
Copy link
Contributor Author

After discussion in the PR, we decided to improve the error message and provide ways to install the package even if a shortcut already exists.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type::bug describes erroneous operation, use severity::* to classify the type
Projects
Status: 🏁 Done
Development

Successfully merging a pull request may close this issue.

1 participant