Skip to content
This repository has been archived by the owner on Jun 20, 2023. It is now read-only.

PlatformIO Home: "Show at startup" checkbox not working #1395

Closed
wurmtal868 opened this issue Jan 16, 2018 · 9 comments
Closed

PlatformIO Home: "Show at startup" checkbox not working #1395

wurmtal868 opened this issue Jan 16, 2018 · 9 comments

Comments

@wurmtal868
Copy link

Configuration

Operating system: Windows 7

PlatformIO Version (platformio --version): Home 0.6.1·Core 3.5.1a6

Description of problem

"Show at startup" has no effect. PlatformIO Home will be shown no matter if the box is checked or not

Steps to Reproduce

  1. Uncheck "Show at startup" on Home tab
  2. Close the HOME tab
  3. Restart Atom.IO

Actual Results

Atom.IO will open PlatformIO Home tab in foreground

Expected Results

Atom.IO should restore the last state of tabs without the PlatformIO Home tab

@ivankravets
Copy link
Member

Did you overwrite HOME environment variable?

@wurmtal868
Copy link
Author

I have a HOME variable defined in Win 7 and PlatformIO is using this folder. I have not defined any HOME in PlatformIO-IDE

@ivankravets
Copy link
Member

ivankravets commented Jan 17, 2018

It seems that JS does not handle it in the same way as Python does. Please add extra http://docs.platformio.org/en/latest/envvars.html#envvar-PLATFORMIO_HOME_DIR environment variable.

Also, don't forget to restart IDE.

@wurmtal868
Copy link
Author

The extra HOME variable did the trick.
This is nothing I would do intuitive.

@ivankravets
Copy link
Member

Ok, let's debug it together.

https://github.com/atom/fs-plus/blob/master/src/fs-plus.coffee#L20

Please provide an output of

echo %HOME%
echo %USERPROFILE%

@wurmtal868
Copy link
Author

Here is the output:

C:\Users\wurmtal>echo %HOME%
c:\home

C:\Users\wurmtal>echo %USERPROFILE%
C:\Users\wurmtal

@ivankravets
Copy link
Member

Good, I made PR atom/fs-plus#44

If they don't merge it, I'll implement own getHomeDirectory for Node.JS.

@ivankravets
Copy link
Member

Should work in the latest release

@wurmtal868
Copy link
Author

I removed the environment variable.
It's working fine.
Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants