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

False positive with inline functions #597

Closed
UVV-gh opened this issue Jul 5, 2024 · 5 comments
Closed

False positive with inline functions #597

UVV-gh opened this issue Jul 5, 2024 · 5 comments

Comments

@UVV-gh
Copy link
Contributor

UVV-gh commented Jul 5, 2024

Seems like this case:
SRC_URI:append = "${@bb.utils.contains('EXTRA_IMAGE_FEATURES', 'my-feature', '', ' file://0001-my-patch.patch', d)}"
is reported as an append without space. Any ideas how to fix it properly?

@priv-kweihmann
Copy link
Owner

The inline functions are evaluated always by their true branch statement - if you would negated the condition and switch true and false branch statement it should be fine.
It admit it's a limitation of the parser used

@priv-kweihmann
Copy link
Owner

The other option is ofc to inject a pseudo space into the true branch statement

@UVV-gh
Copy link
Contributor Author

UVV-gh commented Jul 5, 2024

Any way to disable a warning/error when inline function is used?

@priv-kweihmann
Copy link
Owner

Inline suppression should work (https://github.com/priv-kweihmann/oelint-adv#inline-suppression).

@priv-kweihmann
Copy link
Owner

Support for negative branch check was added with #611 and released with version 6.0.0

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

No branches or pull requests

2 participants