Skip to content

Use global $stdin instead of local variable #284

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

Open
sagiru opened this issue Sep 11, 2019 · 3 comments
Open

Use global $stdin instead of local variable #284

sagiru opened this issue Sep 11, 2019 · 3 comments

Comments

@sagiru
Copy link

sagiru commented Sep 11, 2019

The stdin bypass will use a local variable stdin instead of $stdin and
it will result in the following error:

''''
Traceback (most recent call last):
2: from /usr/lib/ruby/vendor_ruby/svn2git/migration.rb:432:in block (2 levels) in run_command' 1: from /usr/lib/ruby/vendor_ruby/svn2git/migration.rb:432:in loop'
/usr/lib/ruby/vendor_ruby/svn2git/migration.rb:438:in block (3 levels) in run_command': undefined local variable or method stdin' for #Svn2Git::Migration:0x0000555b7c945cc8 (NameError)
Did you mean? String
''''
If will file a merge request to fix it.

[1] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=905832

@lhuria94
Copy link

lhuria94 commented Dec 8, 2019

Did you find a solution for this?

@VenkeyK
Copy link

VenkeyK commented Mar 11, 2020

Hi,
Does anyone find a solution for the above error?

@sagiru
Copy link
Author

sagiru commented Mar 12, 2020

Hi,
Does anyone find a solution for the above error?

Hi,

i filed a pullrequest (285), maybe it's helpful.

Regards
Sascha

skull-squadron added a commit to skull-squadron/svn2git that referenced this issue Jan 14, 2024
nirvdrum#308

The proper way to do subprocess I/O is to do less.
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

4 participants
@sagiru @lhuria94 @VenkeyK and others