Skip to content

CWD Tilde handling #287

@vyeevani

Description

@vyeevani

Not sure if this is necessary a feature or just a fix for some so I'll classify it as a feature for now. Fail to find "lldb-mi" when the cwd has a "~" expansion in it.

Activity

vyeevani

vyeevani commented on Sep 6, 2021

@vyeevani
Author

According to this, the child process developers rejected handling cwd tilde expansion so it would have to be handled inside this repo: nodejs/node#684

GitMensch

GitMensch commented on Sep 6, 2021

@GitMensch
Collaborator

The feature implementation like that sounds easy:

cwd = cwd.replace(/^~/, os.homedir());

But I think this then may also be applied not to cwd, but also to the program and gdb/lldb executable and the argument list - if those don't work already.

vyeevani

vyeevani commented on Sep 6, 2021

@vyeevani
Author

I’ve made this PR to resolve this with untildify so that the support isn’t hard coded here as a string replacement. #288

vyeevani

vyeevani commented on Sep 7, 2021

@vyeevani
Author

@GitMensch do you happen to have write access to get this merged?

GitMensch

GitMensch commented on Sep 7, 2021

@GitMensch
Collaborator

@vyeevani yes, but I won't do it now as @WebFreak001 had more or less the same questions that I had, please see the PR comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

      Participants

      @GitMensch@vyeevani

      Issue actions

        CWD Tilde handling · Issue #287 · WebFreak001/code-debug