I see two ways to do this.

First: Since web-ext is a nodejs binary, you could create the web-ext rpm. If you do this, make sure to use the nodejs bundled library script,[1] to bundle the build-time, and run-time dependencies. You would then just add web-ext as a build dependency when you are building your package.

Second: You could bundle web-ext as a build-only bundle. You might need to edit the bundler script, but it often works without modifications. If you go this route, you will need to run the bundler script each time you do an update of your main package. This will catch any security updates that have been released since last time. A sample spec file will help some.[2]

It's looking like you are leaning towards the second option.

Troy

[1] - https://src.fedoraproject.org/rpms/nodejs-packaging/raw/rawhide/f/nodejs-packaging-bundler

[2] - https://docs.fedoraproject.org/en-US/packaging-guidelines/Node.js/#_example_spec

On Wed, Jun 2, 2021 at 4:19 AM Onuralp SEZER <thunderbirdtr@fedoraproject.org> wrote:

Hello all,

Fedora KDE has a plasma-browser-integration[0] package it doesn't include the "actual browser" extension[1] in our repository.
I want to add the plasma-browser-integration/firefox extension[2][3] "xpi" into the "plasma-browser-integration" package.
However, I need to download the "web-ext"[4] package and a bunch of additional requirements in it. Is it ok to download
all the requirements directly, show them as a separate source, get the xpi file directly, and delete the others, without
doing package entire nodejs dependencies? What do you think? Am I missing some obvious solution, or is this a sane
way forward? Any feedback and help would appreciate it.

Current plasma browser integration spec: https://src.fedoraproject.org/rpms/plasma-browser-integration

[0] : https://community.kde.org/Plasma/Browser_Integration
[1] : https://addons.mozilla.org/en-US/firefox/addon/plasma-integration/
[2] : https://invent.kde.org/plasma/plasma-browser-integration
[3] : https://invent.kde.org/plasma/plasma-browser-integration/-/tree/master/extension
[4] : https://www.npmjs.com/package/web-ext

Thank you.
Onuralp
_______________________________________________
nodejs mailing list -- nodejs@lists.fedoraproject.org
To unsubscribe send an email to nodejs-leave@lists.fedoraproject.org
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/nodejs@lists.fedoraproject.org
Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure