Re: [PATCH 1/2] depmod: --symbol-prefix actually requires an argument

"Andrey Mazo" <mazo@xxxxxxxx> · Mon, 17 Dec 2012 15:48:12 +0400

Thank you!

On Mon, 17 Dec 2012 08:13:29 +0400, Lucas De Marchi <lucas.demarchi@xxxxxxxxxxxxxx> wrote:

On Wed, Dec 12, 2012 at 10:36 AM, Andrey Mazo <mazo@xxxxxxxx> wrote:
-P requires and uses an argument but its long counterpart --symbol-prefix does not:
depmod: option '--symbol-prefix' doesn't allow an argument
---
 tools/depmod.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/tools/depmod.c b/tools/depmod.c
index 682e662..f7d71f8 100644
--- a/tools/depmod.c
+++ b/tools/depmod.c
@@ -63,7 +63,7 @@ static const struct option cmdopts[] = {
        { "verbose", no_argument, 0, 'v' },
        { "show", no_argument, 0, 'n' },
        { "dry-run", no_argument, 0, 'n' },
-       { "symbol-prefix", no_argument, 0, 'P' },
+       { "symbol-prefix", required_argument, 0, 'P' },
        { "warn", no_argument, 0, 'w' },
        { "map", no_argument, 0, 'm' }, /* deprecated */
        { "version", no_argument, 0, 'V' },
--
1.7.8.6


Patch has been applied. Thanks.


Lucas De Marchi

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





