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

packagelists: don't remove ppp-related packages #36

Merged
merged 1 commit into from
Jan 10, 2021
Merged

Conversation

Akira25
Copy link
Member

@Akira25 Akira25 commented Jan 2, 2021

The removal of dsl-related packages was presumeably introduced
2014 to save space on 4MB-Devices. As Falter does not care for
those, we may readd them.

Have a look at: freifunk-berlin/falter-packages#92 (comment)

Before merging this, we should have some further discussion on the
standard WAN-Interface (on DSL-Modem or LAN1)

@PolynomialDivision
Copy link
Contributor

If u have the time, it is always nice to know how much diskspace this costs. ;)

@Akira25
Copy link
Member Author

Akira25 commented Jan 2, 2021

On my GL-AR150 this took around 300kb.

@pmelange
Copy link
Contributor

pmelange commented Jan 3, 2021

@Akira25 why is wpad-mini also included in this PR? I feel like it is unrelated. If it is somehow necessary, please either remove wpad-mini in a seperate commit or as a totally different PR.

@pmelange
Copy link
Contributor

pmelange commented Jan 3, 2021

Also, please change the commit comment. pppoe is not only used for DSL, but in other transport layers as well. It would be best just to use pppoe and not mention DSL at all. :)

@Akira25
Copy link
Member Author

Akira25 commented Jan 3, 2021

I assumed, that -wpad-mini is dead code, as package wpad-full should be the standard. But I am not sure on that. I will drop that.

@Akira25
Copy link
Member Author

Akira25 commented Jan 3, 2021

@pmelange I changed commit that to meet your suggestions. :)

@Akira25
Copy link
Member Author

Akira25 commented Jan 4, 2021

@pmelange sven wrote something on luci-proto-ppp. Do we need to add this into our images explicitly? Is it enough, to just not remove those packages or should we explicitly include them into the packagelists?

@pmelange
Copy link
Contributor

pmelange commented Jan 4, 2021

I guess it would need to be added, but the best way is to test. :)

@pmelange
Copy link
Contributor

pmelange commented Jan 4, 2021

@Akira25 Let me know if adding luci-proto-ppp is not needed so that I can merge this.

The removal of ppp-packages was presumeably introduced
2014 to save space on 4MB-Devices. As Falter does not care for
those, we readd them. Especially devices with built-in modem
will profit from this.

Have a look at: freifunk-berlin/falter-packages#92 (comment)
@Akira25
Copy link
Member Author

Akira25 commented Jan 8, 2021

@pmelange I tested on one of Matthias Fritzboxes. We can use the ppp-packages only in luci, if we install that proto-package. It does not depend on anything except libc, so it shouldn't be a big deal. This can be merged now.

@pmelange pmelange merged commit 975d43a into master Jan 10, 2021
@Akira25 Akira25 deleted the fix/dsl-packages branch February 6, 2021 18:50
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.

3 participants