search

Maximus5 / conemu-inside

An example, how to embed http://conemu.github.io/ into another graphical application
57 stars 42 forks source link

Error in the VS extension #42

Closed Therena closed 6 years ago

Therena commented 6 years ago

Hi,

I have a user of the VS extension which gets an error popup from ComEmu which I never saw before. Maybe you have a clue what is going wrong there …

Text the user posted:

I have 64 bit version of ConEmu installed. I installed the extension in VS 2017 Professional. I have set the path but when I try to open conemu window from VS - it shows a black console window with this error.

Unknown switch: /InsideWnd Unknown switch: /NoKeyHooks Unknown switch: /LoadCfgFile

Press Enter or Esc to close console, or wait...

Also it's not opening within Visual Studio.

https://marketplace.visualstudio.com/items?itemName=DavidRoller.ConEmuIntegration#review-details

Thanks.

Regards, David

Maximus5 commented 6 years ago

Looks like these switches were passed to started shell. First of all you should be sure user have installed latest ConEmu (Old Builds) But most probably user put these switches into startup command in your extension settings.

vsha041 commented 6 years ago

Hi Maximums5 - Here is a screenshot of the issue. What do you think? I have latest version of ConEmu installed. I opened a solution and then right clicked on the folder and clicked on "open folder in conemu" option. untitled

vsha041 commented 6 years ago

And this is how the process is started - as you said. But why it is starting like this?

untitled2

vsha041 commented 6 years ago

It works now. I gave a config xml file in the settings option. It works after removing it. But the very firs time I got this error. I didn't get it again. You can close this issue.

ConEmu 180528 [64]

Assertion: ConEmu 180528 [64] PID=10632, TID=650464 C:\Program Files\ConEmu\ConEmu64.exe conemu\src\conemu\vcongroup.cpp:5248

rcCon.right && rcCon.bottom

Press to continue your work. Press to throw exception, ConEmu will be terminated! Press to copy text information to clipboard

and report a bug (open project web page).

Abort Retry Ignore

Maximus5 commented 6 years ago

Extension starts ConEmuC64.exe instead of ConEmu64.exe! Obviously they are absolutely different executables and you get the error. Extension should check for valid name, if it allows user to select executable instead of installation folder.

vsha041 commented 6 years ago

Maximus5 - Please close this ticket as it is working fine.

Therena commented 6 years ago

Extension should check for valid name, if it allows user to select executable instead of installation folder.

For your interest: I added some path validation to the newest release of the extension-