[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] gnu: Add tcllib.
From: |
Efraim Flashner |
Subject: |
Re: [PATCH] gnu: Add tcllib. |
Date: |
Thu, 11 Feb 2016 22:42:21 +0200 |
On Thu, 11 Feb 2016 21:28:33 +0100
Pjotr Prins <address@hidden> wrote:
> To all.
>
> Again, I think we have to be more careful with new committers. All
> comments below are in the trivial category (in my opinion).
>
> I appreciate the work reviewers are doing (thank you!), but we should
> take care to be nice and welcoming so as not to lose people. I
> personally think that the focus on spacing and indentation is way over
> the top. Especially when you realise many packages in the tree are
> not up to current standards. Anyone looked at python.scm?
>
> Can we concentrate on functionality first. And when a package is
> pretty much up to scratch just do the final editing by the actual
> committer?
>
> Ricardo give a great example here, he corrects and tells us. That way
> we learn in a nice way. Submitting packages and abiding by all the
> rules is not trivial. Not to mention that lint is sometimes broken and
> there are many packages in the tree that do *not* abide by current
> rules. I go wrong when I copy something already there. It is not that
> easy.
>
> We need to find a balance between being nice and being correct.
>
> And don't forget: thank you Jan for the contributions :). Please
> continue! We welcome you.
>
> Pj.
>
> On Thu, Feb 11, 2016 at 09:53:47PM +0200, Efraim Flashner wrote:
>
Pjotr- thanks for the reminder :). I know we've been a bit behind on patch
review and I jumped in a bit too much.
Jan- I hope I haven't scared you off or discouraged you. Indeed, thank you
for the patches.
--
Efraim Flashner <address@hidden> אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D 14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted
pgpGg10L76NvJ.pgp
Description: OpenPGP digital signature
- [PATCH] gnu: Add tcllib., Jan Nieuwenhuizen, 2016/02/09
- Re: [PATCH] gnu: Add tcllib., Efraim Flashner, 2016/02/11
- Re: [PATCH] gnu: Add tcllib., Jan Nieuwenhuizen, 2016/02/11
- Re: [PATCH] gnu: Add tcllib., Leo Famulari, 2016/02/12
- Re: [PATCH] gnu: Add tcllib., Leo Famulari, 2016/02/12
- Re: [PATCH] gnu: Add tcllib., Jan Nieuwenhuizen, 2016/02/12
- Re: [PATCH] gnu: Add tcllib., Leo Famulari, 2016/02/12
- Re: [PATCH] gnu: Add tcllib., Leo Famulari, 2016/02/12
- Re: [PATCH] gnu: Add tcllib., Ludovic Courtès, 2016/02/12