[Octave-bug-tracker] [bug #53124] [MXE Octave] install in path that includes spaces

[Markus Mützel]

Follow-up Comment #13, bug #53124 (project octave):

I re-installed Octave 7 (hg id 865f8ddb2c78) with pre-compiled packages (minus
the ones that are incompatible at the moment) to "C:\Program Files\GNU
Octave\Octave-7.0.0".
Trying to re-install the geometry package as an example, I ran into the
following:

>> pkg install -forge geometry
clipper.cpp: In function 'void ClipperLib::InitEdge(ClipperLib::TEdge*,
ClipperLib::TEdge*, ClipperLib::TEdge*, const ClipperLib::IntPoint&)':
clipper.cpp:750:34: warning: 'void* memset(void*, int, size_t)' clearing an
object of non-trivial type 'struct ClipperLib::TEdge'; use assignment or
 value-initialization instead [-Wclass-memaccess]
  750 |   std::memset(e, 0, sizeof(TEdge));
      |                                  ^
clipper.cpp:95:8: note: 'struct ClipperLib::TEdge' declared here
   95 | struct TEdge {
      |        ^~~~~
error: couldn't delete directory C:\Program Files\GNU
Octave\Octave-7.0.0\mingw64\share\octave\packages\geometry-4.0.0: Permission
denied
error: called from
    uninstall at line 132 column 11
    install at line 211 column 9
    pkg at line 513 column 9


Executing the same command when Octave is started with elevated rights, I see
the following:

>> pkg install -forge geometry

clipper.cpp: In function 'void ClipperLib::InitEdge(ClipperLib::TEdge*,
ClipperLib::TEdge*, ClipperLib::TEdge*, const ClipperLib::IntPoint&)':
clipper.cpp:750:34: warning: 'void* memset(void*, int, size_t)' clearing an
object of non-trivial type 'struct ClipperLib::TEdge'; use assignment or
 value-initialization instead [-Wclass-memaccess]
  750 |   std::memset(e, 0, sizeof(TEdge));
      |                                  ^
clipper.cpp:95:8: note: 'struct ClipperLib::TEdge' declared here
   95 | struct TEdge {
      |        ^~~~~
C:\PROGRA~1\GNUOCT~1\OCTAVE~2.0\mingw64\bin\makeinfo: warning: error loading
./Config: Can't locate ./Config at C:\PROGRA~1\GNUOCT~1\OCTAVE~2.0\ming
w64\bin\makeinfo line 345.

C:\PROGRA~1\GNUOCT~1\OCTAVE~2.0\mingw64\bin\makeinfo: warning: error loading
./Config: Can't locate ./Config at C:\PROGRA~1\GNUOCT~1\OCTAVE~2.0\ming
w64\bin\makeinfo line 345.

[snip]

C:\PROGRA~1\GNUOCT~1\OCTAVE~2.0\mingw64\bin\makeinfo: warning: error loading
./Config: Can't locate ./Config at C:\PROGRA~1\GNUOCT~1\OCTAVE~2.0\ming
w64\bin\makeinfo line 345.

C:\PROGRA~1\GNUOCT~1\OCTAVE~2.0\mingw64\bin\makeinfo: warning: error loading
./Config: Can't locate ./Config at C:\PROGRA~1\GNUOCT~1\OCTAVE~2.0\ming
w64\bin\makeinfo line 345.

For information about changes from previous versions of the geometry package,
run 'news geometry'.


The makeinfo issue is probably something different we should look into. But
overall it looks like the package installed fine.
Loading the package and running "pkg test geometry" seems to work fine as
well. The tests (minus the known issues with @svg) also succeed when
subsequently running Octave without elevated rights.

Is it OK if users would need to run Octave with elevated rights when they want
to update global packages?
I think it is. Especially taking PCs into account that are shared between
multiple users.

Btw, I'm seeing the same already for Octave 5.2.0 installed with the official
installer in "C:\Octave\Octave-5.2.0":

>> pkg install -forge geometry

error: the following dependencies were unsatisfied:
   geometry needs matgeom >= 1.0.0
>> pkg install -forge matgeom
couldn't create installation directory
C:\Octave\OCTAVE~1.0\mingw64\share\octave\packages\matgeom-1.
2.2 : Permission denied
error: called from
    install>copy_files at line 490 column 7
    install at line 225 column 7
    pkg at line 441 column 9
>>


And when running with elevated rights:

>> pkg install -forge matgeom

C:\Octave\OCTAVE~1.0\mingw64\bin\makeinfo: warning: error loading ./Config:
Can't locate ./Config at
 C:\Octave\OCTAVE~1.0\mingw64\bin\makeinfo line 345.

C:\Octave\OCTAVE~1.0\mingw64\bin\makeinfo: warning: error loading ./Config:
Can't locate ./Config at
 C:\Octave\OCTAVE~1.0\mingw64\bin\makeinfo line 345.

For information about changes from previous versions of the matgeom package,
run 'news matgeom'.



    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?53124>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/