All of lore.kernel.org
 help / color / mirror / Atom feed
* [Xenomai-core] Xenomai in Debian
@ 2010-02-07  9:01 Roland Stigge
  2010-02-07 14:24 ` Gilles Chanteperdrix
  2010-02-13 14:32 ` Gilles Chanteperdrix
  0 siblings, 2 replies; 38+ messages in thread
From: Roland Stigge @ 2010-02-07  9:01 UTC (permalink / raw)
  To: Xenomai core; +Cc: Jan Kiszka

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

Hi,

I'm just working on the Xenomai 2.5.1 Debian package and encountered 
non-PIC code (evil! ;-) in the shared libraries (skins libraries).

You can see it with e.g.

$ readelf -d src/skins/native/.libs/libnative.so.3.0.0

The "TEXTREL" tag indicates non-PIC code.

$ scanelf -qT src/skins/native/.libs/libnative.so.3.0.0

Shows the function and code offset of the relocation.

With

$ objdump -d src/skins/native/.libs/libnative.so.3.0.0

I found the actual location of the relocation.

I prepared a patch (attached) that fixes it: First, the relocation 
itself by letting GCC create the address in PIC style. Second, "extern 
inline" looks a bit counterintuitive at first, but according to the GCC 
manual, it seems to be what we want (macro behaviour).

Thanks Jan for reminding me!

bye,
   Roland

[-- Attachment #2: xenomai.patch --]
[-- Type: text/x-patch, Size: 414 bytes --]

--- xenomai-2.5.1.orig/include/asm-x86/syscall.h
+++ xenomai-2.5.1/include/asm-x86/syscall.h
@@ -161,9 +161,11 @@
      ".endif\n\t"
      ".endm\n\t");
 
-static inline void __xn_get_eip(void **dest)
+__attribute__((always_inline))
+extern inline void __xn_get_eip(void **dest)
 {
-        asm volatile("movl $1f, %0; 1:": "=m"(*dest));
+addr:
+	*dest = &&addr;
 }
 
 static inline void __xn_get_ebp(void **dest)

^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-07  9:01 [Xenomai-core] Xenomai in Debian Roland Stigge
@ 2010-02-07 14:24 ` Gilles Chanteperdrix
  2010-02-13 14:32 ` Gilles Chanteperdrix
  1 sibling, 0 replies; 38+ messages in thread
From: Gilles Chanteperdrix @ 2010-02-07 14:24 UTC (permalink / raw)
  To: Roland Stigge; +Cc: Jan Kiszka, Xenomai core

Roland Stigge wrote:
> Hi,
> 
> I'm just working on the Xenomai 2.5.1 Debian package and encountered 
> non-PIC code (evil! ;-) in the shared libraries (skins libraries).
> 
> You can see it with e.g.
> 
> $ readelf -d src/skins/native/.libs/libnative.so.3.0.0
> 
> The "TEXTREL" tag indicates non-PIC code.
> 
> $ scanelf -qT src/skins/native/.libs/libnative.so.3.0.0
> 
> Shows the function and code offset of the relocation.
> 
> With
> 
> $ objdump -d src/skins/native/.libs/libnative.so.3.0.0
> 
> I found the actual location of the relocation.
> 
> I prepared a patch (attached) that fixes it: First, the relocation 
> itself by letting GCC create the address in PIC style. Second, "extern 
> inline" looks a bit counterintuitive at first, but according to the GCC 
> manual, it seems to be what we want (macro behaviour).
> 
> Thanks Jan for reminding me!

Glad to hear that debian work on Xenomai has resumed. Merged, thanks.

-- 
					    Gilles.


^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-07  9:01 [Xenomai-core] Xenomai in Debian Roland Stigge
  2010-02-07 14:24 ` Gilles Chanteperdrix
@ 2010-02-13 14:32 ` Gilles Chanteperdrix
  2010-02-13 15:07   ` Roland Stigge
  1 sibling, 1 reply; 38+ messages in thread
From: Gilles Chanteperdrix @ 2010-02-13 14:32 UTC (permalink / raw)
  To: Roland Stigge; +Cc: Xenomai core

Roland Stigge wrote:
> Hi,
> 
> I'm just working on the Xenomai 2.5.1 Debian package and encountered 
> non-PIC code (evil! ;-) in the shared libraries (skins libraries).

Hi Roland,

I just had a look at the Debian package, and there are a few things that
I do not like:
- the package ships with patches for 2.6.24. These I-pipe patches are
outdated and have been publicly announced as being bogus on the Xenomai
mailing lists;
- xenomai 2.5.1 seems to have introduced a conflict with sigtest, my
fault, you seem to have fixed this by prefixing all xenomai binaries
with xenomai-, there was a simple solution which would have been for us
to rename only sigtest.

This looks to me like a bad idea. You are breaking the documented user
interface.

Applications which were compiling for older versions of xenomai using
xeno-config will simply stop working with the new version, not because
of Xenomai incompatibilities, but simply because of a decision of the
maintainer of Xenomai Debian package. Not mentioning how preposterous it
looks to have things called xenomai-xeno-config.

I have already asked publicly for this, but will ask it once again:
please inform the xenomai mailing list of the debian xenomai package
bugs, and get us involved in their resolution. I do not really
understand why you are refusing to do this. As far as I know, we never
refused any patch coming from you.

If you refuse to comply to this simple request, I will have to advise
publicly people not to use the "official" debian packages, including on
the xenomai project's wiki, and in the text documentation which ships
with Xenomai. Because the debian packaging of xenomai simply do not
meets the quality standards of the Xenomai project. I do not mean that
our quality is perfect, but one of the motto of the Xenomai project is
to avoid breaking uselessly the interfaces, and you just did it.

Thanks in advance.
Regards.

-- 
					    Gilles.


^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-13 14:32 ` Gilles Chanteperdrix
@ 2010-02-13 15:07   ` Roland Stigge
  2010-02-13 15:25     ` Gilles Chanteperdrix
                       ` (2 more replies)
  0 siblings, 3 replies; 38+ messages in thread
From: Roland Stigge @ 2010-02-13 15:07 UTC (permalink / raw)
  To: Gilles Chanteperdrix, xenomai-core

Hi Gilles,

first - I'm sorry if you sometimes feel offended by my work on Xenomai 
in Debian. I understand that you are very much connected to your project 
and want to have it working perfectly everywhere.

Unfortunately, my time to work on this is limited and the last uploads 
were work in progress - to provide latest Xenomai in Debian. Further 
work on it was planned for this weekend.

But please also understand that Debian developers will possibly 
prioritize work on upstream packages where they feel their work is 
appreciated. So please think about your tone before sending email and 
driving people away from Xenomai.

Gilles Chanteperdrix wrote:
> - the package ships with patches for 2.6.24. These I-pipe patches are
> outdated and have been publicly announced as being bogus on the Xenomai
> mailing lists;

Thanks for pointing that out! Will be removed.

> - xenomai 2.5.1 seems to have introduced a conflict with sigtest, my
> fault, you seem to have fixed this by prefixing all xenomai binaries
> with xenomai-, there was a simple solution which would have been for us
> to rename only sigtest.

First, I was thinking about this solution also, of course. But many of 
the Xenomai binaries' names are very generic and not easily recognizable 
as related to Xenomai ("latency", "arith", etc.).  (Please consider that 
there are 10000s of binaries in Debian's /usr/bin.) So I decided to 
rename them all, for consistency. Therefore, also "xenomai-xeno-config". 
:-) It's one exception. I was completely aware of this and also didn't 
find it very beautiful.

The alternative would have been a separate directory for them, e.g. 
somewhere under /usr/lib/xenomai. But that would lead to them not being 
available in user's PATHs. So I decided otherwise.

> This looks to me like a bad idea. You are breaking the documented user
> interface.

Feel free to make suggestions for improvement.

> I have already asked publicly for this, but will ask it once again:
> please inform the xenomai mailing list of the debian xenomai package
> bugs, and get us involved in their resolution.

I'm doing this if it concerns you. E.g. your non-PIC bug. Problems with 
Debian packaging are Debian's task. I'm open to all suggestions, but 
please CC me, I'm not following the Xenomai list so closely.

 > Because the debian packaging of xenomai simply do not
 > meets the quality standards of the Xenomai project.

Please stop. Debian sid is the equivalent of a development branch of 
Xenomai. It's unreleased code. I also don't flame you for bugs in 
Xenomai but instead send patches. So please let's work friendly each one 
on his own project and don't flame.

Thanks for considering.

bye,
   Roland



^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-13 15:07   ` Roland Stigge
@ 2010-02-13 15:25     ` Gilles Chanteperdrix
  2010-02-13 18:30       ` Jan Kiszka
  2010-02-14  9:38     ` Philippe Gerum
  2010-03-01 17:04     ` Gilles Chanteperdrix
  2 siblings, 1 reply; 38+ messages in thread
From: Gilles Chanteperdrix @ 2010-02-13 15:25 UTC (permalink / raw)
  To: Roland Stigge; +Cc: xenomai-core

Roland Stigge wrote:
> Hi Gilles,
> 
> first - I'm sorry if you sometimes feel offended by my work on Xenomai 
> in Debian. I understand that you are very much connected to your project 
> and want to have it working perfectly everywhere.
> 
> Unfortunately, my time to work on this is limited and the last uploads 
> were work in progress - to provide latest Xenomai in Debian. Further 
> work on it was planned for this weekend.
> 
> But please also understand that Debian developers will possibly 
> prioritize work on upstream packages where they feel their work is 
> appreciated. So please think about your tone before sending email and 
> driving people away from Xenomai.

What matters for me is Xenomai users, not the Debian package maintainer.
I am almost thinking that I would prefer Xenomai not having an
"official" Debian package which has been shipping for monthes with buggy
adeos patches and lagging behind upstream development.

-- 
					    Gilles.


^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-13 15:25     ` Gilles Chanteperdrix
@ 2010-02-13 18:30       ` Jan Kiszka
  2010-02-13 18:37         ` Gilles Chanteperdrix
  0 siblings, 1 reply; 38+ messages in thread
From: Jan Kiszka @ 2010-02-13 18:30 UTC (permalink / raw)
  To: Gilles Chanteperdrix, Roland Stigge; +Cc: xenomai-core

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

Gilles Chanteperdrix wrote:
> Roland Stigge wrote:
>> Hi Gilles,
>>
>> first - I'm sorry if you sometimes feel offended by my work on Xenomai 
>> in Debian. I understand that you are very much connected to your project 
>> and want to have it working perfectly everywhere.
>>
>> Unfortunately, my time to work on this is limited and the last uploads 
>> were work in progress - to provide latest Xenomai in Debian. Further 
>> work on it was planned for this weekend.
>>
>> But please also understand that Debian developers will possibly 
>> prioritize work on upstream packages where they feel their work is 
>> appreciated. So please think about your tone before sending email and 
>> driving people away from Xenomai.
> 
> What matters for me is Xenomai users, not the Debian package maintainer.
> I am almost thinking that I would prefer Xenomai not having an
> "official" Debian package which has been shipping for monthes with buggy
> adeos patches and lagging behind upstream development.

Hey guys, this leads to nothing.

I agree with Gilles that a distro package that could appear to users
like it's as mature as upstream while it isn't does not help anyone. But
not all the work here is paid, and resources will remain limited. So
Roland's remark is valid as well that not everything can be done
instantly in The Perfect Way.

The only way to resolve this without killing the idea of ready-to-use
Xenomai package is to openly discuss the problems of both sides and try
to find optimal solutions. I also bet Roland would be happy about
patch-based discussions - just like we are for upstream. Just let the
discussion take place here on this list, in an objective manner, and
ideally before things may show up in releases etc.

Jan


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

^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-13 18:30       ` Jan Kiszka
@ 2010-02-13 18:37         ` Gilles Chanteperdrix
  2010-02-13 18:48           ` Jan Kiszka
  0 siblings, 1 reply; 38+ messages in thread
From: Gilles Chanteperdrix @ 2010-02-13 18:37 UTC (permalink / raw)
  To: Jan Kiszka; +Cc: xenomai-core

Jan Kiszka wrote:
> Gilles Chanteperdrix wrote:
>> Roland Stigge wrote:
>>> Hi Gilles,
>>>
>>> first - I'm sorry if you sometimes feel offended by my work on Xenomai 
>>> in Debian. I understand that you are very much connected to your project 
>>> and want to have it working perfectly everywhere.
>>>
>>> Unfortunately, my time to work on this is limited and the last uploads 
>>> were work in progress - to provide latest Xenomai in Debian. Further 
>>> work on it was planned for this weekend.
>>>
>>> But please also understand that Debian developers will possibly 
>>> prioritize work on upstream packages where they feel their work is 
>>> appreciated. So please think about your tone before sending email and 
>>> driving people away from Xenomai.
>> What matters for me is Xenomai users, not the Debian package maintainer.
>> I am almost thinking that I would prefer Xenomai not having an
>> "official" Debian package which has been shipping for monthes with buggy
>> adeos patches and lagging behind upstream development.
> 
> Hey guys, this leads to nothing.
> 
> I agree with Gilles that a distro package that could appear to users
> like it's as mature as upstream while it isn't does not help anyone. But
> not all the work here is paid, and resources will remain limited. So
> Roland's remark is valid as well that not everything can be done
> instantly in The Perfect Way.
> 
> The only way to resolve this without killing the idea of ready-to-use
> Xenomai package is to openly discuss the problems of both sides and try
> to find optimal solutions. I also bet Roland would be happy about
> patch-based discussions - just like we are for upstream. Just let the
> discussion take place here on this list, in an objective manner, and
> ideally before things may show up in releases etc.

As I already said, I think the proper place to discuss all this is on
the Xenomai mailing list. I have no problem renaming sigtest, at all, I
do not even require a patch to do it. The point is that we did not even
knew that it generated a conflict. And for me this is the real problem,
I am all for helping Roland generating good packages, but if he does not
want to talk to us, what choice do we have?

-- 
					    Gilles.


