Re: [meta-arago][master][PATCH] flutter-samples-material-3-demo: Add --no-pub build option

On 3/17/26 4:05 PM, Denys Dmytriyenko wrote:
> On Tue, Mar 17, 2026 at 04:01:38PM -0500, Antonios Christidis via lists.yoctoproject.org wrote:
>> From: Antonios Christidis <[email protected]>
>>
>> Append "--no-pub" option to flutter.
>>
>> There is a patch pending upstream, [1]. The maintainer is inactive/on
>> vacation.
>>
>> The upstream patch modifies a class, there is no clean way to modify
>> a class, so I'm editing FLUTTER_BUILD_ARGS which gets consumed by the
>> class. The end behavior is the same.
>>
>> [1] https://github.com/meta-flutter/meta-flutter/pull/744
>>
>> Signed-off-by: Antonios Christidis <[email protected]>
>> ---
>>   check_yocto_rules.json                                        | 1 +
>>   .../flutter/flutter-samples-material-3-demo-arago.inc         | 1 +
>>   .../flutter/flutter-samples-material-3-demo_%.bbappend        | 4 ++++
>>   3 files changed, 6 insertions(+)
>>   create mode 100644 meta-arago-test/dynamic-layers/flutter-layer/recipes-graphics/flutter/flutter-samples-material-3-demo-arago.inc
>>   create mode 100644 meta-arago-test/dynamic-layers/flutter-layer/recipes-graphics/flutter/flutter-samples-material-3-demo_%.bbappend
>>
>> diff --git a/check_yocto_rules.json b/check_yocto_rules.json
>> index 25c8c952..70bbe734 100644
>> --- a/check_yocto_rules.json
>> +++ b/check_yocto_rules.json
>> @@ -23,6 +23,7 @@
>>               "meta-arago-extras/recipes-extended/sysrepo/libyang_%.bbappend",
>>               "meta-arago-test/dynamic-layers/qt6-layer/recipes-core/packagegroups/ti-test.bbappend",
>>               "meta-arago-test/dynamic-layers/flutter-layer/recipes-graphics/flutter/flutter-wayland-client_%.bbappend",
>> +            "meta-arago-test/dynamic-layers/flutter-layer/recipes-graphics/flutter/flutter-samples-material-3-demo_%.bbappend",
>>               "meta-arago-test/recipes-benchmark/stream/stream_%.bbappend",
>>               "meta-arago-test/recipes-extended/bc/bc_%.bbappend"
>>           ]
>> diff --git a/meta-arago-test/dynamic-layers/flutter-layer/recipes-graphics/flutter/flutter-samples-material-3-demo-arago.inc b/meta-arago-test/dynamic-layers/flutter-layer/recipes-graphics/flutter/flutter-samples-material-3-demo-arago.inc
>> new file mode 100644
>> index 00000000..b2c9e13d
>> --- /dev/null
>> +++ b/meta-arago-test/dynamic-layers/flutter-layer/recipes-graphics/flutter/flutter-samples-material-3-demo-arago.inc
>> @@ -0,0 +1 @@
>> +FLUTTER_BUILD_ARGS:append = " --no-pub"
> Will it work with += here instead of :append?
>
It would not work, the variable has a soft default that I want to abide 
by and append to. If I use "+=" it overrides the soft default value.

I finally got a proper use case for ":append" lol.

>> diff --git a/meta-arago-test/dynamic-layers/flutter-layer/recipes-graphics/flutter/flutter-samples-material-3-demo_%.bbappend b/meta-arago-test/dynamic-layers/flutter-layer/recipes-graphics/flutter/flutter-samples-material-3-demo_%.bbappend
>> new file mode 100644
>> index 00000000..f0ae3799
>> --- /dev/null
>> +++ b/meta-arago-test/dynamic-layers/flutter-layer/recipes-graphics/flutter/flutter-samples-material-3-demo_%.bbappend
>> @@ -0,0 +1,4 @@
>> +FLUTTER_ARAGO = ""
>> +FLUTTER_ARAGO:arago = "flutter-samples-material-3-demo-arago.inc"
>> +
>> +require ${FLUTTER_ARAGO}
>> -- 
>> 2.34.1