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

Initial Archlinux support added, to replace debian sudoers file. #12

Merged
merged 1 commit into from
Nov 19, 2012
Merged

Initial Archlinux support added, to replace debian sudoers file. #12

merged 1 commit into from
Nov 19, 2012

Conversation

aboe76
Copy link
Contributor

@aboe76 aboe76 commented Nov 18, 2012

No description provided.

saz added a commit that referenced this pull request Nov 19, 2012
Initial Archlinux support added, to replace debian sudoers file.
@saz saz merged commit 230a767 into saz:master Nov 19, 2012
@saz
Copy link
Owner

saz commented Nov 19, 2012

Merged. Thanks!

@saz
Copy link
Owner

saz commented Nov 19, 2012

New release uploaded to the forge.

@aboe76
Copy link
Contributor Author

aboe76 commented Nov 19, 2012

Steffen,

I made a little mistake, with the sudoers.archlinux file, it is missing
Newline at the end of the file..
-#includedir /etc/sudoers.d
\ No newline at end of file

see attachment.

2012/11/19 Steffen Zieger notifications@github.com

New release uploaded to the forge.


Reply to this email directly or view it on GitHubhttps://github.com//pull/12#issuecomment-10511939.

@saz
Copy link
Owner

saz commented Nov 29, 2012

Thanks, but the file is totally fine and working for me. I don't think, there is anything to fix.

@kingcody
Copy link
Contributor

kingcody commented Feb 1, 2014

Hi there, I've actually recently started using your module and I can say that the 'no new line' at end of file does cause errors. It seams that the include line does not get processed with out it. I've be able to replicate this on several up-to-date ArchLinux installs and also fix the issue by adding a new line to the end of the template.

Please consider reviewing this again, thanks.

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

3 participants