All of lore.kernel.org
 help / color / mirror / Atom feed
From: Koen Kooi <koen@dominion.thruhere.net>
To: Andreas Oberritter <obi@opendreambox.org>
Cc: Koen Kooi <koen@openembedded.org>,
	"openembedded-devel@lists.openembedded.org"
	<openembedded-devel@lists.openembedded.org>
Subject: Re: latest clutter-recipe
Date: Wed, 2 Feb 2011 15:57:33 +0100	[thread overview]
Message-ID: <1F887C9B-2BD1-436C-8B7B-4162CA51857C@dominion.thruhere.net> (raw)
In-Reply-To: <4D496DBC.6030709@opendreambox.org>

I checked patchwork first and that didn't show any clutter patches. I'll add the gtk and gst ones after fosdem

Op 2 feb. 2011 om 15:44 heeft Andreas Oberritter <obi@opendreambox.org> het volgende geschreven:

> Hello Andreas,
> 
> I can't answer your technical questions regarding the recipe, because
> I'm not using a clutter/gtk/x11 based distro. I'd like to comment the
> rest of your mail, though.
> 
> On 02/01/2011 08:02 PM, Andreas Mueller wrote:
>> it surprised me to see that Koen committed a new recipe for clutter 1.4.2. As you might know I was also working on this and sent some pre-release version here of this (1). What makes me not running in happy mode:
>> 
>> - Many of my previous patches were rejected first because of some commit message. Here we read 'clutter: add 1.4.2' - cool. Which tests were performed?
> 
> In general, contrary to patches which modify existing files, I think
> that patches adding new recipes don't need a detailed commit message.
> Still, a message should describe the content of a commit in a suitable
> and understandable way. In my opinion, "<name>: add <version>"
> sufficiently describes any new recipe, unless there's anything special
> about it.
> 
> In this case, I would have preferred glib-gettextize getting mentioned,
> which was added to clutter.inc.
> 
>> Please don't tell me I am invited to change this situation:  Many patches send by non-commit guys are nit-picked at least for commit message - and then I read 'clutter: add 1.4.2' for a recipe breaking lots of others.
> 
> Well, shit happens, unfortunately. Hopefully it was tested and worked in
> a different environment. Bug reports like this are needed to make it
> work for as many environments as possible. I've added Koen to CC to
> decrease the chance that your bug report gets lost without Koen
> noticing. I think he didn't mean to offend you in any way with his commit.
> 
> Nit-picking happens frequently, and personally I prefer getting negative
> feedback over no feedback at all. Usually it's not very hard to address
> the complaints, although it can be very frustrating to get only a short
> comment on something you've put a lot of time into. My guess is that
> this has occurred at least once to everybody who contributed to open
> source projects.
> 
>> Sorry for this, but to me the this should have been a hobby for having fun - but at the moment it's far from
> 
> So let's try to regain some fun after this problem gets solved! You
> shouldn't take it personally.
> 
>> Andreas
>> 
>> (1) http://lists.linuxtogo.org/pipermail/openembedded-devel/2011-January/029260.html
> 
> Regards,
> Andreas
> 



  reply	other threads:[~2011-02-02 14:59 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-02-01 19:02 latest clutter-recipe Andreas Mueller
2011-02-02 14:44 ` Andreas Oberritter
2011-02-02 14:57   ` Koen Kooi [this message]
2011-02-02 23:21     ` Andreas Mueller

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=1F887C9B-2BD1-436C-8B7B-4162CA51857C@dominion.thruhere.net \
    --to=koen@dominion.thruhere.net \
    --cc=koen@openembedded.org \
    --cc=obi@opendreambox.org \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.