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

Fix compilation on openSUSE #1998

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

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.

None yet

4 participants