Menu

#286 morphix-rebrand doesn't detect missing arguments

open
tools (16)
5
2009-03-05
2009-03-05
No

The morphix-rebrand script needs a minimum of 3 arguments to be passed on the command line. When optional arguments such as -T are used on the command line, the check for missing arguments fails.

The morphix-rebrand script should check that 3 arguments were passed. Each time an optional argument such as -T is seen on the command line, the required argument count should go up by 2.

Discussion

  • Kevin Cozens

    Kevin Cozens - 2009-03-05
    • assigned_to: alextreme --> bmsleight
     
  • Kevin Cozens

    Kevin Cozens - 2009-03-05

    Suggested patch which fixes the reported problem.

     
  • Kevin Cozens

    Kevin Cozens - 2009-03-05

    The attached patch fixes the problem reported by adjusting the number of expected arguments based when any optional arguments are given on the command line.
    File Added: morphix-rebrand.patch

     

Log in to post a comment.