Re: Revert gcc r227962

2016-06-09 Thread Jeff Law

On 05/29/2016 08:33 AM, JonY wrote:

On 5/23/2016 16:56, JonY wrote:

On 5/20/2016 06:36, JonY wrote:

On 5/20/2016 02:11, Jeff Law wrote:

So if we make this change (revert 227962), my understanding is that
cygwin bootstraps will fail because they won't find kernel32 and perhaps
other libraries.

Jeff



I'll need to double check with trunk but gcc-5.3.0 built OK without it.
The other alternative is to search /usr/lib before w32api.




yep it reached stage 3 but failed from another error building target
libraries (libcilkrts), meaning it was able to find the w32api libraries
even with this patch reverted.



Has it been reverted? I managed to bootstrap after disabling the failing
libraries.

I've reverted the patch on the trunk.  THanks for your patience.

jeff>



Re: Revert gcc r227962

2016-05-29 Thread JonY
On 5/23/2016 16:56, JonY wrote:
> On 5/20/2016 06:36, JonY wrote:
>> On 5/20/2016 02:11, Jeff Law wrote:
>>> So if we make this change (revert 227962), my understanding is that
>>> cygwin bootstraps will fail because they won't find kernel32 and perhaps
>>> other libraries.
>>>
>>> Jeff
>>>
>>
>> I'll need to double check with trunk but gcc-5.3.0 built OK without it.
>> The other alternative is to search /usr/lib before w32api.
>>
>>
> 
> yep it reached stage 3 but failed from another error building target
> libraries (libcilkrts), meaning it was able to find the w32api libraries
> even with this patch reverted.
> 

Has it been reverted? I managed to bootstrap after disabling the failing
libraries.




signature.asc
Description: OpenPGP digital signature


Re: Revert gcc r227962

2016-05-23 Thread JonY
On 5/20/2016 06:36, JonY wrote:
> On 5/20/2016 02:11, Jeff Law wrote:
>> So if we make this change (revert 227962), my understanding is that
>> cygwin bootstraps will fail because they won't find kernel32 and perhaps
>> other libraries.
>>
>> Jeff
>>
> 
> I'll need to double check with trunk but gcc-5.3.0 built OK without it.
> The other alternative is to search /usr/lib before w32api.
> 
> 

yep it reached stage 3 but failed from another error building target
libraries (libcilkrts), meaning it was able to find the w32api libraries
even with this patch reverted.




signature.asc
Description: OpenPGP digital signature


Re: Revert gcc r227962

2016-05-19 Thread JonY
On 5/20/2016 02:11, Jeff Law wrote:
> So if we make this change (revert 227962), my understanding is that
> cygwin bootstraps will fail because they won't find kernel32 and perhaps
> other libraries.
> 
> Jeff
> 

I'll need to double check with trunk but gcc-5.3.0 built OK without it.
The other alternative is to search /usr/lib before w32api.




signature.asc
Description: OpenPGP digital signature


Re: Revert gcc r227962

2016-05-19 Thread Jeff Law

On 02/27/2016 03:39 AM, JonY wrote:

On 2/27/2016 05:26, Jeff Law wrote:

On 02/26/2016 04:04 AM, JonY wrote:

Hi,

I've submitted a patch that was committed as r227962, it causes some
unintended side effects (namely libuuid on Cygwin). Can someone please
revert?

Kai still needs some time to setup his gcc development environment.

We'd need to have a better sense of why this is causing problems.  If
Kai could chime in with a description of the problem it would be helpful.



Normally, /usr/lib is searched BEFORE /usr/lib/w32api. The patch caused
w32api to be searched before /usr/lib.

libuuid.a is known to exist in both directories, with the *nix version
in /usr/lib and the import library for Windows in w32api. The w32api
copy is completely unrelated to the *nix version. This break cygwin apps
that expect to link against *nix libuuid.
So if we make this change (revert 227962), my understanding is that 
cygwin bootstraps will fail because they won't find kernel32 and perhaps 
other libraries.


Jeff


Re: Revert gcc r227962

2016-02-27 Thread JonY
On 2/27/2016 05:26, Jeff Law wrote:
> On 02/26/2016 04:04 AM, JonY wrote:
>> Hi,
>>
>> I've submitted a patch that was committed as r227962, it causes some
>> unintended side effects (namely libuuid on Cygwin). Can someone please
>> revert?
>>
>> Kai still needs some time to setup his gcc development environment.
> We'd need to have a better sense of why this is causing problems.  If
> Kai could chime in with a description of the problem it would be helpful.
> 

Normally, /usr/lib is searched BEFORE /usr/lib/w32api. The patch caused
w32api to be searched before /usr/lib.

libuuid.a is known to exist in both directories, with the *nix version
in /usr/lib and the import library for Windows in w32api. The w32api
copy is completely unrelated to the *nix version. This break cygwin apps
that expect to link against *nix libuuid.




signature.asc
Description: OpenPGP digital signature


Re: Revert gcc r227962

2016-02-26 Thread Jeff Law

On 02/26/2016 04:04 AM, JonY wrote:

Hi,

I've submitted a patch that was committed as r227962, it causes some
unintended side effects (namely libuuid on Cygwin). Can someone please
revert?

Kai still needs some time to setup his gcc development environment.
We'd need to have a better sense of why this is causing problems.  If 
Kai could chime in with a description of the problem it would be helpful.


jeff