[prev in list] [next in list] [prev in thread] [next in thread] 

List:       busybox
Subject:    Re: [PATCH] getopt: allow enabling -l independently of
From:       Colin Watson <cjwatson () ubuntu ! com>
Date:       2009-06-19 10:41:11
Message-ID: 20090619104108.GO7367 () riva ! ucam ! org
[Download RAW message or body]

On Fri, Jun 19, 2009 at 12:16:51PM +0200, Denys Vlasenko wrote:
> On Fri, Jun 19, 2009 at 1:07 AM, Colin Watson<cjwatson@ubuntu.com> wrote:
> > I guess I'm a glutton for punishment today since y'all seem to have
> > completely different approaches to things than I do
> 
> This is sort of typical to have different approaches.
> 
> However, your patches *are* accepted. Why do you complain?
> You want them to be accepted in exactly the form you proposed?

Oh, no, I don't expect that from anyone. I was a bit disheartened at
temporarily feeling like I was being treated as an idiot because I
happen to have slightly different priorities, that's all. :-) I have a
thicker skin on order but the shop has sold out at the moment.

My actual goal is just to reduce the patch set I'm carrying, and it all
works out in the end.

> In my experience, it usually is much worse than that
> in many projects: I send a patch and it gets ignored
> or rejected on some non-technical grounds.
> 
> Best case is when I work on creating the patch for a day and
> then it takes weeks or months to get it applied.
> 
> That's what I call "punishment".

I definitely appreciate the extraordinarily quick responses around here!

> You did not include a bloatcheck output. Please resubmit.
> 
> Just kidding.
> 
> That would be a classic answer from e.g. run of the mill GNU project.
> I once got "Your changelog does not have two spaces between date and your name".

I have an ongoing campaign in projects I'm more extensively involved
with to stop developers from making patch submitters go back and forth
just because they got the changelog wrong ...

> Then I renamed old option GETOPT_LONG to LONG_OPTS, it matches
> the meaning closer and does not look so similar
> to your ENABLE_FEATURE_GETOPT_LONG.

Makes sense.

-- 
Colin Watson                                       [cjwatson@ubuntu.com]
_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox
[prev in list] [next in list] [prev in thread] [next in thread]