Which is it -pc- or -unknown-

[Yaakov Selkowitz]

On 2017-10-19 18:49, Steven Penny wrote:
> On Thu, 19 Oct 2017 17:00:12, cyg Simple wrote:
>> So says you!  The vendor portion has been agreed to be -pc- and it isn't
>> -unknown-, a patch then should be created for config.guess to match the
>> agreed upon vendor.  The config.guess script supplies the default to
>> configure for the build and host.  The fact that config.guess supplies
>> x86_64-unknown-cygwin is used by configure is the reason my assumptions
>> are correct.  If -pc- should be used then config.guess needs to change.
> 
> Let us bring some sanity to this discussion/argument. With this repository:
> 
>    git clone --depth 1 git://github.com/php/php-src
>    cd php-src
>    ./buildconf
> 
> Test 1:
> 
>    $ ./configure --host x86_64-pc-cygwin
>    checking build system type... x86_64-unknown-cygwin
>    checking host system type... x86_64-pc-cygwin
>    checking for x86_64-pc-cygwin-gcc... x86_64-pc-cygwin-gcc
>    checking whether the C compiler works... yes

Two things wrong with this:

1) If you specify --host, you need to specify --build as well.
2) If you're not cross-compiling or building a toolchain package, you
shouldn't be specifying either.

> Test 2:
> 
>    $ ./configure --host x86_64-unknown-cygwin
>    checking build system type... x86_64-unknown-cygwin
>    checking host system type... x86_64-unknown-cygwin
>    checking for x86_64-unknown-cygwin-gcc... no
>    checking for cc... cc
>    checking whether the C compiler works... yes
> 
> So yes, specifying "--host x86_64-unknown-cygwin" causes it to not find
> "x86_64-unknown-cygwin-gcc.exe", which makes sense because that doesnt
> exist.

True, when you *need* to specify --build/--host, then x86_64-pc-cygwin
should be used.  When not (as in this case), then omit them.

-- 
Yaakov

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: OpenPGP digital signature
URL: <http://cygwin.com/pipermail/cygwin/attachments/20171020/37c4390e/attachment.sig>