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

On Wed, Mar 18, 2026 at 10:59:51AM -0500, Antonios Christidis wrote:
> 
> 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.

Ack, thanks.


> >>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