[Pkg-e-devel] Bug#535768: closed by Albin Tonnerre <lutin at debian.org> (Bug#535768: fixed in evas 0.9.9.49898-1)

[Mark-Willem Jansen]

Hi,

Just to be clear I am not a developer or maintainer of this package. Hence, not a 
specialist on the working of the code.

>
> Le 05/05/11 13:47, Mark-Willem Jansen a écrit :
>
> > Hi,
>
> Hi,
>
> > Maybe you could rewrite the patch, so the pre-compiler checks the existence
> > of SA_SIGINFO and selects the appropriate code. For example.
>
> [snip]
>
> > This way the patch will not effect the architects that have SA_SIGINFO defined.
>
> I can, but wouldn't that make the code more complex than it needs to be,
> considering that the features provided by SA_SIGINFO aren't used at all?
> I usually try to avoid adding a lot of #ifdef/#endif when it's not needed.

If you are 100% positive the features are not used and the behaviour of the
code does not change, then go with your option. I was just suggesting the 
#ifdef/#endif option as it has no influence on the behaviour of the code on other 
architects.

Maybe you could contact the evas developers, to see what they thing of the patch.
On the enlightenment webpage they provide a bug tracker, a mailing list and they
have some IRC channels on freenode (#e, #edevelop).

>
> Nevertheless, if you think it's better this way, then I'll do it. I'll
> have to check that no other portability problems have appeared in
> libevas since last July, though.

Yes please do check. Evas is holding up the rest of EFL and e17 on hurd-i386.

>
> > Mark-Willem Jansen
>
> Thanks for the answer!
>
> --
> Manuel Menal

Thanks for contributing to the evas

--
Mark-Willem Jansen