Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#57118 closed defect (fixed)

octave portgroup: can't read "categories": wrong # args

Reported by: herbygillot (Herby Gillot) Owned by: jmroot (Joshua Root)
Priority: Normal Milestone:
Component: ports Version: 2.5.3
Keywords: Cc:
Port:

Description

When port lint is run against a number of octave packages, port crashes:

$ port lint octave-audio
--->  Verifying Portfile for octave-audio
Error: Failed to lint octave-audio: can't read "categories": wrong # args: should be "set varName ?newValue?"
Error: Follow https://guide.macports.org/#project.tickets to report a bug.
Error: Processing of port octave-audio failed

Debug output:

OK: Found platform: darwin
Error: Failed to lint octave-audio: can't read "categories": wrong # args: should be "set varName ?newValue?"
DEBUG: Error code: NONE
DEBUG: Backtrace: wrong # args: should be "set varName ?newValue?"
    while executing
"set categories math science"
    ("uplevel" body line 1)
    invoked from within
"uplevel #0 set $optionName $option_defaults($optionName)"
    (procedure "default_check" line 11)
    invoked from within
"default_check categories {} r"
    (read trace on "categories")
    invoked from within
"llength $categories"
    (procedure "portlint::lint_main" line 291)
    invoked from within
"$procedure $targetname"
Error: Follow https://guide.macports.org/#project.tickets to report a bug.
Error: Processing of port octave-audio failed

Change History (6)

comment:1 Changed 6 years ago by mf2k (Frank Schima)

Port: octave-audio octave-benchmark octave-bim octave-bioinfo octave-bsltl octave-cgi octave-civil-engineering octave-communications octave-control octave-data-smoothing octave-database octave-dataframe octave-divand octave-doctest octave-econometrics octave-financial octave-fits octave-fl-core octave-fpl octave-fuzzy-logic-toolkit octave-ga octave-general octave-generate_html octave-geometry octave-gsl octave-ident octave-image octave-informationtheory octave-integration octave-interval octave-io octave-irsa octave-linear-algebra octave-ltfat octave-mapping octave-mechanics octave-miscellaneous octave-missing-functions octave-msh octave-multicore octave-mvn octave-nan octave-ncarray octave-netcdf octave-nnet octave-ocs octave-octclip octave-octproj octave-odebvp octave-optics octave-optim octave-optiminterp octave-outliers octave-parallel octave-plot octave-printpgf octave-quaternion octave-queueing octave-secs1d octave-signal octave-sockets octave-specfun octave-special-matrix octave-spline-gcvspl octave-splines octave-statistics octave-stk octave-strings octave-struct octave-symband octave-symbolic octave-tcl-octave octave-tisean octave-tsa octave-video octave-vrml octave-windows octave-zenity octave-zeromq removed
Priority: LowNormal
Summary: octave packages crash port lintoctave portgroup causes crash in port lint

comment:2 Changed 6 years ago by ryandesign (Ryan Carsten Schmidt)

Summary: octave portgroup causes crash in port lintoctave portgroup: can't read "categories": wrong # args

Of course this is not a crash. It's just an error message.

comment:3 Changed 6 years ago by herbygillot (Herby Gillot)

You're right, my fault.

comment:4 Changed 6 years ago by jmroot (Joshua Root)

In 8434c0eee6ba2867c09524697d6f95f3a095ddf0/macports-base (master):

Remove double-quoting requirement for 'default'

Update quoting of defaults to match and for consistency.

See: #57118

comment:5 Changed 6 years ago by jmroot (Joshua Root)

Owner: set to jmroot
Resolution: fixed
Status: newclosed

In 58369e469425fafca368aacb307b9234f6bc9022/macports-ports (master):

Update 'default' quoting in portgroups

Adapt to changes in base master, while maintaining compatibility with
the current release. Also adjust quoting for consistency: braces where
deferred evaluation is desired, otherwise quotes where the value
contains spaces, or no quoting if it is a single word literal.

Fixes: #57118

comment:6 Changed 6 years ago by jmroot (Joshua Root)

In 0137a4e0dd8b78aedba5055ed916a04b00a90070/macports-base (release-2.5):

Remove double-quoting requirement for 'default'

Update quoting of defaults to match and for consistency.

See: #57118
(cherry picked from commit 8434c0eee6ba2867c09524697d6f95f3a095ddf0)

Note: See TracTickets for help on using tickets.