Summary: | [Bug] pkg-config of librtlsdr (comms/rtl-sdr) unnecessarily forces -std=gnu89 | ||||||
---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | Nico Sonack <nsonack> | ||||
Component: | Individual Port(s) | Assignee: | Ganael LAPLANCHE <martymac> | ||||
Status: | Closed FIXED | ||||||
Severity: | Affects Only Me | CC: | diizzy | ||||
Priority: | --- | Flags: | bugzilla:
maintainer-feedback?
(martymac) |
||||
Version: | Latest | ||||||
Hardware: | Any | ||||||
OS: | Any | ||||||
Attachments: |
|
Description
Nico Sonack
2021-05-09 14:12:28 UTC
Should probably also use https://github.com/librtlsdr/librtlsdr as repo > Should probably also use https://github.com/librtlsdr/librtlsdr as repo
That indeed sounds like a plan. However when I look at the autotools build
of the upstream source, I can see that noone really seems to be concerned
about that because it is utterly broken. The option of less pain would be
to use CMake as well.
Switching to CMake, the build is broken in another way as of v0.8.0 because
of a missing include of limits.h in src/librtlsdr.c. I will go ahead and submit
a patch upstream. Then we might consider upgrading to version 0.8.0.
Hello Nico, Thanks for your report. CFLAGS+= -fgnu89-inline was introduced in 6b27edfc052a4a7c1ffda25689a671e6b90f7b1f to fix build with Clang 4.0. It was later changed to USE_CSTD= gnu89 in 18878f8fafee33f540a03f2744f10716e416e044. We probably do not need it anymore, so your patch seems OK. I'd prefer wait before switching to librtlsdr/librtlsdr to see how the project evolves as : - this project is not Osmocom's official one - it has not received updates since october - most distros still use osmocom version Best regards, Ganael. Thank you for your reply and feedback! Regarding the possible change to librtlsdr/librtlsdr and to people who read this out of interest: I managed to fix the seriously broken things in the autotools build: https://github.com/librtlsdr/librtlsdr/pull/110 I will see what is going to happen there... Until then I am totally okay with staying at the official version and what most package repositories do. A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=e806a5b226b205e1c99b597e3ec4bcc90427b2bb commit e806a5b226b205e1c99b597e3ec4bcc90427b2bb Author: Ganael LAPLANCHE <martymac@FreeBSD.org> AuthorDate: 2021-05-10 10:22:58 +0000 Commit: Ganael LAPLANCHE <martymac@FreeBSD.org> CommitDate: 2021-05-10 10:32:41 +0000 comms/rtl-sdr: Do not force gnu89 C std anymore - gnu89 do not seem to be needed anymore - also, remove useless dependency to gmake PR: 255725 Reported by: Nico Sonack <nsonack@outlook.com> comms/rtl-sdr/Makefile | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)
> I managed to fix the seriously broken things in the autotools build:
> https://github.com/librtlsdr/librtlsdr/pull/110
> I will see what is going to happen there...
Thanks for the link, I'll keep an eye on it, but it seems that pull request has been closed meanwhile ?
> but it seems that pull request has been closed meanwhile ?
Yes, I closed it again, but the patch is still available in case someone actually wants to fix it. Also, I can reopen the PR if necessary.
As the actual topic of this bug report has been addressed, I would suggest to close it. I feel like discussions on an update/change of the repo should be lead
in a separate bug report.
Yes, feel free to open a dedicated PR for that :) Thanks again, Best regards, Ganael. |