Resend: Patch to implement utimes in smbclient along with a test

Richard Sharpe realrichardsharpe at gmail.com
Fri Sep 22 02:22:19 UTC 2017


On Thu, Sep 21, 2017 at 6:01 PM, Volker Lendecke
<Volker.Lendecke at sernet.de> wrote:
> 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.

OK, fixed that one as well.

-- 
Regards,
Richard Sharpe
(何以解憂?唯有杜康。--曹操)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-s3-Add-a-utimes-command-to-smbclient-so-we-can-set-t.patch
Type: application/octet-stream
Size: 8369 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20170921/6ec9751b/0001-s3-Add-a-utimes-command-to-smbclient-so-we-can-set-t.obj>


More information about the samba-technical mailing list