Inline too.

> -----Original Message-----
> From: Rees, Kevron [mailto:[email protected]]
> Sent: Thursday, November 21, 2013 3:42 AM
> To: Yoonsoo Kim
> Cc: [email protected]
> Subject: Re: [Dev] locale wet plugin
>
> inline
>
>
> On Tue, Nov 19, 2013 at 8:22 PM, Yoonsoo Kim <[email protected]>
> wrote:
> > Kevron,
> >
> > There is a separate kinda general privilege for setting system-wide key:
> > http://tizen.org/privilege/setting
>
> If I'm not mistaken (correct me if I'm wrong), but privileges are
> tightly correlated with the actual wrt-plugin.  Meaning, the privilege
> and so that provides that privilege are inseparable.  I cannot provide
> another plugin that provides this privilege if one already exists.
> Doing so might result in ambiguous behavior if it will even work at
> all.
>
I don't know why each privilege is tightly correlated with the actual 
wrt-plugin. I'll look into the design. Basically feature and privilege are 
different concepts. So there is no reason each privilege should be tightly 
correlated with a wrt-plugin, that is, a feature.

> > You can find out the related Tizen Native APIs at
> > https://developer.tizen.org/dev-guide/2.2.1/org.tizen.native.apireference/cl
> >
> assTizen_1_1System_1_1SettingInfo.html#a6c18036fb5b846859ae1e2bb0115ae4
> 6
> > https://developer.tizen.org/dev-
> guide/2.2.1/org.tizen.native.appprogramming/
> > html/guide/system/settings_info.htm
> >
>
> I don't see how the native API is helpful here since we are talking
> about web APIs.  This also looks like OSP which we cannot use in IVI
> anyway.
>
> > You can also find out the related Tizen Device APIs at
> > https://developer.tizen.org/dev-guide/2.2.1/org.tizen.web.device.apireferenc
> > e/tizen/systemsetting.html
> >
>
> There doesn't appear to be anything related to Locale here.
> SystemSettingType
> (https://developer.tizen.org/dev-
> guide/2.2.1/org.tizen.web.device.apireference/tizen/systemsetting.html#::Syst
> emSetting::SystemSettingType)
> seems too narrow to allow for a settings web application.
>
I was just saying that there was already a privilege for setting system-wide 
properties. You can use http://tizen.org/privilege/setting privilege for 
setting locale too. The privilege is a concept that application developers are 
facing and should be easily conceptually understandable. I can't understand 
why only the locale setting should have a separate privilege. As I mentioned 
before, the privilege is a part of API and should be reviewed in the context 
of API. You know the API review process has already been shared with the list. 
Still, it's not recommended to add a separate privilege for locale setting.

> > If you want to add another privilege, it should be reviewed in the context
> > of feature and API first.
> > And IVI-specific thing should not be merged into common wrt-security
> module.
> > For example, "vehicle" privilege.
> >
>
> What would you suggest?  We fork wrt-security in our profile?  We have
> ivi-specific plugins.  We cannot use these plugins without adding the
> correct <privilege...> section to our web app's wgt file.  The wgt
> file will not install unless we add the privilege to the wrt-security
> package.  I don't like how it works.  I find the wrt-security package
> redundant with SMACK and now PolicyKit in place.  But this is the only
> process by which we can enable our IVI-specific web APIs.
>
Now I understand what you mean. Wrt-security is configurable for 
profile-specific privileges but privilege manager is not configurable for each 
profile. That's the problem you are facing. Let me think this problem for a 
moment.

BR,
Yoonsoo

> BR,
> Kevron
>
> > One more thing: A patch should not be self-reviewed and self-merged. I've
> > just found that several patches have
> > been self-reviewed and self-merged even he is not the maintainer of
> > wrt-security module.
> > https://review.tizen.org/gerrit/#/c/11041
> > https://review.tizen.org/gerrit/#/c/11042/
> > This should not happen again.
> >
> > BR
> > ---
> > Yoonsoo Kim(ys71.kim AT samsung.com)
> > Principal SW Engineer/Software Architect
> > Mobile R&D Office
> > Samsung Electronics Co., Ltd.
> >
> >> -----Original Message-----
> >> From: [email protected] [mailto:[email protected]]
> On
> >> Behalf Of Rees, Kevron
> >> Sent: Wednesday, November 20, 2013 11:40 AM
> >> To: [email protected]
> >> Subject: [Dev] locale wet plugin
> >>
> >> I just submitted쟦ttps://review.tizen.org/gerrit/12605 for review. 쟕his
> > updates
> >> the wrt-security package and enables the locale privilege.
> >>
> >> A settings application written in html would need to be able to set the
> > system
> >> locale. 쟙e have such an application in ivi. 잸dditionally, web
> > applications
> >> need to be notified when a locale changes. 쟕his api allows both a
> > settings
> >> app to set the locale and applications to be notified when the lacale
> >> changes.
> >>
> >> This plugin can currently be found in the wrt-plugins-ivi package in the
> > ivi
> >> profile.
> >>
> >> Questions comments welcome.
> >>
> >> Kevron
> >
> >


_______________________________________________
Dev mailing list
[email protected]
https://lists.tizen.org/listinfo/dev

Reply via email to