^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-13 18:37         ` Gilles Chanteperdrix
@ 2010-02-13 18:48           ` Jan Kiszka
  2010-02-13 21:15             ` Gilles Chanteperdrix
  0 siblings, 1 reply; 38+ messages in thread
From: Jan Kiszka @ 2010-02-13 18:48 UTC (permalink / raw)
  To: Gilles Chanteperdrix; +Cc: xenomai-core

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

Gilles Chanteperdrix wrote:
> Jan Kiszka wrote:
>> Gilles Chanteperdrix wrote:
>>> Roland Stigge wrote:
>>>> Hi Gilles,
>>>>
>>>> first - I'm sorry if you sometimes feel offended by my work on Xenomai 
>>>> in Debian. I understand that you are very much connected to your project 
>>>> and want to have it working perfectly everywhere.
>>>>
>>>> Unfortunately, my time to work on this is limited and the last uploads 
>>>> were work in progress - to provide latest Xenomai in Debian. Further 
>>>> work on it was planned for this weekend.
>>>>
>>>> But please also understand that Debian developers will possibly 
>>>> prioritize work on upstream packages where they feel their work is 
>>>> appreciated. So please think about your tone before sending email and 
>>>> driving people away from Xenomai.
>>> What matters for me is Xenomai users, not the Debian package maintainer.
>>> I am almost thinking that I would prefer Xenomai not having an
>>> "official" Debian package which has been shipping for monthes with buggy
>>> adeos patches and lagging behind upstream development.
>> Hey guys, this leads to nothing.
>>
>> I agree with Gilles that a distro package that could appear to users
>> like it's as mature as upstream while it isn't does not help anyone. But
>> not all the work here is paid, and resources will remain limited. So
>> Roland's remark is valid as well that not everything can be done
>> instantly in The Perfect Way.
>>
>> The only way to resolve this without killing the idea of ready-to-use
>> Xenomai package is to openly discuss the problems of both sides and try
>> to find optimal solutions. I also bet Roland would be happy about
>> patch-based discussions - just like we are for upstream. Just let the
>> discussion take place here on this list, in an objective manner, and
>> ideally before things may show up in releases etc.
> 
> As I already said, I think the proper place to discuss all this is on
> the Xenomai mailing list. I have no problem renaming sigtest, at all, I
> do not even require a patch to do it. The point is that we did not even
> knew that it generated a conflict. And for me this is the real problem,
> I am all for helping Roland generating good packages, but if he does not
> want to talk to us, what choice do we have?
> 

We can only ask him kindly to remember xenomai@xenomai.org.

Jan


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

^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-13 18:48           ` Jan Kiszka
@ 2010-02-13 21:15             ` Gilles Chanteperdrix
  0 siblings, 0 replies; 38+ messages in thread
From: Gilles Chanteperdrix @ 2010-02-13 21:15 UTC (permalink / raw)
  To: Jan Kiszka; +Cc: xenomai-core

Jan Kiszka wrote:
> Gilles Chanteperdrix wrote:
>> Jan Kiszka wrote:
>>> Gilles Chanteperdrix wrote:
>>>> Roland Stigge wrote:
>>>>> Hi Gilles,
>>>>>
>>>>> first - I'm sorry if you sometimes feel offended by my work on Xenomai 
>>>>> in Debian. I understand that you are very much connected to your project 
>>>>> and want to have it working perfectly everywhere.
>>>>>
>>>>> Unfortunately, my time to work on this is limited and the last uploads 
>>>>> were work in progress - to provide latest Xenomai in Debian. Further 
>>>>> work on it was planned for this weekend.
>>>>>
>>>>> But please also understand that Debian developers will possibly 
>>>>> prioritize work on upstream packages where they feel their work is 
>>>>> appreciated. So please think about your tone before sending email and 
>>>>> driving people away from Xenomai.
>>>> What matters for me is Xenomai users, not the Debian package maintainer.
>>>> I am almost thinking that I would prefer Xenomai not having an
>>>> "official" Debian package which has been shipping for monthes with buggy
>>>> adeos patches and lagging behind upstream development.
>>> Hey guys, this leads to nothing.
>>>
>>> I agree with Gilles that a distro package that could appear to users
>>> like it's as mature as upstream while it isn't does not help anyone. But
>>> not all the work here is paid, and resources will remain limited. So
>>> Roland's remark is valid as well that not everything can be done
>>> instantly in The Perfect Way.
>>>
>>> The only way to resolve this without killing the idea of ready-to-use
>>> Xenomai package is to openly discuss the problems of both sides and try
>>> to find optimal solutions. I also bet Roland would be happy about
>>> patch-based discussions - just like we are for upstream. Just let the
>>> discussion take place here on this list, in an objective manner, and
>>> ideally before things may show up in releases etc.
>> As I already said, I think the proper place to discuss all this is on
>> the Xenomai mailing list. I have no problem renaming sigtest, at all, I
>> do not even require a patch to do it. The point is that we did not even
>> knew that it generated a conflict. And for me this is the real problem,
>> I am all for helping Roland generating good packages, but if he does not
>> want to talk to us, what choice do we have?
>>
> 
> We can only ask him kindly to remember xenomai@xenomai.org.

This looks pointless, as we already had that argument in may 2008.

-- 
					    Gilles.


^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-13 15:07   ` Roland Stigge
  2010-02-13 15:25     ` Gilles Chanteperdrix
@ 2010-02-14  9:38     ` Philippe Gerum
  2010-02-23 16:52       ` Stefan Kisdaroczi
  2010-02-25 13:44       ` Stefan Kisdaroczi
  2010-03-01 17:04     ` Gilles Chanteperdrix
  2 siblings, 2 replies; 38+ messages in thread
From: Philippe Gerum @ 2010-02-14  9:38 UTC (permalink / raw)
  To: Roland Stigge; +Cc: xenomai-core


Hi Roland,

On Sat, 2010-02-13 at 16:07 +0100, Roland Stigge wrote: 
> Hi Gilles,
> 
> first - I'm sorry if you sometimes feel offended by my work on Xenomai 
> in Debian. I understand that you are very much connected to your project 
> and want to have it working perfectly everywhere.
> 
> Unfortunately, my time to work on this is limited and the last uploads 
> were work in progress - to provide latest Xenomai in Debian. Further 
> work on it was planned for this weekend.
> 
> But please also understand that Debian developers will possibly 
> prioritize work on upstream packages where they feel their work is 
> appreciated. So please think about your tone before sending email and 
> driving people away from Xenomai.
> 
> Gilles Chanteperdrix wrote:
> > - the package ships with patches for 2.6.24. These I-pipe patches are
> > outdated and have been publicly announced as being bogus on the Xenomai
> > mailing lists;
> 
> Thanks for pointing that out! Will be removed.
> 
> > - xenomai 2.5.1 seems to have introduced a conflict with sigtest, my
> > fault, you seem to have fixed this by prefixing all xenomai binaries
> > with xenomai-, there was a simple solution which would have been for us
> > to rename only sigtest.
> 
> First, I was thinking about this solution also, of course. But many of 
> the Xenomai binaries' names are very generic and not easily recognizable 
> as related to Xenomai ("latency", "arith", etc.).  (Please consider that 
> there are 10000s of binaries in Debian's /usr/bin.) So I decided to 
> rename them all, for consistency. Therefore, also "xenomai-xeno-config". 
> :-) It's one exception. I was completely aware of this and also didn't 
> find it very beautiful.
> 
> The alternative would have been a separate directory for them, e.g. 
> somewhere under /usr/lib/xenomai. But that would lead to them not being 
> available in user's PATHs. So I decided otherwise.
> 
> > This looks to me like a bad idea. You are breaking the documented user
> > interface.
> 
> Feel free to make suggestions for improvement.
> 
> > I have already asked publicly for this, but will ask it once again:
> > please inform the xenomai mailing list of the debian xenomai package
> > bugs, and get us involved in their resolution.
> 
> I'm doing this if it concerns you. E.g. your non-PIC bug. Problems with 
> Debian packaging are Debian's task. I'm open to all suggestions, but 
> please CC me, I'm not following the Xenomai list so closely.
> 
>  > Because the debian packaging of xenomai simply do not
>  > meets the quality standards of the Xenomai project.
> 
> Please stop. Debian sid is the equivalent of a development branch of 
> Xenomai. It's unreleased code. I also don't flame you for bugs in 
> Xenomai but instead send patches. So please let's work friendly each one 
> on his own project and don't flame.
> 

Ok, let's end the ballistic game. I think Gilles is very concerned by
the fact that introducing inconsistencies or any ambiguity in downstream
packages will certainly backfire upstream, with endless posting on the
xenomai mailing lists from users being lost in space. Which makes sense.

Personally, I would really recommend to move the Xenomai stuff in its
own sub-dir whenever possible, as a first remedy; requiring PATH update
is much better than forking the Xenomai namespace.
In the future, maybe we could simply provide a wrapper script accepting
sub-commands, such as "xeno latency, xeno sigtest" etc, to be put
into /usr/bin by distros, which would hide the actual location of those
binaries?

In any case, thanks for your work so far. We probably need to discuss
the packaging issues on this list, so that we get both consistency and
usability in the future.

Gilles and Roland, if this is fine with you, I'll handle the liaison
role with upstream packagers, so please CC me explicitly on those mails.
We'll sort out this issue, it doesn't look that bad anyway.

-- 
Philippe.




^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-14  9:38     ` Philippe Gerum
@ 2010-02-23 16:52       ` Stefan Kisdaroczi
  2010-02-23 17:46         ` Philippe Gerum
  2010-02-25 13:44       ` Stefan Kisdaroczi
  1 sibling, 1 reply; 38+ messages in thread
From: Stefan Kisdaroczi @ 2010-02-23 16:52 UTC (permalink / raw)
  To: xenomai


[-- Attachment #1.1: Type: text/plain, Size: 1631 bytes --]

Hi,

Am 14.02.2010 10:38, schrieb Philippe Gerum:
> 
> <snip>
> In the future, maybe we could simply provide a wrapper script accepting
> sub-commands, such as "xeno latency, xeno sigtest" etc, to be put
> into /usr/bin by distros, which would hide the actual location of those
> binaries?
>
> In any case, thanks for your work so far. We probably need to discuss
> the packaging issues on this list, so that we get both consistency and
> usability in the future.
> 
> Gilles and Roland, if this is fine with you, I'll handle the liaison
> role with upstream packagers, so please CC me explicitly on those mails.
> We'll sort out this issue, it doesn't look that bad anyway.

Roland added a xeno wrapper to the debian.org xenomai package 2.5.1-3.

I synced now the debian/ directories from debian.org and xenomai.org:
 - For debian.org I sent patches to the Debian bugtracker [1] [2].
   Another patch for dpkg-cross support [3] I sent to Roland privately.
 - For xenomai.org I attached patches to this mail (against -2.5.git).

If both parties apply the patches the debian directories are in sync,
except some minor differences in the debian/control file, see patch
do-not-commit-please.patch. I would like to keep these changes out so
that the xenomai.org packages are compatible with Debian 5.0 Lenny.
The debian.org packages are for Debian 6.0 Squeeze.

Thanks
kisda

[1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=571099
[2] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=571104
[3] http://git.xenomai.org/?p=xenomai-2.5.git;a=commitdiff;h=5bcd18f714f4cbeaaac0cc4a08e6c9f375aa3b77

[-- Attachment #1.2: 0001-debian-copyright-fix-typo-and-add-project-url.patch --]
[-- Type: text/plain, Size: 1293 bytes --]

From f8bfbe147654f9eb240b0e94d774185940444b8d Mon Sep 17 00:00:00 2001
From: Stefan Kisdaroczi <kisda@domain.hid>
Date: Tue, 23 Feb 2010 13:06:52 +0100
Subject: [PATCH] debian: copyright: fix typo and add project url

---
 debian/copyright |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/debian/copyright b/debian/copyright
index 683e276..b3980df 100644
--- a/debian/copyright
+++ b/debian/copyright
@@ -3,6 +3,8 @@ On: Sat Mar 3 12:00 GMT 2007
 
 The primary author of the upstream package is Philippe Gerum.
 
+It was downloaded from http://www.xenomai.org/
+
 Copyright (C) 2001,2002,2003,2004,2005,2006,2007,2008 Philippe Gerum <rpm@domain.hid>.
 Copyright (C) 2005 Dmitry Adamushko <dmitry.adamushko@domain.hid>
 Copyright (C) 2001,2003,2004,2005,2006,2008 Gilles Chanteperdrix <gilles.chanteperdrix@domain.hid>
@@ -11,7 +13,7 @@ Copyright (C) 2006 Wolfgang Grandegger <wg@domain.hidom>
 
 License:
 
-Xenmai is licensed under GPL version 2, the user space libraries are LGPL
+Xenomai is licensed under GPL version 2, the user space libraries are LGPL
 version 2.1.
 On Debian systems, the complete texts of the GNU General Public License v2
 and the GNU Lesser General Public License v2 can be found in the file
-- 
1.5.6.5


[-- Attachment #1.3: 0002-debian-libxenomai1-sync-from-debian.org.patch --]
[-- Type: text/plain, Size: 1537 bytes --]

From d3827b9eda17d8332748767b2ae5282f5fcb283d Mon Sep 17 00:00:00 2001
From: Stefan Kisdaroczi <kisda@domain.hid>
Date: Tue, 23 Feb 2010 16:21:23 +0100
Subject: [PATCH] debian: libxenomai1: sync from debian.org

---
 debian/libxenomai1.lintian  |    2 +-
 debian/libxenomai1.postinst |    2 +-
 debian/libxenomai1.postrm   |    2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/debian/libxenomai1.lintian b/debian/libxenomai1.lintian
index 42a4149..0f6a514 100644
--- a/debian/libxenomai1.lintian
+++ b/debian/libxenomai1.lintian
@@ -1,2 +1,2 @@
 # no contained shared library names refer to "xenomai", therefore own name
-libxenomai1: package-name-doesnt-match-sonames libnative1 libpsos0 libpthread-rt1 librtai0 librtdk0 librtdm1 libuitron0 libvrtx0 libvxworks1
+libxenomai1: package-name-doesnt-match-sonames libanalogy1 libnative3 libpsos0 libpthread-rt1 librtai0 librtdk0 librtdm1 libuitron0 libvrtx0 libvxworks1
diff --git a/debian/libxenomai1.postinst b/debian/libxenomai1.postinst
index dfdaa46..8afc6fc 100644
--- a/debian/libxenomai1.postinst
+++ b/debian/libxenomai1.postinst
@@ -1,4 +1,4 @@
-#!/bin/sh
+#!/bin/sh -e
 
 rm -f /etc/udev/rules.d/xenomai.rules
 ln -sf ../xenomai.rules /etc/udev/rules.d/xenomai.rules
diff --git a/debian/libxenomai1.postrm b/debian/libxenomai1.postrm
index a269ef5..3559eb5 100644
--- a/debian/libxenomai1.postrm
+++ b/debian/libxenomai1.postrm
@@ -1,4 +1,4 @@
-#!/bin/sh
+#!/bin/sh -e
 
 case "$1" in
   purge | remove)
-- 
1.5.6.5


[-- Attachment #1.4: 0003-debian-linux-patch-xenomai.README.Debian-sync-from.patch --]
[-- Type: text/plain, Size: 1354 bytes --]

From c05d1fbfdd9360785b82be3d4437fe2b0e39b647 Mon Sep 17 00:00:00 2001
From: Stefan Kisdaroczi <kisda@domain.hid>
Date: Tue, 23 Feb 2010 16:28:39 +0100
Subject: [PATCH] debian: linux-patch-xenomai.README.Debian: sync from debian.org

---
 debian/linux-patch-xenomai.README.Debian |    8 +++++---
 1 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/debian/linux-patch-xenomai.README.Debian b/debian/linux-patch-xenomai.README.Debian
index a6aaa6b..3304bf5 100644
--- a/debian/linux-patch-xenomai.README.Debian
+++ b/debian/linux-patch-xenomai.README.Debian
@@ -4,12 +4,14 @@ Xenomai kernel patches in Debian
 With this package, you can patch and build kernels suitable for usage with
 Xenomai. This can be done with, e.g.:
 
-# cd /usr/src/linux-source-2.6.24
-# fakeroot make-kpkg --initrd --added-patches xenomai kernel_image
+# cd /usr/src/linux-source-2.6.32
+# /usr/src/kernel-patches/i386/apply/xenomai
+# make config
+# make-kpkg --rootcmd fakeroot --initrd kernel_image
 
 This is intended for vanilla and Debian kernel sources.
 
 NOTE: In the kernel config, PARAVIRT should be turned off for Xenomai kernels
       to compile successfully.
 
- -- Roland Stigge <stigge@domain.hid>, Fri, 11 Apr 2008 09:02:28 +0200
+ -- Roland Stigge <stigge@domain.hid>, Sun, 14 Feb 2010 15:14:40 +0100
-- 
1.5.6.5


[-- Attachment #1.5: 0004-debian-Build-Depends-debhelper-7.patch --]
[-- Type: text/plain, Size: 924 bytes --]

From 4b0484a5d7c20f13276b8d609f5bc4dd55b7563e Mon Sep 17 00:00:00 2001
From: Stefan Kisdaroczi <kisda@domain.hid>
Date: Tue, 23 Feb 2010 16:42:40 +0100
Subject: [PATCH] debian: Build-Depends: debhelper (>= 7)

---
 debian/compat  |    2 +-
 debian/control |    2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/debian/compat b/debian/compat
index 7ed6ff8..7f8f011 100644
--- a/debian/compat
+++ b/debian/compat
@@ -1 +1 @@
-5
+7
diff --git a/debian/control b/debian/control
index e7fdfef..ab4636d 100644
--- a/debian/control
+++ b/debian/control
@@ -2,7 +2,7 @@ Source: xenomai
 Section: devel
 Priority: extra
 Maintainer: Roland Stigge <stigge@domain.hid>
-Build-Depends: debhelper (>= 5), dh-kpatches, findutils (>= 4.2.28)
+Build-Depends: debhelper (>= 7), dh-kpatches, findutils (>= 4.2.28)
 Standards-Version: 3.8.0
 Homepage: http://www.xenomai.org/
 
-- 
1.5.6.5


[-- Attachment #1.6: 0005-debian-wrapper-script-usr-bin-xeno-to-call-executa.patch --]
[-- Type: text/plain, Size: 3523 bytes --]

From 3f67cba6b3789ec1bbcda2678f28fe47dee7e9de Mon Sep 17 00:00:00 2001
From: Stefan Kisdaroczi <kisda@domain.hid>
Date: Tue, 23 Feb 2010 17:07:13 +0100
Subject: [PATCH] debian: wrapper script /usr/bin/xeno to call executables in /usr/lib/xenomai/

---
 debian/rules                    |    9 ++++++---
 debian/xeno                     |   10 ++++++++++
 debian/xeno.1                   |   17 +++++++++++++++++
 debian/xenomai-runtime.dirs     |    1 +
 debian/xenomai-runtime.manpages |    1 +
 5 files changed, 35 insertions(+), 3 deletions(-)
 create mode 100644 debian/xeno
 create mode 100644 debian/xeno.1
 create mode 100644 debian/xenomai-runtime.dirs
 create mode 100644 debian/xenomai-runtime.manpages

diff --git a/debian/rules b/debian/rules
index fa93f62..ee6cde7 100755
--- a/debian/rules
+++ b/debian/rules
@@ -87,13 +87,16 @@ clean:
 install: build
 	dh_testdir
 	dh_testroot
-	dh_clean -k
+	dh_prep
 	dh_installdirs
 	$(MAKE) install DESTDIR=$(CURDIR)/debian/tmp/
 	dh_install --sourcedir=$(CURDIR)/debian/tmp
 	# xeno-config should be only in libxenomai-dev
 	rm -f $(CURDIR)/debian/xenomai-runtime/usr/bin/xeno-config
 	rm -f $(CURDIR)/debian/xenomai-runtime/usr/share/man/man1/xeno-config.1
+	mv $(CURDIR)/debian/xenomai-runtime/usr/bin/* \
+		$(CURDIR)/debian/xenomai-runtime/usr/lib/xenomai/
+	install -m 755 debian/xeno $(CURDIR)/debian/xenomai-runtime/usr/bin/xeno
 	for f in $(CURDIR)/ksrc/nucleus/udev/*.rules ; do \
 	    cat $$f >> $(CURDIR)/debian/libxenomai1/etc/udev/xenomai.rules ; \
 	done
@@ -108,8 +111,8 @@ binary-indep: build install
 	dh_testdir -i
 	dh_testroot -i
 	dh_installdocs -i -A CREDITS README.INSTALL TROUBLESHOOTING
-	dh_installchangelogs -i
 	dh_link -i
+	dh_installchangelogs -i
 	dh_strip -i
 	dh_compress -i -X.pdf
 	dh_fixperms -i
@@ -131,8 +134,8 @@ binary-arch: build install
 	dh_testroot -s
 	dh_installman -s
 	dh_installdocs -s -A CREDITS README.INSTALL TROUBLESHOOTING
-	dh_installchangelogs -s
 	dh_link -s
+	dh_installchangelogs -s
 	dh_strip -s
 	dh_compress -s
 	dh_fixperms -s
diff --git a/debian/xeno b/debian/xeno
new file mode 100644
index 0000000..721914e
--- /dev/null
+++ b/debian/xeno
@@ -0,0 +1,10 @@
+#!/bin/sh
+#
+# Wrapper script to call original /usr/bin executables from /usr/lib/xenomai
+#
+# e.g. "xeno latency"
+#
+
+BINDIR=/usr/lib/xenomai
+
+"$BINDIR/$@"
diff --git a/debian/xeno.1 b/debian/xeno.1
new file mode 100644
index 0000000..cec55b1
--- /dev/null
+++ b/debian/xeno.1
@@ -0,0 +1,17 @@
+.TH xeno 1 "14 February 2010" "Version 1" "Xeno Manual"
+
+.SH NAME
+xeno \- Wrapper for Xenomai executables
+
+.SH SYNOPSIS
+xeno [xenomai command]
+
+.SH DESCRIPTION
+.B xeno
+is a wrapper script that makes the common Xenomai userland executables
+available to the user because they don't reside in /usr/bin anymore to prevent
+filename collisions.
+
+.SH AUTHOR
+The wrapper script xeno and this manpage were initially written by Roland
+Stigge <stigge@domain.hid> for the Debian project but may be used elsewhere.
diff --git a/debian/xenomai-runtime.dirs b/debian/xenomai-runtime.dirs
new file mode 100644
index 0000000..6bea564
--- /dev/null
+++ b/debian/xenomai-runtime.dirs
@@ -0,0 +1 @@
+usr/lib/xenomai
diff --git a/debian/xenomai-runtime.manpages b/debian/xenomai-runtime.manpages
new file mode 100644
index 0000000..9555541
--- /dev/null
+++ b/debian/xenomai-runtime.manpages
@@ -0,0 +1 @@
+debian/xeno.1
-- 
1.5.6.5


[-- Attachment #1.7: 0006-debian-changelog-sync-from-debian.org.patch --]
[-- Type: text/plain, Size: 2069 bytes --]

From 43b82d769fc78a7d397eb80ff7c90dfd9f10db96 Mon Sep 17 00:00:00 2001
From: Stefan Kisdaroczi <kisda@domain.hid>
Date: Tue, 23 Feb 2010 17:12:23 +0100
Subject: [PATCH] debian: changelog: sync from debian.org

---
 debian/changelog |   40 ++++++++++++++++++++++++++++++++++++++++
 1 files changed, 40 insertions(+), 0 deletions(-)

diff --git a/debian/changelog b/debian/changelog
index b43955d..45736a4 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,43 @@
+xenomai (2.5.1-3) unstable; urgency=low
+
+  * xenomai-runtime: Replaced "xenomai-" prefixed executables with
+    wrapper script /usr/bin/xeno to call original executables from
+    /usr/lib/xenomai/, e.g. "xeno latency"
+  * debian/prepare-patch.sh: Synchronized from upstream (Closes: #569721)
+  * Removed 2.6.24 adeos-ipipe patches
+  * Enabled patching/use of Debian's kernel (2.6.32)
+
+ -- Roland Stigge <stigge@domain.hid>  Sun, 14 Feb 2010 13:35:15 +0100
+
+xenomai (2.5.1-2) unstable; urgency=low
+
+  * xenomai-runtime: Renamed /usr/bin/* to /usr/bin/xenomai-* (Closes: #569357)
+
+ -- Roland Stigge <stigge@domain.hid>  Fri, 12 Feb 2010 21:18:00 +0100
+
+xenomai (2.5.1-1) unstable; urgency=low
+
+  * New upstream release (Closes: #553334, #566566, #536731)
+  * debian/control: Standards-Version: 3.8.4
+
+ -- Roland Stigge <stigge@domain.hid>  Sat, 06 Feb 2010 11:05:50 +0100
+
+xenomai (2.4.8-2) unstable; urgency=low
+
+  * debian/libxenomai1.{postinst,postrm}: "set -e"
+  * debian/control: linux-patch-xenomai: Section: kernel
+
+ -- Roland Stigge <stigge@domain.hid>  Wed, 24 Jun 2009 17:47:41 +0200
+
+xenomai (2.4.8-1) unstable; urgency=low
+
+  * New upstream release
+    - includes limits.h include as in patch from Peter Green (Closes: #527654)
+  * debian/control:
+    - Standards-Version: 3.8.2
+
+ -- Roland Stigge <stigge@domain.hid>  Mon, 22 Jun 2009 22:50:23 +0200
+
 xenomai (2.4.4-3) unstable; urgency=low
 
   * debian/rules: Fixed concurrent ./configure + make runs, thanks to Albin
-- 
1.5.6.5


[-- Attachment #1.8: do-not-commit-please.patch --]
[-- Type: text/plain, Size: 1971 bytes --]

--- xenomai-2.5.1.orig/debian/control	2010-02-23 16:40:07.000000000 +0100
+++ xenomai-2.5.1/debian/control	2010-02-23 14:34:06.000000000 +0100
@@ -3,13 +3,13 @@ Section: devel
 Priority: extra
 Maintainer: Roland Stigge <stigge@domain.hid>
 Build-Depends: debhelper (>= 7), dh-kpatches, findutils (>= 4.2.28)
-Standards-Version: 3.8.0
+Standards-Version: 3.8.4
 Homepage: http://www.xenomai.org/
 
 Package: xenomai-runtime
 Section: devel
 Architecture: amd64 arm armeb armel i386 powerpc
-Depends: ${shlibs:Depends}
+Depends: ${shlibs:Depends}, ${misc:Depends}
 Suggests: linux-patch-xenomai, xenomai-doc
 Replaces: xenomai
 Conflicts: xenomai
@@ -25,9 +25,9 @@ Description: Xenomai runtime utilities
  realtime system.
 
 Package: linux-patch-xenomai
-Section: devel
+Section: kernel
 Architecture: all
-Depends: ${kpatch:Depends}
+Depends: ${kpatch:Depends}, ${misc:Depends}
 Suggests: xenomai, linux-source-2.6, kernel-package
 Description: Linux kernel patches for Xenomai
  Xenomai is a real-time development framework cooperating with the Linux
@@ -48,7 +48,7 @@ Description: Linux kernel patches for Xe
 Package: libxenomai1
 Section: libs
 Architecture: amd64 arm armeb armel i386 powerpc
-Depends: ${shlibs:Depends}
+Depends: ${shlibs:Depends}, ${misc:Depends}
 Suggests: linux-patch-xenomai, xenomai-doc
 Replaces: xenomai
 Conflicts: xenomai
@@ -65,7 +65,7 @@ Description: Shared libraries for Xenoma
 Package: libxenomai-dev
 Section: libdevel
 Architecture: amd64 arm armeb armel i386 powerpc
-Depends: libxenomai1 (= ${binary:Version})
+Depends: libxenomai1 (= ${binary:Version}), ${misc:Depends}
 Suggests: linux-patch-xenomai, xenomai-doc
 Replaces: xenomai
 Conflicts: xenomai
@@ -83,7 +83,7 @@ Description: Headers and static libs for
 Package: xenomai-doc
 Section: doc
 Architecture: all
-Depends:
+Depends: ${misc:Depends}
 Suggests: xenomai
 Conflicts: xenomai-docs
 Replaces: xenomai-docs

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

^ permalink raw reply related	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-23 16:52       ` Stefan Kisdaroczi
@ 2010-02-23 17:46         ` Philippe Gerum
  2010-02-24 13:06           ` Stefan Kisdaroczi
  0 siblings, 1 reply; 38+ messages in thread
