On 28/8/18 10:35 pm, Sebastian Huber wrote:
> Module:    rtems-libbsd
> Branch:    master
> Commit:    b2e1123733db8fd1a83a893e980a9fe4da357bb6
> Changeset: 
> http://git.rtems.org/rtems-libbsd/commit/?id=b2e1123733db8fd1a83a893e980a9fe4da357bb6
> 
> Author:    Sebastian Huber <sebastian.hu...@embedded-brains.de>
> Date:      Tue Aug 28 14:14:27 2018 +0200
> 
> userspace-header-gen.py: Sort file content
> 
> Sort the file content to make it less dependent on compiler code
> generation variances.
> 
> ---
> 
>  userspace-header-gen.py | 29 +++++++++++++++++++++--------
>  1 file changed, 21 insertions(+), 8 deletions(-)
> 
> diff --git a/userspace-header-gen.py b/userspace-header-gen.py
> index f41feb4..8c7ef76 100755
> --- a/userspace-header-gen.py
> +++ b/userspace-header-gen.py
> @@ -1,4 +1,4 @@
> -#!/usr/bin/env python
> +#!/usr/bin/env python3

I think this type of change should have a discussion. For the record I think the
platform's default 'python' is what we use. When the plaform, distro, OS or
whatever moves we move with it. I do not wish to spend time and energy on the
merits of python versions, we should support both while they are defaults on any
platforms we support.

Also the commit message does not cover this change and it is something that
should be noted in the message with a ticket.

Please revert just this bit.

Thanks
Chris
_______________________________________________
devel mailing list
devel@rtems.org
http://lists.rtems.org/mailman/listinfo/devel

Reply via email to