On Mon, Jul 6, 2009 at 18:52, Dan Nicholson<dbn.lists@xxxxxxxxx> wrote:
> On Mon, Jul 6, 2009 at 6:27 AM, Kay Sievers<kay.sievers@xxxxxxxx> wrote:
>> On Mon, Jul 6, 2009 at 15:20, Thomas Bartosik<thomas.bartosik@xxxxxx> wrote:
>>> consider the following two commands:
>>>
>>> udevadm info --attribute-walk --query=all --path=$(udevadm info --query=path --name=/dev/sdb1)
>>>
>>> udevadm info --query=all --attribute-walk --path=$(udevadm info --query=path --name=/dev/sdb1)
>>>
>>> They produce different results. I don't think this is as expected (at least I did not expect it, tried aome time and thought all the ATTRS fields do not exist anymore since the conversion from udevinfo to udevadm info.. until I found this out!)
>>
>> Sure, one is an --attribute-walk, the other is --query. It makes no
>> sense to specify both at the same time.
>
> Maybe info should bomb if multiple action arguments are supplied?
> Would be pretty simple:
>
> if (action != ACTION_NONE) {
> fprintf(stderr, "action already specified\n");
> rc = 2;
> goto exit;
> }
Sure, send a patch if you like.
Kay
--
To unsubscribe from this list: send the line "unsubscribe linux-hotplug" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
