> On Dec 5, 2015, at 6:35 AM, Ryan Schmidt <[email protected]> wrote:
> 
> 
> On Dec 4, 2015, at 9:15 PM, Mark Brethen wrote:
> 
>> I have source with no configure and this snippet pertaining to Mac in the 
>> makefile:
>> 
>>              ifeq "$(uname)" "Darwin"
>>                      # (2) Mac OS X, using GCC as the C compiler
>>                      export ARCHOS    = x86_LINUX
>>                      export OCAMLDIR  = /usr/local/bin
>>                      export OBJEXT    = o
>>                      export CASE_FILE = case_ins_mac
>>                      export GLOB      = glob_unix
>>                      # Very simplistic arch test
>>                      ifeq "$(shell uname -m)" "x86_64"
>>                              export WEIDU_ARCH = amd64
>>                              export ARCH_FILE = arch_osx
>>                      else ifeq "$(shell uname -m)" "i386"
>>                              export WEIDU_ARCH = x86
>>                              export ARCH_FILE = arch_osx
>>                      else
>>                              export WEIDU_ARCH = mac # PPC
>>                              export ARCH_FILE = arch_legacy_osx
>>                      endif
>>                      export WEIDU_OS = osx
>>                      export USEOPT = .opt
>>                      PROJECT_CMODULES = nix
>>                      export TKDIR = obj-tk
>> 
>> 
>> OCAMLDIR needs to be set, but what about the rest?
> 
> Well the "very simplistic arch test" is of course wrong. "uname -m" tells you 
> the arch of the kernel, which is not necessarily the arch of userland 
> software. In any case, we don't want build systems deciding for themselves 
> what arch(s) to user; they shall obey the arch(s) MacPorts tells them to use.

I see this quite often with multi-platform source. What should be done?

> Also, I assume "arch_osx" and "arch_legacy_osx" are filenames; I wonder what 
> the difference in the contents of those files is.
> 

Just one line:

--- arch_osx.ml 2015-08-09 09:37:56.000000000 -0500
+++ arch_legacy_osx.ml  2015-08-09 09:37:56.000000000 -0500
@@ -48,7 +48,7 @@
 
 let create_process_env = Unix.create_process_env
 
-let biff_path_separator = "\\\\" (* backslash_to_slash hopefully sorts this 
out *)
+let biff_path_separator = ":"
 
 (* On OSX, TOB baldur.ini has:
    [Alias]
@@ -59,7 +59,6 @@
    CD3:=:CD3:
    CD4:=:CD4:
  *)
-
 let cd_regexp = Str.regexp "\\(CD[0-9]\\)"
 (* doesn't work:
    let cd_regexp = Str.regexp "^[CH]D[0-9]+.*=\\([^\r\n]*\\)"  *)


Mark




_______________________________________________
macports-dev mailing list
[email protected]
https://lists.macosforge.org/mailman/listinfo/macports-dev

Reply via email to