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

fix ycp killing bug with changing \r with space #19

Closed
wants to merge 1 commit into from

Conversation

jreidinger
Copy link
Member

bugfix lost forgotten to create sr

@mchf
Copy link
Member

mchf commented Jun 7, 2017

lgtm but mess in repo

@mvidner
Copy link
Member

mvidner commented Jun 12, 2017

@jreidinger @mchf Both of you write so broken English that I don't understand what you mean to say! You will make me fix the grammar checker so that I don't have to put up with this...

@mvidner
Copy link
Member

mvidner commented Jun 26, 2017

@jreidinger Please add a changelog entry. Which bugfix do you mean, is there a bnc#? Which SR do you mean?

@jreidinger
Copy link
Member Author

@mvidner wrong accronym, I mean forgotten branch. There is not bug created for it, just result of found bug in ycp killer, which expend such characters like \n \r or \t. So I search old ycp sources for explicit usage of this characters and then create fix branch for all of them, just forget to create pull request for this one.

@teclator
Copy link
Contributor

After checking the code, the changes are already there so closing it after discussing with @jreidinger on IRC

@teclator teclator closed this Apr 13, 2018
@dgdavid dgdavid deleted the fix_killing branch July 19, 2019 12:08
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.

None yet

4 participants