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

change variable name for clarity #379

Closed
wants to merge 1 commit into from

Conversation

tschettervictor
Copy link
Contributor

changed "rustdesk_hbbs_ip" to "rustdesk_hbbr_server" as that is what the variable is referring to according to the docs

changed "rustdesk_hbbs_ip" to "rustdesk_hbbr_server" as that is what the variable is referring to according to the docs
@tschettervictor
Copy link
Contributor Author

After messaging with the the maintainer of the port, I'm not 100% sure how much this PR will affect existing users.

The maintainer @madpilot78 believes it would be too much hassle to change the variable, as that might force existing users to adjust their setup, while I believe the variable should be changed for the sake of having things easy to understand and implement. If a variable is not specific as to what it is referring to, that caused some confusion. It did for me anyway, that is why I'm pushing this PR.

That said, I will leave the decision up to @rustdesk to decide.

@madpilot78
Copy link
Contributor

I think some context is missing.

What @tschettervictor means with "the port" is, the FreeBSD port existing in the FreeBSD Ports Collection, specifically this one: https://www.freshports.org/net/rustdesk-server/

I maintain that port and took the rc scripts from the rustdesk-server distribution, adapting them just slightly for the package. I left the names of the variables unmodified.

Since this package is installed by users and they have configured their systems with these variables, my issue with changing the variables is, as described, that users will suddenly need to modify their systems configuration when upgrading the package. Some users will end up with broken installation because they are not aware or forget to change this configuration, and this is something we try to avoid in the FreeBSD ports (and base system too).

This could also affect the users of other packages on other OSes.

I also think the variable name is not wrong as is. Variables names are no substitute for documentation. I improved the variable description in the rc script in the port here, I could file a PR for a similar change.

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