Skip to content

Minor fixes#1240

Merged
a-gave merged 5 commits intolibremesh:masterfrom
a-gave:minor_fixes
Apr 2, 2026
Merged

Minor fixes#1240
a-gave merged 5 commits intolibremesh:masterfrom
a-gave:minor_fixes

Conversation

@a-gave
Copy link
Copy Markdown
Contributor

@a-gave a-gave commented Jan 28, 2026

Minor patch list:

  • docs: README add back the logo
  • ci: build packages for the upcoming_branch
  • lime-system: update add-lime-repos
  • lime-report: adapt to get packagelist via ubus

Copy link
Copy Markdown
Member

@ilario ilario left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I checked the code of packages/lime-system/files/etc/uci-defaults/92_add-lime-repos and it looks great. Did not test flashing a router with it. Regarding the Github actions, you @a-gave are the boss :)

@ilario
Copy link
Copy Markdown
Member

ilario commented Feb 24, 2026

There's a small conflict with the renamed lime-report file, as soon as the conflict is solved it is good to merge, in my opinion.

- replace opkg list-installed with 'ubus call rpc-sys packagelist'
  which reads directly the /usr/lib/${package_manger}/status and
  works also with apk
- introduce a dependency on jq to sort correctly the packagelist
- rename lime-report.sh to lime-report, it was already installed
  with that name before that the package was converted to use
  libremesh.mk
@a-gave
Copy link
Copy Markdown
Contributor Author

a-gave commented Mar 8, 2026

I did some further modifications:

  • avoided touching the readme for now
  • keeped the name of the script lime-report.sh
  • apk feeds weren't configured because the apk command was in /usr/bin/apk not in /bin/apk

Will do further checks that everything is ok then and report back

@ilario
Copy link
Copy Markdown
Member

ilario commented Mar 12, 2026

Will do further checks that everything is ok then and report back

Thanks! Feel free to merge when you are satisfied with the pull request :)

a-gave added 2 commits April 2, 2026 13:02
remove a duplicated function introduced in libremesh#1233
without it the `option apup_ssid 'LibreMesh.org'` is included also in wifi-ifaces
using other modes.
@a-gave a-gave merged commit 26e46a7 into libremesh:master Apr 2, 2026
1 check passed
@a-gave
Copy link
Copy Markdown
Contributor Author

a-gave commented Apr 2, 2026

Merged!

@ilario
Copy link
Copy Markdown
Member

ilario commented Apr 2, 2026

Thanks!

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