Skip to content

Feature owm sh own package#473

Merged
Noki merged 1 commit into
mainfrom
feature_owm_sh_own_package
Mar 18, 2025
Merged

Feature owm sh own package#473
Noki merged 1 commit into
mainfrom
feature_owm_sh_own_package

Conversation

@akira25
Copy link
Copy Markdown
Member

@akira25 akira25 commented Mar 10, 2025

Kompiliert Ja/Nein: (Bitte gib Prozessorarchitektur, Modell und Falter/OpenWrtversion an)
Läuft live Ja/Nein: (Bitte gib Prozessorarchitektur, Modell und Falter/OpenWrtversion an und wie du deine Änderungen getestet hast)

Beschreibung deiner Änderungen:

Auf Nachfrage von @Noki ist das owm-script nun in einem eigenen Paket, das keine unnötigen luci-Dpenencies mehr braucht.

Comment thread packages/falter-berlin-owmsh/Makefile Outdated
@Noki
Copy link
Copy Markdown
Member

Noki commented Mar 18, 2025

It might be a good idea to add this package without removing the old one for now. That way we can change the package and delete the old one once we confirm everything is working.

We would like to use the owm publish script on notes with low flash too. As
the former placement in the luci-package pulled a lot of luci dependencies
on install, we move this into its own package.
@akira25 akira25 force-pushed the feature_owm_sh_own_package branch from 7b78eb4 to b3a3937 Compare March 18, 2025 12:42
@akira25 akira25 force-pushed the feature_owm_sh_own_package branch from b3a3937 to 76eef5b Compare March 18, 2025 12:47
@akira25
Copy link
Copy Markdown
Member Author

akira25 commented Mar 18, 2025

It might be a good idea to add this package without removing the old one for now. That way we can change the package and delete the old one once we confirm everything is working.

Sound good. I've created #474 for that and droped the respective commit here. Once rebased on main, that MR should contain only the commit in question.

@Noki Noki merged commit 36f3535 into main Mar 18, 2025
@Noki Noki deleted the feature_owm_sh_own_package branch March 18, 2025 13:57
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