From: Mike Frysinger <vapier@gentoo.org>
To: gentoo-dev@lists.gentoo.org
Cc: Justin Lecher <jlec@gentoo.org>
Subject: Re: [gentoo-dev] Re: [gentoo-commits] repo/gentoo:master commit in: eclass/
Date: Sat, 31 Oct 2015 03:06:21 -0400 [thread overview]
Message-ID: <20151031070621.GO5154@vapier.lan> (raw)
In-Reply-To: <20151030182008.44e0c324.mgorny@gentoo.org>
[-- Attachment #1: Type: text/plain, Size: 1826 bytes --]
On 30 Oct 2015 18:20, Michał Górny wrote:
> On Fri, 30 Oct 2015 12:03:59 +0000 (UTC) "Justin Lecher" wrote:
> > commit: df8e399c9bac2dc30d7cf69c2462a81729a3ae69
> > Author: Justin Lecher <jlec <AT> gentoo <DOT> org>
> > AuthorDate: Fri Oct 30 10:18:05 2015 +0000
> > Commit: Justin Lecher <jlec <AT> gentoo <DOT> org>
> > CommitDate: Fri Oct 30 12:03:49 2015 +0000
> > URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=df8e399c
> >
> > eclass: Use consistent place for then in if clause
>
> Excuse me but who are you exactly to take a random eclass and commit
> random style changes inside without even bothering to contact
> the author?
your attitude is wrong and is a great example of why people in the project
are hesitant to touch anything anymore
> > Signed-off-by: Justin Lecher <jlec <AT> gentoo.org>
> >
> > eclass/distutils-r1.eclass | 3 +--
> > 1 file changed, 1 insertion(+), 2 deletions(-)
> >
> > diff --git a/eclass/distutils-r1.eclass b/eclass/distutils-r1.eclass
> > index 185dd4f..dbd27a7 100644
> > --- a/eclass/distutils-r1.eclass
> > +++ b/eclass/distutils-r1.eclass
> > @@ -322,8 +322,7 @@ distutils-r1_python_prepare_all() {
> >
> > _distutils-r1_disable_ez_setup
> >
> > - if [[ ${DISTUTILS_IN_SOURCE_BUILD} && ! ${DISTUTILS_SINGLE_IMPL} ]]
> > - then
> > + if [[ ${DISTUTILS_IN_SOURCE_BUILD} && ! ${DISTUTILS_SINGLE_IMPL} ]]; then
>
> This was intentionally wrapped to stay within 72-column line width. Not
> saying the eclass is perfect in keeping text width, especially with
> others committing random changes to it, but that's no reason to
> introduce further offenders.
Gentoo has never had a hard 80-col rule let alone 72-cols. forcing a wrap
here makes no sense and the new version is an improvement.
-mike
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
next prev parent reply other threads:[~2015-10-31 7:06 UTC|newest]
Thread overview: 119+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1446206629.df8e399c9bac2dc30d7cf69c2462a81729a3ae69.jlec@gentoo>
2015-10-30 17:20 ` [gentoo-dev] Re: [gentoo-commits] repo/gentoo:master commit in: eclass/ Michał Górny
2015-10-31 7:06 ` Mike Frysinger [this message]
2015-10-31 8:08 ` Michał Górny
2015-10-31 15:05 ` Gregory Woodbury
2015-11-10 23:53 ` Mike Frysinger
2015-11-11 9:09 ` Michał Górny
2015-11-11 14:42 ` Luca Barbato
2015-11-01 7:15 ` Ian Delaney
[not found] <1567619929.63486fef43c2e0dee3b4128db18fd1a6b4bf9381.tupone@gentoo>
2019-09-04 18:59 ` Michał Górny
2019-09-04 22:30 ` David Seifert
2019-09-04 23:26 ` Thomas Deutschmann
2019-09-05 4:02 ` Michał Górny
2019-09-05 13:14 ` Thomas Deutschmann
2019-09-05 14:04 ` Gordon Pettey
2019-09-05 19:26 ` Kent Fredric
2019-09-05 19:28 ` James Le Cuirot
2019-09-05 19:47 ` Michał Górny
2019-09-05 19:49 ` Michael Everitt
2019-09-05 20:08 ` Kent Fredric
2019-09-05 20:16 ` Michał Górny
2019-09-05 22:47 ` Thomas Deutschmann
2019-09-06 3:45 ` Mike Gilbert
2019-09-06 6:35 ` Alfredo Tupone
2019-09-06 5:49 ` Michał Górny
2019-09-06 4:02 ` Mike Gilbert
[not found] <1564451381.6f680e4fe73925ae130343e02adb416cb799ce7d.mattst88@gentoo>
2019-07-30 5:20 ` Michał Górny
2019-07-30 5:29 ` Matt Turner
[not found] <1526675772.d780c05e4459175eb314c82de9f3b998e2b4fc6e.asturm@gentoo>
2018-05-18 21:53 ` Michał Górny
[not found] <1515158020.1e09cba657ccb2b8e6fbcbeb28c5c593da2127eb.polynomial-c@gentoo>
2018-01-05 13:36 ` Michał Górny
[not found] <1494961831.d1b05bcf81e5058c20615179fb83a90c45daa487.vapier@gentoo>
2017-05-16 19:20 ` Michał Górny
2017-05-16 19:36 ` Fabian Groffen
2017-05-21 14:02 ` Kent Fredric
2017-05-21 14:46 ` Martin Vaeth
2017-05-21 16:29 ` William Hubbs
2017-05-21 17:34 ` Michał Górny
2017-05-21 18:18 ` Martin Vaeth
2017-05-21 19:32 ` Kent Fredric
2017-05-21 20:38 ` M. J. Everitt
2017-05-21 23:28 ` M. J. Everitt
2017-05-21 17:46 ` Raymond Jennings
2017-05-22 1:13 ` Luis Ressel
[not found] <1463773635.548f4ee19c6b0fe0d5e87e84a5a82c421229e0ce.johu@gentoo>
2016-05-20 19:55 ` Michał Górny
[not found] <1462655928.66afcab271f65b97330e610040ad3acc1b812a03.hd_brummy@gentoo>
2016-05-07 21:25 ` Michał Górny
2016-05-07 21:48 ` Ryan Hill
2016-05-09 23:10 ` Doug Goldstein
2016-05-13 8:52 ` Ian Delaney
2016-05-14 9:55 ` Andreas K. Huettel
2016-05-14 11:35 ` Ciaran McCreesh
2016-05-14 11:55 ` M. J. Everitt
2016-05-14 15:04 ` Gordon Pettey
2016-05-14 16:53 ` Chí-Thanh Christopher Nguyễn
2016-05-14 16:59 ` M. J. Everitt
2016-05-14 17:06 ` Rich Freeman
2016-05-14 17:07 ` landis blackwell
2016-05-14 17:52 ` Rich Freeman
2016-05-14 19:21 ` M. J. Everitt
2016-05-14 20:23 ` Rich Freeman
2016-05-15 10:47 ` Daniel Campbell
2016-05-14 17:08 ` M. J. Everitt
2016-05-14 20:06 ` Andreas K. Huettel
2016-05-14 11:55 ` Aaron Bauman
2016-05-14 13:54 ` Rich Freeman
2016-05-14 23:40 ` Aaron Bauman
2016-05-14 23:48 ` Ciaran McCreesh
2016-05-15 0:23 ` Aaron Bauman
2016-05-15 1:04 ` Rich Freeman
2016-05-15 1:12 ` M. J. Everitt
2016-05-16 7:56 ` Ian Delaney
2016-05-16 8:20 ` Consus
2016-05-16 8:30 ` Aaron Bauman
2016-05-16 9:05 ` Ciaran McCreesh
2016-05-16 16:21 ` Andrew Savchenko
2016-05-16 16:32 ` Andreas K. Huettel
2016-05-15 0:59 ` Rich Freeman
2016-05-15 1:12 ` M. J. Everitt
2016-05-15 9:53 ` Ryan Hill
2016-05-15 11:04 ` Daniel Campbell
2016-05-15 22:55 ` Duncan
2016-05-15 23:07 ` Daniel Campbell
2016-05-16 0:12 ` M. J. Everitt
2016-05-15 18:43 ` Andreas K. Huettel
2016-05-15 10:29 ` Michał Górny
2016-05-15 11:16 ` Daniel Campbell
2016-05-15 22:46 ` Duncan
2016-05-15 23:19 ` Rich Freeman
2016-05-15 9:05 ` Jeroen Roovers
2016-05-15 9:15 ` Brian Dolbec
2016-05-15 11:18 ` Daniel Campbell
2016-05-15 14:27 ` Brian Dolbec
[not found] <1460833703.ad0c2ab2bdbd34f4550e49c56cfd5974d6a2c07a.blueness@gentoo>
2016-04-16 19:05 ` Michał Górny
2016-04-16 19:13 ` Anthony G. Basile
2016-04-16 19:16 ` Rich Freeman
2016-04-16 19:18 ` Anthony G. Basile
2016-04-16 19:41 ` Anthony G. Basile
2016-04-16 19:27 ` Anthony G. Basile
2016-04-16 19:31 ` Anthony G. Basile
2016-04-16 22:24 ` Mike Gilbert
2016-04-16 22:36 ` Rich Freeman
2016-04-16 22:46 ` Mike Gilbert
2016-04-16 22:54 ` Anthony G. Basile
2016-04-16 22:50 ` Anthony G. Basile
2016-04-17 8:15 ` Fabian Groffen
2016-04-17 8:28 ` Anthony G. Basile
2016-04-17 22:50 ` Anthony G. Basile
2016-04-18 2:09 ` Luca Barbato
[not found] <1448180855.a144d7480f781f21323943d87e6a56136add3830.mr_bones_@gentoo>
2015-11-22 8:38 ` Michał Górny
[not found] <1447717075.aac24917ebe254a23990f0d7fff9f6f570b99d15.mpagano@gentoo>
2015-11-17 7:19 ` Michał Górny
2015-11-17 20:37 ` Raymond Jennings
2015-11-17 23:42 ` Rich Freeman
2015-11-17 21:59 ` Mike Pagano
[not found] <1447458773.ad4c142684afb096e8fff2937ae5c5c3385dd22e.mgorny@gentoo>
2015-11-16 9:01 ` Alexis Ballier
2015-11-16 9:06 ` Justin Lecher (jlec)
2015-11-16 9:14 ` Alexis Ballier
2015-11-16 9:29 ` Justin Lecher (jlec)
2015-11-16 9:52 ` Alexis Ballier
2015-11-16 12:08 ` Rich Freeman
[not found] <1444389412.5220bb29741e1685b42a6312c0b7bf2821672040.aballier@gentoo>
2015-10-09 15:32 ` hasufell
2015-10-09 16:21 ` Alexis Ballier
2015-10-09 16:25 ` hasufell
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20151031070621.GO5154@vapier.lan \
--to=vapier@gentoo.org \
--cc=gentoo-dev@lists.gentoo.org \
--cc=jlec@gentoo.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox