All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Alexis Lothoré via buildroot" <buildroot@buildroot.org>
To: "Thomas Perale" <thomas.perale@mind.be>,
	"Alexis Lothoré" <alexis.lothore@bootlin.com>
Cc: "Thomas Petazzoni" <thomas.petazzoni@bootlin.com>,
	<buildroot@buildroot.org>,
	"Nicolas Carrier" <nicolas.carrier@nav-timing.safrangroup.com>
Subject: Re: [Buildroot] [PATCH 2/2] package/sass: new package
Date: Wed, 12 Aug 2026 09:07:52 +0200	[thread overview]
Message-ID: <DKMRXZZLICJM.Z4H3U5DCRQ56@bootlin.com> (raw)
In-Reply-To: <20260810210451.30439-1-thomas.perale@mind.be>

Hello Thomas, Thomas,

On Mon Aug 10, 2026 at 11:04 PM CEST, Thomas Perale wrote:
> Hello,
>
> In reply of:
>> On Fri Aug 7, 2026 at 1:55 PM CEST, Thomas Perale wrote:

[...]

>> So in this case I've got no package.json, I only have a composer.json
>> requiring mini_asset, and mini_asset does not automatically pull those
>> dependencies. Mini-asset is configured through a .ini file, in which we
>> can use different "filters" (eg: Sass), and each filter has different
>> requirements, but it is up to the user to bring those dependencies on
>> host. One thing that I can check however, close to your suggestion, is
>> whether there is a different CSS "filter" that could be brought
>> automatically by composer and that generates something close to what
>> "webapp" is currently outputting.
>
> I see, thanks for the clarification. I didn't know about 'mini-asset'.
> I think your project can still define a package.json for the 'frontend' part
> with uglify and sass dev dependencies and make 'mini-assets' point to the
> 'node_modules' directory local "bin".
> But if you don't want to rely on 'package.json' or want vendoring then the best
> choice is probably adding the package to Buildroot indeed.

A small update on this:

While exploring the option of making the webapp build system manage the
dependencies installation (as Thomas Perale suggested), we found out
that we may be able to handle those directly with composer, without
having to add a package.json: there seems to be alternatives to sass and
uglifyjs that are packaged directly as php modules  (and so, that can be
pulled directly with composer install) and that mini-asset supports:
- sass -> scssphp
- uglifyjs -> jshrink

Some quick PoC validated that we are able to generate correctly the
frontend assets with those alternatives, so we will likely go this way
and get rid of those custom npm host packages.

If we ever happen to finally need those npm modules, Thomas Petazzoni
also hinted me about the npm fetcher mechanism implemented in bitbake
([1]) that could be inspired from and ported to buildroot.

This series can then be dropped; thanks anyway for taking time to review
it, and for the discussions/suggestions !

Alexis

[1] https://docs.yoctoproject.org/bitbake/2.18/bitbake-user-manual/bitbake-user-manual-fetching.html#npm-fetcher-npm

> Best regards,
> PERALE Thomas




-- 
Alexis Lothoré, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

  reply	other threads:[~2026-08-12  7:08 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-06 21:24 [Buildroot] [PATCH 0/2] package: bring two host tools Alexis Lothoré via buildroot
2026-08-06 21:24 ` [Buildroot] [PATCH 1/2] package/uglifyjs: new package Alexis Lothoré via buildroot
2026-08-07  7:44   ` Thomas Petazzoni via buildroot
2026-08-10  8:44     ` Alexis Lothoré via buildroot
2026-08-06 21:24 ` [Buildroot] [PATCH 2/2] package/sass: " Alexis Lothoré via buildroot
2026-08-07  8:29   ` Thomas Petazzoni via buildroot
2026-08-07 11:55     ` Thomas Perale via buildroot
2026-08-10 12:25       ` Alexis Lothoré via buildroot
2026-08-10 21:04         ` Thomas Perale via buildroot
2026-08-12  7:07           ` Alexis Lothoré via buildroot [this message]
2026-08-10  9:29     ` Alexis Lothoré via buildroot
2026-08-10 12:32       ` Thomas Petazzoni via buildroot

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=DKMRXZZLICJM.Z4H3U5DCRQ56@bootlin.com \
    --to=buildroot@buildroot.org \
    --cc=alexis.lothore@bootlin.com \
    --cc=nicolas.carrier@nav-timing.safrangroup.com \
    --cc=thomas.perale@mind.be \
    --cc=thomas.petazzoni@bootlin.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.