config file is stored in /root

Bug #125649 reported by James Troup
6
Affects Status Importance Assigned to Milestone
oprofile (Ubuntu)
Fix Released
Medium
Soren Hansen

Bug Description

Binary package hint: oprofile

The config file of oprofiled is stored in /root/.oprofile/daemonrc. This seems, err, less than optimal and must surely violate the FHS, Debian and/or Ubuntu policy.

Related branches

Revision history for this message
Scott Kitterman (kitterman) wrote :

Are you running in a real root acount instead of using sudo? I looked and none of the oprofile .debs ship a config file so it looks like the application is creating it based on what it things the environment is.

Changed in oprofile:
status: New → Incomplete
Revision history for this message
James Troup (elmo) wrote :

Yes, we're running it as root.

Revision history for this message
Scott Kitterman (kitterman) wrote :

In that case this is perfectly normal. Many applications store user specific configuration files like that.

Changed in oprofile:
status: Incomplete → Invalid
Revision history for this message
James Troup (elmo) wrote :

Errr, no, oprofiled is a daemon that runs as root. It absolutely should not store it's config in /root/. If upstream defaults to that, we should fix it. That's what Debian/Ubuntu packaging is about. If you still disagee, I invite you to show me any other daemon that stores it's config file in /root/.

Changed in oprofile:
status: Invalid → New
Revision history for this message
Scott Kitterman (kitterman) wrote :

Nevermind. I agree with you.

Changed in oprofile:
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
Soren Hansen (soren) wrote :

I'm all over it.

Changed in oprofile:
assignee: nobody → shawarma
Revision history for this message
Soren Hansen (soren) wrote :

oprofile (0.9.2-4ubuntu1) gutsy; urgency=low

  * Move configuration file to /etc/oprofile, update documentation, and
    handle the migration in postinst and migrate any existing config on
    upgrades (LP: #125649).
  * DebianMaintainerField.

 -- Soren Hansen <email address hidden> Sat, 14 Jul 2007 10:24:15 +0200

Changed in oprofile:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.