[meego-packaging] [meego-commits] 13835: Changes to devel:multimedia/pulseaudio

[Marko Saukko]

On 02/25/2011 08:13 PM, Kalle Lampila wrote:
> On Fri, Feb 18, 2011 at 17:48, Clark, Joel<joel.clark at intel.com>  wrote:
>> Regardless of the architecture diagram, resource policy should not be included until it does not break other common core components.
>
> I have request add requirement to make sure that there is configs when
> pulseaudio-policy-enforcement is installed.
>
> regards
> Kalle

I'm a bit against this "Require: policy-settings" line in the spec. My 
reasoning:

- With this line we require every vertical and every adaptation to have 
package that provides policy-settings already in the very beginning of 
the project. This will make the adaptation harder and will cause the 
need for people of making empty policy-settings-adaptation-x-dummy 
packages that do not actually provide any files. This is pretty much the 
same as not having the requirement in the first place.

- Also after adding multiple packages that provides the same thing and 
having this as a part of "Require:" field, requires that when user 
installs the pulseaudio-policy-enforcement he need also say which one of 
the policy-settings packages he wants to install. Because none of the 
tools can't make the decision between provides packages automatically, 
we need to have the policy-settings-[adaptation|vertical] package in the 
package group or the kickstart anyway to get the image build. So yet 
again the forcing of the settings does not do much good.

- Also there is a case that when there is only one policy-settings 
packages it will be installed automatically even if the settings are not 
meant for the specific platform (settings are noarch packages anyway 
right?). However, this condition will not most probably stay there long.

- Lastly if the settings are missing it really should not crash the 
system but instead disable the policies or inform user gracefully in my 
opinion.

Regards,
Marko

>
>> regards
>> Joel
>>
>>
>> -----Original Message-----
>> From: meego-packaging-bounces at meego.com [mailto:meego-packaging-bounces at meego.com] On Behalf Of Kalle Lampila
>> Sent: Friday, February 18, 2011 4:51 AM
>> To: Zhang, Vivian
>> Cc: meego-packaging at meego.com; meego-commits at meego.com
>> Subject: Re: [meego-packaging] [meego-commits] 13835: Changes to devel:multimedia/pulseaudio
>>
>> On Fri, Feb 18, 2011 at 10:24, Zhang, Vivian<vivian.zhang at intel.com>  wrote:
>>> Will the resource policy be used in Netbook/Tablet/IVI? If not, auto loading module-policy-enforcement is not necessary.
>>> And also the module is loaded failed for missing policy.conf on netbook/tablet, which caused the PA crash, I will hold on accepting the modifications before root causing PA crash issue.
>>>
>>> Thanks
>>> vivian
>>
>> Hi,
>>
>> Resource policy is part of common core architecture. So It should be
>> every image.
>>
>> http://meego.com/developers/meego-architecture/meego-architecture-domain-view
>>
>> --
>> Kalle Lampila
>> _______________________________________________
>> MeeGo-packaging mailing list
>> MeeGo-packaging at meego.com
>> http://lists.meego.com/listinfo/meego-packaging
>>
> _______________________________________________
> MeeGo-packaging mailing list
> MeeGo-packaging at meego.com
> http://lists.meego.com/listinfo/meego-packaging