Bug#321502: marked as done (run-parts syntax in /etc/X11/gdm/Xsession)

Sat Oct 8 10:33:11 UTC 2005

Your message dated Sat, 8 Oct 2005 03:26:16 -0700
with message-id <20051008102616.GI22785 at cyberhqz.com>
and subject line Bug#321502: run-parts syntax in /etc/X11/gdm/Xsession
has caused the attached Bug report to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what I am
talking about this indicates a serious mail system misconfiguration
somewhere.  Please contact me immediately.)

Debian bug tracking system administrator
(administrator, Debian Bugs database)

--------------------------------------
Received: (at submit) by bugs.debian.org; 5 Aug 2005 21:53:30 +0000
>From matthew at tarazi.net Fri Aug 05 14:53:30 2005
Return-path: <matthew at tarazi.net>
Received: from web30513.mail.mud.yahoo.com [68.142.201.241] 
	by spohr.debian.org with smtp (Exim 3.36 1 (Debian))
	id 1E1A8A-0000Gk-00; Fri, 05 Aug 2005 14:53:30 -0700
Received: (qmail 4358 invoked by uid 60001); 5 Aug 2005 21:52:59 -0000
Message-ID: <20050805215259.4356.qmail at web30513.mail.mud.yahoo.com>
Received: from [24.12.102.193] by web30513.mail.mud.yahoo.com via HTTP; Fri, 05 Aug 2005 14:52:59 PDT
Date: Fri, 5 Aug 2005 14:52:59 -0700 (PDT)
From: Matthew Tarazi <matthew at tarazi.net>
Subject: run-parts syntax in /etc/X11/gdm/Xsession
To: submit at bugs.debian.org
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Transfer-Encoding: 8bit
Delivered-To: submit at bugs.debian.org
X-Spam-Checker-Version: SpamAssassin 2.60-bugs.debian.org_2005_01_02 
	(1.212-2003-09-23-exp) on spohr.debian.org
X-Spam-Level: 
X-Spam-Status: No, hits=-6.4 required=4.0 tests=BAYES_00,HAS_PACKAGE,
	UPPERCASE_25_50 autolearn=no version=2.60-bugs.debian.org_2005_01_02

Package: gdm
Version: 2.6.0.8-1 

The code in /etc/X11/gdm/Xsession currently has:

SESSIONFILES=$(run_parts $SYSSESSIONDIR)
if [ -n "$SESSIONFILES" ]; then
  for SESSIONFILE in $SESSIONFILES; do
    . $SESSIONFILE
  done
fi

Should the first line here be:

SESSIONFILES=$(run_parts --list $SYSSESSIONDIR)

run-parts without --list will not return the file names, and so
SESSIONFILES will be null. and the for loop will never be exectued

I am currently runing Debian testing (etch)

---------------------------------------
Received: (at 321502-done) by bugs.debian.org; 8 Oct 2005 10:26:17 +0000
>From rmurray at cyberhqz.com Sat Oct 08 03:26:17 2005
Return-path: <rmurray at cyberhqz.com>
Received: from h24-80-164-222.sbm.shawcable.net (straylight.cyberhqz.com) [24.80.164.222] (postfix)
	by spohr.debian.org with esmtp (Exim 3.36 1 (Debian))
	id 1EOBuD-000678-00; Sat, 08 Oct 2005 03:26:17 -0700
Received: from localhost (localhost.localdomain [127.0.0.1])
	by straylight.cyberhqz.com (Postfix) with ESMTP id 663464F144;
	Sat,  8 Oct 2005 03:26:17 -0700 (PDT)
Received: from straylight.cyberhqz.com ([127.0.0.1])
	by localhost (straylight [127.0.0.1]) (amavisd-new, port 10024)
	with ESMTP id 32475-06; Sat, 8 Oct 2005 03:26:16 -0700 (PDT)
Received: by straylight.cyberhqz.com (Postfix, from userid 1000)
	id 8DC984F145; Sat,  8 Oct 2005 03:26:16 -0700 (PDT)
Date: Sat, 8 Oct 2005 03:26:16 -0700
From: Ryan Murray <rmurray at debian.org>
To: Matthew Tarazi <matthew at tarazi.net>, 321502-done at bugs.debian.org
Subject: Re: Bug#321502: run-parts syntax in /etc/X11/gdm/Xsession
Message-ID: <20051008102616.GI22785 at cyberhqz.com>
References: <20050805215259.4356.qmail at web30513.mail.mud.yahoo.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20050805215259.4356.qmail at web30513.mail.mud.yahoo.com>
User-Agent: Mutt/1.5.11
Delivered-To: 321502-done at bugs.debian.org
X-Spam-Checker-Version: SpamAssassin 2.60-bugs.debian.org_2005_01_02 
	(1.212-2003-09-23-exp) on spohr.debian.org
X-Spam-Level: 
X-Spam-Status: No, hits=-4.4 required=4.0 tests=BAYES_00,HAS_BUG_NUMBER,
	UPPERCASE_25_50 autolearn=no version=2.60-bugs.debian.org_2005_01_02

On Fri, Aug 05, 2005 at 02:52:59PM -0700, Matthew Tarazi wrote:
> The code in /etc/X11/gdm/Xsession currently has:
> 
> SESSIONFILES=$(run_parts $SYSSESSIONDIR)
> if [ -n "$SESSIONFILES" ]; then
>   for SESSIONFILE in $SESSIONFILES; do
>     . $SESSIONFILE
>   done
> fi
> 
> 
> Should the first line here be:
> 
> SESSIONFILES=$(run_parts --list $SYSSESSIONDIR)
> 
> run-parts without --list will not return the file names, and so
> SESSIONFILES will be null. and the for loop will never be exectued

No, as run_parts is a shell function defined in the same file, not a call
to the external binary run-parts