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;
}
Or it could also just warn and drop the second argument.
--
Dan
--
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
