[MacPorts] #714: Variant disable option doesn't work

[MacPorts]

#714: Variant disable option doesn't work
-----------------------------+--------------------------------
  Reporter:  aqua_dabbler@…  |      Owner:  macports-tickets@…
      Type:  defect          |     Status:  closed
  Priority:  Normal          |  Milestone:
 Component:  base            |    Version:  1.0
Resolution:  invalid         |   Keywords:
      Port:                  |
-----------------------------+--------------------------------
Description changed by ryandesign@…:

Old description:

> The current port command cannot detect variant disable option unless
> you specify "--" argument at command line.
>

> $ sudo port install escm -gauche +guile
> Usage: /opt/local/bin/port [-vdqof] [action] [-D portdir] [options]
> $ sudo port -- install escm -gauche +guile
> --->  Fetching escm
> --->  Verifying checksum for escm
> --->  Extracting escm
> --->  Configuring escm
> --->  Building escm with target escm doc
> --->  Installing escm
>

> The following patch changes the behavior a little (-D option has to be
> placed before action) but fix this issue and port's argument parser is
> still simple.
>

> --- port.tcl.orig       Sun Jul 20 12:09:06 2003
> +++ port.tcl    Sun Jul 20 12:10:32 2003
> @@ -147,6 +147,7 @@
>
>         # action
>      } elseif {[regexp {^([A-Za-z0-9/._\-^$\[\[?\(\)\\|\+\*]+)$} $arg
> match opt] == 1} {
> +       set separator 1
>         if [info exists action] {
>             set portname $opt
>         } else {

New description:

 The current port command cannot detect variant disable option unless
 you specify "--" argument at command line.

 {{{
 $ sudo port install escm -gauche +guile
 Usage: /opt/local/bin/port [-vdqof] [action] [-D portdir] [options]
 $ sudo port -- install escm -gauche +guile
 --->  Fetching escm
 --->  Verifying checksum for escm
 --->  Extracting escm
 --->  Configuring escm
 --->  Building escm with target escm doc
 --->  Installing escm
 }}}

 The following patch changes the behavior a little (-D option has to be
 placed before action) but fix this issue and port's argument parser is
 still simple.

 {{{
 --- port.tcl.orig       Sun Jul 20 12:09:06 2003
 +++ port.tcl    Sun Jul 20 12:10:32 2003
 @@ -147,6 +147,7 @@

         # action
      } elseif {[regexp {^([A-Za-z0-9/._\-^$\[\[?\(\)\\|\+\*]+)$} $arg
 match opt] == 1} {
 +       set separator 1
         if [info exists action] {
             set portname $opt
         } else {
 }}}

--

-- 
Ticket URL: <https://trac.macports.org/ticket/714#comment:2>
MacPorts <https://www.macports.org/>
Ports system for OS X