Re: [Pki-devel] [PATCH] 0036 Use SimpleProperties to handle raw profile format

On 5/19/2015 8:20 AM, Fraser Tweedale wrote: >This patch fixes an issue when handling the "raw" (property-list) >profile format. There seems to be a dependency issue. The SimpleProperties belongs to the server package, and the pki CLI belongs to the tools package which may exist on a client machine without the server packages. Does build work? I think there are two ways to resolve this: 1. Move SimpleProperties into the common package (e.g. com.netscape.certsrv.base) and undo the new dependencies added for cmscore. 2. We keep two profile formats: * SimpleProperties for profile configs stored in instance folder * Properties for profile configs accessed via REST The profile configs stored in LDAP can use either format. The REST service/profile subsystem will act as a translator between the two formats. With option #2 a problem with might happen if we retrieve a profile config via REST then store it directly in the instance folder or in LDAP, or vice versa. If we stick to one mechanism we wouldn't see a problem. Considering we're going to remove the profile configs in the instance folder eventually (and possibly changing the LDAP format to be platform agnostic) there would be less risk of intermixing the two formats. But if it's still considered a risk anyway then we should go with option #1. -- Endi S. Dewata