Hi Norbert,

On Fri, Jul 22, 2011 at 7:18 PM, Norbert Thiebaud <nthieb...@gmail.com> wrote:
> Ok it is easier to discuss based on actual code... so:
>
> the attaches patch, allow for --with-mac-sdk=10.x to be specified
> the default is the lowest SDK available in /Developer/SDKs

regarding the default see the other posts.

just regarding
-MACDEVSDK*=/Developer/SDKs/MacOSX10.4u.sdk
+MACDEVSDK*=$(MACSDK_PATH)

time to get rid of that from the external modules. That is basically a
hack for Hamburg's non-configure based environment.

(same for the .IF "$(SYSBASE)"!="" checks in various makefiles)

get rid of that special treatment in the individual modules' makefiles.

And probably a switch --with-macosx-deployment-target would be more
appropriate/consistent

> The patch does not check that your CC anc CCX make sens based on the
> SDK selected (ToDO list)

Well, but it should/must....
Either by simple version check as it is currently done, or by a
compile/link test.
Configure must not pass when the combination of SDK and compiler is
not suitable.

ciao
Christain
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to