linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Johannes Berg <johannes@sipsolutions.net>
To: "Stefanik Gábor" <netrolller.3d@gmail.com>
Cc: "Adam Cécile (Le_Vert)" <gandalf@le-vert.net>,
	"Jochen Friedrich" <jochen@scram.de>,
	linux-wireless@vger.kernel.org,
	"Thomas d'Otreppe" <tdotreppe@aircrack-ng.org>
Subject: Re: iw packaging
Date: Wed, 17 Sep 2008 22:31:07 +0200	[thread overview]
Message-ID: <1221683467.7412.6.camel@localhost> (raw)
In-Reply-To: <69e28c910809171319r30d763cse24744a198e41f92@mail.gmail.com> (sfid-20080917_221957_497607_0A215EEE)

On Wed, 2008-09-17 at 22:19 +0200, Stefanik G=C3=A1bor wrote:

> So, the actual command used, for example, is:
> iw dev wlan0 interface add mon0 type monitor

that seems fine.

> Similarly, the code used for removing interfaces is
> IW_ERROR=3D`iw dev "$iface" interface del 2>&1 | grep "nl80211 not fo=
und"`
> If I interpret your mail correctly, then this should be changed to:
> IW_ERROR=3D`iw dev "$iface" del 2>&1 | grep "nl80211 not found"`

I don't know why you insist on that grep, it should exit with a non-zer=
o
status code.... but you can use both forms, I added the invisible alias=
=2E

> Will the same syntax change be made for the add feature? I mean this:
> iw dev wlan0 add mon0 type monitor

No. What are you adding?

> Also, the following syntax would be even better:
> iw --add mon0 --dev=3Dwlan0 --type=3Dmonitor
> or:
> iw -dwlan0 -tmonitor add mon0
>=20
> Or at least allowing stuff like this:
> iw type monitor add mon0 dev wlan0

No. All of those are gross and not very object oriented.

> An experimental branch of aircrack-ng contains a slightly different
> version, which also has these additional changes:
> -Mesh support has been removed (as I couldn't get it compile with
> -Wall -Werror, which is a requirement for inclusion into aircrack-ng)

That's an extremely stupid requirement as warnings may pop up on variou=
s
platforms which you cannot anticipate, making it hard for users.


Anyway, you're free to fork it and do whatever you like that aircrack-n=
g
needs, but please don't call it iw and don't encourage debian to ship i=
t
as 'iw'.

johannes

--
To unsubscribe from this list: send the line "unsubscribe linux-wireles=
s" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2008-09-17 20:31 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-09-16 20:29 iw packaging Johannes Berg
2008-09-16 20:46 ` Stefanik Gábor
2008-09-16 20:50 ` Pavel Roskin
2008-09-16 21:11   ` Johannes Berg
2008-09-16 21:15     ` Johannes Berg
2008-09-16 21:23       ` Pavel Roskin
2008-09-16 21:38         ` Johannes Berg
2008-09-16 21:48           ` Pavel Roskin
2008-09-16 21:56             ` Johannes Berg
2008-09-16 22:14               ` Johannes Berg
2008-09-16 23:20                 ` Pavel Roskin
2008-09-17  7:30                   ` Johannes Berg
2008-09-17 10:41 ` Jochen Friedrich
2008-09-17 10:43   ` Johannes Berg
2008-09-17 11:05     ` "Adam Cécile (Le_Vert)"
2008-09-17 11:07       ` Johannes Berg
2008-09-17 11:11         ` "Adam Cécile (Le_Vert)"
2008-09-17 11:18           ` Johannes Berg
2008-09-17 11:25             ` Johannes Berg
2008-09-17 20:19             ` Stefanik Gábor
2008-09-17 20:31               ` Johannes Berg [this message]
2008-09-17 21:46                 ` Davide Pesavento
2008-09-17 21:51                   ` Stefanik Gábor
2008-09-17 21:52                   ` Johannes Berg
2008-09-17 21:58                     ` Stefanik Gábor
2008-09-17 22:03                       ` Luis R. Rodriguez
2008-09-17 22:06                         ` Stefanik Gábor
2008-09-17 22:07                           ` Johannes Berg
2008-09-17 22:04                       ` Johannes Berg
2008-09-17 16:23 ` Marcel Holtmann
2008-09-17 16:43   ` Johannes Berg
2008-09-17 18:06     ` Johannes Berg
2008-09-17 19:23       ` Luis R. Rodriguez
2008-09-17 23:36       ` Marcel Holtmann
2008-09-17 23:42         ` Johannes Berg
2008-10-10 21:25           ` Thomas d'Otreppe
2008-09-18  8:09 ` Helmut Schaa
2008-09-27 16:59 ` Davide Pesavento

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=1221683467.7412.6.camel@localhost \
    --to=johannes@sipsolutions.net \
    --cc=gandalf@le-vert.net \
    --cc=jochen@scram.de \
    --cc=linux-wireless@vger.kernel.org \
    --cc=netrolller.3d@gmail.com \
    --cc=tdotreppe@aircrack-ng.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;
as well as URLs for NNTP newsgroup(s).