directory-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Emmanuel Lecharny <elecha...@iktek.com>
Subject Re: [jira] Commented: (DIRLDAP-31) Commons.Cli implemented in Bind
Date Tue, 01 Feb 2005 09:26:57 GMT
I gonna do it, as soon as every classes that extends Opts are rewrote.

Questions :
- { } should be on the same beginning line. It's OK, but what if the
original code does not respect this convention?

- I buy the m_ convention (Jess ?). I don't like a lot the l_ and a_ for
locals and paramters. Whatever, it's important that commons rules apply
to ALL the code. Formatter (Jalopy) are helpfull.
 
- can't we use a script that format the code against those standard
before doing a diff? A hate all those space or linefeed differences ...

- which is you IDE ? Eclipse? IDEA? 

Le mardi 01 février 2005 à 01:32 +0100, Alex Karasulu (JIRA) a écrit :
>      [ http://issues.apache.org/jira/browse/DIRLDAP-31?page=comments#action_58345 ]
>      
> Alex Karasulu commented on DIRLDAP-31:
> --------------------------------------
> 
> Very cool - if you have a (svn diff) patch you can attach it to this JIRA issue.  
> 
> Cheers,
> Alex
> 
> > Commons.Cli implemented in Bind
> > -------------------------------
> >
> >          Key: DIRLDAP-31
> >          URL: http://issues.apache.org/jira/browse/DIRLDAP-31
> >      Project: Directory LDAP
> >         Type: Improvement
> >   Components: Clients
> >     Versions: 0.8.0
> >     Reporter: Emmanuel Lecharny
> >     Assignee: Alex Karasulu
> >     Priority: Minor
> 
> >
> > I've implemented commons.cli in Bind, it works (also used log4j) :
> > FATAL org.apache.ldap.clients.LdapBind  - Error while parsing command line arguments
: Unrecognized option: -k
> > INFO org.apache.ldap.clients.LdapBind  - Command line arguments : -k 127.0.0.1 -p
389 -D cn=Manager,dc=iktek,dc=com -w secret -v 
> > usage: Bind
> >  -D,--binddn      bind DN
> >  -P,--protocol    procotol version (default: 3)
> >  -h,--host        LDAP server
> >  -n,--noupdate    show what would be done but don't actually update
> >  -p,--port        port on LDAP server
> >  -v,--verbose     run in verbose mode (diagnostics to standard output)
> >  -w,--passwd      bind passwd (for simple authentication)
> 



Mime
View raw message