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

Show command adds \n once in a while #138

Closed
oleiade opened this issue Oct 18, 2014 · 2 comments
Closed

Show command adds \n once in a while #138

oleiade opened this issue Oct 18, 2014 · 2 comments
Labels

Comments

@oleiade
Copy link
Owner

oleiade commented Oct 18, 2014

This is a weird bug. Here is the output of the show command repeated mulitple times:

trousseau · (develop) ★  ⟩ ./bin/trousseau --store /tmp/testtesttest show                                                                       ~/D/S/g/s/g/o/trousseau
easy as : 123
hallo : ich bin hungrig
abc : 456


trousseau · (develop) ★  ⟩ ./bin/trousseau --store /tmp/testtesttest show                                                                       ~/D/S/g/s/g/o/trousseau
abc : 456

easy as : 123
hallo : ich bin hungrig

trousseau · (develop) ★  ⟩ ./bin/trousseau --store /tmp/testtesttest show                                                                       ~/D/S/g/s/g/o/trousseau
hallo : ich bin hungrig
abc : 456

easy as : 123

trousseau · (develop) ★  ⟩ ./bin/trousseau --store /tmp/testtesttest show                                                                       
easy as : 123
hallo : ich bin hungrig
abc : 456

Newline statements seem to be inserted pretty randomly in the output. Of course, in order to be able to parse easily the output this should not happen.

@oleiade oleiade added the bug label Oct 18, 2014
@oleiade oleiade added this to the 0.3.5 milestone Oct 18, 2014
@jd1123
Copy link
Contributor

jd1123 commented Oct 20, 2014

@oleiade - i think this is a bug from my pull #137.
I get this bug if i use the stdin feature:
trousseu set key1
not if i use
trousseau set key1 value1

Looking into it....

jd1123 added a commit to jd1123/trousseau that referenced this issue Oct 20, 2014
* edited cmd/trousseu/commands.go to not take the delimiting character
  in ReadString()
@jd1123
Copy link
Contributor

jd1123 commented Oct 20, 2014

indeed it is - fixed in 11ef00d

oleiade pushed a commit that referenced this issue Oct 20, 2014
@oleiade oleiade closed this as completed Oct 20, 2014
oleiade pushed a commit that referenced this issue Nov 9, 2014
* edited cmd/trousseu/commands.go to not take the delimiting character
  in ReadString()
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants