[Freeipa-devel] [PATCH] Multi-valued field support

[Karl MacMillan]

On Thu, 2007-11-08 at 22:15 -0500, Rob Crittenden wrote:
> This patch enables multi-value field support for some attributes on the 
> edit pages (not the new pages). I punted on new because multiple values 
> on the cn makes doing auto-suggest trickier. I'll come back to it.
> 
> I improved error reporting in the GUI by printing the LDAP message as well.
> 
> I also wrote up a short document describing how multi-valued fields work 
> (so I don't forget later).
> 

I reviewed this as well as I could and it looks ok. Should we push the
config of which fields are multi-valued into ldap - whenever the ldap
config gets done - or should we just leave them hard coded?

> Also attached is a required SRPM of the new widget. This RPM just 
> re-packages an already packaged file. TG uses "egg" format for 
> distributing stuff (zip file basically). My RPM just moves data around.

So this will be a separate rpm that we distribute? Or is this general
enough that we should push it to Fedora? Or should we just push this
widget into the ipa-server rpm?

I'm thinking that we need to start version controlling all of the rpms
that we may have to ship. Anyone have thoughts on this? Same repo as the
rest of freeipa or a separate repo?

I'll push this as soon as I understand how this should work.

Karl