> -----Original Message-----
> From: Anders Widell [mailto:[email protected]]
> Sent: den 26 augusti 2014 13:13
> To: Zoran Milinkovic; [email protected]
> Cc: [email protected]
> Subject: Re: [devel] [PATCH 1 of 1] immtools: fix the use of extended names 
> in immcfg [#980]
> 
> Hi!
> 
> Two comments:
> 
> 1) I suppose this bug exists in all tools, so they other ones also need
> to be fixed?
> 
> 2) Maybe the tools should always support long DNs, so that you don't
> have to set the environment variable in the shell?
[Hans] why would you set env vars? The tool should instead have a new option

> 
> regards,
> / Anders Widell
> 
> On 08/25/2014 03:44 PM, Zoran Milinkovic wrote:
> >   osaf/tools/safimm/immcfg/imm_cfg.c |  1 +
> >   1 files changed, 1 insertions(+), 0 deletions(-)
> >
> >
> > The patch enables using osaf_extended_name_* functions in the correct way 
> > before the first saImmOmInitialize() is called.
> >
> > diff --git a/osaf/tools/safimm/immcfg/imm_cfg.c 
> > b/osaf/tools/safimm/immcfg/imm_cfg.c
> > --- a/osaf/tools/safimm/immcfg/imm_cfg.c
> > +++ b/osaf/tools/safimm/immcfg/imm_cfg.c
> > @@ -1539,5 +1539,6 @@ static int imm_operation(int argc, char
> >   }
> >
> >   int main(int argc, char *argv[]) {
> > +   osaf_extended_name_init();
> >     return imm_operation(argc, argv);
> >   }
> >
> > ------------------------------------------------------------------------------
> > Slashdot TV.
> > Video for Nerds.  Stuff that matters.
> > http://tv.slashdot.org/
> > _______________________________________________
> > Opensaf-devel mailing list
> > [email protected]
> > https://lists.sourceforge.net/lists/listinfo/opensaf-devel
> >
> >
> 
> 
> ------------------------------------------------------------------------------
> Slashdot TV.
> Video for Nerds.  Stuff that matters.
> http://tv.slashdot.org/
> _______________________________________________
> Opensaf-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/opensaf-devel

------------------------------------------------------------------------------
Slashdot TV.  
Video for Nerds.  Stuff that matters.
http://tv.slashdot.org/
_______________________________________________
Opensaf-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Reply via email to