Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

various readme improvements #1015

Merged
merged 3 commits into from
May 10, 2023
Merged

various readme improvements #1015

merged 3 commits into from
May 10, 2023

Conversation

aparcar
Copy link
Member

@aparcar aparcar commented Apr 14, 2023

Let's clean things up a little bit and let people use use the ImageBuilder.

It's not used anymore.

Signed-off-by: Paul Spooren <mail@aparcar.org>
Signed-off-by: Paul Spooren <mail@aparcar.org>
Signed-off-by: Paul Spooren <mail@aparcar.org>
@G10h4ck
Copy link
Member

G10h4ck commented Apr 18, 2023

For new WiFi 6 radios, we might end up needing a custom build for hostapd, is this possible with imagebuilder?

@aparcar
Copy link
Member Author

aparcar commented Apr 18, 2023

uhm... why do we need custom builds with the imagebuiler? I'm running LibreMesh successfully on WiFi 6 (and 7) devices with images based on the ImageBuilder.

@ilario
Copy link
Member

ilario commented Apr 26, 2023

Don't you have any problem with dnsmasq-ipv6? #1000 (comment)

I think that apart from that, we are already compatible with the image builder for the core packages (see #1000 for problems with non-core packages).
So for me is good to push the users to use the ImageBuilder (if we confirm that dnsmasq-ipv6 is not a problem).

@pony1k
Copy link
Contributor

pony1k commented May 7, 2023

@ilario Why should there be a problem with dnsmasq-ipv6? When I specify PACKAGES=-dnsmasq [...], imagebuilder builds without dnsmasq.

@ilario
Copy link
Member

ilario commented May 8, 2023

When I specify PACKAGES=-dnsmasq [...], imagebuilder builds without dnsmasq.

Ooh, I didn't know that was possible, nice!

@ilario
Copy link
Member

ilario commented May 9, 2023

Confirmed, you need to add "-dnsmasq", otherwise you have file conflicts with "dnsmasq-dhcpv6".
So I would merge this and then:

  1. add the "-dnsmasq"
  2. add a minimum amount of recommended packages (maybe babeld alone works, but I feel is not tested enough for suggesting that in the readme). So adding "lime-proto-batadv lime-proto-anygw lime-hwd-openwrt-wan" (last one is for having the WAN port configured as WAN)

@aparcar should I merge and do that edits or can you add this to the pull request? Thanks for this amazing work!

@aparcar
Copy link
Member Author

aparcar commented May 9, 2023

Kindly proceed.

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.

4 participants