Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#46516 closed defect (fixed)

octave-tsa @4.2.7: ignored errors during build

Reported by: ryandesign (Ryan Carsten Schmidt) Owned by: Schamschula (Marius Schamschula)
Priority: Normal Milestone:
Component: ports Version: 2.3.99
Keywords: Cc: kurthindenburg (Kurt Hindenburg)
Port: octave-tsa

Description

The following errors are printed while building octave-tsa 4.2.7, though they don't prevent the build from completing:

ls: /usr/local/MATLAB: No such file or directory
ls: /usr/local/MATLAB/R*: No such file or directory
ls: /usr/local/MATLAB/R*: No such file or directory
make: /bin/mexext: Command not found
ls: /usr/local/MATLAB/R*: No such file or directory
make: /bin/mexext: Command not found

It seems wrong that the port would be looking for anything in /usr/local.

Attachments (2)

main.log (197.5 KB) - added by ryandesign (Ryan Carsten Schmidt) 10 years ago.
patch-src_Makefile.diff (7.0 KB) - added by Schamschula (Marius Schamschula) 10 years ago.

Download all attachments as: .zip

Change History (8)

Changed 10 years ago by ryandesign (Ryan Carsten Schmidt)

Attachment: main.log added

comment:1 in reply to:  description ; Changed 10 years ago by Schamschula (Marius Schamschula)

Replying to ryandesign@…:

The following errors are printed while building octave-tsa 4.2.7, though they don't prevent the build from completing:

ls: /usr/local/MATLAB: No such file or directory
ls: /usr/local/MATLAB/R*: No such file or directory
ls: /usr/local/MATLAB/R*: No such file or directory
make: /bin/mexext: Command not found
ls: /usr/local/MATLAB/R*: No such file or directory
make: /bin/mexext: Command not found

It seems wrong that the port would be looking for anything in /usr/local.

Note that there is no MatLab port, as it is an expensive App sold by Mathworks. I see no point in removing the code checking to see if it is installed. Configure scripts do this all the time.

comment:2 in reply to:  1 Changed 10 years ago by larryv (Lawrence Velázquez)

If octave-tsa installs differently depending on whether MATLAB is present, then the code must be removed.

Changed 10 years ago by Schamschula (Marius Schamschula)

Attachment: patch-src_Makefile.diff added

comment:3 Changed 10 years ago by Schamschula (Marius Schamschula)

The attached patch file completely comments out everything related to MatLab.

comment:4 Changed 10 years ago by kurthindenburg (Kurt Hindenburg)

Cc: khindenburg@… added

Cc Me!

comment:5 Changed 10 years ago by kurthindenburg (Kurt Hindenburg)

Resolution: fixed
Status: newclosed

it looks like this has already been fixed

comment:6 in reply to:  5 Changed 10 years ago by Schamschula (Marius Schamschula)

Replying to khindenburg@…:

it looks like this has already been fixed

Superseded by https://trac.macports.org/ticket/46650

fixed in

https://trac.macports.org/changeset/133313

Note: See TracTickets for help on using tickets.