On 11/01/2013 03:18 PM, Tomas Babej wrote: > On 10/31/2013 12:10 PM, Ana Krivokapic wrote: >> On 10/30/2013 04:18 PM, Tomas Babej wrote: >>> Hi, >>> >>> Adds which package to the requirements, since older distros do not have it >>> by default. >>> >>> Part of: https://fedorahosted.org/freeipa/ticket/3833 >>> >>> >>> >>> _______________________________________________ >>> Freeipa-devel mailing list >>> [email protected] >>> https://www.redhat.com/mailman/listinfo/freeipa-devel >> >> You can use the bash built-in command `command`, instead of `which`, to find >> out if a program exists: >> >> command -v cacertdir_rehash >> >> In other words, just replace `which` with `command -v`; there's no need to >> install any additional packages. >> -- >> Regards, >> >> Ana Krivokapic >> Associate Software Engineer >> FreeIPA team >> Red Hat Inc. >> >> >> _______________________________________________ >> Freeipa-devel mailing list >> [email protected] >> https://www.redhat.com/mailman/listinfo/freeipa-devel > > Thanks! > > Updated patch attached. > > > > -- > Tomas Babej > Associate Software Engeneer | Red Hat | Identity Management > RHCE | Brno Site | IRC: tbabej | freeipa.org
Looks good! Please just amend the commit message to reflect the new content of the patch. -- Regards, Ana Krivokapic Associate Software Engineer FreeIPA team Red Hat Inc.
_______________________________________________ Freeipa-devel mailing list [email protected] https://www.redhat.com/mailman/listinfo/freeipa-devel
