executables in share

Bug #336961 reported by Kees Cook
2
Affects Status Importance Assigned to Milestone
screen-profiles (Ubuntu)
Fix Released
Wishlist
Dustin Kirkland 
Karmic
Fix Released
Wishlist
Dustin Kirkland 

Bug Description

Binary package hint: screen-profiles

/usr/share/screen-profiles/bin/cpu-count
/usr/share/screen-profiles/bin/cpu-freq
/usr/share/screen-profiles/bin/ec2-cost
/usr/share/screen-profiles/bin/hostname
/usr/share/screen-profiles/bin/load-average
/usr/share/screen-profiles/bin/mem-available
/usr/share/screen-profiles/bin/mem-used
/usr/share/screen-profiles/bin/reboot-required
/usr/share/screen-profiles/bin/release
/usr/share/screen-profiles/bin/updates-available
/usr/share/screen-profiles/bin/whoami

These programs should be in /var/lib/screen-profiles, per the FHS. Some people mount /usr/share with noexec.

Revision history for this message
Dustin Kirkland  (kirkland) wrote :

Good call.

I've been needing to fix this for a while. I'm going to add a prerm/postinst script to add/remove a symlink to the /usr/share location, to ensure seamless upgrades.

:-Dustin

Changed in screen-profiles:
assignee: nobody → kirkland
importance: Undecided → Wishlist
status: New → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package screen-profiles - 1.32-0ubuntu1

---------------
screen-profiles (1.32-0ubuntu1) jaunty; urgency=low

  * bin/ec2-cost: since route -n is being used, must grep for 0.0.0.0,
    LP: #336955
  * debian/postinst, debian/prerm: add/remove symlink for bin scripts to
    /var/lib/screen-profiles (should be removed before Jaunty GA)
  * profiles/common, debian/install: install helper scripts into
    /var/lib/screen-profiles rather than /usr/share/screen-profiles/bin,
    LP: #336961

 -- Dustin Kirkland <email address hidden> Mon, 02 Mar 2009 17:35:12 -0600

Changed in screen-profiles:
status: In Progress → Fix Released
Revision history for this message
Colin Watson (cjwatson) wrote :

As discussed on IRC, /var/lib/ is wrong since binaries are not variable state data. /usr/lib/ would be an acceptable compromise.

Revision history for this message
Dustin Kirkland  (kirkland) wrote :

Reopening...

Changed in screen-profiles (Ubuntu):
status: Fix Released → Triaged
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (25.8 KiB)

This bug was fixed in the package screen-profiles - 1.48-1

---------------
screen-profiles (1.48-1) unstable; urgency=low

  * upload to unstable (Closes: #518958)
  * add myself to uploaders
  * Bump Standards-Version to 3.8.1

screen-profiles (1.48-0ubuntu1) jaunty; urgency=low

  Drop the distro name in generated profiles
  * debian/install: fix installation of profiles
  * debian/rules: don't create the ubuntu profile symlink
  * debian/screen-profiles-extras.install: no longer need to install
    the distro-specific profiles
  * profiles/generate: generate profiles which do not have ubuntu-
    prepended
  * select-screen-profile: remove the ubuntu specific profile handling
  * screen: fix broken symlinks to ubuntu-* profiles
  * screen-profiles: drop special ubuntu handling
  * screen-profiles-export: drop ubuntu prepending

  * screen: default to the light profile, if unspecified; this should be
    acceptable now that the F9:Menu prompt is always shown, and from there,
    any user can easily change their profile; just touch the windows config
    file since F5 reloads reopens all windows; launch with motd+shell if
    no args to screen are specified
  * keybindings/common: don't force users to name a window on F2 window
    creation
  * debian/postrm: remove the || true on the diversion removal, per Debian
    ITP; I suppose we want to know if the diversion removal failed
  * screen-profiles, screen-profiles-status: default uptime status to 'on',
    generally useful, and pretty compact
  * bin/mem-available: fix subtle bug where whitespace is printed if in a
    totally default (no status file) configuration
  * screen-profiles-status.1, debian/rules: manpage added
  * screen-profiles-export: randomly generate tarball name, if unspecified

screen-profiles (1.47-0ubuntu1) jaunty; urgency=low

  * screen-profiles-status: stop grepping after one match, subtle perf
    improvement
  * bin/logo: lsb_release is really expensive, try /etc/issue first,
    and fall back to lsb_release if that fails
  * bin/date, bin/time: last release installed 644 for some reason?

screen-profiles (1.46-0ubuntu1) jaunty; urgency=low

  * bin/date, bin/time, screen-profiles-status, screen-profiles-export,
    screen-profiles: allow for configuring on/off the date and time
    status, LP: #359050
  * bin/logo: added logos for foresight and xandros
  * bin/battery: use the more intense yellow for lower battery percentage

screen-profiles (1.45-0ubuntu1) jaunty; urgency=low

  [ Dustin Kirkland ]
  * bin/cpu-freq: fix for powerpc LP: #352286, fix for arm LP: #362157,
    always report in GHz, round to nearest hundred
  * bin/uptime: don't print seconds, since we don't update often enough
  * bin/*: route and ifconfig are in /sbin, fully qualify
  * Performance improvements, LP: #360759
    - screen-profiles-status: use home bin scripts, if available first,
      then fallback to system-wide, allows for local overrides; comment
      out nice and ionice calls; this *really* slows down startup
    - bin/release: eliminate one expensive lsb_release call
    - bin/updates-available: background updates checking processes,
      let them write to cache, and let th...

Changed in screen-profiles (Ubuntu Karmic):
status: Triaged → 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.