Bug 23347 - [0.8] asyncns.c uses non-portable getrlimit
Summary: [0.8] asyncns.c uses non-portable getrlimit
Status: RESOLVED FIXED
Alias: None
Product: Telepathy
Classification: Unclassified
Component: gabble (show other bugs)
Version: 0.8
Hardware: Other NetBSD
: medium normal
Assignee: Telepathy bugs list
QA Contact: Telepathy bugs list
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-08-16 11:16 UTC by Thomas Klausner
Modified: 2010-06-27 15:43 UTC (History)
0 users

See Also:
i915 platform:
i915 features:


Attachments
Patch fixing the problem. (227 bytes, patch)
2009-08-16 11:16 UTC, Thomas Klausner
Details | Splinter Review
Patch fixing compilation problem with 0.9.2 (328 bytes, patch)
2010-06-24 13:16 UTC, Thomas Klausner
Details | Splinter Review

Description Thomas Klausner 2009-08-16 11:16:35 UTC
Created attachment 28670 [details] [review]
Patch fixing the problem.

asyncns.c doesn't compile on NetBSD-5.99.15/amd64:
asyncns.c: In function 'close_allv':
asyncns.c:206: error: storage size of 'rl' isn't known
asyncns.c:277: warning: implicit declaration of function 'getrlimit'
asyncns.c:277: error: 'RLIMIT_NOFILE' undeclared (first use in this function)
asyncns.c:277: error: (Each undeclared identifier is reported only once
asyncns.c:277: error: for each function it appears in.)
asyncns.c:206: warning: unused variable 'rl'

The attached patch fixes the problem.
Comment 1 Simon McVittie 2009-08-24 13:23:04 UTC
I'd prefer not to patch asyncns, since it's imported from elsewhere. Does the version from <http://0pointer.de/lennart/projects/libasyncns/> compile correctly on NetBSD? If so, we just need to update our copy; if not, please report the bug to Lennart.
Comment 2 Thomas Klausner 2009-08-24 13:30:26 UTC
libasyncns-0.7 compiles fine.
Its config.h contains:
libasyncns-0.7/config.h:#define HAVE_PTHREAD 1
I don't see that define when compiling telepathy-gabble.
Comment 3 Simon McVittie 2010-06-23 03:18:34 UTC
This bug only affects the 0.8 stable branch; 0.9 uses GibberResolver. I'd recommend using 0.9.x on non-Linux OSs.
Comment 4 Simon McVittie 2010-06-23 03:19:18 UTC
(In reply to comment #3)
> 0.9 uses GibberResolver

... by which I meant GResolver; GibberResolver was the old non-portable code.
Comment 5 Thomas Klausner 2010-06-24 13:16:39 UTC
Created attachment 36477 [details] [review]
Patch fixing compilation problem with 0.9.2

pkgsrc contains 0.9.2 now, but still needs a patch for asyncns.c to compile (attached).
Comment 6 Simon McVittie 2010-06-25 02:59:53 UTC
(In reply to comment #5)
> Created an attachment (id=36477) [details]
> Patch fixing compilation problem with 0.9.2
> 
> pkgsrc contains 0.9.2 now, but still needs a patch for asyncns.c to compile
> (attached).

The current version is 0.9.14, which no longer contains this code, so from our point of view, this isn't an issue for the 0.9 series. If you need to continue to ship an older release for some reason, that patch looks fine for any POSIX OS, but I'd recommend upgrading; if there are other issues that prevent you from using the latest 0.9.x release, please open bugs for them.
Comment 7 Thomas Klausner 2010-06-27 15:43:41 UTC
I'll update to a later version in a few weeks and open new bug reports if I find problems.

Thanks for your help!


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.