Giter Site home page Giter Site logo

Comments (7)

FelixSelter avatar FelixSelter commented on August 18, 2024 1

Yup, you are right. This seems to happen when there are multiple java versions inside your path. The script was supposed to remove them automatically but only does so when there's only one version. I will look into this as fast as possible. For now please remove them manually and everything should work fine. If you have any other questions feel free to ask. Thank you for your contribution
1
2
3

from jenv-for-windows.

serg06 avatar serg06 commented on August 18, 2024

I tried where java and it gave me this:

C:\Program Files\Common Files\Oracle\Java\javapath\java.exe
C:\Program Files (x86)\Common Files\Oracle\Java\javapath\java.exe
C:\tools\jenv\java.bat

Does that mean I should get rid of the first two, so that the output becomes

C:\tools\jenv\java.bat

?

from jenv-for-windows.

FelixSelter avatar FelixSelter commented on August 18, 2024

Yea that's how it's supposed to be. I pushed hopefully a fix. Please test it.

from jenv-for-windows.

FelixSelter avatar FelixSelter commented on August 18, 2024

Now the script should remove at least any java from the user path. There could still be issues if you have java inside your system/machine path but java does not put them there when installed. Where java should only point to jenvs java.bat

from jenv-for-windows.

serg06 avatar serg06 commented on August 18, 2024

@FelixSelter I pulled the latest code, and jenv ran way faster and didn't mess up my terminal, but where java still shows those two .exes. They're in my System's PATH, but I might have added them there manually, I don't remember.

There could still be issues if you have java inside your system/machine path but java does not put them there when installed.

If you don't remove them from the System's PATH, perhaps you could give a warning that the user needs to do it manually?

from jenv-for-windows.

FelixSelter avatar FelixSelter commented on August 18, 2024

Hi, Please update again. I changed the code so it requests you to restart with admin rights so it can automatically correct the system path. Thank you for your suggestion. Can this be closed now?

from jenv-for-windows.

serg06 avatar serg06 commented on August 18, 2024

Seems to work, thanks!

from jenv-for-windows.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.