Start and log error (rather than refusing to start) if -settings flag points to nonexistent file
Maintainer thường phản hồi trong vòng 2 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- Nửa ngày
- Mức phù hợp với người mới
- 62/100
Hướng nghiên cứu
The payload does not name the file, so begin by finding where the -settings command-line argument is parsed and where the settings file is loaded at startup. Reproduce by launching Phoebus with a path that does not exist and note where it exits. Done means startup continues with the normal defaults and logs an error that includes the missing path, with the change covered by a test if one exists for settings loading.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
We are looking to use a base settings.ini file with all the 'defaults', and then override some settings per machine. One option is to distribute multiple versions of Phoebus with different settings.ini files, but this is a pain in many ways.
So seemingly the easiest way to do this is to have our default settings in the install directory (which guarantees they will always be loaded), and then to pass -settings <path>/overrides.ini to the application when it is launched. This works, except that if the file passed to -settings is not found, Phoebus will refuse to start. Since we don't want to force every user to create an empty overrides.ini file, the ideal solution is that Phoebus still starts, and instead logs an error if the file is not found. This seems like more useful functionality in general anyway - from the code it seems like the -settings flag is meant to work as an override
This would be a simple case of adding a try-catch around the relevant code
- Ngôn ngữ chính
- Java
- Star
- 130
- Fork
- 132
- Merge trung bình
- 2 ngày 11 giờ
- Pull request đã merge (30 ngày)
- 19
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Không có hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của ControlSystemStudio/phoebus
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
ControlSystemStudio/phoebus#3983 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Create CONTRIBUTING.mdĐang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
ControlSystemStudio/phoebus#3979 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 80/100
ControlSystemStudio/phoebus#3962 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
PVA/SPVA: track SPVA version independently and move XDG keychain dir pva/<ver> -> spva/<n.n.n>Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
ControlSystemStudio/phoebus#3865 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 52/100
ControlSystemStudio/phoebus#3980 ·
Maintainer thường phản hồi trong vòng 2 ngày
Tất cả issue của ControlSystemStudio/phoebus
Issue tương tự
-
dev/release: verify-release-candidate.sh runs 0 Java tests with Debian's MavenCó thể đã có người làm @kou đã nhận hôm nay. Đang mởType: bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
apache/arrow-adbc#4947 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
lysine-dev/okhttp#9814 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
checkstyle/checkstyle#22153 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 79/100
Maintainer thường phản hồi trong vòng 1 ngày