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

Added additional header template for apt.conf style comments #540

Merged
merged 1 commit into from
Jun 22, 2015
Merged

Added additional header template for apt.conf style comments #540

merged 1 commit into from
Jun 22, 2015

Conversation

szynaka
Copy link

@szynaka szynaka commented Jun 19, 2015

Added additional header comment file for apt.conf files.

apt.conf does not use # as the comment character. Using # as the comment character can cause errors like this:
"Error: /Stage[main]/base::Install/Package[bash]/ensure: change from 3.2-4 to latest failed: Could not get latest version: Execution of '/usr/bin/apt-cache policy bash' returned 100: E: Syntax error /etc/apt/apt.conf.d/15update-stamp:2: Extra junk after value"

From the apt.conf man page:
"Lines starting with // are treated as comments (ignored), as well as all text between /* and */, just like C/C++ comments."
http://manpages.ubuntu.com/manpages/vivid/man5/apt.conf.5.html#contenttoc2

@daenney
Copy link

daenney commented Jun 22, 2015

Uhg, I keep forgetting about apt's wildly inconsistent and special commenting semantics. There's like three different ones 😞.

daenney added a commit that referenced this pull request Jun 22, 2015
Added additional header template for apt.conf style comments
@daenney daenney merged commit a1907ff into puppetlabs:master Jun 22, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants