Skip to content

Fix unresolved imports from krun_sys in muvm binary#237

Closed
MR-UNKNOWN74IDK wants to merge 4 commits into
AsahiLinux:mainfrom
MR-UNKNOWN74IDK:fix/krun-sys-imports
Closed

Fix unresolved imports from krun_sys in muvm binary#237
MR-UNKNOWN74IDK wants to merge 4 commits into
AsahiLinux:mainfrom
MR-UNKNOWN74IDK:fix/krun-sys-imports

Conversation

@MR-UNKNOWN74IDK

Copy link
Copy Markdown

Remove unresolved imports and function calls that no longer exist in
the krun_sys bindings.

  • Remove krun_set_root from the krun_sys imports
  • Remove the krun_set_root() function call that sets the root path

The krun_set_root function is no longer available in the current
version of krun_sys. The root path configuration may now be handled
by a different mechanism or is no longer necessary.

Fixes compilation error:
error[E0432]: unresolved imports krun_sys::krun_set_root

Update copyright year and owner in LICENSE file

Signed-off-by: MR_UNKNOWN74IDK <236201746+MR-UNKNOWN74IDK@users.noreply.github.com>
Updated copyright holder to "The Asahi Linux Contributors".

Signed-off-by: MR_UNKNOWN74IDK <236201746+MR-UNKNOWN74IDK@users.noreply.github.com>
Remove unresolved imports and function calls that no longer exist in
the krun_sys bindings.

- Remove `krun_set_root` from the krun_sys imports
- Remove the `krun_set_root()` function call that sets the root path

The `krun_set_root` function is no longer available in the current
version of krun_sys. The root path configuration may now be handled
by a different mechanism or is no longer necessary.

Fixes compilation error:
  error[E0432]: unresolved imports `krun_sys::krun_set_root`

Signed-off-by: MR_UNKNOWN74IDK <236201746+MR-UNKNOWN74IDK@users.noreply.github.com>
Reverted some information in the license file that already have one PR (PR AsahiLinux#235) for them and are not meant to be in this PR

Signed-off-by: MR_UNKNOWN74IDK <236201746+MR-UNKNOWN74IDK@users.noreply.github.com>
@WhatAmISupposedToPutHere

Copy link
Copy Markdown
Member

#234 already fixes it in a different way.

@MR-UNKNOWN74IDK

Copy link
Copy Markdown
Author

So, what shall I do ?
Do you think I shall revert the change ?

@WhatAmISupposedToPutHere

Copy link
Copy Markdown
Member

Just abandon/close this PR, and use the stable branch of libkrun for your local development

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 this pull request may close these issues.

2 participants