IMO it should be in the feature. Hence +1 for the option #2. Thanks & Regards Danushka Fernando Senior Software Engineer WSO2 inc. http://wso2.com/ Mobile : +94716332729
On Mon, Aug 3, 2015 at 12:36 PM, Niranjan Karunanandham <[email protected]> wrote: > Hi all, > > In Carbon [1], it checks for the tenant-mgt.xml in the > <PRODUCT_HOME>/repository/conf directory, but in the p2.inf of the > tenant-mgt (carbon-multitenancy repo), it is copied to > <PRODUCT_HOME>/repository/conf/multitenancy. Therefore to fix this there > are two solutions: > > 1. Explicitly mention in the bin.xml of the product source to copy the > tenant-mgt.xml from the multitenancy to conf directory (or) > 2. Modify the p2.inf in tenant-mgt feature to copy the tenant-mgt.xml > to conf directly. > > Which one is correct? IMO, the latter fix will resolution the issue and > products need not mention this in their bin.xml. > > > [1] - > https://github.com/wso2/carbon4-kernel/blame/v4.4.1-RC2/core/org.wso2.carbon.user.core/src/main/java/org/wso2/carbon/user/core/config/TenantMgtXMLProcessor.java#L137 > [2] - > https://github.com/wso2/carbon-multitenancy/blob/master/features/tenant-mgt/org.wso2.carbon.tenant.common.server.feature/src/main/resources/p2.inf#L4 > > Regards, > Nira > -- > > *Niranjan Karunanandham* > Senior Software Engineer - WSO2 Inc. > WSO2 Inc.: http://www.wso2.com > > _______________________________________________ > Dev mailing list > [email protected] > http://wso2.org/cgi-bin/mailman/listinfo/dev > >
_______________________________________________ Dev mailing list [email protected] http://wso2.org/cgi-bin/mailman/listinfo/dev
