Re: [devel] [PATCH 0 of 2] Review Request for amf #194

2013-11-13 Thread Hans Feldt
Then I think you should raise a separate ticket and handle it there. /Hans -Original Message- From: Nagendra Kumar [mailto:nagendr...@oracle.com] Sent: den 13 november 2013 10:02 To: Hans Feldt; Praveen Malviya Cc: opensaf-devel@lists.sourceforge.net Subject: RE: SV: RE: SV: RE:

Re: [devel] [PATCH 0 of 2] Review Request for amf #194

2013-11-13 Thread Hans Feldt
From looking at the code, amfnd should be able to handle two susi msgs for the same SU. The last one is just queued and handled once the first one is done. With this patch aren't you basically making all that logic obsolete? /Hans -Original Message- From: Nagendra Kumar

Re: [devel] [PATCH 0 of 2] Review Request for amf #194

2013-11-13 Thread Nagendra Kumar
No. We are only restricting operations at a time. -Nagu -Original Message- From: Hans Feldt [mailto:hans.fe...@ericsson.com] Sent: 13 November 2013 15:06 To: Nagendra Kumar; Praveen Malviya Cc: opensaf-devel@lists.sourceforge.net Subject: RE: SV: RE: [devel] [PATCH 0 of 2] Review Request

Re: [devel] [PATCH 0 of 2] Review Request for amf #194

2013-11-13 Thread Nagendra Kumar
Ack with additional patch sent on 11 Nov. Thanks -Nagu -Original Message- From: Hans Feldt [mailto:hans.fe...@ericsson.com] Sent: 08 November 2013 01:52 To: Praveen Malviya Cc: opensaf-devel@lists.sourceforge.net Subject: [devel] [PATCH 0 of 2] Review Request for amf #194 Summary: Add

[devel] Build problem on default branch

2013-11-13 Thread Anders Widell
Hi! I get the following build problem with the latest on the default branch: osafimmpbed-immpbe_daemon.o: In function `sqlite_prepare_ccb': /home/user/opensaf-staging/osaf/services/saf/immsv/immpbed/immpbe_daemon.cc:88: undefined reference to `pbeTransStarted()' osafimmpbed-immpbe_daemon.o: In

Re: [devel] [PATCH 0 of 2] Review Request for amf #194

2013-11-13 Thread Hans Feldt
Hi, I don't think adding an NPI comp (and CSI) in an PI SU works with/without this patch. The ticket does not state what type of SU it is, that is hidden in the attached campaign (it is NPI). Should we continue working on this ticket or create a new ticket? I will try push these patches

Re: [devel] [PATCH 0 of 2] Review Request for amf #194

2013-11-13 Thread Nagendra Kumar
Hi Hans, I think, you can commit this fix now and can create a separate ticket for the mentioned issue. Thanks -Nagu -Original Message- From: Hans Feldt [mailto:hans.fe...@ericsson.com] Sent: 14 November 2013 13:14 To: Nagendra Kumar; Praveen Malviya Cc: