Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

macOS: CONFIG+=serveronly needs two changes, not one

Đang mở
  • Có gì không ổn?
#3,914 2 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 5 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
78/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
cpp, macos
Lĩnh vực
build-system

Hướng nghiên cứu

Bắt đầu trong Jamulus.pro với danh sách mã nguồn macx và các điều kiện serveronly hiện có, sau đó xem lại bảng đối số tại thời điểm biên dịch và ví dụ Linux trong COMPILING.md. Chạy qmake với CONFIG+=serveronly CONFIG+=headless trên macOS và xác minh rằng bản build server-only được liên kết thành công, đồng thời Jamulus.app/Contents/MacOS/Jamulus --version khởi động ở chế độ máy chủ. Cập nhật tài liệu để mô tả chính xác việc hỗ trợ macOS và các tùy chọn liên quan.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

macOS needs documentation

🤖 AI: #2551 left macOS out of CONFIG+=serveronly with an open invitation — "The MacOS build might be feasible but there were the bundle options already there and I've no way to check what's going on with the MacOS build. If someone else wants to have a go at that, add DEFINES += SERVER_ONLY and see if it works." Now it is run on a Mac, at c862872e, macOS 12.7.6, Qt 5.15.2, command-line qmake+make, x86_64.

That change does most of it, and is not sufficient. With DEFINES += SERVER_ONLY added to the macx branch and nothing else, the define reaches every compile line and all 15 client-side undefined symbols go away. Nine remain, and every one of them is CSoundBase:

$ qmake Jamulus.pro CONFIG+=serveronly CONFIG+=headless && make -j2
Project MESSAGE: Restricting build to server-only due to CONFIG+=serveronly.
Project MESSAGE: Using CoreAudio.
[...]
Undefined symbols for architecture x86_64:
  "CSoundBase::ReinitRequest(int)", referenced from:
      CSound::deviceNotification(unsigned int, unsigned int, AudioObjectPropertyAddress const*, void*) in sound.o
  "CSoundBase::staticMetaObject", referenced from:
      CSound::staticMetaObject in moc_sound.o
  "vtable for CSoundBase", referenced from:
      CSoundBase::~CSoundBase() in sound.o
[6 further CSoundBase symbols, 9 in total]
ld: symbol(s) not found for architecture x86_64

The nine come from the second difference between the branches: the macx branch adds coreaudio-mac/sound.cpp unconditionally, while soundbase.cpp is stripped platform-independently — so CSound is compiled without its base class. win32 does not hit this because its ASIO sources sit inside the else-branch of its own serveronly test. So macOS needs two changes, not one: the missing define, and a serveronly guard around the CoreAudio sources. With both, a plain CONFIG+=serveronly CONFIG+=headless build links and runs:

$ qmake Jamulus.pro CONFIG+=serveronly CONFIG+=headless && make -j2
Project MESSAGE: Restricting build to server-only due to CONFIG+=serveronly.
Project MESSAGE: Server-only build: skipping CoreAudio.
[...]
$ Jamulus.app/Contents/MacOS/Jamulus --version
- Starting in server mode by default (due to compile time option)
 *** Jamulus, Version 3.12.4dev-nogit

With neither change the option half-applies rather than being ignored: five platform-independent !contains(CONFIG, "serveronly") blocks strip client sources while SERVER_ONLY stays undefined, and the link fails with 24 undefined symbols.

No released artifact is affected: the macOS server app is built by mac/deploy_mac.sh with CONFIG+=server_bundle alone, which keeps both sound files — built at the same commit here, 0 errors, JamulusServer starts in server mode.

Documentation is affected either way: the compile-time arguments table marks server_bundle "macOS only" and jackonmac "on macOS", while serveronly carries no platform qualifier and its one worked example, qmake "CONFIG+=headless serveronly", sits under ## Linux.

🤖 This message was written by AI and reviewed by @mcfnord.

Ngôn ngữ chính
C
Star
1.1k
Fork
248
Merge trung bình
9 ngày 3 giờ
Pull request đã merge (30 ngày)
5

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của jamulussoftware/jamulus

Tất cả issue của jamulussoftware/jamulus

Issue tương tự

Thêm issue về C

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.