On 08/24/10 13:28, Federico G. Schwindt wrote:
Your patch for ThreadPool.c is wrong. You didn't remove OpenBSD from the
first ifdef. As for the first part of it, although my diff was wrong you
still want to return the min/max priorities, so it should be something
like this (not a real diff so you'll have to adapt it):
#if (defined(HAVE_SCHED_GET_PRIORITY_MIN)&&
defined(HAVE_SCHED_GET_PRIORITY_MAX)) || defined(__OpenBSD__)
..
+#if !defined(__OpenBSD__)
minPriority = sched_get_priority_min( currentPolicy );
maxPriority = sched_get_priority_max( currentPolicy );
+#else
+ minPriority = 0;
+ maxPriority = 31;
+#endif
..
OpenBSD does not have those functions but does have the priorities.
A few other comments:
- Look at the Makefile.template and try to follow it (spaces, order, etc)
- Switch to new dependency style (I've been hit with the cluestick so why
not pass this knowledge)
Other than that untested.
f.-
* ThreadPool.c updated according to your comments. * New patches to remove the compiler warnings. * Makefile follows template. * Switched to new dependency style.Builds with no warnings now. Didn't notice them previously, was only redirecting stdout to portslogger.
/Markus
mediatomb.tar.gz
Description: application/gzip
