Skip to content
This repository was archived by the owner on Nov 15, 2021. It is now read-only.

Support colon-delimited include paths in command-line front-end#20

Open
ender672 wants to merge 1 commit into
cldwalker:masterfrom
ender672:add_delimiter_to_include_path
Open

Support colon-delimited include paths in command-line front-end#20
ender672 wants to merge 1 commit into
cldwalker:masterfrom
ender672:add_delimiter_to_include_path

Conversation

@ender672

Copy link
Copy Markdown

Similar to the Ruby -I command, rdebug should support multiple load paths
separated by colons.

Similar to the Ruby -I command, rdebug should support multiple load paths
separated by colons.
@cldwalker

Copy link
Copy Markdown
Owner

Thanks for the pull but I need a test that tests this behavior as explained in Contributing of the readme. Warning, the testing suite is passing but pretty ugly. It's on my todo to clean up.

@ender672

ender672 commented Jun 8, 2012

Copy link
Copy Markdown
Author

Where do tests for command line switches go?

@cldwalker

Copy link
Copy Markdown
Owner

I don't see any tests for the executable and bin/rdebug needs to be moved to a class before this and most executable functionality can become testable. Blocked until refactored.

@eric-hu

eric-hu commented Jun 13, 2013

Copy link
Copy Markdown

While working on #57, I came up with a scheme to test rdebug as is. I'm testing it as a separate thread to figure out how rdebug works without changing it, and ensure nothing breaks when I finally get to the refactoring step.

@ender672 your feature should be testable in the same vein. It should be straightforward to test this pull request by working off what I've written on my fork.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants