<div dir="ltr"><br><br>On Wed, Apr 14, 2021 at 10:22 PM Vijay Kumar Banerjee <<a href="mailto:vijay@rtems.org">vijay@rtems.org</a>> wrote:<br>><br>> On Wed, Apr 14, 2021 at 8:40 PM Chris Johns <<a href="mailto:chrisj@rtems.org">chrisj@rtems.org</a>> wrote:<br>> ><br>> > On 15/4/21 5:22 am, Vijay Kumar Banerjee wrote:<br>> > > On Wed, Apr 14, 2021 at 1:10 PM Gedare Bloom <<a href="mailto:gedare@rtems.org">gedare@rtems.org</a>> wrote:<br>> > >><br>> > >> On Wed, Apr 14, 2021 at 12:36 PM Vijay Kumar Banerjee <<a href="mailto:vijay@rtems.org">vijay@rtems.org</a>> wrote:<br>> > >>><br>> > >>> ---<br>> > >>>  netlegacy.py | 27 +++++++++++++--------------<br>> > >>>  1 file changed, 13 insertions(+), 14 deletions(-)<br>> > >>><br>> > >>> diff --git a/netlegacy.py b/netlegacy.py<br>> > >>> index 05cb78a..2936b8f 100644<br>> > >>> --- a/netlegacy.py<br>> > >>> +++ b/netlegacy.py<br>> > >>> @@ -37,7 +37,7 @@ exclude_headers = ['rtems-bsd-user-space.h', 'rtems-bsd-kernel-space.h']<br>> > >>><br>> > >>>  for root, dirs, files in os.walk("."):<br>> > >>>      [dirs.remove(d) for d in list(dirs) if d in exclude_dirs]<br>> > >>> -    dirs.append('./bsps/shared/net')<br>> > >>> +    dirs.append(os.path.join('bsps', 'shared', 'net'))<br>> > >>>      include_files[root[2:]] = []<br>> > >>>      for name in files:<br>> > >>>          ext = os.path.splitext(name)[1]<br>> > >>> @@ -59,10 +59,9 @@ def build(bld):<br>> > >>>      include_path = []<br>> > >>>      ip = ''<br>> > >>>      bsp = bld.env.RTEMS_ARCH_BSP.split('-')[-1]<br>> > >>> -    pppd_source = [os.path.join('./pppd', s)<br>> > >>> -                   for s in os.listdir('./pppd') if s[-2:] == '.c']<br>> > >>> -    telnetd_source = [os.path.join('./telnetd', s)<br>> > >>> -                      for s in os.listdir('telnetd') if s[-2:] == '.c']<br>> > >>> +    pppd_source = [os.path.join('pppd', s)<br>> > >>> +                   for s in os.listdir('pppd')<br>> > >>> +                   if os.path.splitext(s)[1] == '.c']<br>> > >> This indentation looks wrong?<br>> > >><br>> > > This looks good with git show. I'm not sure what happened here, maybe<br>> > > it looks weird due to email formatting?<br>> ><br>> > Embedded tab character?<br>> I'm not sure what's the issue here. Generally, git show output shows<br>> any irregularity. This indentation looks right when I open it on mutt.<br>> This is probably some issue with the email formatting. I see that some<br>> of the removed codes in this patch also have crooked indentation but<br>> they look properly indented upstream.<br>><br>Hi,<br><br><div>Just wanted to add that I pushed the commit in my private repository and the indentation looks alright. Please have a look:</div><div><a href="https://git.rtems.org/vijay/rtems-net-legacy.git/commit/?id=c61fb1cf8517cb5d42568b48721056cf15a3bb40">https://git.rtems.org/vijay/rtems-net-legacy.git/commit/?id=c61fb1cf8517cb5d42568b48721056cf15a3bb40</a></div><div><br></div><div>Best regards,</div><div>Vijay<br></div>> ><br>> > Chris</div>