From: Anders Darander <anders@chargestorm.se>
To: <openembedded-devel@lists.openembedded.org>
Subject: Re: `PR = "r0"`: Add or not to add?
Date: Wed, 31 Aug 2011 14:29:06 +0200 [thread overview]
Message-ID: <20110831122905.GD3273@chargestorm.se> (raw)
In-Reply-To: <j3l8ni$kr3$1@dough.gmane.org>
* Koen Kooi <koen@dominion.thruhere.net> [110831 14:17]:
> Op 31-08-11 13:55, Anders Darander schreef:
> > * Koen Kooi <koen@dominion.thruhere.net> [110831 12:24]:
> >> Op 31-08-11 12:06, Paul Menzel schreef:
> >>> Am Mittwoch, den 31.08.2011, 09:15 +0200 schrieb Koen Kooi:
> >>>> please remove PR in new recipes
> >>> Actually I would favor to leave them in.
> >> Well I don't, and atm I decide if it goes in or not.
> >
> > To sad. It's a lot easier to remember to bump the PR, when PR
> > actually is in the recipe. Thus, including PR=0 will often remove
> > one issue with patches.
> That's what review is for, no?
Sure, but if we can reduce the need for one extra comment, isn't that
good? Especially when it comes to newcommers.
Cheers,
Anders
--
Anders Darander
ChargeStorm AB
next prev parent reply other threads:[~2011-08-31 12:33 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-08-30 23:06 [meta-xfce] thunar-volman: Add initial recipe with version 0.6.0 Andreas Müller
2011-08-31 7:15 ` Koen Kooi
2011-08-31 10:06 ` `PR = "r0"`: Add or not to add? (was: thunar-volman: Add initial recipe with version 0.6.0) Paul Menzel
2011-08-31 10:23 ` `PR = "r0"`: Add or not to add? Koen Kooi
2011-08-31 11:45 ` Frans Meulenbroeks
2011-08-31 11:55 ` Anders Darander
2011-08-31 12:16 ` Koen Kooi
2011-08-31 12:27 ` Paul Eggleton
2011-08-31 12:33 ` Frans Meulenbroeks
2011-08-31 12:36 ` Koen Kooi
2011-08-31 12:43 ` Anders Darander
2011-08-31 13:00 ` Koen Kooi
2011-08-31 13:19 ` Anders Darander
2011-08-31 14:10 ` Frans Meulenbroeks
2011-08-31 12:29 ` Anders Darander [this message]
2011-08-31 12:34 ` Otavio Salvador
2011-08-31 12:45 ` Koen Kooi
2011-08-31 13:02 ` Otavio Salvador
2011-08-31 13:30 ` Koen Kooi
2011-08-31 13:43 ` Otavio Salvador
2011-08-31 22:35 ` Philip Balister
2011-09-01 10:06 ` Frans Meulenbroeks
2011-09-05 10:22 ` Ahsan, Noor
2011-08-31 13:49 ` Paul Menzel
2011-08-31 14:07 ` Koen Kooi
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=20110831122905.GD3273@chargestorm.se \
--to=anders@chargestorm.se \
--cc=openembedded-devel@lists.openembedded.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