From: Philippe Gerum @ 2010-02-23 17:46 UTC (permalink / raw)
  To: Stefan Kisdaroczi; +Cc: xenomai

On Tue, 2010-02-23 at 17:52 +0100, Stefan Kisdaroczi wrote:
> Hi,
> 
> Am 14.02.2010 10:38, schrieb Philippe Gerum:
> > 
> > <snip>
> > In the future, maybe we could simply provide a wrapper script accepting
> > sub-commands, such as "xeno latency, xeno sigtest" etc, to be put
> > into /usr/bin by distros, which would hide the actual location of those
> > binaries?
> >
> > In any case, thanks for your work so far. We probably need to discuss
> > the packaging issues on this list, so that we get both consistency and
> > usability in the future.
> > 
> > Gilles and Roland, if this is fine with you, I'll handle the liaison
> > role with upstream packagers, so please CC me explicitly on those mails.
> > We'll sort out this issue, it doesn't look that bad anyway.
> 
> Roland added a xeno wrapper to the debian.org xenomai package 2.5.1-3.
> 
> I synced now the debian/ directories from debian.org and xenomai.org:
>  - For debian.org I sent patches to the Debian bugtracker [1] [2].
>    Another patch for dpkg-cross support [3] I sent to Roland privately.
>  - For xenomai.org I attached patches to this mail (against -2.5.git).
> 
> If both parties apply the patches the debian directories are in sync,
> except some minor differences in the debian/control file, see patch
> do-not-commit-please.patch. I would like to keep these changes out so
> that the xenomai.org packages are compatible with Debian 5.0 Lenny.
> The debian.org packages are for Debian 6.0 Squeeze.
> 

Merged into my queue (except the last one as mentioned). This will be
pushed upstream to Gilles for 2.5.2. Thanks.

