Skip to content

[dunfell] strongswan: Make PACKAGECONFIG a default value#7

Merged
amstewart merged 1 commit into
ni:nilrt/master/dunfellfrom
jhershbe:patch-1
Apr 21, 2021
Merged

[dunfell] strongswan: Make PACKAGECONFIG a default value#7
amstewart merged 1 commit into
ni:nilrt/master/dunfellfrom
jhershbe:patch-1

Conversation

@jhershbe

Copy link
Copy Markdown

Change from a weak default to a default in the definition of the PACKAGECONFIG.

In https://github.com/flihp/meta-measured/blob/master/networking-layer/recipes-support/strongswan/strongswan_5.%25.bbappend the PACKAGECONFIG is appended to, so if the definition is weak here, the variable will be empty when the bbappend attempts to add to it.

Also have a pull request open on master: openembedded#340

Change from a weak default to a default in the definition of the PACKAGECONFIG.

In https://github.com/flihp/meta-measured/blob/master/networking-layer/recipes-support/strongswan/strongswan_5.%25.bbappend the PACKAGECONFIG is appended to, so if the definition is weak here, the variable will be empty when the bbappend attempts to add to it.
@amstewart

Copy link
Copy Markdown

Is not having this in dunfell blocking something, or can we otherwise wait for it to land upstream before pulling it into the dunfell branch?

@jhershbe

Copy link
Copy Markdown
Author

It's keeping strongswan from working in dunfell, but assuming we can get the change in upstream without too much delay, I think it's alright to wait for that.

@jhershbe

Copy link
Copy Markdown
Author

It has landed upstream.

openembedded@90c04ac

@amstewart

Copy link
Copy Markdown

Great; thanks!

@amstewart amstewart merged commit cd6d0d9 into ni:nilrt/master/dunfell Apr 21, 2021
@jhershbe jhershbe deleted the patch-1 branch April 21, 2021 17:05
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