* alsa-lib 1.0.27 incorrect definition of __kernel_off_t? @ 2013-07-22 21:22 Kevin Groeneveld 2013-07-23 12:55 ` Jaroslav Kysela 0 siblings, 1 reply; 3+ messages in thread From: Kevin Groeneveld @ 2013-07-22 21:22 UTC (permalink / raw) To: alsa-devel I tried doing a cross compile of alsa-lib 1.0.27.2 today and ended up with the following error: error: conflicting types for 'off_t' Commit 9b5beef1 adds the following line to local.h: #define __kernel_off_t off_t Not only is __kernel_off_t already defined elsewhere in my header files, the above line tries to define __kernel_off_t (which I think should be 32 bit on my platform) to off_t which is 64 bit. I just deleted the line and everything seems to be working so far. Kevin ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: alsa-lib 1.0.27 incorrect definition of __kernel_off_t? 2013-07-22 21:22 alsa-lib 1.0.27 incorrect definition of __kernel_off_t? Kevin Groeneveld @ 2013-07-23 12:55 ` Jaroslav Kysela 2013-07-23 14:58 ` Kevin Groeneveld 0 siblings, 1 reply; 3+ messages in thread From: Jaroslav Kysela @ 2013-07-23 12:55 UTC (permalink / raw) To: Kevin Groeneveld; +Cc: alsa-devel Date 22.7.2013 23:22, Kevin Groeneveld wrote: > I tried doing a cross compile of alsa-lib 1.0.27.2 today and ended up with > the following error: > > error: conflicting types for 'off_t' > > Commit 9b5beef1 adds the following line to local.h: > > #define __kernel_off_t off_t > > Not only is __kernel_off_t already defined elsewhere in my header files, > the above line tries to define __kernel_off_t (which I think should be 32 > bit on my platform) to off_t which is 64 bit. > > I just deleted the line and everything seems to be working so far. I tried to fix this issue using <linux/types.h>. Please, check: http://git.alsa-project.org/?p=alsa-lib.git;a=commitdiff;h=b3b53a1bec47aaa2baef724cc9e79adadd603caf Jaroslav -- Jaroslav Kysela <perex@perex.cz> Linux Kernel Sound Maintainer ALSA Project; Red Hat, Inc. ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: alsa-lib 1.0.27 incorrect definition of __kernel_off_t? 2013-07-23 12:55 ` Jaroslav Kysela @ 2013-07-23 14:58 ` Kevin Groeneveld 0 siblings, 0 replies; 3+ messages in thread From: Kevin Groeneveld @ 2013-07-23 14:58 UTC (permalink / raw) Cc: alsa-devel On Tue, Jul 23, 2013 at 8:55 AM, Jaroslav Kysela <perex@perex.cz> wrote: > I tried to fix this issue using <linux/types.h>. Please, check: > > > http://git.alsa-project.org/?p=alsa-lib.git;a=commitdiff;h=b3b53a1bec47aaa2baef724cc9e79adadd603caf > > Yes, that patch fixes the compile error I was getting. Thanks for looking into this. You don't actually need to include <linux/types.h> in local.h. local.h includes asound.h which already includes <linux/types.h>. This is how I just got away with deleting the #define __kernel_off_t line. Kevin ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-07-23 14:58 UTC | newest] Thread overview: 3+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-07-22 21:22 alsa-lib 1.0.27 incorrect definition of __kernel_off_t? Kevin Groeneveld 2013-07-23 12:55 ` Jaroslav Kysela 2013-07-23 14:58 ` Kevin Groeneveld
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.