[Freeipa-devel] [PATCH] Initial Radius Work

[Karl MacMillan]

On Mon, 2007-11-05 at 14:36 -0500, John Dennis wrote:
> Karl MacMillan wrote:
> > 2) What's is get_rpm_nvr_by_name used for? So far we have no explicit
> > rpm deps in the code - I would very much like to keep it that way.
> 
> When configuring a service it's used to determine the service version. 
> That way if there are any config file differences which are dependent on 
> the version the resulting config file generation can properly take 
> account of the version differences.
> 
> To the best of my knowledge we're only requiring (via the spec file), 
> certain packages be installed, or perhaps a minimum revision of a 
> specific package. The ipa-server-install should be able to validate what 
> its configuring and attempt to adapt to it or throw an error saying it 
> encountered an unsupported version of a required service.
> 

Let's deal with that when we come to it - but I don't think parsing rpm
versions is the correct approach. I'd rather look at the actual config
files are grab the version from the application itself.

> We aren't planning on supporting ipa-server installs on non-rpm based 
> systems, right?

Yes - I hope that freeipa will be packaged for other distros.

>  But even if we do the version query can use the 
> mechanism of the host or return None (e.g. I don't know the version).

Like I said - I'd rather just worry about this when the problem arises.
Can you please send a patch to remove that code for now?

Karl