Re: 1.0.16rc2 -build issues

Re: 1.0.16rc2 -build issues

[Alan Horstmann]

On Wednesday 30 January 2008 13:03, you wrote:
> At Wed, 30 Jan 2008 12:41:46 +0000,
>
> Alan Horstmann wrote:
> > On Wednesday 30 January 2008 11:21, Takashi wrote:
> > >
> > > Could you apply the patch below and use ./hgcompile to configure?
> > >
> > > thanks,
> > >
> > > Takashi
> >
> > Patch applied OK.  Unfortunately, hgcompile complains that the Autoconf
> > version is too old on this system -needs >2.59 (looks like I have 1.4o).
> > What is the difference using that rather than straight configure?
>
> You need to regenerate the configure script itself.
>
> >  Is there a
> > way to work round it?
>
> Remove the line "AC_PREREQ(2.59)" in configure.in.

Thanks.  However, I could not get (the old) autoconf to produce a working configure script due to 'AS_HELP_STRING', so I added
	#define CONFIG_HAVE_FFS
to the patched adriver.h and used the original configure script, to proceed for now (which it did).

Unfortunately make didn't get much further, having more ??historic kernel?? trouble in memalloc:

memalloc.c: In function `snd_mem_proc_open':
memalloc.c:611: warning: implicit declaration of function `single_open'
memalloc.c: At top level:
memalloc.c:715: error: `single_release' undeclared here (not in a function)
memalloc.c:715: error: initializer element is not constant
memalloc.c:715: error: (near initialization for `snd_mem_proc_fops.release')
memalloc.c: In function `snd_mem_init':
memalloc.c:731: warning: assignment discards qualifiers from pointer target type

Sorry if this is causing a lot of extra trouble.

Regards  Alan

Re: 1.0.16rc2 -build issues

[Takashi Iwai]

At Wed, 30 Jan 2008 21:03:45 +0000,
Alan Horstmann wrote:
> 
> On Wednesday 30 January 2008 13:03, you wrote:
> > At Wed, 30 Jan 2008 12:41:46 +0000,
> >
> > Alan Horstmann wrote:
> > > On Wednesday 30 January 2008 11:21, Takashi wrote:
> > > >
> > > > Could you apply the patch below and use ./hgcompile to configure?
> > > >
> > > > thanks,
> > > >
> > > > Takashi
> > >
> > > Patch applied OK.  Unfortunately, hgcompile complains that the Autoconf
> > > version is too old on this system -needs >2.59 (looks like I have 1.4o).
> > > What is the difference using that rather than straight configure?
> >
> > You need to regenerate the configure script itself.
> >
> > >  Is there a
> > > way to work round it?
> >
> > Remove the line "AC_PREREQ(2.59)" in configure.in.
> 
> Thanks.  However, I could not get (the old) autoconf to produce a working configure script due to 'AS_HELP_STRING', so I added
> 	#define CONFIG_HAVE_FFS
> to the patched adriver.h and used the original configure script, to proceed for now (which it did).
> 
> Unfortunately make didn't get much further, having more ??historic kernel?? trouble in memalloc:
> 
> memalloc.c: In function `snd_mem_proc_open':
> memalloc.c:611: warning: implicit declaration of function `single_open'
> memalloc.c: At top level:
> memalloc.c:715: error: `single_release' undeclared here (not in a function)
> memalloc.c:715: error: initializer element is not constant
> memalloc.c:715: error: (near initialization for `snd_mem_proc_fops.release')
> memalloc.c: In function `snd_mem_init':
> memalloc.c:731: warning: assignment discards qualifiers from pointer target type

Then the patch below should work.  Give it a try.


Takashi

---

diff -r e600ecb8fbd7 acore/memalloc.patch
--- a/acore/memalloc.patch	Wed Jan 30 08:36:00 2008 +0100
+++ b/acore/memalloc.patch	Thu Jan 31 12:39:11 2008 +0100
@@ -189,7 +189,7 @@
  
  
  #ifdef CONFIG_PROC_FS
-+#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 4, 0)
++#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 0)
  /*
   * proc file interface
   */
@@ -205,7 +205,7 @@
  static int __init snd_mem_init(void)
  {
  #ifdef CONFIG_PROC_FS
-+#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 4, 0)
++#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 0)
  	snd_mem_proc = create_proc_entry(SND_MEM_PROC_FILE, 0644, NULL);
  	if (snd_mem_proc)
  		snd_mem_proc->proc_fops = &snd_mem_proc_fops;
@@ -216,7 +216,7 @@
  
  static void __exit snd_mem_exit(void)
  {
-+#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 4, 0)
++#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 0)
  	remove_proc_entry(SND_MEM_PROC_FILE, NULL);
 +#endif
  	free_all_reserved_pages();

Re: 1.0.16rc2 -build issues

[Alan Horstmann]

On Thursday 31 January 2008 11:38, you wrote:
> At Wed, 30 Jan 2008 21:03:45 +0000,
>
> Alan Horstmann wrote:
> > Unfortunately make didn't get much further, having more ??historic
> > kernel?? trouble in memalloc:
> >
> > memalloc.c: In function `snd_mem_proc_open':
> > memalloc.c:611: warning: implicit declaration of function `single_open'
> > memalloc.c: At top level:
> > memalloc.c:715: error: `single_release' undeclared here (not in a
> > function) memalloc.c:715: error: initializer element is not constant
> > memalloc.c:715: error: (near initialization for
> > `snd_mem_proc_fops.release') memalloc.c: In function `snd_mem_init':
> > memalloc.c:731: warning: assignment discards qualifiers from pointer
> > target type
>
> Then the patch below should work.  Give it a try.

Yup, that nailed it!  Make completes OK now, thanks.

I applied the memalloc.patch patch by hand.  If this is to be applied, notice 
that Line 200 #endif text string also presumably needs changing.

Alsa-lib was OK, but the configure was slightly messy in reporting 
'python-config' Command not found  before a more reasonable message and 
disabling python from the build.

If both the patches are to be applied, would you like me to retest the nightly 
build at some stage(which presumably won't need hgcompile)?  If so can you 
give the link again.

Thanks 

Alan

> ---
>
> diff -r e600ecb8fbd7 acore/memalloc.patch
> --- a/acore/memalloc.patch	Wed Jan 30 08:36:00 2008 +0100
> +++ b/acore/memalloc.patch	Thu Jan 31 12:39:11 2008 +0100
> @@ -189,7 +189,7 @@
>
>
>   #ifdef CONFIG_PROC_FS
> -+#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 4, 0)
> ++#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 0)
>   /*
>    * proc file interface
>    */
> @@ -205,7 +205,7 @@
>   static int __init snd_mem_init(void)
>   {
>   #ifdef CONFIG_PROC_FS
> -+#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 4, 0)
> ++#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 0)
>   	snd_mem_proc = create_proc_entry(SND_MEM_PROC_FILE, 0644, NULL);
>   	if (snd_mem_proc)
>   		snd_mem_proc->proc_fops = &snd_mem_proc_fops;
> @@ -216,7 +216,7 @@
>
>   static void __exit snd_mem_exit(void)
>   {
> -+#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 4, 0)
> ++#if LINUX_VERSION_CODE >= KERNEL_VERSION(2, 6, 0)
>   	remove_proc_entry(SND_MEM_PROC_FILE, NULL);
>  +#endif
>   	free_all_reserved_pages();

Re: 1.0.16rc2 -build issues

[Alan Horstmann]

Hi Takashi,

I tried out yesterdays tarball hg20080207 to confirm DSP2000 Midi stuff before 
pointing another DSP2000 user to it.  Build proceeded OK but I did notice the 
following warnings, which I might have missed previously, or they could be 
new.  Do you care about redefinitions?

pcm_oss.c:639:1: warning: "AFMT_S32_LE" redefined
In file included from pcm_oss.c:43:
/lib/modules/2.4.21-99-athlon/build/include/linux/soundcard.h:562:1: warning: 
this is the location of the previous definition
pcm_oss.c:640:1: warning: "AFMT_S32_BE" redefined
/lib/modules/2.4.21-99-athlon/build/include/linux/soundcard.h:563:1: warning: 
this is the location of the previous definition
pcm_oss.c:641:1: warning: "AFMT_S24_LE" redefined
/lib/modules/2.4.21-99-athlon/build/include/linux/soundcard.h:558:1: warning: 
this is the location of the previous definition
pcm_oss.c:642:1: warning: "AFMT_S24_BE" redefined
/lib/modules/2.4.21-99-athlon/build/include/linux/soundcard.h:559:1: warning: 
this is the location of the previous definition

Regards

Alan

Re: 1.0.16rc2 -build issues

[Clemens Ladisch]

Alan Horstmann wrote:
> I tried out yesterdays tarball hg20080207 to confirm DSP2000 Midi stuff before 
> pointing another DSP2000 user to it.  Build proceeded OK but I did notice the 
> following warnings, which I might have missed previously, or they could be 
> new.  Do you care about redefinitions?
> 
> pcm_oss.c:639:1: warning: "AFMT_S32_LE" redefined
> In file included from pcm_oss.c:43:
> /lib/modules/2.4.21-99-athlon/build/include/linux/soundcard.h:562:1: warning: 
> this is the location of the previous definition

This warnings are caused by harmless whitespace in the 2.4.x soundcard.h
file.


Regards,
Clemens