newmat 10 please enable the use of namespace

[slask@…]

Newmat is capable of using c++ namespaces. This needs to be enabled during build, otherwise one gets linker errors if one uses the namespace in user code.

Namespaces is a good thing, and should in my opinion be enabled by default for these kind of packages. I understand if turning it on breaks other peoples build, but in that case one may use a variant.

I suggest that the use of namespace is turned on, and if this is not possible because of breaking other applications, adding a variant.

I made the change locally with the "port edit" command, appending the lines configure.cxxflags-append -Duse_namespace to the portfile.

[jameskyle@…]

I've added the build options. I tested on my system though and namespaces seem to work with or without this flag.

I get no compile or runtime errors on this regardless of flags, could you provide an example please:

#include <newmat/newmat.h>
#include <iostream>

using namespace std;

namespace foo {
  struct Foo {
    int bar;
    int bah;
    LogAndSign sign;
  };
}

int main (int argc, char const *argv[])
{
  cout << "time be time" << endl;
  foo::Foo myfoo;
  Real baz(26);
  

  cout << myfoo.sign.Value() << endl;
  
  return 0;
}

[ryandesign@…]

James committed the change in r62813 and I committed a revbump in r62851 because this presumably changes how the software builds.