Summary: | net-im/tg_owt: Fails to build on aarch64: CMakeLists.txt:77 (add_library): Target "tg_owt" links to target "tg_owt::libusrsctp" but the target was not found | ||||||
---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | pr | ||||
Component: | Individual Port(s) | Assignee: | Mikael Urankar <mikael> | ||||
Status: | Closed FIXED | ||||||
Severity: | Affects Many People | CC: | henry.hu.sh, mikael | ||||
Priority: | --- | Keywords: | needs-qa | ||||
Version: | Latest | Flags: | henry.hu.sh:
maintainer-feedback+
koobs: merge-quarterly? |
||||
Hardware: | arm64 | ||||||
OS: | Any | ||||||
Attachments: |
|
Description
pr
2021-07-05 16:37:29 UTC
Well, I have no aarch64 machine (I may have some aarch64 SBCs, but they're too weak for telegram), so I can't fix this. Patches are welcome. Created attachment 226265 [details]
patch to files/patch-CMakeLists.txt to remove libusrsctp target for all the architectures
The proposed patch removes tg_owt::libusrsctp target for all the architectures, not only i386 and amd64.
It builds on aarch64 (poudriere builds telegram-desktop ok with net-im/tg_owt). Not tested on other ARMs.
I cannot assess whether the built library nor telegram-desktop work, due to (temporary?) lack of graphic support on my system.
As a side note, you might be willing to add to the port Makefile:
ONLY_FOR_ARCHS= aarch64 amd64 arm armv7 i386
ONLY_FOR_ARCHS_REASON= upstream does not support other archs
I confirm telegram-desktop built on tg_owt works in aarch64. Hi, this is solved by the proposed patch. Can it be committed? Will take a look at the weekend. Need to test how this works on amd64. (In reply to Henry Hu from comment #1) Note: Anyone may test aarch64 (and other architectures) via poudriere build with the QEMU option, which can then create jails with any architecture. Very useful and helpful for QA Comment on attachment 226265 [details]
patch to files/patch-CMakeLists.txt to remove libusrsctp target for all the architectures
^Triage: Please set the maintainer-approval attachment flag (to +) on patches for ports you maintain to signify approval
Attachment -> Details -> maintainer-approval [+]
Comment on attachment 226265 [details]
patch to files/patch-CMakeLists.txt to remove libusrsctp target for all the architectures
Wrong issue, apologies.
Why do we need to disable libusrsctp on FreeBSD? It's listed in LIB_DEPENDS and it's not used? I should have noticed that this patch does not affect i386 and amd64. I didn't consider other platforms when I created the port. Tried to build amd64 and it still builds fine. I'm okay with this patch. Please commit it. (In reply to Mikael Urankar from comment #9) We're not disabling libusrsctp and it's still used. Notice that CMakeLists.txt is also patched to link with libusrsctp, with cmake_external.cmake also patched to provide the include and link directories. The tg_owt carries its own version of libusrsctp, and has its own cmake file to build it. Since we already have it in the ports, I've patched it to use the ports version instead. A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=eda0db32e774b8e58b831ad0b3fe4b1ecc85cd3e commit eda0db32e774b8e58b831ad0b3fe4b1ecc85cd3e Author: Aoek <pr@aoek.com> AuthorDate: 2021-07-18 14:51:49 +0000 Commit: Mikael Urankar <mikael@FreeBSD.org> CommitDate: 2021-07-18 14:56:54 +0000 net-im/tg_owt: Fix build on aarch64 Don't build the bundled libusrsctp, use the one provided by net/libusrsctp like it's done on other arches. PR: 256998 Approved by: Henry Hu (maintainer) net-im/tg_owt/files/patch-CMakeLists.txt | 8 ++++++++ 1 file changed, 8 insertions(+) Thanks! A commit in branch 2021Q3 references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=330c751fd75884ec2891ff920187c991013cf06d commit 330c751fd75884ec2891ff920187c991013cf06d Author: Aoek <pr@aoek.com> AuthorDate: 2021-07-18 14:51:49 +0000 Commit: Mikael Urankar <mikael@FreeBSD.org> CommitDate: 2021-07-18 15:02:24 +0000 net-im/tg_owt: Fix build on aarch64 Don't build the bundled libusrsctp, use the one provided by net/libusrsctp like it's done on other arches. PR: 256998 Approved by: Henry Hu (maintainer) (cherry picked from commit eda0db32e774b8e58b831ad0b3fe4b1ecc85cd3e) net-im/tg_owt/files/patch-CMakeLists.txt | 8 ++++++++ 1 file changed, 8 insertions(+) |