chkwon / PATHSolver.jl

provides a Julia wrapper for the PATH Solver for solving mixed complementarity problems
http://pages.cs.wisc.edu/%7Eferris/path.html
MIT License
50 stars 15 forks source link

Different Windows Issue #35

Closed arnavs closed 5 years ago

arnavs commented 5 years ago

I think the install script fails when your user directory has a space in it, as Windows allows. See the Copy-Item line of:

┌ Error: Error building `PATHSolver`:
│
│ 7-Zip 18.05 (x64) : Copyright (c) 1999-2018 Igor Pavlov : 2018-04-30
│
│ Scanning the drive for archives:
│ 1 file, 3860585 bytes (3771 KiB)
│
│ Extracting archive: C:\Users\Arnav Sood\.julia\packages\PATHSolver\xjazJ\deps\downloads\pathlib.zip
│ --
│ Path = C:\Users\Arnav Sood\.julia\packages\PATHSolver\xjazJ\deps\downloads\pathlib.zip
│ Type = zip
│ Physical Size = 3860585
│ Comment = a11966f36875748820583e41455800470c971171
│
│ Everything is Ok
│
│ Folders: 13
│ Files: 43
│ Size:       11230861
│ Compressed: 3860585
│ Copy-Item : A positional parameter cannot be found that accepts argument
│ 'Sood\.julia\packages\PATHSolver\xjazJ\deps\src\pathlib-4.7.03\lib\win64\path47.dll'.
│ At line:1 char:1
│ + Copy-Item -Path C:\Users\Arnav Sood\.julia\packages\PATHSolver\xjazJ\ ...
│ + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
│     + CategoryInfo          : InvalidArgument: (:) [Copy-Item], ParameterBindingException
│     + FullyQualifiedErrorId : PositionalParameterNotFound,Microsoft.PowerShell.Commands.CopyItemCommand
│
│ [ Info: Attempting to create directory C:\Users\Arnav Sood\.julia\packages\PATHSolver\xjazJ\deps\usr\lib
│ [ Info: Attempting to create directory C:\Users\Arnav Sood\.julia\packages\PATHSolver\xjazJ\deps\src
│ [ Info: Changing directory to C:\Users\Arnav Sood\.julia\packages\PATHSolver\xjazJ\deps\src
│ [ Info: Attempting to create directory C:\Users\Arnav Sood\.julia\packages\PATHSolver\xjazJ\deps\downloads
│ [ Info: Changing directory to C:\Users\Arnav Sood\.julia\packages\PATHSolver\xjazJ\deps\src
│ [ Info: Downloading file https://github.com/ampl/pathlib/archive/4.7.03.zip
│ [ Info: Changing directory to C:\Users\Arnav Sood\.julia\packages\PATHSolver\xjazJ\deps\src
│ [ Info: Changing directory to C:\Users\Arnav Sood\.julia\packages\PATHSolver\xjazJ\deps\src
│ [ Info: Done downloading file https://github.com/ampl/pathlib/archive/4.7.03.zip
│ [ Info: Changing directory to C:\Users\Arnav Sood\.julia\packages\PATHSolver\xjazJ\deps\src
│ [ Info: Attempting to create directory C:\Users\Arnav Sood\.julia\packages\PATHSolver\xjazJ\deps
│ [ Info: Directory C:\Users\Arnav Sood\.julia\packages\PATHSolver\xjazJ\deps already exists
│ [ Info: Changing directory to C:\Users\Arnav Sood\.julia\packages\PATHSolver\xjazJ\deps\src
│ [ Info: Changing directory to C:\Users\Arnav Sood\.julia\packages\PATHSolver\xjazJ\deps\src
│ ERROR: LoadError: failed process: Process(`powershell -NoProfile -Command 'Copy-Item -Path C:\Users\Arnav Sood\.julia\packages\PATHSolver\xjazJ\deps\src\pathlib-4.7.03\lib\win64\path47.dll -Destination C:\Users\Arnav Sood\.julia\packages\PATHSolver\xjazJ\deps\usr\lib -force'`, ProcessExited(1)) [1]
│ Stacktrace:
│  [1] error(::String, ::Base.Process, ::String, ::Int64, ::String) at .\error.jl:42
│  [2] pipeline_error at .\process.jl:785 [inlined]
│  [3] #run#515(::Bool, ::Function, ::Cmd) at .\process.jl:726
│  [4] run(::Cmd) at .\process.jl:724
│  [5] run(::BinDeps.SynchronousStepCollection) at C:\Users\Arnav Sood\.julia\packages\BinDeps\ZEval\src\BinDeps.jl:521 (repeats 2 times)
│  [6] satisfy!(::BinDeps.LibraryDependency, ::Array{DataType,1}) at C:\Users\Arnav Sood\.julia\packages\BinDeps\ZEval\src\dependencies.jl:944
│  [7] satisfy!(::BinDeps.LibraryDependency) at C:\Users\Arnav Sood\.julia\packages\BinDeps\ZEval\src\dependencies.jl:922
│  [8] top-level scope at C:\Users\Arnav Sood\.julia\packages\BinDeps\ZEval\src\dependencies.jl:977
│  [9] include at .\boot.jl:326 [inlined]
│  [10] include_relative(::Module, ::String) at .\loading.jl:1038
│  [11] include(::Module, ::String) at .\sysimg.jl:29
│  [12] include(::String) at .\client.jl:403
│  [13] top-level scope at none:0
│ in expression starting at C:\Users\Arnav Sood\.julia\packages\PATHSolver\xjazJ\deps\build.jl:96

For context:

julia> versioninfo()
Julia Version 1.1.0
Commit 80516ca202 (2019-01-21 21:24 UTC)
Platform Info:
  OS: Windows (x86_64-w64-mingw32)
  CPU: Intel(R) Core(TM) i7-7700 CPU @ 3.60GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-6.0.1 (ORCJIT, skylake)
chkwon commented 5 years ago

Can you try ] add PATHSolver#master and ] build PATHSolver?

arnavs commented 5 years ago

Sure, will give this a try at the office and let you know.

arnavs commented 5 years ago

@chkwon This works great, thanks for the fix.

chkwon commented 5 years ago

Awesome!