[Libreoffice] [PUSHED] [PATCH] Part of FDO #42454 - Remove code associated with unused icons

Noel Power nopower at suse.com
Tue Nov 15 09:23:04 PST 2011


On 15/11/11 16:55, Julien Chaffraix wrote:
[...]
> I tried removing SV_ICON_ID_HTML_DOCUMENT because I also did not see
> it during my grepping. However I would get some errors during make
> dev-install about the missing icons (I don't have the error with me
> sorry).
I think this was some side-affect of some stale build products from not 
doing a clean build ( install works fine for me )
[...]
>> I have restarted my build from clean, assuming the build completes without
>> errors, unless I get some sort of "stop the world will end" I will commit
>> the patches again.
> Don't try it too hard. It shouldn't have broken anything. If it does,
> it means I should have done more research and testing on my side!
builds, installs,  starts & smoketest passes :-) so... pushed again

btw, I see you have stated license information in the past (didn't see 
it for this one ), if you haven't already could you give some sort of 
statement like "unless stated otherwise all my patch contribution to 
libreoffice are under LGPLv3+ / MPL."  Perhaps you could update 
http://wiki.documentfoundation.org/Development/Developers similarly ?

anyway, thanks again for the effort and the nice patch, we appreciate it !!

Noel


More information about the LibreOffice mailing list