[rfc][icedtea-web] fix for RH947647, following the XDG basedir specification

Andrew Azores aazores at redhat.com
Mon Jul 15 11:10:18 PDT 2013


On 07/13/2013 07:21 AM, Jiri Vanek wrote:
> On 07/12/2013 05:25 PM, Jiri Vanek wrote:
>> On 07/12/2013 05:09 PM, Andrew Azores wrote:
>>> On 07/12/2013 10:04 AM, Jiri Vanek wrote:
>>>> Attempt two :)
>>>>
>>>> Well this test nearly cost me my sanity (not that there is to much 
>>>> left but...) but here it is
>>>> again.
>>>>
>>>> As Omair have signed himself off this review (after saying that by 
>>>> concept it is ok) this patch is
>>>> looking for volunteer.
>>>>
>>>> J.
>>>
>>> All looks good to me other than a couple style nitpicks, but nothing 
>>> important. Just things like
>>> missing spaces around + when concatenating strings, hardly worth 
>>> changing :) I think this looks
>>> pretty good as it is.
>>>
>>> I tried to apply the patches in my VM with latest ITW - the tests 
>>> patch succeeded (as expected since
>>> it's just file additions), but the other patch just about entirely 
>>> failed :( I used hg patch -f
>>> --no-commit for this. I tried to use the "patch" command but it 
>>> complained about a malformed line,
>>> so maybe I was just not using it correctly. Not sure what that's 
>>> about, hopefully it's just a
>>> problem on my end though and won't result in some merge conflict!
>>>
>>> Andrew A
>>
>> aargh, yah, I have not updated underlying repo.
>>
>> I will post new patch on top of head and with proper reformatting.
>>
>> Tahnx for check!
>>
>> J.
>>
>
> here we go!
> (note changelog and neews added to patch)
>
> Thanx, J.
>
This one looks good to me!

Andrew A



More information about the distro-pkg-dev mailing list