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

add CWD in server status output #1

Merged
merged 1 commit into from
Nov 24, 2014
Merged

add CWD in server status output #1

merged 1 commit into from
Nov 24, 2014

Conversation

ulidtko
Copy link

@ulidtko ulidtko commented Nov 24, 2014

README mentions that filenames are resolved relative to the server's CWD (current working directory) - rather than to the client's.

While it's technically possible to intrusively determine working path of a foreign process (with tools like readlink /proc/$PID/cwd, or proceexp.exe on Windows) — these methods are rarely userfriendly and mostly inconvenient for regular use (though excellent for troubleshooting/debug).

So, this addition can be very useful to simplify hdevtools setup and plugin integration, especially on Windows.

mvoidex added a commit that referenced this pull request Nov 24, 2014
add CWD in server status output
@mvoidex mvoidex merged commit 381c1aa into mvoidex:master Nov 24, 2014
@mvoidex
Copy link
Owner

mvoidex commented Nov 24, 2014

Thanks!

@ulidtko
Copy link
Author

ulidtko commented Nov 24, 2014

Thanks for the port, too!

I generally prefer to avoid Windows for development work, but sometimes there's no other choice (or it's just too hard). Frankly, your work has been useful and I admit that.

By the way, any plans on merging this into bitc/hdevtools, at least as a windows-port branch?
I think I can prepare a PR easily.

@ulidtko
Copy link
Author

ulidtko commented Nov 24, 2014

Owh, I see... bitc#15

@mvoidex
Copy link
Owner

mvoidex commented Nov 24, 2014

Yes, I support idea of merging

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

Successfully merging this pull request may close these issues.

2 participants