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

ConEmu Inside seems not to react on startup shell configuration #27

Open
Therena opened this issue Jun 23, 2016 · 7 comments
Open

ConEmu Inside seems not to react on startup shell configuration #27

Therena opened this issue Jun 23, 2016 · 7 comments

Comments

@Therena
Copy link

Therena commented Jun 23, 2016

If I set a startup shell in the conemu config and use this config within conemu integration it isn't taken into account.
Setting:
23-06-_2016_21-43-42
Open ConEmu using ConEmu inside
23-06-_2016_21-48-50
This was originally a request from the user xsxxxxx on the page of the Visual Studio extension:
https://visualstudiogallery.msdn.microsoft.com/a0536370-40e4-4141-8f51-5f00d0434012

@Therena
Copy link
Author

Therena commented Jul 1, 2016

See also initial bug request from @PhilipDaniels:
Therena/ConEmuIntegration#6

To solve this bug I need the here requested functionality.

@hypersw
Copy link
Collaborator

hypersw commented Jul 1, 2016

ConEmu issues a command when it starts, as given in the startinfo, and then happens what's written in the label above — the startup task gets overridden with /cmd.
You can specify {Bash::Git bash} as the startup command (the default is {cmd}), and this would open the task. Would it do to read the task name from the config and supply it as the ConsoleProcessCommandLine into the startinfo?

Alternatively, I could make startinfo let an empty string for the ConsoleProcessCommandLine param (currently it asserts the command is not null or empty), which would skip passing the /cmd arg and effectively make conemu execute whatever is its default session named task.

@Therena
Copy link
Author

Therena commented Jul 4, 2016

@hypersw I will check this today evening.

@Therena
Copy link
Author

Therena commented Jul 4, 2016

@hypersw I added the possibility to the VS extension to set the default task in the configuration. This is a quick fix for the issue. But I think it would be nicer for the user to have in future the possibility to let the the configuration empty and conemu loads then the default task from the XML configuration.

For now the issue is solved but I would like to let it open until there is a possibility to use the default task from the conemu XML configuration.

Thanks for your help.

@Therena
Copy link
Author

Therena commented Jul 6, 2016

@hypersw Seems so that if I set default tasks to the ConsoleProcessCommandLine also other settings are affected.

And the initial working directory does not appear to be set, which is a regression compared to
previous versions when using the context-sensitive Open ConEmu menus in solution explorer.

(See comment from PhilipDaniels here: Therena/ConEmuIntegration#6)

Do you know why this could happen or how some fix for this?

@PhilipDaniels
Copy link

@Therena

In previous versions of ConEmuIntegration you were explicitly sending a
"cd" command to do it.

On 6 July 2016 at 12:48, David Roller notifications@github.com wrote:

@hypersw https://github.com/hypersw Seems so that if I set default
tasks to the ConsoleProcessCommandLine also other settings are affected.

And the initial working directory does not appear to be set, which is a
regression compared to
previous versions when using the context-sensitive Open ConEmu menus in
solution explorer.
(See comment from PhilipDaniels here: Therena/ConEmuIntegration#6
Therena/ConEmuIntegration#6)

Do you know why this could happen or how some fix for this?


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#27 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/ABubRGapKy8esa3du1N7xAmwxS306VWIks5qS5X_gaJpZM4I9LMg
.

Philip Daniels.

@Therena
Copy link
Author

Therena commented Jul 6, 2016

@PhilipDaniels Which "cd" command? There wasn't any change regarding the cd commands in the last version. The extension only send "cd" commands in case you use the context menu of the solution explorer.

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

No branches or pull requests

3 participants