Resend: Patch to implement utimes in smbclient along with a test
Volker Lendecke
Volker.Lendecke at SerNet.DE
Fri Sep 22 01:01:01 UTC 2017
On Thu, Sep 21, 2017 at 05:33:00PM -0700, Richard Sharpe wrote:
> On Thu, Sep 21, 2017 at 5:19 PM, Volker Lendecke
> <Volker.Lendecke at sernet.de> wrote:
> > Hi, Richard!
> >
> > On Thu, Sep 21, 2017 at 04:25:54PM -0700, Richard Sharpe via samba-technical wrote:
> >> Attached is a patch to implement a utimes command in smbclient along
> >> with a test.
> >
> > Let me cite a snippet from README.Coding:
> >
> > * Maximum Line Width is 80 Characters
> > The reason is not about people with low-res screens but rather sticking
> > to 80 columns prevents you from easily nesting more than one level of
> > if statements or other code blocks. Use source3/script/count_80_col.pl
> > to check your changes.
>
> Here is a change that fixes the added line that causes problems, I think.
For me it does not, sorry. Line 4964 in source3/client/client.c after
applying this patch is overlong for me. Just by 2 characters, but my
editor shows red for that new line.
Don't get me wrong. I will not NACK your patch over this, so if Jeremy
likes it, he is of course free to put it in.
Sorry,
Volker
--
SerNet GmbH, Bahnhofsallee 1b, 37081 Göttingen
phone: +49-551-370000-0, fax: +49-551-370000-9
AG Göttingen, HRB 2816, GF: Dr. Johannes Loxen
http://www.sernet.de, mailto:kontakt at sernet.de
More information about the samba-technical
mailing list