Il 07/11/2014 12:51, tarungoyal ha scritto:
Hi Riccardo,

Thanks for the patch. I think a little modification is required.

$(if $(ENABLE_HEADLESS),, \ must be modified to: $(if ! $(ENABLE_HEADLESS),,
\

Nope, the original patch was fine :)
The if syntax is: conditional,true branch,false branch

Also after making this change I got this error:
Test name: testFdo37606Copy::Import
equality assertion failed
- Expected: 2
- Actual  : 1

Failures !!!
Run: 20   Failure total: 1   Failures: 1   Errors: 0

I'd say that it's fine for us to ignore that since it's a c&p bug where
we well maybe not implementing all the interfaces.

thanks,
riccardo
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to