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

phoebus.bat: Don't use server mode #2577

Merged
merged 1 commit into from
Mar 11, 2023
Merged

phoebus.bat: Don't use server mode #2577

merged 1 commit into from
Mar 11, 2023

Conversation

kasemir
Copy link
Collaborator

@kasemir kasemir commented Mar 10, 2023

Mention -server 4918 in comment, but don't include it in the default parameters because it prevents opening multiple instances. In addition, that TCP port may be used by other tools, so leave addition of -server... settings to each site.

See also end user questions like https://epics.anl.gov/tech-talk/2023/msg00299.php

Mention `-server 4918` in comment, but don't include it in the default parameters because it prevents opening multiple instances.
In addition, that TCP port may be used by other tools, so leave addition of `-server...` settings to each site.

See also end user questions like https://epics.anl.gov/tech-talk/2023/msg00299.php
@kasemir
Copy link
Collaborator Author

kasemir commented Mar 10, 2023

@shroffk @georgweiss This is a change for those who simply use the 'phoebus.bat' without any local changes.
I assume, however, that most people would expect a new instance whenever invoking 'phoebus.bat'.
Including -server 4918 risks issues with something already using that port, including some hung instance of the launcher, so it's likely better to leave adding -server ... to whoever creates a launcher for local usage.

@shroffk shroffk merged commit a9ed226 into master Mar 11, 2023
@shroffk shroffk deleted the windows_launcher branch September 11, 2023 13:36
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