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

Clean up 1a6d2e9 #1153

Conversation

DimitriPapadopoulos
Copy link
Collaborator

Runtime option pppd-accept-remote will remain available at all time, but will now accept a boolean argument:

  • --pppd-accept-remote=1: invoke pppd with the ipcp-accept-remote option, which is required by pppd ≥ 2.5.0.
  • --pppd-accept-remote=0: invoke pppd without the ipcp-accept-remote option, which is required by pppd < 2.5.0.
  • --pppd-accept-remote is kept as a deprecated, undocumented synonym of --pppd-accept-remote=1.

Build-time option --enable-legacy-pppd will change the default from --pppd-accept-remote=1 to --pppd-accept-remote=0. This might be preferable on older platforms shipping pppd < 2.5.0, such as macOS.

Runtime option `pppd-accept-remote` will remain available at all time,
but will now accept a boolean argument:
* `--pppd-accept-remote=1`: invoke pppd with the `ipcp-accept-remote`
  option, which is required by pppd ≥ 2.5.0.
* `--pppd-accept-remote=0`: invoke pppd without the `ipcp-accept-remote`
  option, which is required by pppd < 2.5.0.
* `--pppd-accept-remote` is kept as a deprecated, undocumented synonym
  of `--pppd-accept-remote=1`.

Build-time option `--enable-legacy-pppd` will change the default from
`--pppd-accept-remote=1` to `--pppd-accept-remote=0`. This might be
preferable on older platforms shipping pppd < 2.5.0, such as macOS.
@DimitriPapadopoulos DimitriPapadopoulos merged commit f4ecda6 into adrienverge:master Nov 5, 2023
5 checks passed
@DimitriPapadopoulos DimitriPapadopoulos deleted the pppd-accept-remote branch November 5, 2023 23:59
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.

1 participant