Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing popd path in make.bat #10733

Open
flysand7 opened this issue Jul 10, 2021 · 3 comments
Open

Missing popd path in make.bat #10733

flysand7 opened this issue Jul 10, 2021 · 3 comments
Labels
Bug This tag is applied to issues which reports bugs.

Comments

@flysand7
Copy link

flysand7 commented Jul 10, 2021

V version: V full version: V 0.2.2 44e78a6
OS: windows, Microsoft Windows 10 Home Single Language v19042 64-bit

Additional potentially-useful details:

This system has MSVC and GCC compilers installed and the command is ran under Visual Studio Developers command prompt, meaning MSVC environment is on. GCC is in PATH.

What did you do?

  1. Run make.bat
  2. Notice how the script leaves a directory in the pushd stack.

Note:

In order to see pushd stack set the windows cmd prompt to a one that displays directory stack ($+ format specifier). An example of such prompt:

prompt $+$p$g 

(copy together with the space at the end)

This will print the number of plus signs corresponding to the height of the directory stack. Running popd decreases the number of pluses, and running pushd increases the number of pluses.

If you want to reset the prompt either restart the shell or type the following command (if you used default prompt):

prompt $p$g 

Note 2:

Alternative way to see this effect is to run the script from some subdirectory of the project root:

..\make.bat

And noticing that the directory after running the script is not restored.

What did you expect to see?

The height of the directory stack should be the same before and after running the make.bat script.

What did you see instead?

The script leaves the compiler root directory in the stack without popping:

c:\programs\v> ./make.bat
+c:\programs\v>
@flysand7 flysand7 added the Bug This tag is applied to issues which reports bugs. label Jul 10, 2021
@danieldaeschle
Copy link
Member

Do you want to make a PR for this if you know the solution?

@flysand7
Copy link
Author

I have looked through the code, unfortunately I could not figure out why that happens. I'm not interested in contributing to V that much

@felipensp
Copy link
Member

I think that has been already fixed. Confirm @JalonSolov @spytheman ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug This tag is applied to issues which reports bugs.
Projects
None yet
Development

No branches or pull requests

3 participants