public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
From: Mike Frysinger <vapier@gentoo.org>
To: "Amadeusz Żołnowski" <aidecoe@aidecoe.name>
Cc: "gentoo-dev" <gentoo-dev@lists.gentoo.org>
Subject: Re: [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in net-misc/openvpn: ChangeLog openvpn-2.1.3.ebuild
Date: Sun, 17 Oct 2010 17:33:57 -0400	[thread overview]
Message-ID: <201010171733.58705.vapier@gentoo.org> (raw)
In-Reply-To: <1287009654-sup-7531@etiriah>

[-- Attachment #1: Type: Text/Plain, Size: 1344 bytes --]

On Wednesday, October 13, 2010 19:08:55 Amadeusz Żołnowski wrote:
> Excerpts from Mike Frysinger's message of Thu Oct 14 00:32:40 +0200 2010:
> > On Wednesday, October 13, 2010 18:13:18 Amadeusz Żołnowski wrote:
> > > Mike Frysinger's message of Wed Oct 13 23:46:43 +0200 2010:
> > > > On Wednesday, October 13, 2010 15:57:17 Amadeusz Żołnowski wrote:
> > > > > And why putting different tasks into one function?
> > > > 
> > > > for the same reason we dont have separate test binaries: test_exist,
> > > > test_file, test_dir, etc...
> > > > 
> > > > it makes more sense in my mind to combine the functionality.
> > > 
> > > So the only argument for having more complicated, less intuitive and
> > > less readable function is the old 'test' program?  Please, reconsider
> > > my solution with more reason.
> > 
> > we prioritize differently.  i prefer unified code with options.
> 
> In which part it's unified?

the file checking & status accumulation.  extending my code to add more 
options in the future is easier as well.

> As I said it doesn't conform much to 'test' convention.

sure it does

this thread is going nowhere.  i believe my proposal is the way to go, and we 
arent arguing over anything of too much value (i.e. bike shedding).  no one 
else has an opinion, so ive gone my route.
-mike

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

  reply	other threads:[~2010-10-17 21:34 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20100927113752.CF4CE20051@flycatcher.gentoo.org>
2010-10-01 13:07 ` [gentoo-dev] Re: [gentoo-commits] gentoo-x86 commit in net-misc/openvpn: ChangeLog openvpn-2.1.3.ebuild Peter Volkov
2010-10-01 13:34   ` Thomas Sachau
2010-10-01 15:17   ` Nirbheek Chauhan
2010-10-02  3:38     ` Ryan Hill
2010-10-02 12:32       ` Nirbheek Chauhan
2010-10-05  4:20     ` Jeroen Roovers
2010-10-12 20:09   ` Dirkjan Ochtman
2010-10-12 20:17     ` Mike Frysinger
2010-10-12 20:26     ` Jeroen Roovers
2010-10-12 20:57       ` Mike Frysinger
2010-10-13  8:23         ` Amadeusz Żołnowski
2010-10-13 14:13           ` Mike Frysinger
2010-10-13 16:36             ` Amadeusz Żołnowski
2010-10-13 16:41             ` Michał Górny
2010-10-13 16:51               ` Amadeusz Żołnowski
2010-10-13 17:26                 ` Michał Górny
2010-10-13 18:04                   ` Mike Frysinger
2010-10-13 18:51                     ` Mike Frysinger
2010-10-13 19:20                       ` Amadeusz Żołnowski
2010-10-13 19:57                         ` Amadeusz Żołnowski
2010-10-13 21:46                           ` Mike Frysinger
2010-10-13 22:13                             ` Amadeusz Żołnowski
2010-10-13 22:32                               ` Mike Frysinger
2010-10-13 23:08                                 ` Amadeusz Żołnowski
2010-10-17 21:33                                   ` Mike Frysinger [this message]
2010-10-17 23:06                                     ` Amadeusz Żołnowski
2010-10-18  7:08                             ` Michał Górny
2010-10-18 18:06                               ` Mike Frysinger
2010-10-18 18:25                                 ` Mike Frysinger
2010-10-19  4:03                                 ` Michał Górny
2010-10-13 19:37                       ` Ulrich Mueller
2010-10-13 21:45                         ` Mike Frysinger
2010-10-17 21:35     ` Mike Frysinger
     [not found] <fyNx7-2Qx-7@gated-at.bofh.it>
     [not found] ` <fCTuh-8on-5@gated-at.bofh.it>
     [not found]   ` <fCTDX-aL-5@gated-at.bofh.it>
     [not found]     ` <fCUgF-1kl-15@gated-at.bofh.it>
2010-10-13  7:08       ` Vaeth
2010-10-13 14:15         ` Mike Frysinger
     [not found] <fD3Dj-k9-9@gated-at.bofh.it>
     [not found] ` <fD3Dj-k9-15@gated-at.bofh.it>
     [not found]   ` <fD3Dj-k9-7@gated-at.bofh.it>
     [not found]     ` <fDbhw-4Y2-5@gated-at.bofh.it>
2010-10-13 16:35       ` Vaeth
2010-10-13 18:07         ` Mike Frysinger

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=201010171733.58705.vapier@gentoo.org \
    --to=vapier@gentoo.org \
    --cc=aidecoe@aidecoe.name \
    --cc=gentoo-dev@lists.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