> Thanks
> kisda
> 
> [1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=571099
> [2] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=571104
> [3] http://git.xenomai.org/?p=xenomai-2.5.git;a=commitdiff;h=5bcd18f714f4cbeaaac0cc4a08e6c9f375aa3b77


-- 
Philippe.




^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-23 17:46         ` Philippe Gerum
@ 2010-02-24 13:06           ` Stefan Kisdaroczi
  2010-02-24 13:11             ` Philippe Gerum
  2010-02-25 12:46             ` Stefan Kisdaroczi
  0 siblings, 2 replies; 38+ messages in thread
From: Stefan Kisdaroczi @ 2010-02-24 13:06 UTC (permalink / raw)
  To: Philippe Gerum; +Cc: xenomai

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

Hi Philippe,

Am 23.02.2010 18:46, schrieb Philippe Gerum:
> On Tue, 2010-02-23 at 17:52 +0100, Stefan Kisdaroczi wrote:
>> Hi,
>>
>> Am 14.02.2010 10:38, schrieb Philippe Gerum:
>>>
>>> <snip>
>>> In the future, maybe we could simply provide a wrapper script accepting
>>> sub-commands, such as "xeno latency, xeno sigtest" etc, to be put
>>> into /usr/bin by distros, which would hide the actual location of those
>>> binaries?
>>>
>>> In any case, thanks for your work so far. We probably need to discuss
>>> the packaging issues on this list, so that we get both consistency and
>>> usability in the future.
>>>
>>> Gilles and Roland, if this is fine with you, I'll handle the liaison
>>> role with upstream packagers, so please CC me explicitly on those mails.
>>> We'll sort out this issue, it doesn't look that bad anyway.
>>
>> Roland added a xeno wrapper to the debian.org xenomai package 2.5.1-3.
>>
>> I synced now the debian/ directories from debian.org and xenomai.org:
>>  - For debian.org I sent patches to the Debian bugtracker [1] [2].
>>    Another patch for dpkg-cross support [3] I sent to Roland privately.
>>  - For xenomai.org I attached patches to this mail (against -2.5.git).
>>
>> If both parties apply the patches the debian directories are in sync,
>> except some minor differences in the debian/control file, see patch
>> do-not-commit-please.patch. I would like to keep these changes out so
>> that the xenomai.org packages are compatible with Debian 5.0 Lenny.
>> The debian.org packages are for Debian 6.0 Squeeze.
>>
> 
> Merged into my queue (except the last one as mentioned). This will be
> pushed upstream to Gilles for 2.5.2. Thanks.

I took a look at your branch for-upstream. Your commit
  scripts: add wrapper script to run standard Xenomai commands
  6e0574791f48cbf8b3421a68c5789254e7d084b7
adds the same wrapper as my patch 0005-debian-wrapper-script-usr-bin-xeno-to-call-executa.patch
  debian: wrapper script /usr/bin/xeno to call executables in /usr/lib/xenomai/
  fbe86cc50d3a65cd23e93d43adba4ed369fe70b1
Please revert the commit of my patch, we need another fix for debian/rules for "your" wrapper.

How do I call configure to install the wrapper in /usr/bin and
the programs like latency, switchtest etc. to /usr/lib/xenomai ?

Stefan

> 
>> Thanks
>> kisda
>>
>> [1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=571099
>> [2] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=571104
>> [3] http://git.xenomai.org/?p=xenomai-2.5.git;a=commitdiff;h=5bcd18f714f4cbeaaac0cc4a08e6c9f375aa3b77
> 
> 



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

^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-24 13:06           ` Stefan Kisdaroczi
@ 2010-02-24 13:11             ` Philippe Gerum
  2010-02-24 13:13               ` Philippe Gerum
  2010-02-25 12:46             ` Stefan Kisdaroczi
  1 sibling, 1 reply; 38+ messages in thread
From: Philippe Gerum @ 2010-02-24 13:11 UTC (permalink / raw)
  To: Stefan Kisdaroczi; +Cc: xenomai

On Wed, 2010-02-24 at 14:06 +0100, Stefan Kisdaroczi wrote:
> Hi Philippe,
> 
> Am 23.02.2010 18:46, schrieb Philippe Gerum:
> > On Tue, 2010-02-23 at 17:52 +0100, Stefan Kisdaroczi wrote:
> >> Hi,
> >>
> >> Am 14.02.2010 10:38, schrieb Philippe Gerum:
> >>>
> >>> <snip>
> >>> In the future, maybe we could simply provide a wrapper script accepting
> >>> sub-commands, such as "xeno latency, xeno sigtest" etc, to be put
> >>> into /usr/bin by distros, which would hide the actual location of those
> >>> binaries?
> >>>
> >>> In any case, thanks for your work so far. We probably need to discuss
> >>> the packaging issues on this list, so that we get both consistency and
> >>> usability in the future.
> >>>
> >>> Gilles and Roland, if this is fine with you, I'll handle the liaison
> >>> role with upstream packagers, so please CC me explicitly on those mails.
> >>> We'll sort out this issue, it doesn't look that bad anyway.
> >>
> >> Roland added a xeno wrapper to the debian.org xenomai package 2.5.1-3.
> >>
> >> I synced now the debian/ directories from debian.org and xenomai.org:
> >>  - For debian.org I sent patches to the Debian bugtracker [1] [2].
> >>    Another patch for dpkg-cross support [3] I sent to Roland privately.
> >>  - For xenomai.org I attached patches to this mail (against -2.5.git).
> >>
> >> If both parties apply the patches the debian directories are in sync,
> >> except some minor differences in the debian/control file, see patch
> >> do-not-commit-please.patch. I would like to keep these changes out so
> >> that the xenomai.org packages are compatible with Debian 5.0 Lenny.
> >> The debian.org packages are for Debian 6.0 Squeeze.
> >>
> > 
> > Merged into my queue (except the last one as mentioned). This will be
> > pushed upstream to Gilles for 2.5.2. Thanks.
> 
> I took a look at your branch for-upstream. Your commit
>   scripts: add wrapper script to run standard Xenomai commands
>   6e0574791f48cbf8b3421a68c5789254e7d084b7
> adds the same wrapper as my patch 0005-debian-wrapper-script-usr-bin-xeno-to-call-executa.patch
>   debian: wrapper script /usr/bin/xeno to call executables in /usr/lib/xenomai/
>   fbe86cc50d3a65cd23e93d43adba4ed369fe70b1
> Please revert the commit of my patch, we need another fix for debian/rules for "your" wrapper.
> 

Ok, I thought your patch set was based on my tree, so I did not check
thoroughly. I did not send any pull request to Gilles, so no harm done.

> How do I call configure to install the wrapper in /usr/bin and
> the programs like latency, switchtest etc. to /usr/lib/xenomai ?
> 

We need some fixage in scripts/wrappers/Makefile.am to do that. I'll
prepare this asap.

> Stefan
> 
> > 
> >> Thanks
> >> kisda
> >>
> >> [1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=571099
> >> [2] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=571104
> >> [3] http://git.xenomai.org/?p=xenomai-2.5.git;a=commitdiff;h=5bcd18f714f4cbeaaac0cc4a08e6c9f375aa3b77
> > 
> > 
> 
> 


-- 
Philippe.




^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-24 13:11             ` Philippe Gerum
@ 2010-02-24 13:13               ` Philippe Gerum
  2010-02-26 13:13                 ` Stefan Kisdaroczi
  0 siblings, 1 reply; 38+ messages in thread
From: Philippe Gerum @ 2010-02-24 13:13 UTC (permalink / raw)
  To: Stefan Kisdaroczi; +Cc: xenomai

On Wed, 2010-02-24 at 14:11 +0100, Philippe Gerum wrote:
> On Wed, 2010-02-24 at 14:06 +0100, Stefan Kisdaroczi wrote:
> > Hi Philippe,
> > 
> > Am 23.02.2010 18:46, schrieb Philippe Gerum:
> > > On Tue, 2010-02-23 at 17:52 +0100, Stefan Kisdaroczi wrote:
> > >> Hi,
> > >>
> > >> Am 14.02.2010 10:38, schrieb Philippe Gerum:
> > >>>
> > >>> <snip>
> > >>> In the future, maybe we could simply provide a wrapper script accepting
> > >>> sub-commands, such as "xeno latency, xeno sigtest" etc, to be put
> > >>> into /usr/bin by distros, which would hide the actual location of those
> > >>> binaries?
> > >>>
> > >>> In any case, thanks for your work so far. We probably need to discuss
> > >>> the packaging issues on this list, so that we get both consistency and
> > >>> usability in the future.
> > >>>
> > >>> Gilles and Roland, if this is fine with you, I'll handle the liaison
> > >>> role with upstream packagers, so please CC me explicitly on those mails.
> > >>> We'll sort out this issue, it doesn't look that bad anyway.
> > >>
> > >> Roland added a xeno wrapper to the debian.org xenomai package 2.5.1-3.
> > >>
> > >> I synced now the debian/ directories from debian.org and xenomai.org:
> > >>  - For debian.org I sent patches to the Debian bugtracker [1] [2].
> > >>    Another patch for dpkg-cross support [3] I sent to Roland privately.
> > >>  - For xenomai.org I attached patches to this mail (against -2.5.git).
> > >>
> > >> If both parties apply the patches the debian directories are in sync,
> > >> except some minor differences in the debian/control file, see patch
> > >> do-not-commit-please.patch. I would like to keep these changes out so
> > >> that the xenomai.org packages are compatible with Debian 5.0 Lenny.
> > >> The debian.org packages are for Debian 6.0 Squeeze.
> > >>
> > > 
> > > Merged into my queue (except the last one as mentioned). This will be
> > > pushed upstream to Gilles for 2.5.2. Thanks.
> > 
> > I took a look at your branch for-upstream. Your commit
> >   scripts: add wrapper script to run standard Xenomai commands
> >   6e0574791f48cbf8b3421a68c5789254e7d084b7
> > adds the same wrapper as my patch 0005-debian-wrapper-script-usr-bin-xeno-to-call-executa.patch
> >   debian: wrapper script /usr/bin/xeno to call executables in /usr/lib/xenomai/
> >   fbe86cc50d3a65cd23e93d43adba4ed369fe70b1
> > Please revert the commit of my patch, we need another fix for debian/rules for "your" wrapper.
> > 
> 
> Ok, I thought your patch set was based on my tree, so I did not check
> thoroughly. I did not send any pull request to Gilles, so no harm done.
> 
> > How do I call configure to install the wrapper in /usr/bin and
> > the programs like latency, switchtest etc. to /usr/lib/xenomai ?
> > 
> 
> We need some fixage in scripts/wrappers/Makefile.am to do that. I'll
> prepare this asap.

scripts/Makefile.am...

> 
> > Stefan
> > 
> > > 
> > >> Thanks
> > >> kisda
> > >>
> > >> [1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=571099
> > >> [2] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=571104
> > >> [3] http://git.xenomai.org/?p=xenomai-2.5.git;a=commitdiff;h=5bcd18f714f4cbeaaac0cc4a08e6c9f375aa3b77
> > > 
> > > 
> > 
> > 
> 
> 


-- 
Philippe.




^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-24 13:06           ` Stefan Kisdaroczi
  2010-02-24 13:11             ` Philippe Gerum
@ 2010-02-25 12:46             ` Stefan Kisdaroczi
  1 sibling, 0 replies; 38+ messages in thread
From: Stefan Kisdaroczi @ 2010-02-25 12:46 UTC (permalink / raw)
  To: xenomai, Philippe Gerum


[-- Attachment #1.1: Type: text/plain, Size: 3055 bytes --]

Am 24.02.2010 14:06, schrieb Stefan Kisdaroczi:
> Hi Philippe,
> 
> Am 23.02.2010 18:46, schrieb Philippe Gerum:
>> On Tue, 2010-02-23 at 17:52 +0100, Stefan Kisdaroczi wrote:
>>> Hi,
>>>
>>> Am 14.02.2010 10:38, schrieb Philippe Gerum:
>>>>
>>>> <snip>
>>>> In the future, maybe we could simply provide a wrapper script accepting
>>>> sub-commands, such as "xeno latency, xeno sigtest" etc, to be put
>>>> into /usr/bin by distros, which would hide the actual location of those
>>>> binaries?
>>>>
>>>> In any case, thanks for your work so far. We probably need to discuss
>>>> the packaging issues on this list, so that we get both consistency and
>>>> usability in the future.
>>>>
>>>> Gilles and Roland, if this is fine with you, I'll handle the liaison
>>>> role with upstream packagers, so please CC me explicitly on those mails.
>>>> We'll sort out this issue, it doesn't look that bad anyway.
>>>
>>> Roland added a xeno wrapper to the debian.org xenomai package 2.5.1-3.
>>>
>>> I synced now the debian/ directories from debian.org and xenomai.org:
>>>  - For debian.org I sent patches to the Debian bugtracker [1] [2].
>>>    Another patch for dpkg-cross support [3] I sent to Roland privately.
>>>  - For xenomai.org I attached patches to this mail (against -2.5.git).
>>>
>>> If both parties apply the patches the debian directories are in sync,
>>> except some minor differences in the debian/control file, see patch
>>> do-not-commit-please.patch. I would like to keep these changes out so
>>> that the xenomai.org packages are compatible with Debian 5.0 Lenny.
>>> The debian.org packages are for Debian 6.0 Squeeze.
>>>
>>
>> Merged into my queue (except the last one as mentioned). This will be
>> pushed upstream to Gilles for 2.5.2. Thanks.
> 
> I took a look at your branch for-upstream. Your commit
>   scripts: add wrapper script to run standard Xenomai commands
>   6e0574791f48cbf8b3421a68c5789254e7d084b7
> adds the same wrapper as my patch 0005-debian-wrapper-script-usr-bin-xeno-to-call-executa.patch
>   debian: wrapper script /usr/bin/xeno to call executables in /usr/lib/xenomai/
>   fbe86cc50d3a65cd23e93d43adba4ed369fe70b1
> Please revert the commit of my patch, we need another fix for debian/rules for "your" wrapper.

Thanks for reverting. Roland has uploaded 2.5.1-4 yesterday.
Attached a patch to stay in sync. Based on your for-upstream branch this time.

kisda

> How do I call configure to install the wrapper in /usr/bin and
> the programs like latency, switchtest etc. to /usr/lib/xenomai ?
> 
> Stefan
> 
>>
>>> Thanks
>>> kisda
>>>
>>> [1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=571099
>>> [2] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=571104
>>> [3] http://git.xenomai.org/?p=xenomai-2.5.git;a=commitdiff;h=5bcd18f714f4cbeaaac0cc4a08e6c9f375aa3b77
>>
>>
> 
> 
> 
> 
> _______________________________________________
> Xenomai-core mailing list
> Xenomai-core@domain.hid
> https://mail.gna.org/listinfo/xenomai-core


[-- Attachment #1.2: 0001-debian-sync-2.5.1-4-from-debian.org.patch --]
[-- Type: text/plain, Size: 1184 bytes --]

From f6b7a0115f3f785f08f8d085e66c8bf8d7057f88 Mon Sep 17 00:00:00 2001
From: Stefan Kisdaroczi <kisda@hispeed.ch>
Date: Thu, 25 Feb 2010 13:27:31 +0100
Subject: [PATCH] debian: sync 2.5.1-4 from debian.org

---
 debian/changelog        |    9 +++++++++
 debian/libxenomai1.dirs |    2 +-
 2 files changed, 10 insertions(+), 1 deletions(-)

diff --git a/debian/changelog b/debian/changelog
index 45736a4..6d2d3cf 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,12 @@
+xenomai (2.5.1-4) unstable; urgency=low
+
+  * Added patches by Stefan Kisdaroczi <kisda@hispeed.ch>:
+    - debian/copyright: Typo and email address (Closes: #571099)
+    - debian/control: ia64 support removed (Closes: #571104)
+    - debian/rules: Added dpkg-cross support
+
+ -- Roland Stigge <stigge@antcom.de>  Wed, 24 Feb 2010 22:20:10 +0100
+
 xenomai (2.5.1-3) unstable; urgency=low
 
   * xenomai-runtime: Replaced "xenomai-" prefixed executables with
diff --git a/debian/libxenomai1.dirs b/debian/libxenomai1.dirs
index d5bb34f..1737ea1 100644
--- a/debian/libxenomai1.dirs
+++ b/debian/libxenomai1.dirs
@@ -1,2 +1,2 @@
-etc/udev
+etc/udev/rules.d
 usr/share/lintian/overrides
-- 
1.5.6.5


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

^ permalink raw reply related	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-14  9:38     ` Philippe Gerum
  2010-02-23 16:52       ` Stefan Kisdaroczi
@ 2010-02-25 13:44       ` Stefan Kisdaroczi
  2010-02-25 13:49         ` Gilles Chanteperdrix
  1 sibling, 1 reply; 38+ messages in thread
From: Stefan Kisdaroczi @ 2010-02-25 13:44 UTC (permalink / raw)
  To: xenomai


[-- Attachment #1.1: Type: text/plain, Size: 735 bytes --]

Hi,

Am 14.02.2010 10:38, schrieb Philippe Gerum:
> <snip>
> In any case, thanks for your work so far. We probably need to discuss
> the packaging issues on this list, so that we get both consistency and
> usability in the future.
> 
> Gilles and Roland, if this is fine with you, I'll handle the liaison
> role with upstream packagers, so please CC me explicitly on those mails.
> We'll sort out this issue, it doesn't look that bad anyway.

udev/xenomai.rules sets the group for realtime devices to "xenomai".
Patch attached to create the group "xenomai" on installation.

Additionally, it would be nice to set "xeno_nucleus.xenomai_gid" to
group "xenomai" too. Is this possible with a udev rule ?

regards
kisda

[-- Attachment #1.2: group-xenomai.patch --]
[-- Type: text/plain, Size: 835 bytes --]

diff --git a/debian/libxenomai1.postinst b/debian/libxenomai1.postinst
index 8afc6fc..97c3476 100644
--- a/debian/libxenomai1.postinst
+++ b/debian/libxenomai1.postinst
@@ -1,6 +1,22 @@
 #!/bin/sh -e
 
-rm -f /etc/udev/rules.d/xenomai.rules
-ln -sf ../xenomai.rules /etc/udev/rules.d/xenomai.rules
+case "$1" in
+    configure)
+        # Add the xenomai group unless it's already there
+        if ! getent group xenomai >/dev/null; then
+            addgroup --quiet --system xenomai || true
+        fi
+        rm -f /etc/udev/rules.d/xenomai.rules
+        ln -sf ../xenomai.rules /etc/udev/rules.d/xenomai.rules
+    ;;
+
+    abort-upgrade|abort-remove|abort-deconfigure)
+    ;;
+
+    *)
+        echo "postinst called with unknown argument \`$1'" >&2
+        exit 1
+    ;;
+esac
 
 #DEBHELPER#

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

^ permalink raw reply related	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-25 13:44       ` Stefan Kisdaroczi
@ 2010-02-25 13:49         ` Gilles Chanteperdrix
  2010-02-25 13:59           ` Stefan Kisdaroczi
  2010-02-25 13:59           ` Jan Kiszka
  0 siblings, 2 replies; 38+ messages in thread
From: Gilles Chanteperdrix @ 2010-02-25 13:49 UTC (permalink / raw)
  To: Stefan Kisdaroczi; +Cc: xenomai

Stefan Kisdaroczi wrote:
> Hi,
> 
> Am 14.02.2010 10:38, schrieb Philippe Gerum:
>> <snip>
>> In any case, thanks for your work so far. We probably need to discuss
>> the packaging issues on this list, so that we get both consistency and
>> usability in the future.
>>
>> Gilles and Roland, if this is fine with you, I'll handle the liaison
>> role with upstream packagers, so please CC me explicitly on those mails.
>> We'll sort out this issue, it doesn't look that bad anyway.
> 
> udev/xenomai.rules sets the group for realtime devices to "xenomai".
> Patch attached to create the group "xenomai" on installation.
> 
> Additionally, it would be nice to set "xeno_nucleus.xenomai_gid" to
> group "xenomai" too. Is this possible with a udev rule ?

Well, no, xeno_nucleus.xenomai_gid is a kernel parameter, so it is boot
loader stuff. However, if xeno_nucleus is compiled as a module, you can
add parameters in /etc/modprobe.d. Well, that would have been the place
to do it some time ago. I may not be completely up-to-date.

> 
> regards
> kisda
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> Xenomai-core mailing list
> Xenomai-core@domain.hid
> https://mail.gna.org/listinfo/xenomai-core


-- 
					    Gilles.


^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-25 13:49         ` Gilles Chanteperdrix
@ 2010-02-25 13:59           ` Stefan Kisdaroczi
  2010-02-25 14:29             ` Stefan Kisdaroczi
       [not found]             ` <4B868CC1.6030103@domain.hid>
  2010-02-25 13:59           ` Jan Kiszka
  1 sibling, 2 replies; 38+ messages in thread
From: Stefan Kisdaroczi @ 2010-02-25 13:59 UTC (permalink / raw)
  To: Gilles Chanteperdrix; +Cc: xenomai

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

Am 25.02.2010 14:49, schrieb Gilles Chanteperdrix:
> Stefan Kisdaroczi wrote:
>> Hi,
>>
>> Am 14.02.2010 10:38, schrieb Philippe Gerum:
>>> <snip>
>>> In any case, thanks for your work so far. We probably need to discuss
>>> the packaging issues on this list, so that we get both consistency and
>>> usability in the future.
>>>
>>> Gilles and Roland, if this is fine with you, I'll handle the liaison
>>> role with upstream packagers, so please CC me explicitly on those mails.
>>> We'll sort out this issue, it doesn't look that bad anyway.
>>
>> udev/xenomai.rules sets the group for realtime devices to "xenomai".
>> Patch attached to create the group "xenomai" on installation.
>>
>> Additionally, it would be nice to set "xeno_nucleus.xenomai_gid" to
>> group "xenomai" too. Is this possible with a udev rule ?
> 
> Well, no, xeno_nucleus.xenomai_gid is a kernel parameter, so it is boot
> loader stuff. However, if xeno_nucleus is compiled as a module, you can
> add parameters in /etc/modprobe.d. Well, that would have been the place
> to do it some time ago. I may not be completely up-to-date.

Hi Gilles,

can a init.d script with
  echo "<gid>" > /sys/module/xeno_nucleus/parameters/xenomai_gid
work in module and builtin case ?

>>
>> regards
>> kisda
>>
>>
>> ------------------------------------------------------------------------
>>
>> _______________________________________________
>> Xenomai-core mailing list
>> Xenomai-core@domain.hid
>> https://mail.gna.org/listinfo/xenomai-core
> 
> 



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

^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-25 13:49         ` Gilles Chanteperdrix
  2010-02-25 13:59           ` Stefan Kisdaroczi
@ 2010-02-25 13:59           ` Jan Kiszka
  1 sibling, 0 replies; 38+ messages in thread
From: Jan Kiszka @ 2010-02-25 13:59 UTC (permalink / raw)
  To: Gilles Chanteperdrix; +Cc: xenomai

Gilles Chanteperdrix wrote:
> Stefan Kisdaroczi wrote:
>> Hi,
>>
>> Am 14.02.2010 10:38, schrieb Philippe Gerum:
>>> <snip>
>>> In any case, thanks for your work so far. We probably need to discuss
>>> the packaging issues on this list, so that we get both consistency and
>>> usability in the future.
>>>
>>> Gilles and Roland, if this is fine with you, I'll handle the liaison
>>> role with upstream packagers, so please CC me explicitly on those mails.
>>> We'll sort out this issue, it doesn't look that bad anyway.
>> udev/xenomai.rules sets the group for realtime devices to "xenomai".
>> Patch attached to create the group "xenomai" on installation.
>>
>> Additionally, it would be nice to set "xeno_nucleus.xenomai_gid" to
>> group "xenomai" too. Is this possible with a udev rule ?
> 
> Well, no, xeno_nucleus.xenomai_gid is a kernel parameter, so it is boot
> loader stuff. However, if xeno_nucleus is compiled as a module, you can
> add parameters in /etc/modprobe.d. Well, that would have been the place
> to do it some time ago. I may not be completely up-to-date.

You can change this [module] parameter at any time:

echo GID > /sys/module/xeno_nucleus/parameters/xenomai_gid

Jan

-- 
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux


^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-25 13:59           ` Stefan Kisdaroczi
@ 2010-02-25 14:29             ` Stefan Kisdaroczi
  2010-02-25 17:18               ` Stefan Kisdaroczi
       [not found]             ` <4B868CC1.6030103@domain.hid>
  1 sibling, 1 reply; 38+ messages in thread
From: Stefan Kisdaroczi @ 2010-02-25 14:29 UTC (permalink / raw)
  To: xenomai

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

Am 25.02.2010 14:59, schrieb Stefan Kisdaroczi:
> Am 25.02.2010 14:49, schrieb Gilles Chanteperdrix:
>> Stefan Kisdaroczi wrote:
>>> Hi,
>>>
>>> Am 14.02.2010 10:38, schrieb Philippe Gerum:
>>>> <snip>
>>>> In any case, thanks for your work so far. We probably need to discuss
>>>> the packaging issues on this list, so that we get both consistency and
>>>> usability in the future.
>>>>
>>>> Gilles and Roland, if this is fine with you, I'll handle the liaison
>>>> role with upstream packagers, so please CC me explicitly on those mails.
>>>> We'll sort out this issue, it doesn't look that bad anyway.
>>>
>>> udev/xenomai.rules sets the group for realtime devices to "xenomai".
>>> Patch attached to create the group "xenomai" on installation.
>>>
>>> Additionally, it would be nice to set "xeno_nucleus.xenomai_gid" to
>>> group "xenomai" too. Is this possible with a udev rule ?
>>
>> Well, no, xeno_nucleus.xenomai_gid is a kernel parameter, so it is boot
>> loader stuff. However, if xeno_nucleus is compiled as a module, you can
>> add parameters in /etc/modprobe.d. Well, that would have been the place
>> to do it some time ago. I may not be completely up-to-date.
> 
> Hi Gilles,
> 
> can a init.d script with
>   echo "<gid>" > /sys/module/xeno_nucleus/parameters/xenomai_gid
> work in module and builtin case ?

it seems that init.d will work for the builtin case and modprobe.d for the
module case. So adding both should work "in all cases" :-)
I'll give it a try ...

>>>
>>> regards
>>> kisda
>>>
>>>
>>> ------------------------------------------------------------------------
>>>
>>> _______________________________________________
>>> Xenomai-core mailing list
>>> Xenomai-core@domain.hid
>>> https://mail.gna.org/listinfo/xenomai-core
>>
>>
> 
> 
> 
> 
> _______________________________________________
> Xenomai-core mailing list
> Xenomai-core@domain.hid
> https://mail.gna.org/listinfo/xenomai-core



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

^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-25 14:29             ` Stefan Kisdaroczi
@ 2010-02-25 17:18               ` Stefan Kisdaroczi
  2010-02-26 13:35                 ` Stefan Kisdaroczi
  0 siblings, 1 reply; 38+ messages in thread
From: Stefan Kisdaroczi @ 2010-02-25 17:18 UTC (permalink / raw)
  To: xenomai


[-- Attachment #1.1: Type: text/plain, Size: 2502 bytes --]

Am 25.02.2010 15:29, schrieb Stefan Kisdaroczi:
> Am 25.02.2010 14:59, schrieb Stefan Kisdaroczi:
>> Am 25.02.2010 14:49, schrieb Gilles Chanteperdrix:
>>> Stefan Kisdaroczi wrote:
>>>> Hi,
>>>>
>>>> Am 14.02.2010 10:38, schrieb Philippe Gerum:
>>>>> <snip>
>>>>> In any case, thanks for your work so far. We probably need to discuss
>>>>> the packaging issues on this list, so that we get both consistency and
>>>>> usability in the future.
>>>>>
>>>>> Gilles and Roland, if this is fine with you, I'll handle the liaison
>>>>> role with upstream packagers, so please CC me explicitly on those mails.
>>>>> We'll sort out this issue, it doesn't look that bad anyway.
>>>>
>>>> udev/xenomai.rules sets the group for realtime devices to "xenomai".
>>>> Patch attached to create the group "xenomai" on installation.
>>>>
>>>> Additionally, it would be nice to set "xeno_nucleus.xenomai_gid" to
>>>> group "xenomai" too. Is this possible with a udev rule ?
>>>
>>> Well, no, xeno_nucleus.xenomai_gid is a kernel parameter, so it is boot
>>> loader stuff. However, if xeno_nucleus is compiled as a module, you can
>>> add parameters in /etc/modprobe.d. Well, that would have been the place
>>> to do it some time ago. I may not be completely up-to-date.
>>
>> Hi Gilles,
>>
>> can a init.d script with
>>   echo "<gid>" > /sys/module/xeno_nucleus/parameters/xenomai_gid
>> work in module and builtin case ?
> 
> it seems that init.d will work for the builtin case and modprobe.d for the
> module case. So adding both should work "in all cases" :-)
> I'll give it a try ...

Attached a patch that adds a init-script /etc/init.d/xenomai to the package libxenomai1.
If group xenomai and the file /sys/module/xeno_nucleus/parameters/xenomai_gid are found,
xenomai_gid is set on startup.

Now looking at modprobe.d ...

>>>>
>>>> regards
>>>> kisda
>>>>
>>>>
>>>> ------------------------------------------------------------------------
>>>>
>>>> _______________________________________________
>>>> Xenomai-core mailing list
>>>> Xenomai-core@domain.hid
>>>> https://mail.gna.org/listinfo/xenomai-core
>>>
>>>
>>
>>
>>
>>
>> _______________________________________________
>> Xenomai-core mailing list
>> Xenomai-core@domain.hid
>> https://mail.gna.org/listinfo/xenomai-core
> 
> 
> 
> 
> _______________________________________________
> Xenomai-core mailing list
> Xenomai-core@domain.hid
> https://mail.gna.org/listinfo/xenomai-core


[-- Attachment #1.2: xenomai-init.patch --]
[-- Type: text/plain, Size: 1485 bytes --]

diff -uNrp xenomai-2.5.1/debian/libxenomai1.xenomai.init xenomai-2.5.1.new/debian/libxenomai1.xenomai.init
--- xenomai-2.5.1/debian/libxenomai1.xenomai.init	1970-01-01 01:00:00.000000000 +0100
+++ xenomai-2.5.1.new/debian/libxenomai1.xenomai.init	2010-02-25 17:13:51.000000000 +0100
@@ -0,0 +1,37 @@
+#!/bin/sh -e
+### BEGIN INIT INFO
+# Provides:          xenomai
+# Required-Start:    mountkernfs
+# Required-Stop:
+# Default-Start:     S
+# Default-Stop:
+# Short-Description: Set xeno_nucleus group
+### END INIT INFO
+
+GROUP=xenomai
+INITNAME=/etc/init.d/xenomai
+FILENAME=/sys/module/xeno_nucleus/parameters/xenomai_gid
+GID=$(getent group $GROUP | cut -d: -f3)
+
+test -e $FILENAME || exit 0
+test -n "$GID" || exit 0
+
+case "$1" in
+  start)
+        echo "$GID" > $FILENAME
+        ;;
+  stop)
+        echo "-1" > $FILENAME
+        ;;
+  restart|force-reload)
+        $0 stop
+        $0 start
+        ;;
+  *)
+        echo "Usage: $INITNAME {start|stop|restart|force-reload}"
+        exit 1
+        ;;
+esac
+
+exit 0
+
diff -uNrp xenomai-2.5.1/debian/rules xenomai-2.5.1.new/debian/rules
--- xenomai-2.5.1/debian/rules	2010-02-25 16:56:05.000000000 +0100
+++ xenomai-2.5.1.new/debian/rules	2010-02-25 17:33:20.000000000 +0100
@@ -132,6 +132,7 @@ binary-indep: build install
 binary-arch: build install
 	dh_testdir -s
 	dh_testroot -s
+	dh_installinit -s --name=xenomai
 	dh_installman -s
 	dh_installdocs -s -A CREDITS README.INSTALL TROUBLESHOOTING
 	dh_link -s

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

^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
       [not found]             ` <4B868CC1.6030103@domain.hid>
@ 2010-02-25 18:31               ` Stefan Kisdaroczi
  0 siblings, 0 replies; 38+ messages in thread
From: Stefan Kisdaroczi @ 2010-02-25 18:31 UTC (permalink / raw)
  To: Jeff Weber; +Cc: xenomai

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

Hi Jeff,

Am 25.02.2010 15:44, schrieb Jeff Weber:
> Hello Stefan:
> 
> I've been watching with interest the recent active work integrating
> Debian and Xenomai.  I am currently porting an embedded application to
> Xenomai, and am surveying candidate Linux distributions.
> 
> I did a test build, install of Xenomai on a Debian Lenny distributionI
> per the HOWTO:
> http://xenomai.org/index.php/Xenomai_quick_build_quide
> 
> and updated my grub menu.lst with the new kernel, and all worked well.
> I did spend a significant amount of time tuning my kernel config file to
> my target board.
> 
> Can you help me understand the advantages of using the Debian packages,
> over the process I followed above?  I am still unclear on the advantages
> of using the Debian Xenomia packages.

Advantages:
- Installing software is easy, maintaining a system is harder. Using debian
  packages helps you upgrading/removing xenomai on a system.
- If you need xenomai on more than one machine, you build once and install
  the same deb's on all machines.
- You are testing different distributions now. If you install distribution xyz
  on your system and want later to switch back to lenny, you just have to
  keep the deb's and you are fine. Nothing to compile.
- You don't need any xenomai knowhow to install a deb. If, for example, your
  co-worker needs to be able to setup xenomai too and he has the deb's a
  simple "dpkg -i *.deb" on a lenny system will do it and it will work,
  and you can enjoy your holidays ;-)

> Which kernels can I use?
> 
> Do I still need to tune a kernel config file?

You still need to tune the config and build a kernel, there is no binary
kernel-image with xenomai support. However, if you use make-kpkg to build
the kernel it will build a *.deb. If you install this deb you dont
have to fiddle with grub menu.lst, the kernel is automatically added.

Please always CC the mailing list. Thank you.

Regards
Stefan

> If not, how is the kernel pre-configured?
> 
> thanks,
> Jeff



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

^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-24 13:13               ` Philippe Gerum
@ 2010-02-26 13:13                 ` Stefan Kisdaroczi
  2010-02-26 13:28                   ` Philippe Gerum
  0 siblings, 1 reply; 38+ messages in thread
From: Stefan Kisdaroczi @ 2010-02-26 13:13 UTC (permalink / raw)
  To: Philippe Gerum; +Cc: xenomai

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

Am 24.02.2010 14:13, schrieb Philippe Gerum:
> On Wed, 2010-02-24 at 14:11 +0100, Philippe Gerum wrote:
>> On Wed, 2010-02-24 at 14:06 +0100, Stefan Kisdaroczi wrote:
>>> Hi Philippe,
>>>
>>> Am 23.02.2010 18:46, schrieb Philippe Gerum:
>>>> On Tue, 2010-02-23 at 17:52 +0100, Stefan Kisdaroczi wrote:
>>>>> Hi,
>>>>>
>>>>> Am 14.02.2010 10:38, schrieb Philippe Gerum:
>>>>>>
>>>>>> <snip>
>>>>>> In the future, maybe we could simply provide a wrapper script accepting
>>>>>> sub-commands, such as "xeno latency, xeno sigtest" etc, to be put
>>>>>> into /usr/bin by distros, which would hide the actual location of those
>>>>>> binaries?
>>>>>>
>>>>>> In any case, thanks for your work so far. We probably need to discuss
>>>>>> the packaging issues on this list, so that we get both consistency and
>>>>>> usability in the future.
>>>>>>
>>>>>> Gilles and Roland, if this is fine with you, I'll handle the liaison
>>>>>> role with upstream packagers, so please CC me explicitly on those mails.
>>>>>> We'll sort out this issue, it doesn't look that bad anyway.
>>>>>
>>>>> Roland added a xeno wrapper to the debian.org xenomai package 2.5.1-3.
>>>>>
>>>>> I synced now the debian/ directories from debian.org and xenomai.org:
>>>>>  - For debian.org I sent patches to the Debian bugtracker [1] [2].
>>>>>    Another patch for dpkg-cross support [3] I sent to Roland privately.
>>>>>  - For xenomai.org I attached patches to this mail (against -2.5.git).
>>>>>
>>>>> If both parties apply the patches the debian directories are in sync,
>>>>> except some minor differences in the debian/control file, see patch
>>>>> do-not-commit-please.patch. I would like to keep these changes out so
>>>>> that the xenomai.org packages are compatible with Debian 5.0 Lenny.
>>>>> The debian.org packages are for Debian 6.0 Squeeze.
>>>>>
>>>>
>>>> Merged into my queue (except the last one as mentioned). This will be
>>>> pushed upstream to Gilles for 2.5.2. Thanks.
>>>
>>> I took a look at your branch for-upstream. Your commit
>>>   scripts: add wrapper script to run standard Xenomai commands
>>>   6e0574791f48cbf8b3421a68c5789254e7d084b7
>>> adds the same wrapper as my patch 0005-debian-wrapper-script-usr-bin-xeno-to-call-executa.patch
>>>   debian: wrapper script /usr/bin/xeno to call executables in /usr/lib/xenomai/
>>>   fbe86cc50d3a65cd23e93d43adba4ed369fe70b1
>>> Please revert the commit of my patch, we need another fix for debian/rules for "your" wrapper.
>>>
>>
>> Ok, I thought your patch set was based on my tree, so I did not check
>> thoroughly. I did not send any pull request to Gilles, so no harm done.
>>
>>> How do I call configure to install the wrapper in /usr/bin and
>>> the programs like latency, switchtest etc. to /usr/lib/xenomai ?
>>>
>>
>> We need some fixage in scripts/wrappers/Makefile.am to do that. I'll
>> prepare this asap.
> 
> scripts/Makefile.am...

Hi Philippe,

I just tried the --with-testdir switch. It worked, but i'm not really sure if
this is the right track.

Roland's packages install all binaries to /usr/lib/xenomai, except xeno and
xeno-config. You state in your commit message more or less the same goal:
"At some point, all remaining executables or scripts left under $prefix/bin should match
xeno*, to further reduce the odds of causing name collisions."

Using --with-testdir all tests (latency,switchtest,...) are now in /usr/lib/xenomai.
To install the "utils" (rtcansend,insn_write,insn_write,cmd_read,...) to the same
directory using --with-testdir sounds not obviously. You could add a second switch
--with-utildir, but a second dir will not work for the xeno-wrapper-script.

I think something like using --bindir=/usr/lib/xenomai and --wrapperdir=/usr/bin
is probably better, as there are less exceptions.

For the test I patched xeno.in: exec @XENO_TEST_DIR@/$@

Stefan

>>
>>> Stefan
>>>
>>>>
>>>>> Thanks
>>>>> kisda
>>>>>
>>>>> [1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=571099
>>>>> [2] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=571104
>>>>> [3] http://git.xenomai.org/?p=xenomai-2.5.git;a=commitdiff;h=5bcd18f714f4cbeaaac0cc4a08e6c9f375aa3b77
>>>>
>>>>
>>>
>>>
>>
>>
> 
> 



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

^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-26 13:13                 ` Stefan Kisdaroczi
@ 2010-02-26 13:28                   ` Philippe Gerum
  2010-02-26 13:48                     ` Stefan Kisdaroczi
  0 siblings, 1 reply; 38+ messages in thread
From: Philippe Gerum @ 2010-02-26 13:28 UTC (permalink / raw)
  To: Stefan Kisdaroczi; +Cc: xenomai

On Fri, 2010-02-26 at 14:13 +0100, Stefan Kisdaroczi wrote:
> Am 24.02.2010 14:13, schrieb Philippe Gerum:
> > On Wed, 2010-02-24 at 14:11 +0100, Philippe Gerum wrote:
> >> On Wed, 2010-02-24 at 14:06 +0100, Stefan Kisdaroczi wrote:
> >>> Hi Philippe,
> >>>
> >>> Am 23.02.2010 18:46, schrieb Philippe Gerum:
> >>>> On Tue, 2010-02-23 at 17:52 +0100, Stefan Kisdaroczi wrote:
> >>>>> Hi,
> >>>>>
> >>>>> Am 14.02.2010 10:38, schrieb Philippe Gerum:
> >>>>>>
> >>>>>> <snip>
> >>>>>> In the future, maybe we could simply provide a wrapper script accepting
> >>>>>> sub-commands, such as "xeno latency, xeno sigtest" etc, to be put
> >>>>>> into /usr/bin by distros, which would hide the actual location of those
> >>>>>> binaries?
> >>>>>>
> >>>>>> In any case, thanks for your work so far. We probably need to discuss
> >>>>>> the packaging issues on this list, so that we get both consistency and
> >>>>>> usability in the future.
> >>>>>>
> >>>>>> Gilles and Roland, if this is fine with you, I'll handle the liaison
> >>>>>> role with upstream packagers, so please CC me explicitly on those mails.
> >>>>>> We'll sort out this issue, it doesn't look that bad anyway.
> >>>>>
> >>>>> Roland added a xeno wrapper to the debian.org xenomai package 2.5.1-3.
> >>>>>
> >>>>> I synced now the debian/ directories from debian.org and xenomai.org:
> >>>>>  - For debian.org I sent patches to the Debian bugtracker [1] [2].
> >>>>>    Another patch for dpkg-cross support [3] I sent to Roland privately.
> >>>>>  - For xenomai.org I attached patches to this mail (against -2.5.git).
> >>>>>
> >>>>> If both parties apply the patches the debian directories are in sync,
> >>>>> except some minor differences in the debian/control file, see patch
> >>>>> do-not-commit-please.patch. I would like to keep these changes out so
> >>>>> that the xenomai.org packages are compatible with Debian 5.0 Lenny.
> >>>>> The debian.org packages are for Debian 6.0 Squeeze.
> >>>>>
> >>>>
> >>>> Merged into my queue (except the last one as mentioned). This will be
> >>>> pushed upstream to Gilles for 2.5.2. Thanks.
> >>>
> >>> I took a look at your branch for-upstream. Your commit
> >>>   scripts: add wrapper script to run standard Xenomai commands
> >>>   6e0574791f48cbf8b3421a68c5789254e7d084b7
> >>> adds the same wrapper as my patch 0005-debian-wrapper-script-usr-bin-xeno-to-call-executa.patch
> >>>   debian: wrapper script /usr/bin/xeno to call executables in /usr/lib/xenomai/
> >>>   fbe86cc50d3a65cd23e93d43adba4ed369fe70b1
> >>> Please revert the commit of my patch, we need another fix for debian/rules for "your" wrapper.
> >>>
> >>
> >> Ok, I thought your patch set was based on my tree, so I did not check
> >> thoroughly. I did not send any pull request to Gilles, so no harm done.
> >>
> >>> How do I call configure to install the wrapper in /usr/bin and
> >>> the programs like latency, switchtest etc. to /usr/lib/xenomai ?
> >>>
> >>
> >> We need some fixage in scripts/wrappers/Makefile.am to do that. I'll
> >> prepare this asap.
> > 
> > scripts/Makefile.am...
> 
> Hi Philippe,
> 
> I just tried the --with-testdir switch. It worked, but i'm not really sure if
> this is the right track.
> 
> Roland's packages install all binaries to /usr/lib/xenomai, except xeno and
> xeno-config. You state in your commit message more or less the same goal:
> "At some point, all remaining executables or scripts left under $prefix/bin should match
> xeno*, to further reduce the odds of causing name collisions."
> 
> Using --with-testdir all tests (latency,switchtest,...) are now in /usr/lib/xenomai.
> To install the "utils" (rtcansend,insn_write,insn_write,cmd_read,...) to the same
> directory using --with-testdir sounds not obviously. You could add a second switch
> --with-utildir, but a second dir will not work for the xeno-wrapper-script.
> 

CAN and other utilities should definitely remain in $bindir. The fact
that they are not prefixed by xeno* is another thing; CAN utilities are
already prefixed, maybe Analogy ones should be named in a bit less
generic way, although they are not raising any conflict today. I wrote
that what's under $bindir "should" match xeno* when a risk of collision
exists, but there is no point in enforcing a stricter rule at this
point. In any case, I don't want to enforce a never-in-bindir rule for
all Xenomai binaries, we can still pick their names in a way that avoids
obvious issues.

The real problem was about tests, for which using rather generic names
made sense. This is what that patch is for.

> I think something like using --bindir=/usr/lib/xenomai and --wrapperdir=/usr/bin
> is probably better, as there are less exceptions.
> 
> For the test I patched xeno.in: exec @XENO_TEST_DIR@/$@
> 
> Stefan
> 
> >>
> >>> Stefan
> >>>
> >>>>
> >>>>> Thanks
> >>>>> kisda
> >>>>>
> >>>>> [1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=571099
> >>>>> [2] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=571104
> >>>>> [3] http://git.xenomai.org/?p=xenomai-2.5.git;a=commitdiff;h=5bcd18f714f4cbeaaac0cc4a08e6c9f375aa3b77
> >>>>
> >>>>
> >>>
> >>>
> >>
> >>
> > 
> > 
> 
> 


-- 
Philippe.




^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-25 17:18               ` Stefan Kisdaroczi
@ 2010-02-26 13:35                 ` Stefan Kisdaroczi
  2010-05-02 16:01                   ` Roland Stigge
  0 siblings, 1 reply; 38+ messages in thread
From: Stefan Kisdaroczi @ 2010-02-26 13:35 UTC (permalink / raw)
  To: xenomai


[-- Attachment #1.1: Type: text/plain, Size: 1194 bytes --]

Am 25.02.2010 18:18, schrieb Stefan Kisdaroczi:
> <snip>
>>>
>>> Hi Gilles,
>>>
>>> can a init.d script with
>>>   echo "<gid>" > /sys/module/xeno_nucleus/parameters/xenomai_gid
>>> work in module and builtin case ?
>>
>> it seems that init.d will work for the builtin case and modprobe.d for the
>> module case. So adding both should work "in all cases" :-)
>> I'll give it a try ...
> 
> Attached a patch that adds a init-script /etc/init.d/xenomai to the package libxenomai1.
> If group xenomai and the file /sys/module/xeno_nucleus/parameters/xenomai_gid are found,
> xenomai_gid is set on startup.
> 
> Now looking at modprobe.d ...

I have attached a patch against debian xenomai version 2.5.1-4, changes:
 - create group xenomai on install
 - added a init-script which sets /sys/.../xenomai_gid if
   /sys/.../xenomai_gid exists
 - added a modprobe-script that adds the xenomai_gid parameter if the user
   did call modprobe without xenomai_gid=

With this changes, all users which are member of the group xenomai are able
to run xenomai apps, with xeno_nucleus builtin or as a module.

Please do not merge anywhere for now, comments are welcome.

Stefan

[-- Attachment #1.2: xenomai-group-debian.patch --]
[-- Type: text/plain, Size: 3814 bytes --]

diff -uNrp xenomai-2.5.1.orig/debian/libxenomai1.dirs xenomai-2.5.1/debian/libxenomai1.dirs
--- xenomai-2.5.1.orig/debian/libxenomai1.dirs	2010-02-26 01:01:07.000000000 +0100
+++ xenomai-2.5.1/debian/libxenomai1.dirs	2010-02-26 01:16:13.000000000 +0100
@@ -1,2 +1,3 @@
+etc/modprobe.d
 etc/udev/rules.d
 usr/share/lintian/overrides
diff -uNrp xenomai-2.5.1.orig/debian/libxenomai1.modprobe xenomai-2.5.1/debian/libxenomai1.modprobe
--- xenomai-2.5.1.orig/debian/libxenomai1.modprobe	1970-01-01 01:00:00.000000000 +0100
+++ xenomai-2.5.1/debian/libxenomai1.modprobe	2010-02-26 01:07:41.000000000 +0100
@@ -0,0 +1,3 @@
+install xeno_nucleus /sbin/modprobe --ignore-install xeno_nucleus $CMDLINE_OPTS \
+  $(/usr/bin/test $(/bin/echo -n '$CMDLINE_OPTS' | /bin/grep xenomai_gid) \
+    || /usr/bin/getent group xenomai | /usr/bin/cut -d: -f3 | /bin/sed -e 's/^/xenomai_gid\=/')
diff -uNrp xenomai-2.5.1.orig/debian/libxenomai1.postinst xenomai-2.5.1/debian/libxenomai1.postinst
--- xenomai-2.5.1.orig/debian/libxenomai1.postinst	2010-02-26 01:01:07.000000000 +0100
+++ xenomai-2.5.1/debian/libxenomai1.postinst	2010-02-26 01:06:00.000000000 +0100
@@ -1,6 +1,22 @@
 #!/bin/sh -e
 
-rm -f /etc/udev/rules.d/xenomai.rules
-ln -sf ../xenomai.rules /etc/udev/rules.d/xenomai.rules
+case "$1" in
+    configure)
+        # Add the xenomai group unless it's already there
+        if ! getent group xenomai >/dev/null; then
+            addgroup --quiet --system xenomai || true
+        fi
+        rm -f /etc/udev/rules.d/xenomai.rules
+        ln -sf ../xenomai.rules /etc/udev/rules.d/xenomai.rules
+    ;;
+
+    abort-upgrade|abort-remove|abort-deconfigure)
+    ;;
+
+    *)
+        echo "postinst called with unknown argument \`$1'" >&2
+        exit 1
+    ;;
+esac
 
 #DEBHELPER#
diff -uNrp xenomai-2.5.1.orig/debian/libxenomai1.xenomai.init xenomai-2.5.1/debian/libxenomai1.xenomai.init
--- xenomai-2.5.1.orig/debian/libxenomai1.xenomai.init	1970-01-01 01:00:00.000000000 +0100
+++ xenomai-2.5.1/debian/libxenomai1.xenomai.init	2010-02-26 01:48:28.000000000 +0100
@@ -0,0 +1,36 @@
+#!/bin/sh -e
+### BEGIN INIT INFO
+# Provides:          xenomai
+# Required-Start:    mountkernfs
+# Required-Stop:
+# Default-Start:     S
+# Default-Stop:
+# Short-Description: Set xeno_nucleus group
+### END INIT INFO
+
+GROUP=xenomai
+INITNAME=/etc/init.d/xenomai
+FILENAME=/sys/module/xeno_nucleus/parameters/xenomai_gid
+GID=$(getent group $GROUP | cut -d: -f3)
+
+test -e $FILENAME || exit 0
+test -n "$GID" || exit 0
+
+case "$1" in
+  start)
+        echo "$GID" > $FILENAME
+        ;;
+  stop)
+        echo "-1" > $FILENAME
+        ;;
+  restart|force-reload)
+        $0 start
+        ;;
+  *)
+        echo "Usage: $INITNAME {start|stop|restart|force-reload}"
+        exit 1
+        ;;
+esac
+
+exit 0
+
diff -uNrp xenomai-2.5.1.orig/debian/rules xenomai-2.5.1/debian/rules
--- xenomai-2.5.1.orig/debian/rules	2010-02-26 01:01:07.000000000 +0100
+++ xenomai-2.5.1/debian/rules	2010-02-26 01:17:19.000000000 +0100
@@ -100,6 +100,7 @@ install: build
 	for f in $(CURDIR)/ksrc/nucleus/udev/*.rules ; do \
 	    cat $$f >> $(CURDIR)/debian/libxenomai1/etc/udev/xenomai.rules ; \
 	done
+	install -m 644 debian/libxenomai1.modprobe $(CURDIR)/debian/libxenomai1/etc/modprobe.d/xenomai
 	# remove empty directory
 	rm -rf $(CURDIR)/debian/xenomai-doc/usr/share/doc/xenomai-doc/ps
 	cp debian/libxenomai1.lintian $(CURDIR)/debian/libxenomai1/usr/share/lintian/overrides/libxenomai1
@@ -132,6 +133,7 @@ binary-indep: build install
 binary-arch: build install
 	dh_testdir -s
 	dh_testroot -s
+	dh_installinit -s --name=xenomai
 	dh_installman -s
 	dh_installdocs -s -A CREDITS README.INSTALL TROUBLESHOOTING
 	dh_link -s

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

^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-26 13:28                   ` Philippe Gerum
@ 2010-02-26 13:48                     ` Stefan Kisdaroczi
  2010-02-26 14:07                       ` Philippe Gerum
  0 siblings, 1 reply; 38+ messages in thread
From: Stefan Kisdaroczi @ 2010-02-26 13:48 UTC (permalink / raw)
  To: Philippe Gerum; +Cc: xenomai

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

Am 26.02.2010 14:28, schrieb Philippe Gerum:
> On Fri, 2010-02-26 at 14:13 +0100, Stefan Kisdaroczi wrote:
>> Am 24.02.2010 14:13, schrieb Philippe Gerum:
>>> On Wed, 2010-02-24 at 14:11 +0100, Philippe Gerum wrote:
>>>> On Wed, 2010-02-24 at 14:06 +0100, Stefan Kisdaroczi wrote:
>>>>> Hi Philippe,
>>>>>
>>>>> Am 23.02.2010 18:46, schrieb Philippe Gerum:
>>>>>> On Tue, 2010-02-23 at 17:52 +0100, Stefan Kisdaroczi wrote:
>>>>>>> Hi,
>>>>>>>
>>>>>>> Am 14.02.2010 10:38, schrieb Philippe Gerum:
>>>>>>>>
>>>>>>>> <snip>
>>>>>>>> In the future, maybe we could simply provide a wrapper script accepting
>>>>>>>> sub-commands, such as "xeno latency, xeno sigtest" etc, to be put
>>>>>>>> into /usr/bin by distros, which would hide the actual location of those
>>>>>>>> binaries?
>>>>>>>>
>>>>>>>> In any case, thanks for your work so far. We probably need to discuss
>>>>>>>> the packaging issues on this list, so that we get both consistency and
>>>>>>>> usability in the future.
>>>>>>>>
>>>>>>>> Gilles and Roland, if this is fine with you, I'll handle the liaison
>>>>>>>> role with upstream packagers, so please CC me explicitly on those mails.
>>>>>>>> We'll sort out this issue, it doesn't look that bad anyway.
>>>>>>>
>>>>>>> Roland added a xeno wrapper to the debian.org xenomai package 2.5.1-3.
>>>>>>>
>>>>>>> I synced now the debian/ directories from debian.org and xenomai.org:
>>>>>>>  - For debian.org I sent patches to the Debian bugtracker [1] [2].
>>>>>>>    Another patch for dpkg-cross support [3] I sent to Roland privately.
>>>>>>>  - For xenomai.org I attached patches to this mail (against -2.5.git).
>>>>>>>
>>>>>>> If both parties apply the patches the debian directories are in sync,
>>>>>>> except some minor differences in the debian/control file, see patch
>>>>>>> do-not-commit-please.patch. I would like to keep these changes out so
>>>>>>> that the xenomai.org packages are compatible with Debian 5.0 Lenny.
>>>>>>> The debian.org packages are for Debian 6.0 Squeeze.
>>>>>>>
>>>>>>
>>>>>> Merged into my queue (except the last one as mentioned). This will be
>>>>>> pushed upstream to Gilles for 2.5.2. Thanks.
>>>>>
>>>>> I took a look at your branch for-upstream. Your commit
>>>>>   scripts: add wrapper script to run standard Xenomai commands
>>>>>   6e0574791f48cbf8b3421a68c5789254e7d084b7
>>>>> adds the same wrapper as my patch 0005-debian-wrapper-script-usr-bin-xeno-to-call-executa.patch
>>>>>   debian: wrapper script /usr/bin/xeno to call executables in /usr/lib/xenomai/
>>>>>   fbe86cc50d3a65cd23e93d43adba4ed369fe70b1
>>>>> Please revert the commit of my patch, we need another fix for debian/rules for "your" wrapper.
>>>>>
>>>>
>>>> Ok, I thought your patch set was based on my tree, so I did not check
>>>> thoroughly. I did not send any pull request to Gilles, so no harm done.
>>>>
>>>>> How do I call configure to install the wrapper in /usr/bin and
>>>>> the programs like latency, switchtest etc. to /usr/lib/xenomai ?
>>>>>
>>>>
>>>> We need some fixage in scripts/wrappers/Makefile.am to do that. I'll
>>>> prepare this asap.
>>>
>>> scripts/Makefile.am...
>>
>> Hi Philippe,
>>
>> I just tried the --with-testdir switch. It worked, but i'm not really sure if
>> this is the right track.
>>
>> Roland's packages install all binaries to /usr/lib/xenomai, except xeno and
>> xeno-config. You state in your commit message more or less the same goal:
>> "At some point, all remaining executables or scripts left under $prefix/bin should match
>> xeno*, to further reduce the odds of causing name collisions."
>>
>> Using --with-testdir all tests (latency,switchtest,...) are now in /usr/lib/xenomai.
>> To install the "utils" (rtcansend,insn_write,insn_write,cmd_read,...) to the same
>> directory using --with-testdir sounds not obviously. You could add a second switch
>> --with-utildir, but a second dir will not work for the xeno-wrapper-script.
>>
> 
> CAN and other utilities should definitely remain in $bindir. The fact
> that they are not prefixed by xeno* is another thing; CAN utilities are
> already prefixed, maybe Analogy ones should be named in a bit less
> generic way, although they are not raising any conflict today. I wrote
> that what's under $bindir "should" match xeno* when a risk of collision
> exists, but there is no point in enforcing a stricter rule at this
> point. In any case, I don't want to enforce a never-in-bindir rule for
> all Xenomai binaries, we can still pick their names in a way that avoids
> obvious issues.
> 
> The real problem was about tests, for which using rather generic names
> made sense. This is what that patch is for.

ok, got the goal now, thanks for the explanation.
But "xeno.in" needs a fix to use @XENO_TEST_DIR@, no?

>> I think something like using --bindir=/usr/lib/xenomai and --wrapperdir=/usr/bin
>> is probably better, as there are less exceptions.
>>
>> For the test I patched xeno.in: exec @XENO_TEST_DIR@/$@


>>
>> Stefan
>>
>>>>
>>>>> Stefan
>>>>>
>>>>>>
>>>>>>> Thanks
>>>>>>> kisda
>>>>>>>
>>>>>>> [1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=571099
>>>>>>> [2] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=571104
>>>>>>> [3] http://git.xenomai.org/?p=xenomai-2.5.git;a=commitdiff;h=5bcd18f714f4cbeaaac0cc4a08e6c9f375aa3b77
>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>
>>>>
>>>
>>>
>>
>>
> 
> 



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

^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-26 13:48                     ` Stefan Kisdaroczi
@ 2010-02-26 14:07                       ` Philippe Gerum
  2010-03-03 16:54                         ` Stefan Kisdaroczi
  0 siblings, 1 reply; 38+ messages in thread
From: Philippe Gerum @ 2010-02-26 14:07 UTC (permalink / raw)
  To: Stefan Kisdaroczi; +Cc: xenomai

On Fri, 2010-02-26 at 14:48 +0100, Stefan Kisdaroczi wrote:
> Am 26.02.2010 14:28, schrieb Philippe Gerum:
> > On Fri, 2010-02-26 at 14:13 +0100, Stefan Kisdaroczi wrote:
> >> Am 24.02.2010 14:13, schrieb Philippe Gerum:
> >>> On Wed, 2010-02-24 at 14:11 +0100, Philippe Gerum wrote:
> >>>> On Wed, 2010-02-24 at 14:06 +0100, Stefan Kisdaroczi wrote:
> >>>>> Hi Philippe,
> >>>>>
> >>>>> Am 23.02.2010 18:46, schrieb Philippe Gerum:
> >>>>>> On Tue, 2010-02-23 at 17:52 +0100, Stefan Kisdaroczi wrote:
> >>>>>>> Hi,
> >>>>>>>
> >>>>>>> Am 14.02.2010 10:38, schrieb Philippe Gerum:
> >>>>>>>>
> >>>>>>>> <snip>
> >>>>>>>> In the future, maybe we could simply provide a wrapper script accepting
> >>>>>>>> sub-commands, such as "xeno latency, xeno sigtest" etc, to be put
> >>>>>>>> into /usr/bin by distros, which would hide the actual location of those
> >>>>>>>> binaries?
> >>>>>>>>
> >>>>>>>> In any case, thanks for your work so far. We probably need to discuss
> >>>>>>>> the packaging issues on this list, so that we get both consistency and
> >>>>>>>> usability in the future.
> >>>>>>>>
> >>>>>>>> Gilles and Roland, if this is fine with you, I'll handle the liaison
> >>>>>>>> role with upstream packagers, so please CC me explicitly on those mails.
> >>>>>>>> We'll sort out this issue, it doesn't look that bad anyway.
> >>>>>>>
> >>>>>>> Roland added a xeno wrapper to the debian.org xenomai package 2.5.1-3.
> >>>>>>>
> >>>>>>> I synced now the debian/ directories from debian.org and xenomai.org:
> >>>>>>>  - For debian.org I sent patches to the Debian bugtracker [1] [2].
> >>>>>>>    Another patch for dpkg-cross support [3] I sent to Roland privately.
> >>>>>>>  - For xenomai.org I attached patches to this mail (against -2.5.git).
> >>>>>>>
> >>>>>>> If both parties apply the patches the debian directories are in sync,
> >>>>>>> except some minor differences in the debian/control file, see patch
> >>>>>>> do-not-commit-please.patch. I would like to keep these changes out so
> >>>>>>> that the xenomai.org packages are compatible with Debian 5.0 Lenny.
> >>>>>>> The debian.org packages are for Debian 6.0 Squeeze.
> >>>>>>>
> >>>>>>
> >>>>>> Merged into my queue (except the last one as mentioned). This will be
> >>>>>> pushed upstream to Gilles for 2.5.2. Thanks.
> >>>>>
> >>>>> I took a look at your branch for-upstream. Your commit
> >>>>>   scripts: add wrapper script to run standard Xenomai commands
> >>>>>   6e0574791f48cbf8b3421a68c5789254e7d084b7
> >>>>> adds the same wrapper as my patch 0005-debian-wrapper-script-usr-bin-xeno-to-call-executa.patch
> >>>>>   debian: wrapper script /usr/bin/xeno to call executables in /usr/lib/xenomai/
> >>>>>   fbe86cc50d3a65cd23e93d43adba4ed369fe70b1
> >>>>> Please revert the commit of my patch, we need another fix for debian/rules for "your" wrapper.
> >>>>>
> >>>>
> >>>> Ok, I thought your patch set was based on my tree, so I did not check
> >>>> thoroughly. I did not send any pull request to Gilles, so no harm done.
> >>>>
> >>>>> How do I call configure to install the wrapper in /usr/bin and
> >>>>> the programs like latency, switchtest etc. to /usr/lib/xenomai ?
> >>>>>
> >>>>
> >>>> We need some fixage in scripts/wrappers/Makefile.am to do that. I'll
> >>>> prepare this asap.
> >>>
> >>> scripts/Makefile.am...
> >>
> >> Hi Philippe,
> >>
> >> I just tried the --with-testdir switch. It worked, but i'm not really sure if
> >> this is the right track.
> >>
> >> Roland's packages install all binaries to /usr/lib/xenomai, except xeno and
> >> xeno-config. You state in your commit message more or less the same goal:
> >> "At some point, all remaining executables or scripts left under $prefix/bin should match
> >> xeno*, to further reduce the odds of causing name collisions."
> >>
> >> Using --with-testdir all tests (latency,switchtest,...) are now in /usr/lib/xenomai.
> >> To install the "utils" (rtcansend,insn_write,insn_write,cmd_read,...) to the same
> >> directory using --with-testdir sounds not obviously. You could add a second switch
> >> --with-utildir, but a second dir will not work for the xeno-wrapper-script.
> >>
> > 
> > CAN and other utilities should definitely remain in $bindir. The fact
> > that they are not prefixed by xeno* is another thing; CAN utilities are
> > already prefixed, maybe Analogy ones should be named in a bit less
> > generic way, although they are not raising any conflict today. I wrote
> > that what's under $bindir "should" match xeno* when a risk of collision
> > exists, but there is no point in enforcing a stricter rule at this
> > point. In any case, I don't want to enforce a never-in-bindir rule for
> > all Xenomai binaries, we can still pick their names in a way that avoids
> > obvious issues.
> > 
> > The real problem was about tests, for which using rather generic names
> > made sense. This is what that patch is for.
> 
> ok, got the goal now, thanks for the explanation.
> But "xeno.in" needs a fix to use @XENO_TEST_DIR@, no?
> 

Yes, I overlooked that. In fact, I think we may not even need the new
xeno wrapper at all, but we probably want to rewrite xeno-test to wrap
to what is in XENO_TEST_DIR now.

I would suggest to hold the changes to the debian/ area for now, until
the dust has settled a bit. We are trying to fix long-standing problems
in the way we allow people to test their setup. 

> >> I think something like using --bindir=/usr/lib/xenomai and --wrapperdir=/usr/bin
> >> is probably better, as there are less exceptions.
> >>
> >> For the test I patched xeno.in: exec @XENO_TEST_DIR@/$@
> 
> 
> >>
> >> Stefan
> >>
> >>>>
> >>>>> Stefan
> >>>>>
> >>>>>>
> >>>>>>> Thanks
> >>>>>>> kisda
> >>>>>>>
> >>>>>>> [1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=571099
> >>>>>>> [2] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=571104
> >>>>>>> [3] http://git.xenomai.org/?p=xenomai-2.5.git;a=commitdiff;h=5bcd18f714f4cbeaaac0cc4a08e6c9f375aa3b77
> >>>>>>
> >>>>>>
> >>>>>
> >>>>>
> >>>>
> >>>>
> >>>
> >>>
> >>
> >>
> > 
> > 
> 
> 


-- 
Philippe.




^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-13 15:07   ` Roland Stigge
  2010-02-13 15:25     ` Gilles Chanteperdrix
  2010-02-14  9:38     ` Philippe Gerum
@ 2010-03-01 17:04     ` Gilles Chanteperdrix
  2010-03-01 21:45       ` Roland Stigge
  2 siblings, 1 reply; 38+ messages in thread
From: Gilles Chanteperdrix @ 2010-03-01 17:04 UTC (permalink / raw)
  To: Roland Stigge; +Cc: xenomai-core

Roland Stigge wrote:
> Hi Gilles,
> 
> first - I'm sorry if you sometimes feel offended by my work on Xenomai 
> in Debian. I understand that you are very much connected to your project 
> and want to have it working perfectly everywhere.
> 
> Unfortunately, my time to work on this is limited and the last uploads 
> were work in progress - to provide latest Xenomai in Debian. Further 
> work on it was planned for this weekend.
> 
> But please also understand that Debian developers will possibly 
> prioritize work on upstream packages where they feel their work is 
> appreciated. So please think about your tone before sending email and 
> driving people away from Xenomai.

Ok. Please accept my sincere apologies. I understand that I am in no
position to ask anything the way I did it.

Trying to be more positive, would it help you, or anybody else if we
created a xenomai-announce mailing list, where we would only publish
releases and releasees informations ?

-- 
					    Gilles.


^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-03-01 17:04     ` Gilles Chanteperdrix
@ 2010-03-01 21:45       ` Roland Stigge
  0 siblings, 0 replies; 38+ messages in thread
From: Roland Stigge @ 2010-03-01 21:45 UTC (permalink / raw)
  To: Gilles Chanteperdrix; +Cc: xenomai-core

Hi Gilles,

thanks for your mail!

And don't worry - there's an updated Xenomai version in Debian sid now
thanks to everyone from the Xenomai dev list. Sven put quite some effort
into syncing the two projects. If there are any problems, feel free to
"bug" me. ;-)

Regarding the xenomai-announce list, this sounds like a good idea since
many projects have it, including Debian. For me personally, my work much
depends on free ressources and Debian release schedule urgency. Btw: The
Debian watch system automatically generates an email when xenomai has
got a new release. :-) So also no hurry here.

See you,

bye,
  Roland



^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-26 14:07                       ` Philippe Gerum
@ 2010-03-03 16:54                         ` Stefan Kisdaroczi
  2010-03-03 17:21                           ` Philippe Gerum
  0 siblings, 1 reply; 38+ messages in thread
From: Stefan Kisdaroczi @ 2010-03-03 16:54 UTC (permalink / raw)
  To: Philippe Gerum; +Cc: xenomai


[-- Attachment #1.1: Type: text/plain, Size: 6368 bytes --]

Am 26.02.2010 15:07, schrieb Philippe Gerum:
> On Fri, 2010-02-26 at 14:48 +0100, Stefan Kisdaroczi wrote:
>> Am 26.02.2010 14:28, schrieb Philippe Gerum:
>>> On Fri, 2010-02-26 at 14:13 +0100, Stefan Kisdaroczi wrote:
>>>> Am 24.02.2010 14:13, schrieb Philippe Gerum:
>>>>> On Wed, 2010-02-24 at 14:11 +0100, Philippe Gerum wrote:
>>>>>> On Wed, 2010-02-24 at 14:06 +0100, Stefan Kisdaroczi wrote:
>>>>>>> Hi Philippe,
>>>>>>>
>>>>>>> Am 23.02.2010 18:46, schrieb Philippe Gerum:
>>>>>>>> On Tue, 2010-02-23 at 17:52 +0100, Stefan Kisdaroczi wrote:
>>>>>>>>> Hi,
>>>>>>>>>
>>>>>>>>> Am 14.02.2010 10:38, schrieb Philippe Gerum:
>>>>>>>>>>
>>>>>>>>>> <snip>
>>>>>>>>>> In the future, maybe we could simply provide a wrapper script accepting
>>>>>>>>>> sub-commands, such as "xeno latency, xeno sigtest" etc, to be put
>>>>>>>>>> into /usr/bin by distros, which would hide the actual location of those
>>>>>>>>>> binaries?
>>>>>>>>>>
>>>>>>>>>> In any case, thanks for your work so far. We probably need to discuss
>>>>>>>>>> the packaging issues on this list, so that we get both consistency and
>>>>>>>>>> usability in the future.
>>>>>>>>>>
>>>>>>>>>> Gilles and Roland, if this is fine with you, I'll handle the liaison
>>>>>>>>>> role with upstream packagers, so please CC me explicitly on those mails.
>>>>>>>>>> We'll sort out this issue, it doesn't look that bad anyway.
>>>>>>>>>
>>>>>>>>> Roland added a xeno wrapper to the debian.org xenomai package 2.5.1-3.
>>>>>>>>>
>>>>>>>>> I synced now the debian/ directories from debian.org and xenomai.org:
>>>>>>>>>  - For debian.org I sent patches to the Debian bugtracker [1] [2].
>>>>>>>>>    Another patch for dpkg-cross support [3] I sent to Roland privately.
>>>>>>>>>  - For xenomai.org I attached patches to this mail (against -2.5.git).
>>>>>>>>>
>>>>>>>>> If both parties apply the patches the debian directories are in sync,
>>>>>>>>> except some minor differences in the debian/control file, see patch
>>>>>>>>> do-not-commit-please.patch. I would like to keep these changes out so
>>>>>>>>> that the xenomai.org packages are compatible with Debian 5.0 Lenny.
>>>>>>>>> The debian.org packages are for Debian 6.0 Squeeze.
>>>>>>>>>
>>>>>>>>
>>>>>>>> Merged into my queue (except the last one as mentioned). This will be
>>>>>>>> pushed upstream to Gilles for 2.5.2. Thanks.
>>>>>>>
>>>>>>> I took a look at your branch for-upstream. Your commit
>>>>>>>   scripts: add wrapper script to run standard Xenomai commands
>>>>>>>   6e0574791f48cbf8b3421a68c5789254e7d084b7
>>>>>>> adds the same wrapper as my patch 0005-debian-wrapper-script-usr-bin-xeno-to-call-executa.patch
>>>>>>>   debian: wrapper script /usr/bin/xeno to call executables in /usr/lib/xenomai/
>>>>>>>   fbe86cc50d3a65cd23e93d43adba4ed369fe70b1
>>>>>>> Please revert the commit of my patch, we need another fix for debian/rules for "your" wrapper.
>>>>>>>
>>>>>>
>>>>>> Ok, I thought your patch set was based on my tree, so I did not check
>>>>>> thoroughly. I did not send any pull request to Gilles, so no harm done.
>>>>>>
>>>>>>> How do I call configure to install the wrapper in /usr/bin and
>>>>>>> the programs like latency, switchtest etc. to /usr/lib/xenomai ?
>>>>>>>
>>>>>>
>>>>>> We need some fixage in scripts/wrappers/Makefile.am to do that. I'll
>>>>>> prepare this asap.
>>>>>
>>>>> scripts/Makefile.am...
>>>>
>>>> Hi Philippe,
>>>>
>>>> I just tried the --with-testdir switch. It worked, but i'm not really sure if
>>>> this is the right track.
>>>>
>>>> Roland's packages install all binaries to /usr/lib/xenomai, except xeno and
>>>> xeno-config. You state in your commit message more or less the same goal:
>>>> "At some point, all remaining executables or scripts left under $prefix/bin should match
>>>> xeno*, to further reduce the odds of causing name collisions."
>>>>
>>>> Using --with-testdir all tests (latency,switchtest,...) are now in /usr/lib/xenomai.
>>>> To install the "utils" (rtcansend,insn_write,insn_write,cmd_read,...) to the same
>>>> directory using --with-testdir sounds not obviously. You could add a second switch
>>>> --with-utildir, but a second dir will not work for the xeno-wrapper-script.
>>>>
>>>
>>> CAN and other utilities should definitely remain in $bindir. The fact
>>> that they are not prefixed by xeno* is another thing; CAN utilities are
>>> already prefixed, maybe Analogy ones should be named in a bit less
>>> generic way, although they are not raising any conflict today. I wrote
>>> that what's under $bindir "should" match xeno* when a risk of collision
>>> exists, but there is no point in enforcing a stricter rule at this
>>> point. In any case, I don't want to enforce a never-in-bindir rule for
>>> all Xenomai binaries, we can still pick their names in a way that avoids
>>> obvious issues.
>>>
>>> The real problem was about tests, for which using rather generic names
>>> made sense. This is what that patch is for.
>>
>> ok, got the goal now, thanks for the explanation.
>> But "xeno.in" needs a fix to use @XENO_TEST_DIR@, no?
>>
> 
> Yes, I overlooked that. In fact, I think we may not even need the new
> xeno wrapper at all, but we probably want to rewrite xeno-test to wrap
> to what is in XENO_TEST_DIR now.
> 
> I would suggest to hold the changes to the debian/ area for now, until

Agree with holding back wrapper-changes, but please consider the attached patch
with small fixes, thanks. (against rpm/for-upstream)

> the dust has settled a bit. We are trying to fix long-standing problems
> in the way we allow people to test their setup. 
> 
>>>> I think something like using --bindir=/usr/lib/xenomai and --wrapperdir=/usr/bin
>>>> is probably better, as there are less exceptions.
>>>>
>>>> For the test I patched xeno.in: exec @XENO_TEST_DIR@/$@
>>
>>
>>>>
>>>> Stefan
>>>>
>>>>>>
>>>>>>> Stefan
>>>>>>>
>>>>>>>>
>>>>>>>>> Thanks
>>>>>>>>> kisda
>>>>>>>>>
>>>>>>>>> [1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=571099
>>>>>>>>> [2] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=571104
>>>>>>>>> [3] http://git.xenomai.org/?p=xenomai-2.5.git;a=commitdiff;h=5bcd18f714f4cbeaaac0cc4a08e6c9f375aa3b77
>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>
>>>>
>>>
>>>
>>
>>
> 
> 


[-- Attachment #1.2: 0001-debian-fix-package-libxenomai1.patch --]
[-- Type: text/plain, Size: 2674 bytes --]

From a730e95eca146e944e376e0d28bcfa536dd6a062 Mon Sep 17 00:00:00 2001
From: Stefan Kisdaroczi <kisda@hispeed.ch>
Date: Wed, 3 Mar 2010 17:17:35 +0100
Subject: [PATCH] debian: fix package libxenomai1

---
 debian/changelog           |    9 +++++++++
 debian/libxenomai1.dirs    |    2 +-
 debian/libxenomai1.lintian |    2 +-
 debian/rules               |    6 +++---
 4 files changed, 14 insertions(+), 5 deletions(-)

diff --git a/debian/changelog b/debian/changelog
index 45736a4..6d2d3cf 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,12 @@
+xenomai (2.5.1-4) unstable; urgency=low
+
+  * Added patches by Stefan Kisdaroczi <kisda@hispeed.ch>:
+    - debian/copyright: Typo and email address (Closes: #571099)
+    - debian/control: ia64 support removed (Closes: #571104)
+    - debian/rules: Added dpkg-cross support
+
+ -- Roland Stigge <stigge@antcom.de>  Wed, 24 Feb 2010 22:20:10 +0100
+
 xenomai (2.5.1-3) unstable; urgency=low
 
   * xenomai-runtime: Replaced "xenomai-" prefixed executables with
diff --git a/debian/libxenomai1.dirs b/debian/libxenomai1.dirs
index d5bb34f..1737ea1 100644
--- a/debian/libxenomai1.dirs
+++ b/debian/libxenomai1.dirs
@@ -1,2 +1,2 @@
-etc/udev
+etc/udev/rules.d
 usr/share/lintian/overrides
diff --git a/debian/libxenomai1.lintian b/debian/libxenomai1.lintian
index 0f6a514..fe0da46 100644
--- a/debian/libxenomai1.lintian
+++ b/debian/libxenomai1.lintian
@@ -1,2 +1,2 @@
 # no contained shared library names refer to "xenomai", therefore own name
-libxenomai1: package-name-doesnt-match-sonames libanalogy1 libnative3 libpsos0 libpthread-rt1 librtai0 librtdk0 librtdm1 libuitron0 libvrtx0 libvxworks1
+libxenomai1: package-name-doesnt-match-sonames libanalogy1 libnative3 libpsos0 libpthread-rt1 librtai0 librtdk0 librtdm1 libuitron0 libvrtx0 libvxworks1 libxenomai0
diff --git a/debian/rules b/debian/rules
index fa93f62..a538800 100755
--- a/debian/rules
+++ b/debian/rules
@@ -87,7 +87,7 @@ clean:
 install: build
 	dh_testdir
 	dh_testroot
-	dh_clean -k
+	dh_prep
 	dh_installdirs
 	$(MAKE) install DESTDIR=$(CURDIR)/debian/tmp/
 	dh_install --sourcedir=$(CURDIR)/debian/tmp
@@ -108,8 +108,8 @@ binary-indep: build install
 	dh_testdir -i
 	dh_testroot -i
 	dh_installdocs -i -A CREDITS README.INSTALL TROUBLESHOOTING
-	dh_installchangelogs -i
 	dh_link -i
+	dh_installchangelogs -i
 	dh_strip -i
 	dh_compress -i -X.pdf
 	dh_fixperms -i
@@ -131,8 +131,8 @@ binary-arch: build install
 	dh_testroot -s
 	dh_installman -s
 	dh_installdocs -s -A CREDITS README.INSTALL TROUBLESHOOTING
-	dh_installchangelogs -s
 	dh_link -s
+	dh_installchangelogs -s
 	dh_strip -s
 	dh_compress -s
 	dh_fixperms -s
-- 
1.5.6.5


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

^ permalink raw reply related	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-03-03 16:54                         ` Stefan Kisdaroczi
@ 2010-03-03 17:21                           ` Philippe Gerum
  0 siblings, 0 replies; 38+ messages in thread
From: Philippe Gerum @ 2010-03-03 17:21 UTC (permalink / raw)
  To: Stefan Kisdaroczi; +Cc: xenomai

On Wed, 2010-03-03 at 17:54 +0100, Stefan Kisdaroczi wrote:
> Am 26.02.2010 15:07, schrieb Philippe Gerum:
> > On Fri, 2010-02-26 at 14:48 +0100, Stefan Kisdaroczi wrote:
> >> Am 26.02.2010 14:28, schrieb Philippe Gerum:
> >>> On Fri, 2010-02-26 at 14:13 +0100, Stefan Kisdaroczi wrote:
> >>>> Am 24.02.2010 14:13, schrieb Philippe Gerum:
> >>>>> On Wed, 2010-02-24 at 14:11 +0100, Philippe Gerum wrote:
> >>>>>> On Wed, 2010-02-24 at 14:06 +0100, Stefan Kisdaroczi wrote:
> >>>>>>> Hi Philippe,
> >>>>>>>
> >>>>>>> Am 23.02.2010 18:46, schrieb Philippe Gerum:
> >>>>>>>> On Tue, 2010-02-23 at 17:52 +0100, Stefan Kisdaroczi wrote:
> >>>>>>>>> Hi,
> >>>>>>>>>
> >>>>>>>>> Am 14.02.2010 10:38, schrieb Philippe Gerum:
> >>>>>>>>>>
> >>>>>>>>>> <snip>
> >>>>>>>>>> In the future, maybe we could simply provide a wrapper script accepting
> >>>>>>>>>> sub-commands, such as "xeno latency, xeno sigtest" etc, to be put
> >>>>>>>>>> into /usr/bin by distros, which would hide the actual location of those
> >>>>>>>>>> binaries?
> >>>>>>>>>>
> >>>>>>>>>> In any case, thanks for your work so far. We probably need to discuss
> >>>>>>>>>> the packaging issues on this list, so that we get both consistency and
> >>>>>>>>>> usability in the future.
> >>>>>>>>>>
> >>>>>>>>>> Gilles and Roland, if this is fine with you, I'll handle the liaison
> >>>>>>>>>> role with upstream packagers, so please CC me explicitly on those mails.
> >>>>>>>>>> We'll sort out this issue, it doesn't look that bad anyway.
> >>>>>>>>>
> >>>>>>>>> Roland added a xeno wrapper to the debian.org xenomai package 2.5.1-3.
> >>>>>>>>>
> >>>>>>>>> I synced now the debian/ directories from debian.org and xenomai.org:
> >>>>>>>>>  - For debian.org I sent patches to the Debian bugtracker [1] [2].
> >>>>>>>>>    Another patch for dpkg-cross support [3] I sent to Roland privately.
> >>>>>>>>>  - For xenomai.org I attached patches to this mail (against -2.5.git).
> >>>>>>>>>
> >>>>>>>>> If both parties apply the patches the debian directories are in sync,
> >>>>>>>>> except some minor differences in the debian/control file, see patch
> >>>>>>>>> do-not-commit-please.patch. I would like to keep these changes out so
> >>>>>>>>> that the xenomai.org packages are compatible with Debian 5.0 Lenny.
> >>>>>>>>> The debian.org packages are for Debian 6.0 Squeeze.
> >>>>>>>>>
> >>>>>>>>
> >>>>>>>> Merged into my queue (except the last one as mentioned). This will be
> >>>>>>>> pushed upstream to Gilles for 2.5.2. Thanks.
> >>>>>>>
> >>>>>>> I took a look at your branch for-upstream. Your commit
> >>>>>>>   scripts: add wrapper script to run standard Xenomai commands
> >>>>>>>   6e0574791f48cbf8b3421a68c5789254e7d084b7
> >>>>>>> adds the same wrapper as my patch 0005-debian-wrapper-script-usr-bin-xeno-to-call-executa.patch
> >>>>>>>   debian: wrapper script /usr/bin/xeno to call executables in /usr/lib/xenomai/
> >>>>>>>   fbe86cc50d3a65cd23e93d43adba4ed369fe70b1
> >>>>>>> Please revert the commit of my patch, we need another fix for debian/rules for "your" wrapper.
> >>>>>>>
> >>>>>>
> >>>>>> Ok, I thought your patch set was based on my tree, so I did not check
> >>>>>> thoroughly. I did not send any pull request to Gilles, so no harm done.
> >>>>>>
> >>>>>>> How do I call configure to install the wrapper in /usr/bin and
> >>>>>>> the programs like latency, switchtest etc. to /usr/lib/xenomai ?
> >>>>>>>
> >>>>>>
> >>>>>> We need some fixage in scripts/wrappers/Makefile.am to do that. I'll
> >>>>>> prepare this asap.
> >>>>>
> >>>>> scripts/Makefile.am...
> >>>>
> >>>> Hi Philippe,
> >>>>
> >>>> I just tried the --with-testdir switch. It worked, but i'm not really sure if
> >>>> this is the right track.
> >>>>
> >>>> Roland's packages install all binaries to /usr/lib/xenomai, except xeno and
> >>>> xeno-config. You state in your commit message more or less the same goal:
> >>>> "At some point, all remaining executables or scripts left under $prefix/bin should match
> >>>> xeno*, to further reduce the odds of causing name collisions."
> >>>>
> >>>> Using --with-testdir all tests (latency,switchtest,...) are now in /usr/lib/xenomai.
> >>>> To install the "utils" (rtcansend,insn_write,insn_write,cmd_read,...) to the same
> >>>> directory using --with-testdir sounds not obviously. You could add a second switch
> >>>> --with-utildir, but a second dir will not work for the xeno-wrapper-script.
> >>>>
> >>>
> >>> CAN and other utilities should definitely remain in $bindir. The fact
> >>> that they are not prefixed by xeno* is another thing; CAN utilities are
> >>> already prefixed, maybe Analogy ones should be named in a bit less
> >>> generic way, although they are not raising any conflict today. I wrote
> >>> that what's under $bindir "should" match xeno* when a risk of collision
> >>> exists, but there is no point in enforcing a stricter rule at this
> >>> point. In any case, I don't want to enforce a never-in-bindir rule for
> >>> all Xenomai binaries, we can still pick their names in a way that avoids
> >>> obvious issues.
> >>>
> >>> The real problem was about tests, for which using rather generic names
> >>> made sense. This is what that patch is for.
> >>
> >> ok, got the goal now, thanks for the explanation.
> >> But "xeno.in" needs a fix to use @XENO_TEST_DIR@, no?
> >>
> > 
> > Yes, I overlooked that. In fact, I think we may not even need the new
> > xeno wrapper at all, but we probably want to rewrite xeno-test to wrap
> > to what is in XENO_TEST_DIR now.
> > 
> > I would suggest to hold the changes to the debian/ area for now, until
> 
> Agree with holding back wrapper-changes, but please consider the attached patch
> with small fixes, thanks. (against rpm/for-upstream)
> 

Merged, thanks.



-- 
Philippe.




^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2010-02-26 13:35                 ` Stefan Kisdaroczi
@ 2010-05-02 16:01                   ` Roland Stigge
  0 siblings, 0 replies; 38+ messages in thread
From: Roland Stigge @ 2010-05-02 16:01 UTC (permalink / raw)
  To: Stefan Kisdaroczi; +Cc: xenomai

Hi Stefan,

Stefan Kisdaroczi wrote:
> I have attached a patch against debian xenomai version 2.5.1-4, changes:
>  - create group xenomai on install
>  - added a init-script which sets /sys/.../xenomai_gid if
>    /sys/.../xenomai_gid exists
>  - added a modprobe-script that adds the xenomai_gid parameter if the user
>    did call modprobe without xenomai_gid=
> 
> With this changes, all users which are member of the group xenomai are able
> to run xenomai apps, with xeno_nucleus builtin or as a module.

Looks fine. Just uploaded into Debian. (And renamed
/etc/modprobe.d/xenomai to xenomai.conf.)

Thanks!

bye,
  Roland


^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2008-02-26 13:20     ` Gilles Chanteperdrix
@ 2008-02-26 15:32       ` Roland Stigge
  0 siblings, 0 replies; 38+ messages in thread
From: Roland Stigge @ 2008-02-26 15:32 UTC (permalink / raw)
  To: Gilles Chanteperdrix; +Cc: xenomai

Gilles Chanteperdrix wrote:
>>  The binary package list changed to adjust to Debian conventions:
>>
>>  xenomai-runtime
>>  linux-patch-xenomai
>>  libxenomai1
>>  libxenomai-dev
>>  xenomai-doc
> 
> So, I need to install all these packages, then continue at make-kpkg ?

Similar to the Wiki description before. You certainly don't _need_
xenomai-doc for make-kpkg. Some of the other packages can be missing,
will need to check later. But I'm sure you already know it better:
xenomai-runtime is all the executables (except xeno-config),
libxenomai-dev is the installed header files, xeno-config and static
libs, libxenomai1 is the shared lib.

bye,
  Roland


^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2008-02-26 10:43   ` Roland Stigge
@ 2008-02-26 13:20     ` Gilles Chanteperdrix
  2008-02-26 15:32       ` Roland Stigge
  0 siblings, 1 reply; 38+ messages in thread
From: Gilles Chanteperdrix @ 2008-02-26 13:20 UTC (permalink / raw)
  To: Roland Stigge; +Cc: xenomai

On Tue, Feb 26, 2008 at 11:43 AM, Roland Stigge <stigge@domain.hid> wrote:
> Hi Gilles,
>
>
>  Gilles Chanteperdrix wrote:
>  > The current help on Xenomai Debian packages is:
>  > http://www.xenomai.org/index.php/Building_Debian_packages
>  >
>  > Is this documentation still valid when using pre-built packages ?
>
>  Thanks for the note!
>
>  The binary package list changed to adjust to Debian conventions:
>
>  xenomai-runtime
>  linux-patch-xenomai
>  libxenomai1
>  libxenomai-dev
>  xenomai-doc

So, I need to install all these packages, then continue at make-kpkg ?

-- 
                                               Gilles Chanteperdrix


^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2008-02-26 10:26 ` Gilles Chanteperdrix
@ 2008-02-26 10:43   ` Roland Stigge
  2008-02-26 13:20     ` Gilles Chanteperdrix
  0 siblings, 1 reply; 38+ messages in thread
From: Roland Stigge @ 2008-02-26 10:43 UTC (permalink / raw)
  To: Gilles Chanteperdrix; +Cc: xenomai

Hi Gilles,

Gilles Chanteperdrix wrote:
> The current help on Xenomai Debian packages is:
> http://www.xenomai.org/index.php/Building_Debian_packages
> 
> Is this documentation still valid when using pre-built packages ?

Thanks for the note!

The binary package list changed to adjust to Debian conventions:

xenomai-runtime
linux-patch-xenomai
libxenomai1
libxenomai-dev
xenomai-doc

bye,
  Roland


^ permalink raw reply	[flat|nested] 38+ messages in thread

* Re: [Xenomai-core] Xenomai in Debian
  2008-02-26  8:25 Roland Stigge
@ 2008-02-26 10:26 ` Gilles Chanteperdrix
  2008-02-26 10:43   ` Roland Stigge
  0 siblings, 1 reply; 38+ messages in thread
From: Gilles Chanteperdrix @ 2008-02-26 10:26 UTC (permalink / raw)
  To: Roland Stigge; +Cc: xenomai

On Tue, Feb 26, 2008 at 9:25 AM, Roland Stigge <stigge@domain.hid> wrote:
> Hi,
>
>  thanks to the previous Debian packaging work of Paul Corner (and of
>  course, all the Xenomai core developers), we now have xenomai officially
>  in Debian (i.e. Debian "sid", ready for the next Debian release "lenny").
>
>  For Debian related bugs, please use the Bug Tracking System at
>  bugs.debian.org. Everything else should stay the same as before, except
>  that all the Debian related files under debian/ are maintained inside
>  Debian (and can be backported to Xenomai "upstream", if desired in
>  Xenomai SVN).

The current help on Xenomai Debian packages is:
http://www.xenomai.org/index.php/Building_Debian_packages

Is this documentation still valid when using pre-built packages ?

-- 
                                               Gilles Chanteperdrix


^ permalink raw reply	[flat|nested] 38+ messages in thread

* [Xenomai-core] Xenomai in Debian
@ 2008-02-26  8:25 Roland Stigge
  2008-02-26 10:26 ` Gilles Chanteperdrix
  0 siblings, 1 reply; 38+ messages in thread
From: Roland Stigge @ 2008-02-26  8:25 UTC (permalink / raw)
  To: xenomai

Hi,

thanks to the previous Debian packaging work of Paul Corner (and of
course, all the Xenomai core developers), we now have xenomai officially
in Debian (i.e. Debian "sid", ready for the next Debian release "lenny").

For Debian related bugs, please use the Bug Tracking System at
bugs.debian.org. Everything else should stay the same as before, except
that all the Debian related files under debian/ are maintained inside
Debian (and can be backported to Xenomai "upstream", if desired in
Xenomai SVN).

Have fun,

Roland


^ permalink raw reply	[flat|nested] 38+ messages in thread

end of thread, other threads:[~2010-05-02 16:01 UTC | newest]

Thread overview: 38+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-02-07  9:01 [Xenomai-core] Xenomai in Debian Roland Stigge
2010-02-07 14:24 ` Gilles Chanteperdrix
2010-02-13 14:32 ` Gilles Chanteperdrix
2010-02-13 15:07   ` Roland Stigge
2010-02-13 15:25     ` Gilles Chanteperdrix
2010-02-13 18:30       ` Jan Kiszka
2010-02-13 18:37         ` Gilles Chanteperdrix
2010-02-13 18:48           ` Jan Kiszka
2010-02-13 21:15             ` Gilles Chanteperdrix
2010-02-14  9:38     ` Philippe Gerum
2010-02-23 16:52       ` Stefan Kisdaroczi
2010-02-23 17:46         ` Philippe Gerum
2010-02-24 13:06           ` Stefan Kisdaroczi
2010-02-24 13:11             ` Philippe Gerum
2010-02-24 13:13               ` Philippe Gerum
2010-02-26 13:13                 ` Stefan Kisdaroczi
2010-02-26 13:28                   ` Philippe Gerum
2010-02-26 13:48                     ` Stefan Kisdaroczi
2010-02-26 14:07                       ` Philippe Gerum
2010-03-03 16:54                         ` Stefan Kisdaroczi
2010-03-03 17:21                           ` Philippe Gerum
2010-02-25 12:46             ` Stefan Kisdaroczi
2010-02-25 13:44       ` Stefan Kisdaroczi
2010-02-25 13:49         ` Gilles Chanteperdrix
2010-02-25 13:59           ` Stefan Kisdaroczi
2010-02-25 14:29             ` Stefan Kisdaroczi
2010-02-25 17:18               ` Stefan Kisdaroczi
2010-02-26 13:35                 ` Stefan Kisdaroczi
2010-05-02 16:01                   ` Roland Stigge
     [not found]             ` <4B868CC1.6030103@domain.hid>
2010-02-25 18:31               ` Stefan Kisdaroczi
2010-02-25 13:59           ` Jan Kiszka
2010-03-01 17:04     ` Gilles Chanteperdrix
2010-03-01 21:45       ` Roland Stigge
  -- strict thread matches above, loose matches on Subject: below --
2008-02-26  8:25 Roland Stigge
2008-02-26 10:26 ` Gilles Chanteperdrix
2008-02-26 10:43   ` Roland Stigge
2008-02-26 13:20     ` Gilles Chanteperdrix
2008-02-26 15:32       ` Roland Stigge

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.