openwrt / packages

Community maintained packages for OpenWrt. Documentation for submitting pull requests is in CONTRIBUTING.md
GNU General Public License v2.0
3.97k stars 3.47k forks source link

rp-pppoe: Packaging wish: include pppoe client executable #3699

Closed arnysch closed 6 years ago

arnysch commented 7 years ago

Environment: rp-pppoe package update f6611ab on 23 Oct

For DSL testing and checking, I find the the user space client program 'pppoe' most helpful ("pppoe -A").

Though pppoe is built in rp-pppoe, it is not packaged for installation.

I added a package for it to my local rp-pppoe/Makefile manually (URL below as an example). It would be nice if it could come by default in OpenWrt/LEDE.

Just some ideas:

For example: http://arny.tjps.eu/OpenWrt/misc/20161224/Makefile

danielfdickinson commented 7 years ago

@arnysch Would you be interested in taking over maintainership? I don't have a need for this package any more, but don't want to orphan it for those that do so have been in 'maintenance mode' with this package.

arnysch commented 7 years ago

Hi Daniel,

due to little time I shy away from becoming a maintainer. Because this means taking over responsibility :-) Just once in a while I mess about with OpenWrt for my little home router. I do not know the maintainer procedures to heed.

Also, from rp-pppoe I only need the client pppoe, just for testing connectivity above the level of DSL sync.

Testing all rp-pppoe functionality would be too much for me, so I could only package and wait for other people suggestions about what to modify.

Having said this:

Regards, Arnold

Am 12.01.2017 um 02:07 schrieb Daniel Dickinson:

@arnysch https://github.com/arnysch Would you be interested in taking over maintainership? I don't have a need for this package any more, but don't want to orphan it for those that do so have been in 'maintenance mode' with this package.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/openwrt/packages/issues/3699#issuecomment-272046233, or mute the thread https://github.com/notifications/unsubscribe-auth/AXm6Cm8j6sVKIzrmmhoW6Ojx3ArN2wB6ks5rRXzogaJpZM4LVRKj.

danielfdickinson commented 7 years ago

Would you be more comfortable with doing a co-maintainer thing? You could do this patch, and help with DSL testing (I don't have DSL; I was using this to test DSL-type pppoe when I was doing devel that I waned to make sure work for more use cases than just static and DHCP. I think this package is not a critical package because it's not widely used. It should be mostly a case of doing version updates and dealing with compile issues due to trunk changes (or changes in the new versions).

arnysch commented 6 years ago

Just discovered this existing alternative:

CONFIG_PACKAGE_pppoe-discovery:

This tool performs the same discovery process as pppoe, but does not initiate a session. Can be useful to debug pppoe. https://ppp.samba.org/ Felix Fietkau nbd@nbd.name

Symbol: PACKAGE_pppoe-discovery [=y] Type : tristate Prompt: pppoe-discovery........................ Perform a PPPoE-discovery process Location: -> Network│
-> ppp-multilink........................ PPP daemon (with multilink support) (PACKAGE_ppp-multilink [=n])