Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Feb 13, 2024. It is now read-only.

Conversation

leviathanch
Copy link

Hi
On openSUSE those small changes are needed for successfully building an RPM.
Please consider pulling this patch.
Thanks!

@morphis
Copy link
Member

morphis commented Feb 13, 2022

@leviathanch Looks like your branch contains other camera related changes too. I guess your intention was not to include them in this PR, right?

@leviathanch
Copy link
Author

leviathanch commented Feb 13, 2022 via email

@Fuseteam
Copy link

@leviathanch so those camera commits aren't intented for this PR?

@leviathanch
Copy link
Author

leviathanch commented Feb 14, 2022 via email

@Fuseteam
Copy link

the camera commits ended up as a part of this PR, you may want to remove them

@leviathanch
Copy link
Author

Oops...

@leviathanch
Copy link
Author

Fixed it

@Fuseteam
Copy link

Is this commit also intended to be part of this PR 000df0d @leviathanch?

@leviathanch
Copy link
Author

Yeah. Because otherwise Anbox doesn't work with the current LXC version openSUSE

@Fuseteam
Copy link

That sounds more like this PR depends on #1827.
hmmm at this point it's up to @morphis how he wants to handle it

Copy link
Member

@morphis morphis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants