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

wpewebkit: Fixes for cross-compilation #243

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dwrobel
Copy link
Contributor

@dwrobel dwrobel commented Mar 23, 2021

  • extend native bbclass,
  • fix dependencies for target (avoid using -native counterparts),
  • do not pull-in qt unless 'qtwpe' config is enabled,
  • synchronize RRECOMMENDS to PACKAGECONFIG selection.

Signed-off-by: Damian Wrobel [email protected]

 - extend native bbclass,
 - fix dependencies for target (avoid using -native counterparts),
 - do not pull-in qt unless 'qtwpe' config is enabled,
 - synchronize RRECOMMENDS to PACKAGECONFIG selection.

Signed-off-by: Damian Wrobel <[email protected]>
inherit ${@'cmake_qt5' if 'qt5-layer' in d.getVar('BBFILE_COLLECTIONS').split() else ''}
inherit cmake pkgconfig perlnative python3native ${@bb.utils.contains('PACKAGECONFIG','qtwpe','cmake_qt5','',d)}

BBCLASSEXTEND = "native"
Copy link
Member

Choose a reason for hiding this comment

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

What is the point to make this recipe native? What other component needs wpewebkit built natively in the host?

Copy link

Choose a reason for hiding this comment

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

Maybe to use to run some automation or similar? in any case, the extend doesn't hurt.

Copy link
Member

Choose a reason for hiding this comment

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

Sure, just I want to be sure we are not missing something else. If the change if for that reason I could suggest a small change in the commit description pointing in this direction. The current commit message makes reference to a unknown build error (wpewebkit: Fixes for cross-compilation). @dwrobel could you make this small change?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

could you make this small change?

Could you be more precise what should I change/add?

Copy link
Member

Choose a reason for hiding this comment

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

to update the commit log message with a more clear message instead of to make reference to a unknown build error.

recipes-browser/wpewebkit/wpewebkit.inc Show resolved Hide resolved
recipes-browser/wpewebkit/wpewebkit.inc Show resolved Hide resolved
recipes-browser/wpewebkit/wpewebkit.inc Show resolved Hide resolved
recipes-browser/wpewebkit/wpewebkit.inc Show resolved Hide resolved
recipes-browser/wpewebkit/wpewebkit.inc Show resolved Hide resolved
@github-actions
Copy link

This PR is stale because it has been open 90 days with no activity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants