Re: [Xenomai-help] kernel 2.6.32.11 with xenomai 2.5.3 fails to boot on ubuntu lucid system

[Stefan Kisdaroczi <kisda@domain.hid>]

[-- Attachment #1: Type: text/plain, Size: 2398 bytes --]

On 18.08.2010 11:06, Gilles Chanteperdrix wrote:
> Paul wrote:
>   
>> On Wednesday 18 August 2010, Gilles Chanteperdrix wrote:
>>     
>>> Stefan Kisdaroczi wrote:
>>>       
>>>> On 17.08.2010 15:51, Hemal C.Bavishi wrote:
>>>>         
>>>>> When I tried to compile it with the latest version of kernel with
>>>>> xenomai 2.5.4, I am getting following errors in Xenomai (disable
>>>>> CONFIG_SMP, enable CONFIG_X86_UP_APIC and CONFIG_X86_UP_IOAPIC
>>>>> (*).)
>>>>>           
>>>> Just tested, got the same build error with 2.6.34.
>>>> If I patch 2.6.34 with prepare-kernel [1] it compiles,
>>>> if I use the debian packaged patch generated with prepare-patch [2]
>>>> it fails.
>>>>
>>>> I guess a fix is needed in prepare-patch for 2.6.34, but no time to
>>>> look closer now.
>>>>
>>>> Stefan
>>>>
>>>> [1] xenomai-2.5.4/scripts/prepare-kernel.sh
>>>> [2] xenomai-2.5.4/debian/prepare-patch.sh
>>>>         
>>> prepare-kernel.sh has a "--outpatch" option, which seems to be able
>>> to generate patches, so, would not it be possible to modify
>>> prepare-patch to simply call prepare-kernel.sh with the --outpatch
>>> option?
>>>
>>> This way, we will not have to duplicate into prepare-patch.sh the
>>> modifications we make to prepare-kernel.sh.
>>>       
>> The debian/prepare-patch.sh does not require a kernel source tree and 
>> generates patches for multiple kernels & arches - It is a fudge, but it 
>> works for the most part and does not impose dependencies of multiple 
>> kernel source trees on package build systems.
>>     
> Ok. Understood. The thing is that prepare-patch.sh is broken, so now may
> be a good occasion to merge its functionality in prepare-kernel.sh, so
> that we do not duplicate the code in these two really non-trivial scripts.
>   

Hi Gilles,

There is another copy. The debian/ directory from the xenomai tree is
not used for debian packages at debian.org. The Debian Maintainer Roland
Stigge has his own debian/ directory.
I submitted a bugreport including the patch for this bug to debian [1].
If we move the prepare-patch.sh out of the debian/ dir (suggested by
Roland), that would not be necessary.

I suggest to move debian/prepare-patch.sh to
scripts/prepare-debian-patch.sh.
I'll create a patch if you agree.

[1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=593585

Stefan



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 262 bytes --]