Re: [Sblim-devel] Pegasus version that works with SBLIM

[Viktor M. <MIH...@de...>]

Hi all,

I felt that it would be a good idea to shove this thread back into the
mailing list, so that everyone can benefit from the discussions. Please use
this message to carry on with this thread should it be necessary. Thanks a
bunch.

Mit freundlichen Grüßen / Kind Regards
   Viktor Mihajlovski

Linux Technology Center

   IBM Laboratory Böblingen, Germany
   Phone +49-7031-16-2560
   E-Mail mih...@de...
----- Forwarded by Viktor Mihajlovski/Germany/IBM on 27.02.02 10:13 -----
                                                                                                                       
                    Markus Müller                                                                                      
                    <sedgewick_de@yaho       To:     fra...@ho...                                         
                    o.de>                    cc:     Viktor Mihajlovski/Germany/IBM@IBMDE, Adrian                      
                    Sent by: Markus           Schuur/Germany/IBM@IBMDE, Ulrich Kurz/Germany/IBM@IBMDE, Heidi           
                    Hildebrand/Germany        Neumann/Germany/IBM@IBMDE                                                
                    /IBM@IBMDE               Subject:     Re: [Sblim-devel] Pegasus version that works with SBLIM      
                                                                                                                       
                                                                                                                       
                    27.02.02 10:48                                                                                     
                                                                                                                       
                                                                                                                       




Frank Olsen wrote:
>
> >From: Markus Müller <sed...@ya...>
> >To: Frank Olsen <fra...@ho...>
> >Subject: Re: [Sblim-devel] Pegasus version that works with SBLIM
> >Date: Tue, 26 Feb 2002 17:12:52 +0100
> >
> >Hi Frank,
> >
> >I've just verified that the NPI can be compiled without problems,
> >provided the paths to pegasus libraries and included files are set
> >correctly in the makefile (phew).
> >
>
> Hi Markus,
>
> Hopefully I didn't confuse you -- I did manage to compile NPI itself. My
> problem arised when starting to compile the other modules (base, fsvol,
> ...). The problems I referred to concerns the `base' module (written by
> Heidi Neumann and not updated very recently). I just thought that maybe
> other modules than `npi' were not up-to-date with the latest Pegasus CVS
> since there have been quite a few changes in the Pegasus CVS recently...
>
> >Can you make sure that you include NPIProvider.h as the very first file.
> >NPIProvider.h includes Pegasus/Common/Config.h which in turn sets the
> >variable _GNU_SOURCE before include the system file <features.h> . If
> >another include file comes first, features.h is also included, but
> >without the environment variable set correctly. The result is that
> >Unix98 extensions, for example pthread_crit_*, pthread_rwlock_* are left
> >undefined and the occurrence in IPC.h, the threading wrapper, gives an
> >error.
>
> So, this may solve the problem in the `base' module I hope. I'll have a
> look.
>
> The specific file causing problems was
> `base/pegasus/PegLinux_UnitaryComputerSystem.cpp'. There _is_ another
> include:
>
> #include <iostream>
> #include "Pegasus/Common/Config.h"
> #include "NPIProvider.h"
>
> Hm, maybe there's a problem in that Config.h will be included twice?
>
> Anyway, there seems to be issues with modules other than `npi'.
>
> Thanks again for your help!
>
> Regards,
> Frank


Hi Frank,

what's good for the Base modules, i.e. NPI for Pegasus, is also good for
all "applications" of the NPI. There is no problem with Config.h
included twice, because an #define PEGASUS_CONFIG_h prevents duplicate
inclusion. The real problem here is that iostream is included first. If
you move #include "Pegasus/Common/Config.h" to the front you should be
fine.

Regards

Markus

_________________________________________________________
Do You Yahoo!?
Get your free @yahoo.com address at http://mail.yahoo.com