s3: Patch to smbclient to add a utimes command for setting the Windows times
Volker Lendecke
Volker.Lendecke at SerNet.DE
Fri Sep 15 00:17:55 UTC 2017
On Thu, Sep 14, 2017 at 12:21:16PM -0700, Richard Sharpe via samba-technical wrote:
> > ENOPATCH :-).
>
> Sorry. Should be attached now.
Standard review reply from me: This has lines beyond 80 characters. We
have a snippet in README.Coding about that:
* 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.
Can you make sure that you get the lines into that range?
Thanks, 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