Re: classdef problems

[Michael Goffioul]

On Thu, Sep 12, 2013 at 7:59 AM, Richard Crozier <address@hidden> wrote:

On 12/09/2013 12:20, John W. Eaton wrote:

On 09/12/2013 07:05 AM, Richard Crozier wrote:

On 12/09/2013 11:43, Richard Crozier wrote:

Quoting Richard Crozier <address@hidden> on Thu, 12 Sep 2013
11:26:12 +0100:

I'm using 498b2dd1bd56 (classdef).

Ok, apologies, I've realised my errors where largely due to using
addpath(genpath('.')) when testing rather than the full path name. I
still think there may be a bug in there somewhere though, would it be
possible for octave to resolve paths to overcome the ingenuity of fools
such as myself?

What did addpath (genpath ('.')) do incorrectly?  Did it add + and @
directories to the path?  Should it not do that?  If not, is it genpath
or addpath that should be fixed?  Or should it just warn you?  I can see
arguments for all possibilities, but probably the only thing that
matters is WDMD, because if we don't do what Matlab does, we will likely
see bug reports about the differences.

jwe

It added directories starting with '.' to the path. When I changed directory it could no longer find the paths, at least this is my guess at what is happening from the load path errors I am getting. I've checked and Matlab exhibits the same behaviour if you add relative paths, then change directory. addpath didn't do anything incorrectly, it did exactly what I asked, but I was asking something slightly silly.

However, further to this, I think there really is still some kind of issue with package folders in classdef, if I correctly add the package directory, I get issues:


address@hidden ~/src/nova_package $ octave-cli

warning: function ./+oandm/system.m shadows a built-in function
GNU Octave, version 3.7.5
Copyright (C) 2013 John W. Eaton and others.
This is free software; see the source code for copying conditions.
There is ABSOLUTELY NO WARRANTY; not even for MERCHANTABILITY or
FITNESS FOR A PARTICULAR PURPOSE.  For details, type 'warranty'.

Octave was configured for "i686-pc-linux-gnu".

Additional information about Octave is available at http://www.octave.org.

Please contribute if you find this software useful.
For more information, visit http://www.octave.org/get-involved.html

Read http://www.octave.org/bugs.html to learn how to submit bug reports.

For information about changes from previous versions, type 'news'.

octave-cli:1> addpath(genpath ("/home/richard/src/<snip>"))
warning: function /home/richard/src/nova_package/renewnet-foundry/matlab-octave/tb-multicore/gethostname.m shadows a built-in function
warning: function /home/richard/src/<snip>/+oandm/system.m shadows a built-in function

Could you report what genpath ("/home/richard/src/<snip>") actually generates, or what's the content of path after the addpath call?

Michael.