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

Update to FHS version 3 #56

Merged
merged 2 commits into from Oct 19, 2021
Merged

Update to FHS version 3 #56

merged 2 commits into from Oct 19, 2021

Conversation

eribertomota
Copy link
Contributor

Rationale:

  1. mrtg.ok is a library (a set of information) or, if you prefer, a
    "database". The best for it place is /var/lib/mrtg.

  2. PID files must be moved from /var/run to /run.

  3. lock files should be put in /var/lock.

  4. The log file was moved to /var/log/mrtg/ to allow non-root users
    to create logs and to organize better the rotated logs.

  5. The main config file was moved to /etc/mrtg/ to make ease the use
    of separate configuration files and to provide a central location
    for configuration.

Rationale:

1. mrtg.ok is a library (a set of information) or, if you prefer, a
   "database". The best for it place is /var/lib/mrtg.

2. PID files must be moved from /var/run to /run.

3. lock files should be put in /var/lock.

4. The log file was moved to /var/log/mrtg/ to allow non-root users
   to create logs and to organize better the rotated logs.

5. The main config file was moved to /etc/mrtg/ to make ease the use
   of separate configuration files and to provide a central location
   for configuration.
@oetiker
Copy link
Owner

oetiker commented Oct 18, 2021

BUT, could you please also update the CHANGES file

@eribertomota
Copy link
Contributor Author

BUT, could you please also update the CHANGES file

Sure. I decided to update the changelog after each approval for a couple of changes to avoid mistakes. Is it ok for your? I like to write books and changelogs, so it is easy for me.

@oetiker
Copy link
Owner

oetiker commented Oct 19, 2021

for now it is fine yes :)

@oetiker oetiker merged commit 84dee09 into oetiker:master Oct 19, 2021
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

2 participants