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

optionalDependencies doesn't work properly on Ubuntu 22.04 #2959

Closed
lizable opened this issue Dec 17, 2024 · 0 comments · Fixed by #2960
Closed

optionalDependencies doesn't work properly on Ubuntu 22.04 #2959

lizable opened this issue Dec 17, 2024 · 0 comments · Fixed by #2960
Assignees

Comments

@lizable
Copy link
Contributor

lizable commented Dec 17, 2024

What Operating System(s) are you seeing this problem on?

Linux (x86-64)

Backend.AI version

24.09.4

Describe the bug

Despites the workaround of the issue, reported by @studioego (#2942), I got an error originated from "macos-alias" and "electron" on Backend.AI Installation.
Image

To Reproduce

  1. Clone Backend.AI Repo (https://github.com/lablup/backend.ai)
  2. Execute shell scripts install-dev.sh in scripts directory.
  3. Error occurs.

Expected Behavior

If macos-alias pkg is only for certain OS (such as macOS), It should be skipped during installation.

Anything else?

No response

@lizable lizable self-assigned this Dec 17, 2024
lizable added a commit that referenced this issue Dec 17, 2024
<!--
Please precisely, concisely, and concretely describe what this PR changes, the rationale behind codes,
and how it affects the users and other developers.
-->

This PR resolves [#2959](#2959).

### Test
> Prerequisites: you need to install pnpm in target machine. (Install guide: https://pnpm.io/installation)

1. Execute command `pnpm i`
2. Check whether installation is successfully done or not.

**Checklist:** (if applicable)

- [ ] Mention to the original issue
- [ ] Documentation
- [ ] Minium required manager version
- [ ] Specific setting for review (eg., KB link, endpoint or how to setup)
- [ ] Minimum requirements to check during review
- [ ] Test case(s) to demonstrate the difference of before/after
@lizable lizable closed this as completed Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant