Re: HEAD amd64 seems unhappy

2018-02-04 Thread Michael Tuexen 
> On 4. Feb 2018, at 21:40, Dimitry Andric  wrote:
> 
> On 4 Feb 2018, at 20:59, Michael Tuexen  wrote:
>> 
>>> On 4. Feb 2018, at 11:24, Dimitry Andric  wrote:
>>> On 4 Feb 2018, at 03:38, Josh Paetzel  wrote:
 
 --- all_subdir_lib/libngatm ---
 /usr/src/sys/contrib/ngatm/netnatm/api/cc_port.c:71:28: error: result of 
 comparison 'u_int' (aka 'unsigned int') > 4294967295 is always fal
 se [-Werror,-Wtautological-type-limit-compare]
if (portno == 0 || portno > 0x)
   ~~ ^ ~~
 1 error generated.
 *** [cc_port.o] Error code 1
 
 This code hasn't changed in ages so I can only assume it's clang or such.
>>> 
>>> This is because of r328816, which requires you to update your toolchain
>>> to at least r328381.  E.g. please run either "make buildworld" or "make
>>> kernel-toolchain" before building your kernel.
>> I get this error when doing a buildworld... Using r328861.
> 
> This should now be fixed by r328863.  The MK_SYSTEM_COMPILER logic did
> not build a cross-tools toolchain, so you were still getting warnings
> from the (old) compiler in your base system.
I can confirm that it works now.

Thanks for the quick fix!

Best regards
Michael
> 
> -Dimitry
> 

___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Re: HEAD amd64 seems unhappy

2018-02-04 Thread Dimitry Andric 
On 4 Feb 2018, at 20:59, Michael Tuexen  wrote:
> 
>> On 4. Feb 2018, at 11:24, Dimitry Andric  wrote:
>> On 4 Feb 2018, at 03:38, Josh Paetzel  wrote:
>>> 
>>> --- all_subdir_lib/libngatm ---
>>> /usr/src/sys/contrib/ngatm/netnatm/api/cc_port.c:71:28: error: result of 
>>> comparison 'u_int' (aka 'unsigned int') > 4294967295 is always fal
>>> se [-Werror,-Wtautological-type-limit-compare]
>>> if (portno == 0 || portno > 0x)
>>>~~ ^ ~~
>>> 1 error generated.
>>> *** [cc_port.o] Error code 1
>>> 
>>> This code hasn't changed in ages so I can only assume it's clang or such.
>> 
>> This is because of r328816, which requires you to update your toolchain
>> to at least r328381.  E.g. please run either "make buildworld" or "make
>> kernel-toolchain" before building your kernel.
> I get this error when doing a buildworld... Using r328861.

This should now be fixed by r328863.  The MK_SYSTEM_COMPILER logic did
not build a cross-tools toolchain, so you were still getting warnings
from the (old) compiler in your base system.

-Dimitry



signature.asc
Description: Message signed with OpenPGP

Re: HEAD amd64 seems unhappy

2018-02-04 Thread Michael Tuexen 
> On 4. Feb 2018, at 11:24, Dimitry Andric  wrote:
> 
> On 4 Feb 2018, at 03:38, Josh Paetzel  wrote:
>> 
>> --- all_subdir_lib/libngatm ---
>> /usr/src/sys/contrib/ngatm/netnatm/api/cc_port.c:71:28: error: result of 
>> comparison 'u_int' (aka 'unsigned int') > 4294967295 is always fal
>> se [-Werror,-Wtautological-type-limit-compare]
>>  if (portno == 0 || portno > 0x)
>> ~~ ^ ~~
>> 1 error generated.
>> *** [cc_port.o] Error code 1
>> 
>> This code hasn't changed in ages so I can only assume it's clang or such.
> 
> This is because of r328816, which requires you to update your toolchain
> to at least r328381.  E.g. please run either "make buildworld" or "make
> kernel-toolchain" before building your kernel.
I get this error when doing a buildworld... Using r328861.

Best regards
Michael
> 
> -Dimitry
> 

___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Re: HEAD amd64 seems unhappy

2018-02-04 Thread Stefan Hagen 

Dimitry Andric wrote:
On 4 Feb 2018, at 03:38, Josh Paetzel  wrote:


--- all_subdir_lib/libngatm ---
/usr/src/sys/contrib/ngatm/netnatm/api/cc_port.c:71:28: error: result of 
comparison 'u_int' (aka 'unsigned int') > 4294967295 is always fal
se [-Werror,-Wtautological-type-limit-compare]
   if (portno == 0 || portno > 0x)
  ~~ ^ ~~
1 error generated.
*** [cc_port.o] Error code 1

This code hasn't changed in ages so I can only assume it's clang or such.


This is because of r328816, which requires you to update your toolchain
to at least r328381.  E.g. please run either "make buildworld" or "make
kernel-toolchain" before building your kernel.


I get the same error. It actually comes up at "make buildworld".
Running "make kernel-toolchain" before did not help. I'm on r328860.

Best Regards,
Stefan
___
freebsd-current@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscr...@freebsd.org"

Re: HEAD amd64 seems unhappy

2018-02-04 Thread Dimitry Andric 
On 4 Feb 2018, at 03:38, Josh Paetzel  wrote:
> 
> --- all_subdir_lib/libngatm ---
> /usr/src/sys/contrib/ngatm/netnatm/api/cc_port.c:71:28: error: result of 
> comparison 'u_int' (aka 'unsigned int') > 4294967295 is always fal
> se [-Werror,-Wtautological-type-limit-compare]
>if (portno == 0 || portno > 0x)
>   ~~ ^ ~~
> 1 error generated.
> *** [cc_port.o] Error code 1
> 
> This code hasn't changed in ages so I can only assume it's clang or such.

This is because of r328816, which requires you to update your toolchain
to at least r328381.  E.g. please run either "make buildworld" or "make
kernel-toolchain" before building your kernel.

-Dimitry



signature.asc
Description: Message signed with OpenPGP

Re: HEAD amd64 seems unhappy

2018-02-04 Thread David Wolfskill 
On Sat, Feb 03, 2018 at 08:38:22PM -0600, Josh Paetzel wrote:
> --- all_subdir_lib/libngatm ---
> /usr/src/sys/contrib/ngatm/netnatm/api/cc_port.c:71:28: error: result of 
> comparison 'u_int' (aka 'unsigned int') > 4294967295 is always fal
> se [-Werror,-Wtautological-type-limit-compare]
> if (portno == 0 || portno > 0x)
>~~ ^ ~~
> 1 error generated.
> *** [cc_port.o] Error code 1
> 
> This code hasn't changed in ages so I can only assume it's clang or such.
> 

My update of head/amd64 from r328830 to r328838 was rather quick
and quite painless this morning (00:05:15 elapsed time... on my
laptop; build machine was < 00:05:00, as I didn't need to re-authenticate
to sudo for the shutdown).

So: I didn't see the issue.

Peace,
david
-- 
David H. Wolfskill  da...@catwhisker.org
The circus around that memo helps confirm that Mr. Trump is unfit for office.

See http://www.catwhisker.org/~david/publickey.gpg for my public key.


signature.asc
Description: PGP signature