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 newline under each section #18

Merged
merged 1 commit into from
Jun 6, 2016
Merged

Fix newline under each section #18

merged 1 commit into from
Jun 6, 2016

Conversation

sgnl05
Copy link
Owner

@sgnl05 sgnl05 commented Jun 1, 2016

@edestecd I'm trying to fix an issue where every section in sssd.conf gets written without being split by a new line. Do you have this issue? Can you please test how this commit affects your setup?

@sgnl05 sgnl05 added this to the 0.3.1 milestone Jun 1, 2016
@edestecd
Copy link
Collaborator

edestecd commented Jun 1, 2016

@sgnl05 yes I saw that awhile back. I think maybe the ruby 1.9.3 ordering introduced it.
I didn't complain at the time, b/c it didn't break anything. sssd does not seem to care.

It definitely used to have newlines between sections and I do think it is more readable that way.
What you have looks correct to me...
We have one big change in the pipeline at the moment, but I'll give it a go after that. Should be in the next day or so.

@edestecd
Copy link
Collaborator

edestecd commented Jun 6, 2016

@sgnl05 I tested this in our environment with no issue. Login continues to work as expected on RHEL 5,6,7 and Ununtu 14.04 LTS

@edestecd edestecd merged commit 0d677cc into master Jun 6, 2016
@sgnl05
Copy link
Owner Author

sgnl05 commented Jun 6, 2016

Great, thanks!

sgnl05 added a commit that referenced this pull request Aug 23, 2016
@sgnl05 sgnl05 deleted the newline branch October 29, 2016 19:12
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.

2 participants