Bug 159391

Summary: x11/gdm needs hal dep, rc.d fixes
Product: Ports & Packages Reporter: Doug Barton <dougb>
Component: Individual Port(s)Assignee: freebsd-gnome (Nobody) <gnome>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
file.diff none

Description Doug Barton freebsd_committer freebsd_triage 2011-08-02 20:20:10 UTC 
	The rc.d script calls lshal which is provided by hal, so the
	port needs to have that dependency. (Noticed when installing
	gdm as part of xfce4.)

	The rc.d script itself has some problems, which are addressed
	in the following patch:

	1. If gnome_enable is not set, gdm_enable will remain undefined.
	2. Change the order of calling load_rc_config() and setting the
	   default variables to the typical order for ports
	3. Not only is the checkyesno for gdm_enable in gdm_start() not
	   needed, it causes problems if, for example the user does
	   '/usr/local/etc/rc.d/gdm onestart' from the command line.
	4. Switch to using 'unlink' instead of 'rm -f'
	5. If lshal does not exist, error out instead of looping
	6. Mark a variable local

Fix: Apply the following patch:
How-To-Repeat: 	DNA
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2011-08-02 20:20:20 UTC 
Responsible Changed
From-To: freebsd-ports-bugs->gnome

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 2 dfilter service freebsd_committer freebsd_triage 2012-03-16 03:45:32 UTC 
mezz        2012-03-16 03:45:21 UTC

  FreeBSD ports repository

  Modified files:
    x11/gdm              Makefile 
    x11/gdm/files        gdm.in 
  Log:
  Fix the rc.d script:
  
   1. If gnome_enable is not set, gdm_enable will remain undefined.
   2. Change the order of calling load_rc_config() and setting the
   default variables to the typical order for ports
   3. Not only is the checkyesno for gdm_enable in gdm_start() not
   needed, it causes problems if, for example the user does
   '/usr/local/etc/rc.d/gdm onestart' from the command line.
   4. Switch to using 'unlink' instead of 'rm -f'
   5. If lshal does not exist, error out instead of looping
   6. Mark a variable local
  
  Bump the PORTREVISION.
  
  PR:             ports/159391
  Submitted by:   dougb
  Feature safe:   yes
  
  Revision  Changes    Path
  1.141     +1 -1      ports/x11/gdm/Makefile
  1.22      +17 -12    ports/x11/gdm/files/gdm.in
_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "cvs-all-unsubscribe@freebsd.org"
Comment 3 Jeremy Messenger freebsd_committer freebsd_triage 2012-03-17 18:12:41 UTC 
State Changed
From-To: open->closed

Committed, thanks!